Hi Srinath, On 4/26/2019 2:20 AM, Srinath Mannam wrote: > iProc config read flag has to enable for PAXBv2 instead of PAXB. > > Fixes: f78e60a29d4ff ("PCI: iproc: Reject unconfigured physical functions from PAXC") > Signed-off-by: Srinath Mannam <srinath.mannam@xxxxxxxxxxxx> > --- > drivers/pci/controller/pcie-iproc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/pcie-iproc.c b/drivers/pci/controller/pcie-iproc.c > index c20fd6b..9d5cbc7 100644 > --- a/drivers/pci/controller/pcie-iproc.c > +++ b/drivers/pci/controller/pcie-iproc.c > @@ -1347,7 +1347,6 @@ static int iproc_pcie_rev_init(struct iproc_pcie *pcie) > break; > case IPROC_PCIE_PAXB: > regs = iproc_pcie_reg_paxb; > - pcie->iproc_cfg_read = true; > pcie->has_apb_err_disable = true; > if (pcie->need_ob_cfg) { > pcie->ob_map = paxb_ob_map; > @@ -1356,6 +1355,7 @@ static int iproc_pcie_rev_init(struct iproc_pcie *pcie) > break; > case IPROC_PCIE_PAXB_V2: > regs = iproc_pcie_reg_paxb_v2; > + pcie->iproc_cfg_read = true; > pcie->has_apb_err_disable = true; > if (pcie->need_ob_cfg) { > pcie->ob_map = paxb_v2_ob_map; > Fix looks good to me. Thanks! Reviewed-by: Ray Jui <ray.jui@xxxxxxxxxxxx>