drm: omapdrm: dss: Pass DSS pointer to dss clock functions

This removes the need to access the global DSS private data in those
functions (both for the current accesses and the future ones that will
be introduced when allocating the DSS device dynamically).

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>

diff --git a/drivers/gpu/drm/omapdrm/dss/dispc.c b/drivers/gpu/drm/omapdrm/dss/dispc.c
index 048b2e4d1f40110c8a9501cc0708d195aac336aa..8d0de1b790b7dfbb674a27269b2878c6da8bc306 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/dispc.c
+++ b/drivers/gpu/drm/omapdrm/dss/dispc.c
@@ -3116,7 +3116,7 @@ static unsigned long dispc_fclk_rate(void)
        src = dss_get_dispc_clk_source(dispc.dss);
 
        if (src == DSS_CLK_SRC_FCK) {
-               r = dss_get_dispc_clk_rate();
+               r = dss_get_dispc_clk_rate(dispc.dss);
        } else {
                struct dss_pll *pll;
                unsigned int clkout_idx;
@@ -3143,7 +3143,7 @@ static unsigned long dispc_mgr_lclk_rate(enum omap_channel channel)
        src = dss_get_lcd_clk_source(dispc.dss, channel);
 
        if (src == DSS_CLK_SRC_FCK) {
-               r = dss_get_dispc_clk_rate();
+               r = dss_get_dispc_clk_rate(dispc.dss);
        } else {
                struct dss_pll *pll;
                unsigned int clkout_idx;
@@ -3499,7 +3499,7 @@ bool dispc_div_calc(unsigned long dispc_freq,
        pckd_hw_min = dispc.feat->min_pcd;
        pckd_hw_max = 255;
 
-       lck_max = dss_get_max_fck_rate();
+       lck_max = dss_get_max_fck_rate(dispc.dss);
 
        pck_min = pck_min ? pck_min : 1;
        pck_max = pck_max ? pck_max : ULONG_MAX;
@@ -4460,7 +4460,7 @@ static void dispc_errata_i734_wa(void)
 
        /* Set up and enable display manager for LCD1 */
        dispc_mgr_setup(OMAP_DSS_CHANNEL_LCD, &i734.mgri);
-       dispc_calc_clock_rates(dss_get_dispc_clk_rate(),
+       dispc_calc_clock_rates(dss_get_dispc_clk_rate(dispc.dss),
                               &lcd_conf.clock_info);
        dispc_mgr_set_lcd_config(OMAP_DSS_CHANNEL_LCD, &lcd_conf);
        dispc_mgr_set_timings(OMAP_DSS_CHANNEL_LCD, &i734.vm);

diff --git a/drivers/gpu/drm/omapdrm/dss/dpi.c b/drivers/gpu/drm/omapdrm/dss/dpi.c
index e7f50fabca6feab146970e2ec959852fd0e623af..ba5adfb7ee706c9b41b6246ef37d0b835f3088c5 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/dpi.c
+++ b/drivers/gpu/drm/omapdrm/dss/dpi.c
@@ -207,7 +207,7 @@ static bool dpi_calc_pll_cb(int n, int m, unsigned long fint,
        ctx->pll_cinfo.clkdco = clkdco;
 
        return dss_pll_hsdiv_calc_a(ctx->pll, clkdco,
-               ctx->pck_min, dss_get_max_fck_rate(),
+               ctx->pck_min, dss_get_max_fck_rate(ctx->pll->dss),
                dpi_calc_hsdiv_cb, ctx);
 }
 
@@ -256,7 +256,8 @@ static bool dpi_pll_clk_calc(struct dpi_data *dpi, unsigned long pck,
        }
 }
 
-static bool dpi_dss_clk_calc(unsigned long pck, struct dpi_clk_calc_ctx *ctx)
+static bool dpi_dss_clk_calc(struct dpi_data *dpi, unsigned long pck,
+                            struct dpi_clk_calc_ctx *ctx)
 {
        int i;
 
@@ -277,7 +278,8 @@ static bool dpi_dss_clk_calc(unsigned long pck, struct dpi_clk_calc_ctx *ctx)
                        ctx->pck_min = 0;
                ctx->pck_max = pck + 1000 * i * i * i;
 
-               ok = dss_div_calc(pck, ctx->pck_min, dpi_calc_dss_cb, ctx);
+               ok = dss_div_calc(dpi->dss, pck, ctx->pck_min,
+                                 dpi_calc_dss_cb, ctx);
                if (ok)
                        return ok;
        }
@@ -321,11 +323,11 @@ static int dpi_set_dispc_clk(struct dpi_data *dpi, unsigned long pck_req,
        int r;
        bool ok;
 
-       ok = dpi_dss_clk_calc(pck_req, &ctx);
+       ok = dpi_dss_clk_calc(dpi, pck_req, &ctx);
        if (!ok)
                return -EINVAL;
 
-       r = dss_set_fck_rate(ctx.fck);
+       r = dss_set_fck_rate(dpi->dss, ctx.fck);
        if (r)
                return r;
 
@@ -530,7 +532,7 @@ static int dpi_check_timings(struct omap_dss_device *dssdev,
 
                fck = ctx.pll_cinfo.clkout[ctx.clkout_idx];
        } else {
-               ok = dpi_dss_clk_calc(vm->pixelclock, &ctx);
+               ok = dpi_dss_clk_calc(dpi, vm->pixelclock, &ctx);
                if (!ok)
                        return -EINVAL;
 

diff --git a/drivers/gpu/drm/omapdrm/dss/dss.c b/drivers/gpu/drm/omapdrm/dss/dss.c
index bdf8f66002b682ca40bc79a838bc41b059a69be2..0d292da6757db39b1312f808ab1fb934a46e30d7 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/dss.c
+++ b/drivers/gpu/drm/omapdrm/dss/dss.c
@@ -594,8 +594,8 @@ enum dss_clk_source dss_get_lcd_clk_source(struct dss_device *dss,
        }
 }
 
-bool dss_div_calc(unsigned long pck, unsigned long fck_min,
-               dss_div_calc_func func, void *data)
+bool dss_div_calc(struct dss_device *dss, unsigned long pck,
+                 unsigned long fck_min, dss_div_calc_func func, void *data)
 {
        int fckd, fckd_start, fckd_stop;
        unsigned long fck;
@@ -604,24 +604,24 @@ bool dss_div_calc(unsigned long pck, unsigned long fck_min,
        unsigned long prate;
        unsigned int m;
 
-       fck_hw_max = dss.feat->fck_freq_max;
+       fck_hw_max = dss->feat->fck_freq_max;
 
-       if (dss.parent_clk == NULL) {
+       if (dss->parent_clk == NULL) {
                unsigned int pckd;
 
                pckd = fck_hw_max / pck;
 
                fck = pck * pckd;
 
-               fck = clk_round_rate(dss.dss_clk, fck);
+               fck = clk_round_rate(dss->dss_clk, fck);
 
                return func(fck, data);
        }
 
-       fckd_hw_max = dss.feat->fck_div_max;
+       fckd_hw_max = dss->feat->fck_div_max;
 
-       m = dss.feat->dss_fck_multiplier;
-       prate = clk_get_rate(dss.parent_clk);
+       m = dss->feat->dss_fck_multiplier;
+       prate = clk_get_rate(dss->parent_clk);
 
        fck_min = fck_min ? fck_min : 1;
 
@@ -638,33 +638,32 @@ bool dss_div_calc(unsigned long pck, unsigned long fck_min,
        return false;
 }
 
-int dss_set_fck_rate(unsigned long rate)
+int dss_set_fck_rate(struct dss_device *dss, unsigned long rate)
 {
        int r;
 
        DSSDBG("set fck to %lu\n", rate);
 
-       r = clk_set_rate(dss.dss_clk, rate);
+       r = clk_set_rate(dss->dss_clk, rate);
        if (r)
                return r;
 
-       dss.dss_clk_rate = clk_get_rate(dss.dss_clk);
+       dss->dss_clk_rate = clk_get_rate(dss->dss_clk);
 
-       WARN_ONCE(dss.dss_clk_rate != rate,
-                       "clk rate mismatch: %lu != %lu", dss.dss_clk_rate,
-                       rate);
+       WARN_ONCE(dss->dss_clk_rate != rate, "clk rate mismatch: %lu != %lu",
+                 dss->dss_clk_rate, rate);
 
        return 0;
 }
 
-unsigned long dss_get_dispc_clk_rate(void)
+unsigned long dss_get_dispc_clk_rate(struct dss_device *dss)
 {
-       return dss.dss_clk_rate;
+       return dss->dss_clk_rate;
 }
 
-unsigned long dss_get_max_fck_rate(void)
+unsigned long dss_get_max_fck_rate(struct dss_device *dss)
 {
-       return dss.feat->fck_freq_max;
+       return dss->feat->fck_freq_max;
 }
 
 enum omap_dss_output_id dss_get_supported_outputs(enum omap_channel channel)
@@ -691,7 +690,7 @@ static int dss_setup_default_clock(void)
                fck = DIV_ROUND_UP(prate, fck_div) * dss.feat->dss_fck_multiplier;
        }
 
-       r = dss_set_fck_rate(fck);
+       r = dss_set_fck_rate(&dss, fck);
        if (r)
                return r;
 

diff --git a/drivers/gpu/drm/omapdrm/dss/dss.h b/drivers/gpu/drm/omapdrm/dss/dss.h
index fc70c3c5ef11ff294f62f2a1b64c261f347854f8..1d0edf2d145e5bb060808625d74d02044d1aca00 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/dss.h
+++ b/drivers/gpu/drm/omapdrm/dss/dss.h
@@ -297,8 +297,8 @@ struct dss_device *dss_get_device(struct device *dev);
 int dss_runtime_get(struct dss_device *dss);
 void dss_runtime_put(struct dss_device *dss);
 
-unsigned long dss_get_dispc_clk_rate(void);
-unsigned long dss_get_max_fck_rate(void);
+unsigned long dss_get_dispc_clk_rate(struct dss_device *dss);
+unsigned long dss_get_max_fck_rate(struct dss_device *dss);
 enum omap_dss_output_id dss_get_supported_outputs(enum omap_channel channel);
 int dss_dpi_select_source(struct dss_device *dss, int port,
                          enum omap_channel channel);
@@ -332,11 +332,11 @@ enum dss_clk_source dss_get_lcd_clk_source(struct dss_device *dss,
 void dss_set_venc_output(enum omap_dss_venc_type type);
 void dss_set_dac_pwrdn_bgz(bool enable);
 
-int dss_set_fck_rate(unsigned long rate);
+int dss_set_fck_rate(struct dss_device *dss, unsigned long rate);
 
 typedef bool (*dss_div_calc_func)(unsigned long fck, void *data);
-bool dss_div_calc(unsigned long pck, unsigned long fck_min,
-               dss_div_calc_func func, void *data);
+bool dss_div_calc(struct dss_device *dss, unsigned long pck,
+                 unsigned long fck_min, dss_div_calc_func func, void *data);
 
 /* SDI */
 #ifdef CONFIG_OMAP2_DSS_SDI

diff --git a/drivers/gpu/drm/omapdrm/dss/sdi.c b/drivers/gpu/drm/omapdrm/dss/sdi.c
index f0564daa3831ec13dcccda197e5e1b530b455ce2..6f39e0ff305527ba5fbfd6251337aacc371bdf76 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/sdi.c
+++ b/drivers/gpu/drm/omapdrm/dss/sdi.c
@@ -99,7 +99,8 @@ static int sdi_calc_clock_div(unsigned long pclk,
                        ctx.pck_min = 0;
                ctx.pck_max = pclk + 1000 * i * i * i;
 
-               ok = dss_div_calc(pclk, ctx.pck_min, dpi_calc_dss_cb, &ctx);
+               ok = dss_div_calc(sdi.dss, pclk, ctx.pck_min,
+                                 dpi_calc_dss_cb, &ctx);
                if (ok) {
                        *fck = ctx.fck;
                        *dispc_cinfo = ctx.dispc_cinfo;
@@ -169,7 +170,7 @@ static int sdi_display_enable(struct omap_dss_device *dssdev)
 
        dss_mgr_set_timings(channel, vm);
 
-       r = dss_set_fck_rate(fck);
+       r = dss_set_fck_rate(sdi.dss, fck);
        if (r)
                goto err_set_dss_clock_div;