pciconf: incorrect description for Marvell 88SX6101 chip

Andriy Gapon avg at icyb.net.ua
Sat Jan 10 05:57:41 PST 2009


on 21/11/2008 15:14 Andriy Gapon said the following:
> I wasn't sure where this belongs, so writing here.
> This is stable/7 on Intel DG33TL:
> 
> $ pciconf -lv
> ...
> atapci0 at pci0:3:0:0:     class=0x01018f card=0x610111ab chip=0x610111ab
> rev=0xb2 hdr=0x00
>     vendor     = 'Marvell Semiconductor (Was: Galileo Technology Ltd)'
>     device     = '6101 SATA2 Controller'
>     class      = mass storage
>     subclass   = ATA
> ...
> 
> This is actually a PATA controller (SATA is provided by ICH9).
> lspci is more correct:
> $ lspci -v
> ...
> 03:00.0 IDE interface: Marvell Technology Group Ltd. 88SE6101
> single-port PATA133 interface (rev b2) (prog-if 8f [Master SecP SecO
> PriP PriO])
>         Subsystem: Marvell Technology Group Ltd. 88SE6101 single-port
> PATA133 interface
> ...
> 
> ATA driver is also cool:
> atapci0: <Marvell 88SX6101 UDMA133 controller>
> 

This is very minor but still...
Here's a tiny patch.
diff --git a/share/misc/pci_vendors b/share/misc/pci_vendors
index 8286310..c92d12e 100644
--- a/share/misc/pci_vendors
+++ b/share/misc/pci_vendors
@@ -4606,7 +4606,7 @@
 	6041	MV88SX6041 Marvell Technology Group Ltd. MV88SX6041 4-port SATA
II PCI-X Controller (rev 03)
 	6042	MV88SX6042 4-port SATA II PCI-X Controller
 	6081	MV88SX6081 8-port SATA II PCI-X Controller
-	6101	6101 SATA2 Controller
+	6101	MV88SX6101 1-port UltraATA/133 Controller
 	6111	6111 SATA2 Controller
 	6120	6120 SATA2 Controller
 	6121	6121 SATA2 Controller



-- 
Andriy Gapon


More information about the freebsd-stable mailing list