On Sat, 06 Mar 2010 20:39:11 -0700 Bjorn Helgaas <bjorn.helgaas@xxxxxx> wrote: > Linus says: > > > I have nothing against this patch-series, but I'm not really much of a > > pnp person, so I'd like to see this at a minimum being acked by people who > > are, and who would use this. > > > In fact, since the primary _reason_ for this seems to be to improve on the > > _CRS parsing code for PCI, I'd have expected that the people involved > > with previous _CRS issues be involved, and perhaps this whole thing could > > go through the PCI tree? But Yinghai Lu and Jesse Barnes weren't even > > cc'd. > > > But maybe this really is more of an ACPI issue. I dunno. I get the feeling > > that ACPI doesn't really care, though - the subsystem that actually gets > > _affected_ by all this in the end is PCI. > > > Anyway, more acks, please. And preferably involve the PCI people too. > > So here you go :-) Please comment. > > Linus is right; eventually I would like to use this to improve the PCI > host bridge _CRS parsing. I didn't think to include you at first because > that goal is still so far off that I can only dimly envision it. What I > *can* see as possible steps are these: > > - add PNP support for bus number ranges and windows (these patches) > - initialize PNP (at least PNPACPI, probably PNPBIOS & ISAPNP also) > earlier, before PCI probing > - get rid of acpi_pci_root_start() by folding it into acpi_pci_root_add() > (this is the big one and will require lots of PCI and ACPI hotplug > cleanup that I don't know how to do yet) > - convert pci_root.c from an ACPI driver to a PNP driver > - remove pci_root.c _CRS parsing because PNPACPI already does it > > The whole series is here: http://lkml.org/lkml/2010/3/5/243 and the > cover is below. Sounds like a good direction to me, assuming we can work through the issues in this merge window (and it seems like you are). Acked-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
