From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alvaro Lopes Subject: Re: Re: ACPI PCI bus mismatch ? Date: Mon, 10 Feb 2003 18:13:03 +0000 Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Message-ID: <3E47EBAF.1040103@alvie.com> References: <20030210161754.GI25625@poup.poupinou.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20030210161754.GI25625-j6u/t2rXLliUoIHC/UFpr9i2O/JbrIOy@public.gmane.org> Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Help: List-Post: List-Subscribe: , List-Unsubscribe: , List-Archive: To: Ducrot Bruno Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: linux-acpi@vger.kernel.org Ducrot Bruno wrote: >Hi! > >On Thu, Feb 06, 2003 at 12:39:43PM -0800, Grover, Andrew wrote: > =20 > >>>From: Alvaro Lopes [mailto:alvieboy-JkNoWtNHh5AAvxtiuMwx3w@public.gmane.org]=20 >>>OK, I tracked the problem down to acpi_os_derive_pci_id(). If=20 >>>I remove=20 >>>the call to acpi_os_derive_pci_id() in=20 >>>acpi_ev_pci_config_region_setup()=20 >>>it works fine. >>> >>>Will do some tests on 2.5.x now and let you know. >>> =20 >>> >>Grrreat. >> >>We stuck that code in because some IA64 systems needed it to work. >> >>Sounds like we need to re-think all this. A hack for right now might be >>to only call derive on IA64 boxes. >> =20 >> > > >It is KO for me. > >I sticked: >printk(KERN_DEBUG "BEFORE: 0x%x 0x%x 0x%x 0x%x\n", pci_id->segment, pci_= id->bus, > pci_id->device, pci_i= d->function); >acpi_os_derive_pci_id(...); >printk(KERN_DEBUG "AFTER: 0x%x 0x%x 0x%x 0x%x\n", pci_id->segment, pci_i= d->bus, > pci_id->device, pci_i= d->function); > >in drivers/acpi/events/evrgnini.c::ev_pci_config_region_setup() > >and it show me that pci_id->bus is *always* 0 at BEFORE, and will be >correct in AFTER (tested by creating 2 OPs, in fact). > =20 > Perhaps a stupid question, but: If we are reading the busID for the brigde itself, shouldn't we use the=20 "Primary bus number" @ 0x18, instead of "Secondary bus number" @ 0x19 ? =C1lvaro --=20 =C1lvaro Lopes=20 --------------------- A .sig is just a .sig ------------------------------------------------------- This SF.NET email is sponsored by: SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See! http://www.vasoftware.com