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@xxxxxxxxxx>
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> 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
