drm/msm: dsi: Handle dual-channel for 6G as well
authorSean Paul <seanpaul@chromium.org>
Wed, 25 Jul 2018 20:34:45 +0000 (16:34 -0400)
committerRob Clark <robdclark@gmail.com>
Mon, 30 Jul 2018 12:50:12 +0000 (08:50 -0400)
This fixes up a collision between introducing dual-channel support and
the dsi refactors. This patch applies the same dual-channel
considerations and pclk calculations to both v2 and 6G, with a bit of
abstracting for good measure.

Signed-off-by: Sean Paul <seanpaul@chromium.org>
Signed-off-by: Rob Clark <robdclark@gmail.com>
drivers/gpu/drm/msm/dsi/dsi_host.c

index 319501d..96fb5f6 100644 (file)
@@ -664,11 +664,9 @@ void dsi_link_clk_disable_v2(struct msm_dsi_host *msm_host)
        clk_disable_unprepare(msm_host->byte_clk);
 }
 
-int dsi_calc_clk_rate_6g(struct msm_dsi_host *msm_host, bool is_dual_dsi)
+static u32 dsi_get_pclk_rate(struct msm_dsi_host *msm_host, bool is_dual_dsi)
 {
        struct drm_display_mode *mode = msm_host->mode;
-       u8 lanes = msm_host->lanes;
-       u32 bpp = dsi_get_bpp(msm_host->format);
        u32 pclk_rate;
 
        pclk_rate = mode->clock * 1000;
@@ -676,61 +674,61 @@ int dsi_calc_clk_rate_6g(struct msm_dsi_host *msm_host, bool is_dual_dsi)
        /*
         * For dual DSI mode, the current DRM mode has the complete width of the
         * panel. Since, the complete panel is driven by two DSI controllers,
-        * theclock rates have to be split between the two dsi controllers.
+        * the clock rates have to be split between the two dsi controllers.
         * Adjust the byte and pixel clock rates for each dsi host accordingly.
         */
        if (is_dual_dsi)
                pclk_rate /= 2;
 
-       if (lanes > 0) {
-               msm_host->byte_clk_rate = (pclk_rate * bpp) / (8 * lanes);
-       } else {
+       return pclk_rate;
+}
+
+static void dsi_calc_pclk(struct msm_dsi_host *msm_host, bool is_dual_dsi)
+{
+       u8 lanes = msm_host->lanes;
+       u32 bpp = dsi_get_bpp(msm_host->format);
+       u32 pclk_rate = dsi_get_pclk_rate(msm_host, is_dual_dsi);
+       u64 pclk_bpp = (u64)pclk_rate * bpp;
+
+       if (lanes == 0) {
                pr_err("%s: forcing mdss_dsi lanes to 1\n", __func__);
-               msm_host->byte_clk_rate = (pclk_rate * bpp) / 8;
+               lanes = 1;
        }
 
-       DBG("pclk=%d, bclk=%d", pclk_rate, msm_host->byte_clk_rate);
+       do_div(pclk_bpp, (8 * lanes));
 
-       msm_host->esc_clk_rate = clk_get_rate(msm_host->esc_clk);
+       msm_host->pixel_clk_rate = pclk_rate;
+       msm_host->byte_clk_rate = pclk_bpp;
+
+       DBG("pclk=%d, bclk=%d", msm_host->pixel_clk_rate,
+                               msm_host->byte_clk_rate);
+
+}
+
+int dsi_calc_clk_rate_6g(struct msm_dsi_host *msm_host, bool is_dual_dsi)
+{
+       if (!msm_host->mode) {
+               pr_err("%s: mode not set\n", __func__);
+               return -EINVAL;
+       }
 
+       dsi_calc_pclk(msm_host, is_dual_dsi);
+       msm_host->esc_clk_rate = clk_get_rate(msm_host->esc_clk);
        return 0;
 }
 
 int dsi_calc_clk_rate_v2(struct msm_dsi_host *msm_host, bool is_dual_dsi)
 {
-       struct drm_display_mode *mode = msm_host->mode;
-       u8 lanes = msm_host->lanes;
        u32 bpp = dsi_get_bpp(msm_host->format);
-       u32 pclk_rate;
        u64 pclk_bpp;
        unsigned int esc_mhz, esc_div;
        unsigned long byte_mhz;
 
-       pclk_rate = mode->clock * 1000;
-
-       /*
-        * For dual DSI mode, the current DRM mode has the complete width of the
-        * panel. Since, the complete panel is driven by two DSI controllers,
-        * theclock rates have to be split between the two dsi controllers.
-        * Adjust the byte and pixel clock rates for each dsi host accordingly.
-        */
-       if (is_dual_dsi)
-               pclk_rate /= 2;
-
-       pclk_bpp = pclk_rate * bpp;
-       if (lanes > 0) {
-               do_div(pclk_bpp, (8 * lanes));
-       } else {
-               pr_err("%s: forcing mdss_dsi lanes to 1\n", __func__);
-               do_div(pclk_bpp, 8);
-       }
-       msm_host->pixel_clk_rate = pclk_rate;
-       msm_host->byte_clk_rate = pclk_bpp;
-
-       DBG("pclk=%d, bclk=%d", msm_host->pixel_clk_rate,
-                               msm_host->byte_clk_rate);
+       dsi_calc_pclk(msm_host, is_dual_dsi);
 
-       msm_host->src_clk_rate = (pclk_rate * bpp) / 8;
+       pclk_bpp = (u64)dsi_get_pclk_rate(msm_host, is_dual_dsi) * bpp;
+       do_div(pclk_bpp, 8);
+       msm_host->src_clk_rate = pclk_bpp;
 
        /*
         * esc clock is byte clock followed by a 4 bit divider,