On Mon, 3 May 2010, Bjorn Helgaas wrote: > > My *plan* was to make pci_root.c a PNP driver, so it would use the > pre-parsed PNP resources to find the host bridge windows rather than > re-parsing the ACPI _CRS info by hand. Those PNP resources are not in > the resource tree when the driver first sees them (which might be a > deficiency of our current PNP core), so I don't think there's currently > a way for the driver to distinguish IO from MEM unless it's in the > resource itself. > > At this stage of the release, my preference would be to use the patch > above (with the intention of coming back later for some more work), or > to just revert fa35b4926 (PNPACPI window support) and maybe 9d7cca04 > (generic window support). If we revert those patches, I'll have to > figure out another way to clean up pci_root.c. Oh, I'm fine with your patch, I just reacted to it wondering whether maybe there was some cleaner way of solving the problem. I guess annotating the resource itself is ok. So don't take my comment as a NAK, more of a "hmm, how about.. " Linus -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
