The DSI specific dss_param_range are moved from struct omap_dss_features to
corresponding struct dsi_param_range, which is initialized in struct dsi_feats
thereby enabling local access.

Signed-off-by: Chandrabhanu Mahapatra <cmahapa...@ti.com>
---
 drivers/video/omap2/dss/dsi.c          |   70 ++++++++++++++++++++++++++++----
 drivers/video/omap2/dss/dss_features.c |   27 ------------
 drivers/video/omap2/dss/dss_features.h |    7 ----
 3 files changed, 62 insertions(+), 42 deletions(-)

diff --git a/drivers/video/omap2/dss/dsi.c b/drivers/video/omap2/dss/dsi.c
index 2d387cb..f1f617c 100644
--- a/drivers/video/omap2/dss/dsi.c
+++ b/drivers/video/omap2/dss/dsi.c
@@ -351,8 +351,19 @@ enum dsi_feat_reg_field {
        FEAT_REG_DSIPLL_REGM_DSI,
 };
 
+enum dsi_range_param {
+       FEAT_PARAM_DSIPLL_REGN,
+       FEAT_PARAM_DSIPLL_REGM,
+       FEAT_PARAM_DSIPLL_REGM_DISPC,
+       FEAT_PARAM_DSIPLL_REGM_DSI,
+       FEAT_PARAM_DSIPLL_FINT,
+       FEAT_PARAM_DSIPLL_LPDIV,
+       FEAT_PARAM_DSI_FCK,
+};
+
 struct feats {
        const struct register_field *reg_fields;
+       const struct param_range *params;
 };
 
 static const struct feats *dsi_feat;
@@ -1518,7 +1529,7 @@ static void dsi_pll_calc_dsi_fck(struct platform_device 
*dsidev,
 {
        unsigned long max_dsi_fck;
 
-       max_dsi_fck = dss_feat_get_param_max(FEAT_PARAM_DSI_FCK);
+       max_dsi_fck = dsi_feat->params[FEAT_PARAM_DSI_FCK].max;
 
        cinfo->regm_dsi = DIV_ROUND_UP(cinfo->clkin4ddr, max_dsi_fck);
        cinfo->dsi_pll_hsdiv_dsi_clk = cinfo->clkin4ddr / cinfo->regm_dsi;
@@ -5082,14 +5093,14 @@ static void dsi_calc_clock_param_ranges(struct 
platform_device *dsidev)
 {
        struct dsi_data *dsi = dsi_get_dsidrv_data(dsidev);
 
-       dsi->regn_max = dss_feat_get_param_max(FEAT_PARAM_DSIPLL_REGN);
-       dsi->regm_max = dss_feat_get_param_max(FEAT_PARAM_DSIPLL_REGM);
+       dsi->regn_max = dsi_feat->params[FEAT_PARAM_DSIPLL_REGN].max;
+       dsi->regm_max = dsi_feat->params[FEAT_PARAM_DSIPLL_REGM].max;
        dsi->regm_dispc_max =
-               dss_feat_get_param_max(FEAT_PARAM_DSIPLL_REGM_DISPC);
-       dsi->regm_dsi_max = dss_feat_get_param_max(FEAT_PARAM_DSIPLL_REGM_DSI);
-       dsi->fint_min = dss_feat_get_param_min(FEAT_PARAM_DSIPLL_FINT);
-       dsi->fint_max = dss_feat_get_param_max(FEAT_PARAM_DSIPLL_FINT);
-       dsi->lpdiv_max = dss_feat_get_param_max(FEAT_PARAM_DSIPLL_LPDIV);
+               dsi_feat->params[FEAT_PARAM_DSIPLL_REGM_DISPC].max;
+       dsi->regm_dsi_max = dsi_feat->params[FEAT_PARAM_DSIPLL_REGM_DSI].max;
+       dsi->fint_min = dsi_feat->params[FEAT_PARAM_DSIPLL_FINT].min;
+       dsi->fint_max = dsi_feat->params[FEAT_PARAM_DSIPLL_FINT].max;
+       dsi->lpdiv_max = dsi_feat->params[FEAT_PARAM_DSIPLL_LPDIV].max;
 }
 
 static int dsi_get_clocks(struct platform_device *dsidev)
@@ -5242,20 +5253,63 @@ static struct register_field omap5_dsi_reg_fields[] = {
        [FEAT_REG_DSIPLL_REGM_DSI]              = { 30, 26 },
 };
 
+static struct param_range omap2_dsi_param_range[] = {
+       [FEAT_PARAM_DSIPLL_REGN]                = { 0, 0 },
+       [FEAT_PARAM_DSIPLL_REGM]                = { 0, 0 },
+       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, 0 },
+       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, 0 },
+       [FEAT_PARAM_DSIPLL_FINT]                = { 0, 0 },
+       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, 0 },
+};
+
+static struct param_range omap3_dsi_param_range[] = {
+       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 7) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 11) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 4) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 4) - 1 },
+       [FEAT_PARAM_DSIPLL_FINT]                = { 750000, 2100000 },
+       [FEAT_PARAM_DSIPLL_LPDIV]               = { 1, (1 << 13) - 1},
+       [FEAT_PARAM_DSI_FCK]                    = { 0, 173000000 },
+};
+
+static struct param_range omap4_dsi_param_range[] = {
+       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 8) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 12) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 5) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 5) - 1 },
+       [FEAT_PARAM_DSIPLL_FINT]                = { 500000, 2500000 },
+       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, (1 << 13) - 1 },
+       [FEAT_PARAM_DSI_FCK]                    = { 0, 170000000 },
+};
+
+static struct param_range omap5_dsi_param_range[] = {
+       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 8) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 12) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 5) - 1 },
+       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 5) - 1 },
+       [FEAT_PARAM_DSIPLL_FINT]                = { 500000, 2500000 },
+       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, (1 << 13) - 1 },
+       [FEAT_PARAM_DSI_FCK]                    = { 0, 170000000 },
+};
+
 static const struct feats omap24xx_dsi_feats __initconst = {
        .reg_fields             =       omap2_dsi_reg_fields,
+       .params                 =       omap2_dsi_param_range,
 };
 
 static const struct feats omap34xx_dsi_feats __initconst = {
        .reg_fields             =       omap3_dsi_reg_fields,
+       .params                 =       omap3_dsi_param_range,
 };
 
 static const struct feats omap44xx_dsi_feats __initconst = {
        .reg_fields             =       omap4_dsi_reg_fields,
+       .params                 =       omap4_dsi_param_range,
 };
 
 static const struct feats omap54xx_dsi_feats __initconst = {
        .reg_fields             =       omap5_dsi_reg_fields,
+       .params                 =       omap5_dsi_param_range,
 };
 
 static int __init dsi_init_features(struct platform_device *dsidev)
diff --git a/drivers/video/omap2/dss/dss_features.c 
b/drivers/video/omap2/dss/dss_features.c
index 8e6defb..75dddb2 100644
--- a/drivers/video/omap2/dss/dss_features.c
+++ b/drivers/video/omap2/dss/dss_features.c
@@ -369,45 +369,18 @@ static const char * const omap5_dss_clk_source_names[] = {
 
 static const struct dss_param_range omap2_dss_param_range[] = {
        [FEAT_PARAM_DSS_FCK]                    = { 0, 173000000 },
-       [FEAT_PARAM_DSIPLL_REGN]                = { 0, 0 },
-       [FEAT_PARAM_DSIPLL_REGM]                = { 0, 0 },
-       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, 0 },
-       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, 0 },
-       [FEAT_PARAM_DSIPLL_FINT]                = { 0, 0 },
-       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, 0 },
 };
 
 static const struct dss_param_range omap3_dss_param_range[] = {
        [FEAT_PARAM_DSS_FCK]                    = { 0, 173000000 },
-       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 7) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 11) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 4) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 4) - 1 },
-       [FEAT_PARAM_DSIPLL_FINT]                = { 750000, 2100000 },
-       [FEAT_PARAM_DSIPLL_LPDIV]               = { 1, (1 << 13) - 1},
-       [FEAT_PARAM_DSI_FCK]                    = { 0, 173000000 },
 };
 
 static const struct dss_param_range omap4_dss_param_range[] = {
        [FEAT_PARAM_DSS_FCK]                    = { 0, 186000000 },
-       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 8) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 12) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 5) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 5) - 1 },
-       [FEAT_PARAM_DSIPLL_FINT]                = { 500000, 2500000 },
-       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, (1 << 13) - 1 },
-       [FEAT_PARAM_DSI_FCK]                    = { 0, 170000000 },
 };
 
 static const struct dss_param_range omap5_dss_param_range[] = {
        [FEAT_PARAM_DSS_FCK]                    = { 0, 200000000 },
-       [FEAT_PARAM_DSIPLL_REGN]                = { 0, (1 << 8) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM]                = { 0, (1 << 12) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DISPC]          = { 0, (1 << 5) - 1 },
-       [FEAT_PARAM_DSIPLL_REGM_DSI]            = { 0, (1 << 5) - 1 },
-       [FEAT_PARAM_DSIPLL_FINT]                = { 500000, 2500000 },
-       [FEAT_PARAM_DSIPLL_LPDIV]               = { 0, (1 << 13) - 1 },
-       [FEAT_PARAM_DSI_FCK]                    = { 0, 170000000 },
 };
 
 static const enum dss_feat_id omap2_dss_feat_list[] = {
diff --git a/drivers/video/omap2/dss/dss_features.h 
b/drivers/video/omap2/dss/dss_features.h
index 3e82404..d9f69c7 100644
--- a/drivers/video/omap2/dss/dss_features.h
+++ b/drivers/video/omap2/dss/dss_features.h
@@ -77,13 +77,6 @@ enum dss_feat_reg_field {
 
 enum dss_range_param {
        FEAT_PARAM_DSS_FCK,
-       FEAT_PARAM_DSIPLL_REGN,
-       FEAT_PARAM_DSIPLL_REGM,
-       FEAT_PARAM_DSIPLL_REGM_DISPC,
-       FEAT_PARAM_DSIPLL_REGM_DSI,
-       FEAT_PARAM_DSIPLL_FINT,
-       FEAT_PARAM_DSIPLL_LPDIV,
-       FEAT_PARAM_DSI_FCK,
 };
 
 /* DSS Feature Functions */
-- 
1.7.10

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to