On 2015/3/13 22:48, Bjorn Helgaas wrote:

On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau [off-list ref] wrote:

quoted

On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote:

quoted

On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote:

quoted

...
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.

Sure, I will repost it today.

Bjorn

.

-- 
Thanks!
Yijing