On Wed, Jan 9, 2013 at 10:53 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > On Wed, Jan 9, 2013 at 9:35 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: >>> I don't know, that could be separated patcheset after we conclude >>> pci root bus hotplug support. >> >> The main reason I review patches before merging them is to identify >> issues that we can fix before they affect everybody. It makes my life >> a lot easier if I don't have to keep track of pending unaddressed >> review comments. Is there an advantage to waiting and doing this work >> later? >> >>>>> In general these look good. My main concern is that they only touch >>>>> x86, without touching the similar code in frv, microblaze, mn10300, >>>>> and powerpc. > > the reason why we need to change those codes for x86, we want to make it support > pci root bus hotplug. So it would be reasonable for us to align other > platform to x86 > changes after pci root bus hotplug change is completely done. OK, I opened https://bugzilla.kernel.org/show_bug.cgi?id=52531 as a way to keep track of this consistency issue and merged pci/yinghai-survey-resources to my -next branch. -- 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