[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAErSpo5TjhNv1W=PxSPO6qXfhW1Mi4dMp83vj82=O_hFOq1vow@mail.gmail.com>
Date: Tue, 6 Mar 2012 16:13:42 -0700
From: Bjorn Helgaas <bhelgaas@...gle.com>
To: Yinghai Lu <yinghai@...nel.org>
Cc: Jesse Barnes <jbarnes@...tuousgeek.org>, x86 <x86@...nel.org>,
linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 23/23] x86, PCI: add pcibios_root_rescan
On Tue, Mar 6, 2012 at 12:14 AM, Yinghai Lu <yinghai@...nel.org> wrote:
> need use it to rescan root bus that was not added via acpi probe.
>
> Signed-off-by: Yinghai Lu <yinghai@...nel.org>
> ---
> arch/x86/pci/legacy.c | 25 +++++++++++++++++++++++++
> 1 files changed, 25 insertions(+), 0 deletions(-)
>
> diff --git a/arch/x86/pci/legacy.c b/arch/x86/pci/legacy.c
> index aab0e41..7a7c78f 100644
> --- a/arch/x86/pci/legacy.c
> +++ b/arch/x86/pci/legacy.c
> @@ -79,3 +79,28 @@ int __init pci_subsys_init(void)
> return 0;
> }
> subsys_initcall(pci_subsys_init);
> +
> +void __devinit pcibios_root_rescan(void)
> +{
> + int busn;
> + struct pci_bus *bus;
> +
> + if (pcibios_last_bus <= 0 || pcibios_last_bus > 0xff)
> + return;
> +
> + for (busn = 0; busn <= pcibios_last_bus; busn++) {
> + bus = pci_find_bus(0, busn);
> +
> + if (bus)
> + continue;
> +
> + bus = __pcibios_scan_specific_bus(busn, false);
> +
> + if (!bus)
> + continue;
> +
> + pci_assign_unassigned_bus_resources(bus);
> +
> + pci_bus_add_devices(bus);
> + }
> +}
Does anybody call this?
Do we need it? I assume we only care about host bridge hotplug via ACPI on x86.
Please wait a while (at least a day or two) before reposting your
series so people have time to take a look at the current one. It's a
real nuisance to be working through a large series like this and get
bombed with a repost of the whole thing that tweaks some minor detail.
I'd rather take some time and get as much of the discussion out of
the way on V1 as possible before getting to V2, V3, V4, etc.
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists