| Message ID | 20200522234832.954484-15-robh@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | PCI controller probe cleanups | expand |
On 23. 05. 20 1:48, Rob Herring wrote: > The xilinx-nwl host driver does the same host registration and bus scanning > calls as pci_host_probe, so let's use it instead. > > The only difference is pci_assign_unassigned_bus_resources() was called > instead of pci_bus_size_bridges() and pci_bus_assign_resources(). This > should be the same. > > Cc: Michal Simek <michal.simek@xilinx.com> > Signed-off-by: Rob Herring <robh@kernel.org> > --- > drivers/pci/controller/pcie-xilinx-nwl.c | 14 +------------- > 1 file changed, 1 insertion(+), 13 deletions(-) > > diff --git a/drivers/pci/controller/pcie-xilinx-nwl.c b/drivers/pci/controller/pcie-xilinx-nwl.c > index 9bd1427f2fd6..32a0b08d6da5 100644 > --- a/drivers/pci/controller/pcie-xilinx-nwl.c > +++ b/drivers/pci/controller/pcie-xilinx-nwl.c > @@ -817,8 +817,6 @@ static int nwl_pcie_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct nwl_pcie *pcie; > - struct pci_bus *bus; > - struct pci_bus *child; > struct pci_host_bridge *bridge; > int err; > > @@ -871,17 +869,7 @@ static int nwl_pcie_probe(struct platform_device *pdev) > } > } > > - err = pci_scan_root_bus_bridge(bridge); > - if (err) > - return err; > - > - bus = bridge->bus; > - > - pci_assign_unassigned_bus_resources(bus); > - list_for_each_entry(child, &bus->children, node) > - pcie_bus_configure_settings(child); > - pci_bus_add_devices(bus); > - return 0; > + return pci_host_probe(bridge); > } > > static struct platform_driver nwl_pcie_driver = { > Bharat: Please review. Thanks, Michal
diff --git a/drivers/pci/controller/pcie-xilinx-nwl.c b/drivers/pci/controller/pcie-xilinx-nwl.c index 9bd1427f2fd6..32a0b08d6da5 100644 --- a/drivers/pci/controller/pcie-xilinx-nwl.c +++ b/drivers/pci/controller/pcie-xilinx-nwl.c @@ -817,8 +817,6 @@ static int nwl_pcie_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; struct nwl_pcie *pcie; - struct pci_bus *bus; - struct pci_bus *child; struct pci_host_bridge *bridge; int err; @@ -871,17 +869,7 @@ static int nwl_pcie_probe(struct platform_device *pdev) } } - err = pci_scan_root_bus_bridge(bridge); - if (err) - return err; - - bus = bridge->bus; - - pci_assign_unassigned_bus_resources(bus); - list_for_each_entry(child, &bus->children, node) - pcie_bus_configure_settings(child); - pci_bus_add_devices(bus); - return 0; + return pci_host_probe(bridge); } static struct platform_driver nwl_pcie_driver = {
The xilinx-nwl host driver does the same host registration and bus scanning calls as pci_host_probe, so let's use it instead. The only difference is pci_assign_unassigned_bus_resources() was called instead of pci_bus_size_bridges() and pci_bus_assign_resources(). This should be the same. Cc: Michal Simek <michal.simek@xilinx.com> Signed-off-by: Rob Herring <robh@kernel.org> --- drivers/pci/controller/pcie-xilinx-nwl.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-)