On Sun, Jul 01, 2018 at 11:54:47AM +0800, Shawn Guo wrote: > Function dw_pcie_host_init() already initializes the root_bus_nr field > of 'struct pcie_port', so the -1 assignment prior to calling > dw_pcie_host_init() in platform specific driver is not really needed. > Drop it. > > Signed-off-by: Shawn Guo <shawn.guo@xxxxxxxxxx> Acked-by: Jesper Nilsson <jesper.nilsson@xxxxxxxx> > --- > drivers/pci/controller/dwc/pcie-artpec6.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/pci/controller/dwc/pcie-artpec6.c b/drivers/pci/controller/dwc/pcie-artpec6.c > index 321b56cfd5d0..128b182648b3 100644 > --- a/drivers/pci/controller/dwc/pcie-artpec6.c > +++ b/drivers/pci/controller/dwc/pcie-artpec6.c > @@ -399,7 +399,6 @@ static int artpec6_add_pcie_port(struct artpec6_pcie *artpec6_pcie, > } > } > > - pp->root_bus_nr = -1; > pp->ops = &artpec6_pcie_host_ops; > > ret = dw_pcie_host_init(pp); > -- > 1.9.1 /^JN - Jesper Nilsson -- Jesper Nilsson -- jesper.nilsson@xxxxxxxx