Hi Shawn, Thanks for the patch! On 05/09/2018 08:39 AM, Shawn Guo wrote: > The root_bus_nr initialization has already been done in function > dw_pcie_host_init(). The -1 assignment in platform driver before > calling into dw_pcie_host_init() is not really needed. Drop it. > > Signed-off-by: Shawn Guo <shawn.guo@xxxxxxxxxx> > Cc: Stanimir Varbanov <svarbanov@xxxxxxxxxx> > --- > drivers/pci/dwc/pcie-qcom.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/pci/dwc/pcie-qcom.c b/drivers/pci/dwc/pcie-qcom.c > index 5897af7d3355..a32aa024e907 100644 > --- a/drivers/pci/dwc/pcie-qcom.c > +++ b/drivers/pci/dwc/pcie-qcom.c > @@ -1247,7 +1247,6 @@ static int qcom_pcie_probe(struct platform_device *pdev) > if (ret) > return ret; > > - pp->root_bus_nr = -1; > pp->ops = &qcom_pcie_dw_ops; > > if (IS_ENABLED(CONFIG_PCI_MSI)) { > Acked-by: Stanimir Varbanov <svarbanov@xxxxxxxxxx> -- regards, Stan