On Wed, Feb 22, 2012 at 1:41 PM, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> wrote: > On Wed, 2012-02-22 at 11:19 -0700, Bjorn Helgaas wrote: >> We already use pci_flags, so this just sets pci_flags directly and removes >> the intermediate step of figuring out pci_probe_only, then using it to set >> pci_flags. > > Ah yes, those flags are now common indeed. > >> The PCI core provides a pci_flags definition (currently __weak), so drop >> the powerpc definitions in favor of that. Note that we must set the ppc64 >> default (PCI_PROBE_ONLY set) in the 64-bit setup_arch() before calling >> the platform .setup_arch() method, which may override the default. > > Ah wait, I have a patch about to hit -next that flips the ppc64 > default... > > Maybe it's easier if you just pick it up and stick it in your series, > that will avoid integration clashes.. > > http://patchwork.ozlabs.org/patch/141225/ Sure, I'll pick up that patch and repost the series. I'll wait until tomorrow or so in case anybody else has comments. Thanks, Bjorn -- 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