bus: omap_l3_noc: introduce concept of submodule

While OMAP4 and OMAP5 had 3 separate clock domains, DRA7 has only 2
and the first one then is internally divided into 2 sub clock domains.

To better represent this in the driver, we use the concept of submodule.

The address defintions in the devicetree is as per the high level
clock domain(module) base, the sub clockdomain/subdomain which shares
the same register space of a clockdomain is marked in the SoC data as
L3_BASE_IS_SUBMODULE.

L3_BASE_IS_SUBMODULE is used as an indication that it's base address is
the same as the parent module and offsets are considered from the same
base address as they are usually intermingled.

Other than the base address, the submodule is same as a module as it is
functionally so.

Signed-off-by: Nishanth Menon <nm@ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
Acked-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Tested-by: Darren Etheridge <detheridge@ti.com>
Tested-by: Sekhar Nori <nsekhar@ti.com>

1 files changed, 12 insertions, 5 deletions

diff --git a/drivers/bus/omap_l3_noc.c b/drivers/bus/omap_l3_noc.c
index 08344b03fda6..0eba07ac6008 100644
--- a/drivers/bus/omap_l3_noc.c
+++ b/drivers/bus/omap_l3_noc.c
@@ -238,7 +238,7 @@ static int omap_l3_probe(struct platform_device *pdev)
 {
         const struct of_device_id *of_id;
         static struct omap_l3 *l3;
-        int ret, i;
+        int ret, i, res_idx;
 
         of_id = of_match_device(l3_noc_match, &pdev->dev);
         if (!of_id) {
@@ -255,15 +255,22 @@ static int omap_l3_probe(struct platform_device *pdev)
         platform_set_drvdata(pdev, l3);
 
         /* Get mem resources */
-        for (i = 0; i < l3->num_modules; i++) {
-                struct resource *res = platform_get_resource(pdev,
-                                                             IORESOURCE_MEM, i);
-
+        for (i = 0, res_idx = 0; i < l3->num_modules; i++) {
+                struct resource *res;
+
+                if (l3->l3_base[i] == L3_BASE_IS_SUBMODULE) {
+                        /* First entry cannot be submodule */
+                        BUG_ON(i == 0);
+                        l3->l3_base[i] = l3->l3_base[i - 1];
+                        continue;
+                }
+                res = platform_get_resource(pdev, IORESOURCE_MEM, res_idx);
                 l3->l3_base[i] = devm_ioremap_resource(&pdev->dev, res);
                 if (IS_ERR(l3->l3_base[i])) {
                         dev_err(l3->dev, "ioremap %d failed\n", i);
                         return PTR_ERR(l3->l3_base[i]);
                 }
+                res_idx++;
         }
 
         /*