OMAP2PLUS: DSS2: FEATURES: Fix usage of dss_reg_field and dss_clk_source_name

The structures dss_reg_field and dss_clk_source_name have enum members which
specify the register field and the clock source respectively. These members are
not used to choose the correct result in the corresponding feature functions.
Remove these members and change the features array declaration to incorporate
these enums.

The structure dss_clk_source_name without the enum member is just a pointer to
an string. Remove the structure and use a character pointer directly.

Signed-off-by: Archit Taneja <archit@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

diff --git a/drivers/video/omap2/dss/dss.c b/drivers/video/omap2/dss/dss.c
index aed9345e8adadd8e812f16886595bdbf5948c557..06ad3129085309ed4f51cd543ed3d7f12770f831 100644 (file)
--- a/drivers/video/omap2/dss/dss.c
+++ b/drivers/video/omap2/dss/dss.c
@@ -82,10 +82,10 @@ static struct {
        u32             ctx[DSS_SZ_REGS / sizeof(u32)];
 } dss;
 
-static const struct dss_clk_source_name dss_generic_clk_source_names[] = {
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC, "DSI_PLL_HSDIV_DISPC" },
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DSI, "DSI_PLL_HSDIV_DSI" },
-       { DSS_CLK_SRC_FCK, "DSS_FCK" },
+static const char * const dss_generic_clk_source_names[] = {
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC]       = "DSI_PLL_HSDIV_DISPC",
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DSI]         = "DSI_PLL_HSDIV_DSI",
+       [DSS_CLK_SRC_FCK]                       = "DSS_FCK",
 };
 
 static void dss_clk_enable_all_no_ctx(void);
@@ -232,7 +232,7 @@ void dss_sdi_disable(void)
 
 const char *dss_get_generic_clk_source_name(enum dss_clk_source clk_src)
 {
-       return dss_generic_clk_source_names[clk_src].clksrc_name;
+       return dss_generic_clk_source_names[clk_src];
 }
 
 void dss_dump_clocks(struct seq_file *s)

diff --git a/drivers/video/omap2/dss/dss.h b/drivers/video/omap2/dss/dss.h
index b845468d9706218ca4bfa119fab6a01c47b2b652..26a43c93e4893ba0e95df3504de3b51a95720dec 100644 (file)
--- a/drivers/video/omap2/dss/dss.h
+++ b/drivers/video/omap2/dss/dss.h
@@ -126,12 +126,6 @@ enum dss_clk_source {
                                                 * OMAP4: DSS_FCLK */
 };
 
-/* Correlates clock source name and dss_clk_source member */
-struct dss_clk_source_name {
-       enum dss_clk_source clksrc;
-       const char *clksrc_name;
-};
-
 struct dss_clock_info {
        /* rates that we get with dividers below */
        unsigned long fck;

diff --git a/drivers/video/omap2/dss/dss_features.c b/drivers/video/omap2/dss/dss_features.c
index 6eb6ec62a000202d54ab77bf9550f81e7b218876..8de3344842d7e3c8af0281154d56a88f4099f871 100644 (file)
--- a/drivers/video/omap2/dss/dss_features.c
+++ b/drivers/video/omap2/dss/dss_features.c
@@ -30,7 +30,6 @@
 
 /* Defines a generic omap register field */
 struct dss_reg_field {
-       enum dss_feat_reg_field id;
        u8 start, end;
 };
 
@@ -45,43 +44,43 @@ struct omap_dss_features {
        const unsigned long max_dss_fck;
        const enum omap_display_type *supported_displays;
        const enum omap_color_mode *supported_color_modes;
-       const struct dss_clk_source_name *clksrc_names;
+       const char * const *clksrc_names;
 };
 
 /* This struct is assigned to one of the below during initialization */
 static struct omap_dss_features *omap_current_dss_features;
 
 static const struct dss_reg_field omap2_dss_reg_fields[] = {
-       { FEAT_REG_FIRHINC, 11, 0 },
-       { FEAT_REG_FIRVINC, 27, 16 },
-       { FEAT_REG_FIFOLOWTHRESHOLD, 8, 0 },
-       { FEAT_REG_FIFOHIGHTHRESHOLD, 24, 16 },
-       { FEAT_REG_FIFOSIZE, 8, 0 },
-       { FEAT_REG_HORIZONTALACCU, 9, 0 },
-       { FEAT_REG_VERTICALACCU, 25, 16 },
-       { FEAT_REG_DISPC_CLK_SWITCH, 0, 0 },
+       [FEAT_REG_FIRHINC]              = { 11, 0 },
+       [FEAT_REG_FIRVINC]              = { 27, 16 },
+       [FEAT_REG_FIFOLOWTHRESHOLD]     = { 8, 0 },
+       [FEAT_REG_FIFOHIGHTHRESHOLD]    = { 24, 16 },
+       [FEAT_REG_FIFOSIZE]             = { 8, 0 },
+       [FEAT_REG_HORIZONTALACCU]       = { 9, 0 },
+       [FEAT_REG_VERTICALACCU]         = { 25, 16 },
+       [FEAT_REG_DISPC_CLK_SWITCH]     = { 0, 0 },
 };
 
 static const struct dss_reg_field omap3_dss_reg_fields[] = {
-       { FEAT_REG_FIRHINC, 12, 0 },
-       { FEAT_REG_FIRVINC, 28, 16 },
-       { FEAT_REG_FIFOLOWTHRESHOLD, 11, 0 },
-       { FEAT_REG_FIFOHIGHTHRESHOLD, 27, 16 },
-       { FEAT_REG_FIFOSIZE, 10, 0 },
-       { FEAT_REG_HORIZONTALACCU, 9, 0 },
-       { FEAT_REG_VERTICALACCU, 25, 16 },
-       { FEAT_REG_DISPC_CLK_SWITCH, 0, 0 },
+       [FEAT_REG_FIRHINC]              = { 12, 0 },
+       [FEAT_REG_FIRVINC]              = { 28, 16 },
+       [FEAT_REG_FIFOLOWTHRESHOLD]     = { 11, 0 },
+       [FEAT_REG_FIFOHIGHTHRESHOLD]    = { 27, 16 },
+       [FEAT_REG_FIFOSIZE]             = { 10, 0 },
+       [FEAT_REG_HORIZONTALACCU]       = { 9, 0 },
+       [FEAT_REG_VERTICALACCU]         = { 25, 16 },
+       [FEAT_REG_DISPC_CLK_SWITCH]     = { 0, 0 },
 };
 
 static const struct dss_reg_field omap4_dss_reg_fields[] = {
-       { FEAT_REG_FIRHINC, 12, 0 },
-       { FEAT_REG_FIRVINC, 28, 16 },
-       { FEAT_REG_FIFOLOWTHRESHOLD, 15, 0 },
-       { FEAT_REG_FIFOHIGHTHRESHOLD, 31, 16 },
-       { FEAT_REG_FIFOSIZE, 15, 0 },
-       { FEAT_REG_HORIZONTALACCU, 10, 0 },
-       { FEAT_REG_VERTICALACCU, 26, 16 },
-       { FEAT_REG_DISPC_CLK_SWITCH, 9, 8 },
+       [FEAT_REG_FIRHINC]              = { 12, 0 },
+       [FEAT_REG_FIRVINC]              = { 28, 16 },
+       [FEAT_REG_FIFOLOWTHRESHOLD]     = { 15, 0 },
+       [FEAT_REG_FIFOHIGHTHRESHOLD]    = { 31, 16 },
+       [FEAT_REG_FIFOSIZE]             = { 15, 0 },
+       [FEAT_REG_HORIZONTALACCU]       = { 10, 0 },
+       [FEAT_REG_VERTICALACCU]         = { 26, 16 },
+       [FEAT_REG_DISPC_CLK_SWITCH]     = { 9, 8 },
 };
 
 static const enum omap_display_type omap2_dss_supported_displays[] = {
@@ -162,22 +161,22 @@ static const enum omap_color_mode omap3_dss_supported_color_modes[] = {
        OMAP_DSS_COLOR_RGBA32 | OMAP_DSS_COLOR_RGBX32,
 };
 
-static const struct dss_clk_source_name omap2_dss_clk_source_names[] = {
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC, "N/A" },
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DSI, "N/A" },
-       { DSS_CLK_SRC_FCK, "DSS_FCLK1" },
+static const char * const omap2_dss_clk_source_names[] = {
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC]       = "N/A",
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DSI]         = "N/A",
+       [DSS_CLK_SRC_FCK]                       = "DSS_FCLK1",
 };
 
-static const struct dss_clk_source_name omap3_dss_clk_source_names[] = {
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC, "DSI1_PLL_FCLK" },
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DSI, "DSI2_PLL_FCLK" },
-       { DSS_CLK_SRC_FCK, "DSS1_ALWON_FCLK" },
+static const char * const omap3_dss_clk_source_names[] = {
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC]       = "DSI1_PLL_FCLK",
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DSI]         = "DSI2_PLL_FCLK",
+       [DSS_CLK_SRC_FCK]                       = "DSS1_ALWON_FCLK",
 };
 
-static const struct dss_clk_source_name omap4_dss_clk_source_names[] = {
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC, "PLL1_CLK1" },
-       { DSS_CLK_SRC_DSI_PLL_HSDIV_DSI, "PLL1_CLK2" },
-       { DSS_CLK_SRC_FCK, "DSS_FCLK" },
+static const char * const omap4_dss_clk_source_names[] = {
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DISPC]       = "PLL1_CLK1",
+       [DSS_CLK_SRC_DSI_PLL_HSDIV_DSI]         = "PLL1_CLK2",
+       [DSS_CLK_SRC_FCK]                       = "DSS_FCLK",
 };
 
 /* OMAP2 DSS Features */
@@ -290,7 +289,7 @@ bool dss_feat_color_mode_supported(enum omap_plane plane,
 
 const char *dss_feat_get_clk_source_name(enum dss_clk_source id)
 {
-       return omap_current_dss_features->clksrc_names[id].clksrc_name;
+       return omap_current_dss_features->clksrc_names[id];
 }
 
 /* DSS has_feature check */