From owner-freebsd-bugs@FreeBSD.ORG Tue Sep 16 11:30:25 2003 Return-Path: Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 370F316A4BF for ; Tue, 16 Sep 2003 11:30:25 -0700 (PDT) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 3799643FF3 for ; Tue, 16 Sep 2003 11:30:24 -0700 (PDT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.12.9/8.12.9) with ESMTP id h8GIUNUp075121 for ; Tue, 16 Sep 2003 11:30:23 -0700 (PDT) (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.12.9/8.12.9/Submit) id h8GIUNxu075120; Tue, 16 Sep 2003 11:30:23 -0700 (PDT) Date: Tue, 16 Sep 2003 11:30:23 -0700 (PDT) Message-Id: <200309161830.h8GIUNxu075120@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Simon Walton Subject: Re: kern/56924: [Patch] ATA controller seems to have wrong ID for ServerWorkschipset X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: Simon Walton List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 16 Sep 2003 18:30:25 -0000 The following reply was made to PR kern/56924; it has been noted by GNATS. From: Simon Walton To: Soren Schmidt Cc: FreeBSD-gnats-submit@FreeBSD.ORG Subject: Re: kern/56924: [Patch] ATA controller seems to have wrong ID for ServerWorkschipset Date: Tue, 16 Sep 2003 11:24:28 -0700 Soren Schmidt wrote: > > It seems Simon Walton wrote: > > The current code seems to check for the chip's subid, not the chip id: > > > > pciconf -lv > > ... > > atapci0@pci0:15:1: class=0x01018a card=0x02121166 chip=0x02131166 > > rev=0xa0 > > hdr=0x00 > > vendor = 'Reliance Computer Corp./ServerWorks' > > class = mass storage > > subclass = ATA > > Nope, what you have is a CSB6 chip, the code in 4.7 only knows about > the CSB5... > > -Søren OK. Is this fixed in the 4.x branch? I see the same values in the CVS repository. Simon