pci: layerscape: Add size check for config resource
authorWasim Khan <wasim.khan@nxp.com>
Mon, 28 Sep 2020 10:56:13 +0000 (16:26 +0530)
committerPriyanka Jain <priyanka.jain@nxp.com>
Thu, 10 Dec 2020 08:26:39 +0000 (13:56 +0530)
resource "config" is required to have minimum 8KB space
as per hardware documentation.

Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Signed-off-by: Wasim Khan <wasim.khan@nxp.com>
Reviewed-by: Priyanka Jain <priyanka.jain@nxp.com>
drivers/pci/pcie_layerscape_rc.c

index f9e3089..cdfcad6 100644 (file)
@@ -314,6 +314,13 @@ static int ls_pcie_probe(struct udevice *dev)
                return ret;
        }
 
+       cfg_size = fdt_resource_size(&pcie_rc->cfg_res);
+       if (cfg_size < SZ_8K) {
+               printf("PCIe%d: %s Invalid size(0x%llx) for resource \"config\",expected minimum 0x%x\n",
+                      PCIE_SRDS_PRTCL(pcie->idx), dev->name, (u64)cfg_size, SZ_8K);
+               return 0;
+       }
+
        /*
         * Fix the pcie memory map address and PF control registers address
         * for LS2088A series SoCs
@@ -323,7 +330,6 @@ static int ls_pcie_probe(struct udevice *dev)
        if (svr == SVR_LS2088A || svr == SVR_LS2084A ||
            svr == SVR_LS2048A || svr == SVR_LS2044A ||
            svr == SVR_LS2081A || svr == SVR_LS2041A) {
-               cfg_size = fdt_resource_size(&pcie_rc->cfg_res);
                pcie_rc->cfg_res.start = LS2088A_PCIE1_PHYS_ADDR +
                                         LS2088A_PCIE_PHYS_SIZE * pcie->idx;
                pcie_rc->cfg_res.end = pcie_rc->cfg_res.start + cfg_size;