drm/tilcdc: rename req_rate to pclk_rate
authorDario Binacchi <dariobin@libero.it>
Mon, 22 Mar 2021 21:33:35 +0000 (22:33 +0100)
committerJyri Sarha <jyri.sarha@iki.fi>
Wed, 24 Mar 2021 10:05:28 +0000 (12:05 +0200)
The req_rate name is a little misleading, so let's rename to pclk_rate
(pixel clock rate).

Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Jyri Sarha <jyri.sarha@iki.fi>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Signed-off-by: Jyri Sarha <jyri.sarha@iki.fi>
Link: https://patchwork.freedesktop.org/patch/msgid/20210322213337.26667-2-dariobin@libero.it
drivers/gpu/drm/tilcdc/tilcdc_crtc.c

index 21e7b3d..3d0b7fb 100644 (file)
@@ -203,18 +203,18 @@ static void tilcdc_crtc_set_clk(struct drm_crtc *crtc)
        struct drm_device *dev = crtc->dev;
        struct tilcdc_drm_private *priv = dev->dev_private;
        struct tilcdc_crtc *tilcdc_crtc = to_tilcdc_crtc(crtc);
-       unsigned long clk_rate, real_rate, req_rate;
+       unsigned long clk_rate, real_rate, pclk_rate;
        unsigned int clkdiv;
        int ret;
 
        clkdiv = 2; /* first try using a standard divider of 2 */
 
        /* mode.clock is in KHz, set_rate wants parameter in Hz */
-       req_rate = crtc->mode.clock * 1000;
+       pclk_rate = crtc->mode.clock * 1000;
 
-       ret = clk_set_rate(priv->clk, req_rate * clkdiv);
+       ret = clk_set_rate(priv->clk, pclk_rate * clkdiv);
        clk_rate = clk_get_rate(priv->clk);
-       if (ret < 0 || tilcdc_pclk_diff(req_rate, clk_rate) > 5) {
+       if (ret < 0 || tilcdc_pclk_diff(pclk_rate, clk_rate) > 5) {
                /*
                 * If we fail to set the clock rate (some architectures don't
                 * use the common clock framework yet and may not implement
@@ -229,7 +229,7 @@ static void tilcdc_crtc_set_clk(struct drm_crtc *crtc)
                        return;
                }
 
-               clkdiv = DIV_ROUND_CLOSEST(clk_rate, req_rate);
+               clkdiv = DIV_ROUND_CLOSEST(clk_rate, pclk_rate);
 
                /*
                 * Emit a warning if the real clock rate resulting from the
@@ -238,7 +238,7 @@ static void tilcdc_crtc_set_clk(struct drm_crtc *crtc)
                 * 5% is an arbitrary value - LCDs are usually quite tolerant
                 * about pixel clock rates.
                 */
-               real_rate = clkdiv * req_rate;
+               real_rate = clkdiv * pclk_rate;
 
                if (tilcdc_pclk_diff(clk_rate, real_rate) > 5) {
                        dev_warn(dev->dev,