bus: omap_l3_noc: convert flagmux information into a structure
authorNishanth Menon <nm@ti.com>
Mon, 14 Apr 2014 14:57:50 +0000 (09:57 -0500)
committerNishanth Menon <nm@ti.com>
Mon, 5 May 2014 19:33:40 +0000 (14:33 -0500)
This allows us to encompass target information and flag mux offset that
points to the target information into a singular structure. This saves
us the need to look up two different arrays indexed by module ID for
information.

This allows us to reduce the static target information allocation to
just the ones that are documented.

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>
drivers/bus/omap_l3_noc.c
drivers/bus/omap_l3_noc.h

index 7e0a988ad579085370313584f9911fa6066fc409..9524452ee12c810e56e3ab40e2b0ec7bfefa8f63 100644 (file)
@@ -60,6 +60,7 @@ static irqreturn_t l3_interrupt_handler(int irq, void *_l3)
        void __iomem *l3_targ_stderr, *l3_targ_slvofslsb, *l3_targ_mstaddr;
        char *target_name, *master_name = "UN IDENTIFIED";
        struct l3_target_data *l3_targ_inst;
+       struct l3_flagmux_data *flag_mux;
        struct l3_masters_data *master;
 
        /* Get the Type of interrupt */
@@ -71,7 +72,8 @@ static irqreturn_t l3_interrupt_handler(int irq, void *_l3)
                 * to determine the source
                 */
                base = l3->l3_base[i];
-               err_reg = readl_relaxed(base + l3->l3_flagmux[i] +
+               flag_mux = l3->l3_flagmux[i];
+               err_reg = readl_relaxed(base + flag_mux->offset +
                                        L3_FLAGMUX_REGERR0 + (inttype << 3));
 
                /* Get the corresponding error and analyse */
@@ -82,9 +84,13 @@ static irqreturn_t l3_interrupt_handler(int irq, void *_l3)
                        /* We DONOT expect err_src to go out of bounds */
                        BUG_ON(err_src > MAX_CLKDM_TARGETS);
 
-                       l3_targ_inst = &l3->l3_targ[i][err_src];
-                       target_name = l3_targ_inst->name;
-                       l3_targ_base = base + l3_targ_inst->offset;
+                       if (err_src < flag_mux->num_targ_data) {
+                               l3_targ_inst = &flag_mux->l3_targ[err_src];
+                               target_name = l3_targ_inst->name;
+                               l3_targ_base = base + l3_targ_inst->offset;
+                       } else {
+                               target_name = L3_TARGET_NOT_SUPPORTED;
+                       }
 
                        /*
                         * If we do not know of a register offset to decode
@@ -104,7 +110,7 @@ static irqreturn_t l3_interrupt_handler(int irq, void *_l3)
                                        inttype ? "debug" : "application",
                                        err_src, i, "(unclearable)");
 
-                               mask_reg = base + l3->l3_flagmux[i] +
+                               mask_reg = base + flag_mux->offset +
                                           L3_FLAGMUX_MASK0 + (inttype << 3);
                                mask_val = readl_relaxed(mask_reg);
                                mask_val &= ~(1 << err_src);
index e60865fe5965ace5d27493be2bc3e227b97e4596..64869fe656e5465e0ef65c170f263c7050296a91 100644 (file)
@@ -17,8 +17,8 @@
 #ifndef __OMAP_L3_NOC_H
 #define __OMAP_L3_NOC_H
 
-#define OMAP_L3_MODULES                        3
 #define MAX_L3_MODULES                 3
+#define MAX_CLKDM_TARGETS              31
 
 #define CLEAR_STDERR_LOG               (1 << 31)
 #define CUSTOM_ERROR                   0x2
@@ -36,8 +36,6 @@
 
 #define L3_TARGET_NOT_SUPPORTED                NULL
 
-#define MAX_CLKDM_TARGETS              31
-
 /**
  * struct l3_masters_data - L3 Master information
  * @id:                ID of the L3 Master
@@ -60,21 +58,32 @@ struct l3_target_data {
        char *name;
 };
 
+/**
+ * struct l3_flagmux_data - Flag Mux information
+ * @offset:    offset from base for flagmux register
+ * @l3_targ:   array indexed by flagmux index (bit offset) pointing to the
+ *             target data. unsupported ones are marked with
+ *             L3_TARGET_NOT_SUPPORTED
+ * @num_targ_data: number of entries in target data
+ */
+struct l3_flagmux_data {
+       u32 offset;
+       struct l3_target_data *l3_targ;
+       u8 num_targ_data;
+};
+
 
 /**
  * struct omap_l3 - Description of data relevant for L3 bus.
  * @dev:       device representing the bus (populated runtime)
  * @l3_base:   base addresses of modules (populated runtime)
- * @l3_flag_mux: array containing offsets to flag mux per module
+ * @l3_flag_mux: array containing flag mux data per module
  *              offset from corresponding module base indexed per
  *              module.
  * @num_modules: number of clock domains / modules.
  * @l3_masters:        array pointing to master data containing name and register
  *             offset for the master.
  * @num_master: number of masters
- * @l3_targ:   array indexed by flagmux index (bit offset) pointing to the
- *             target data. unsupported ones are marked with
- *             L3_TARGET_NOT_SUPPORTED
  * @debug_irq: irq number of the debug interrupt (populated runtime)
  * @app_irq:   irq number of the application interrupt (populated runtime)
  */
@@ -82,25 +91,17 @@ struct omap_l3 {
        struct device *dev;
 
        void __iomem *l3_base[MAX_L3_MODULES];
-       u32 *l3_flagmux;
+       struct l3_flagmux_data **l3_flagmux;
        int num_modules;
 
        struct l3_masters_data *l3_masters;
        int num_masters;
 
-       struct l3_target_data **l3_targ;
-
        int debug_irq;
        int app_irq;
 };
 
-static u32 omap_l3_flagmux[OMAP_L3_MODULES] = {
-       0x500,
-       0x1000,
-       0X0200
-};
-
-static struct l3_target_data omap_l3_target_data_clk1[MAX_CLKDM_TARGETS] = {
+static struct l3_target_data omap_l3_target_data_clk1[] = {
        {0x100, "DMM1",},
        {0x200, "DMM2",},
        {0x300, "ABE",},
@@ -110,7 +111,14 @@ static struct l3_target_data omap_l3_target_data_clk1[MAX_CLKDM_TARGETS] = {
        {0x900, "L4WAKEUP",},
 };
 
-static struct l3_target_data omap_l3_target_data_clk2[MAX_CLKDM_TARGETS] = {
+static struct l3_flagmux_data omap_l3_flagmux_clk1 = {
+       .offset = 0x500,
+       .l3_targ = omap_l3_target_data_clk1,
+       .num_targ_data = ARRAY_SIZE(omap_l3_target_data_clk1),
+};
+
+
+static struct l3_target_data omap_l3_target_data_clk2[] = {
        {0x500, "CORTEXM3",},
        {0x300, "DSS",},
        {0x100, "GPMC",},
@@ -134,12 +142,25 @@ static struct l3_target_data omap_l3_target_data_clk2[MAX_CLKDM_TARGETS] = {
        {0x1700, "LLI",},
 };
 
-static struct l3_target_data omap_l3_target_data_clk3[MAX_CLKDM_TARGETS] = {
+static struct l3_flagmux_data omap_l3_flagmux_clk2 = {
+       .offset = 0x1000,
+       .l3_targ = omap_l3_target_data_clk2,
+       .num_targ_data = ARRAY_SIZE(omap_l3_target_data_clk2),
+};
+
+
+static struct l3_target_data omap_l3_target_data_clk3[] = {
        {0x0100, "EMUSS",},
        {0x0300, "DEBUG SOURCE",},
        {0x0,   "HOST CLK3",},
 };
 
+static struct l3_flagmux_data omap_l3_flagmux_clk3 = {
+       .offset = 0x0200,
+       .l3_targ = omap_l3_target_data_clk3,
+       .num_targ_data = ARRAY_SIZE(omap_l3_target_data_clk3),
+};
+
 static struct l3_masters_data omap_l3_masters[] = {
        { 0x0 , "MPU"},
        { 0x10, "CS_ADP"},
@@ -168,18 +189,17 @@ static struct l3_masters_data omap_l3_masters[] = {
        { 0xC8, "USBHOSTFS"}
 };
 
-static struct l3_target_data *omap_l3_targ[OMAP_L3_MODULES] = {
-       omap_l3_target_data_clk1,
-       omap_l3_target_data_clk2,
-       omap_l3_target_data_clk3,
+static struct l3_flagmux_data *omap_l3_flagmux[] = {
+       &omap_l3_flagmux_clk1,
+       &omap_l3_flagmux_clk2,
+       &omap_l3_flagmux_clk3,
 };
 
 static const struct omap_l3 omap_l3_data = {
        .l3_flagmux = omap_l3_flagmux,
-       .num_modules = OMAP_L3_MODULES,
+       .num_modules = ARRAY_SIZE(omap_l3_flagmux),
        .l3_masters = omap_l3_masters,
        .num_masters = ARRAY_SIZE(omap_l3_masters),
-       .l3_targ = omap_l3_targ,
 };
 
 #endif /* __OMAP_L3_NOC_H */