From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Wed, 17 Jan 2001 05:15:19 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Wed, 17 Jan 2001 05:15:10 -0500 Received: from irz301.inf.tu-dresden.de ([141.76.2.1]:53032 "EHLO irz301.inf.tu-dresden.de") by vger.kernel.org with ESMTP id ; Wed, 17 Jan 2001 05:15:00 -0500 Date: Wed, 17 Jan 2001 11:14:49 +0100 From: Adam Lackorzynski To: Martin Mares Cc: Kernel Mailing List Subject: Re: [PATCH] PCI-Devices and ServerWorks chipset Message-ID: <20010117111449.A5195@inf.tu-dresden.de> In-Reply-To: <20010116181207.A1325@inf.tu-dresden.de> <20010117095221.A553@albireo.ucw.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20010117095221.A553@albireo.ucw.cz>; from mj@suse.cz on Wed, Jan 17, 2001 at 09:52:21AM +0100 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Hi! On Wed Jan 17, 2001 at 09:52:21 +0100, Martin Mares wrote: > > The patch below (against vanilla 2.4.0) makes Linux recognize > > PCI-Devices sitting in another PCI bus than 0 (or 1). > > > > This was tested on a Netfinity 7100-8666 using a ServerWorks chipset. > > I don't have the ServerWorks chipset documentation at hand, but I think your > patch is wrong -- it doesn't make any sense to scan a bus _range_. The registers > 0x44 and 0x45 are probably ID's of two primary buses and the code should scan > both of them, but not the space between them. I was just inspired by the comment in this function and knew that this "patch" was probably wrong but it was enough to get someone else' eyes watching on it ... :-) BTW, one line of dmesg says: PCI: PCI BIOS revision 2.10 entry at 0xfd5cc, last bus=6 So that "6" in 0x45 made some sense to me... > Anyway, could you send me a `lspci -MH1 -vvx' and `lspci -vvxxx -s0:0', please? $ lspci -MH1 -vvx 00:00.0 Host bridge: ServerWorks CNB20HE (rev 21) Control: I/O- Mem- BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- Status: Cap- 66Mhz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR-