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