From owner-freebsd-stable@FreeBSD.ORG Sat Jan 10 13:57:40 2009 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8BEF41065670 for ; Sat, 10 Jan 2009 13:57:40 +0000 (UTC) (envelope-from avg@icyb.net.ua) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id BB4B68FC18 for ; Sat, 10 Jan 2009 13:57:39 +0000 (UTC) (envelope-from avg@icyb.net.ua) Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua [212.40.38.101]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id PAA21678 for ; Sat, 10 Jan 2009 15:57:36 +0200 (EET) (envelope-from avg@icyb.net.ua) Message-ID: <4968A950.8000806@icyb.net.ua> Date: Sat, 10 Jan 2009 15:57:36 +0200 From: Andriy Gapon User-Agent: Thunderbird 2.0.0.18 (X11/20081124) MIME-Version: 1.0 To: FreeBSD Stable References: <4926B42E.2010408@icyb.net.ua> In-Reply-To: <4926B42E.2010408@icyb.net.ua> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: pciconf: incorrect description for Marvell 88SX6101 chip X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 10 Jan 2009 13:57:41 -0000 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@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: > 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