On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau <liviu@xxxxxxxxxxx> wrote: > On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote: >> On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote: >> > ... >> > Yijing Wang (29): >> > PCI: Rip out pci_bus_add_devices() from pci_scan_bus() >> > PCI: Rip out pci_bus_add_devices() from pci_scan_root_bus() >> > sparc/PCI: Claim bus resources before pci_bus_add_devices() >> >> I put the above patches plus a cleanup patch on my pci/enumeration branch. >> Please rebase your next revision to that. If there's anything else >> non-controversial that we can pull in to chip away at this, let me know. > > Hi Bjorn and Yijing, > > Sorry for not providing feedback earlier on this series (or any previous ones), > I'm just crawling out from under a rock of doing graphics drivers :) > > The patch ripping out pci_bus_add_devices() from pci_scan_root_bus() misses out > the users of that function from drivers/pci/host. If Yijing is going to refresh > the series it is probably worth starting with adding back the call into pcie-xilinx.c > and pci-versatile.c. Huh, sure enough, I missed those, too. Yijing, when you add those, can you pull my branch, update the patch, and repost it? That way you'll keep my changelog and cleanup updates. 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