On Wed, Mar 21, 2018 at 02:42:46PM +0000, Lorenzo Pieralisi wrote: > From: Rob Herring <robh@xxxxxxxxxx> > > The iproc driver is using ARM's struct pci_sys_data simply to store a > private data pointer. This is completely unnecessary, so store the > private data directly in bus->sysdata as is done on arm64. Oh, thank you, this is awesome! struct pci_sys_data is still referenced in mvebu and rcar. I didn't look closely enough to see if they could be similarly simplified. I guess not, because they use pci_common_init_dev(). They look tantalizingly close to being able to use pci_scan_root_bus_bridge() like everybody else, though. Bjorn
