We need to program both port registers during dual link disable path.

v2: Address review comments by Jani
    - Used a for loop instead of do-while loop.

Signed-off-by: Gaurav K Singh <gaurav.k.si...@intel.com>
Signed-off-by: Shobhit Kumar <shobhit.ku...@intel.com>
---
 drivers/gpu/drm/i915/intel_dsi.c |   68 +++++++++++++++++++++++---------------
 1 file changed, 41 insertions(+), 27 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c
index e00dcd8..512ccbc 100644
--- a/drivers/gpu/drm/i915/intel_dsi.c
+++ b/drivers/gpu/drm/i915/intel_dsi.c
@@ -303,8 +303,8 @@ static void intel_dsi_disable(struct intel_encoder *encoder)
        struct intel_crtc *intel_crtc = to_intel_crtc(encoder->base.crtc);
        struct intel_dsi *intel_dsi = enc_to_intel_dsi(&encoder->base);
        int port = PIPE_MAPPED_TO_MIPI_PORT(intel_crtc->pipe);
-
        u32 temp;
+       int count;
 
        DRM_DEBUG_KMS("\n");
 
@@ -315,23 +315,27 @@ static void intel_dsi_disable(struct intel_encoder 
*encoder)
                msleep(2);
        }
 
-       /* Panel commands can be sent when clock is in LP11 */
-       I915_WRITE(MIPI_DEVICE_READY(port), 0x0);
+       for (count = 0; count < (intel_dsi->dual_link ? 2 : 1); count++) {
+               /* Panel commands can be sent when clock is in LP11 */
+               I915_WRITE(MIPI_DEVICE_READY(port), 0x0);
 
-       temp = I915_READ(MIPI_CTRL(port));
-       temp &= ~ESCAPE_CLOCK_DIVIDER_MASK;
-       I915_WRITE(MIPI_CTRL(port), temp |
-                  intel_dsi->escape_clk_div <<
-                  ESCAPE_CLOCK_DIVIDER_SHIFT);
+               temp = I915_READ(MIPI_CTRL(port));
+               temp &= ~ESCAPE_CLOCK_DIVIDER_MASK;
+               I915_WRITE(MIPI_CTRL(port), temp |
+                          intel_dsi->escape_clk_div <<
+                          ESCAPE_CLOCK_DIVIDER_SHIFT);
 
-       I915_WRITE(MIPI_EOT_DISABLE(port), CLOCKSTOP);
+               I915_WRITE(MIPI_EOT_DISABLE(port), CLOCKSTOP);
 
-       temp = I915_READ(MIPI_DSI_FUNC_PRG(port));
-       temp &= ~VID_MODE_FORMAT_MASK;
-       I915_WRITE(MIPI_DSI_FUNC_PRG(port), temp);
+               temp = I915_READ(MIPI_DSI_FUNC_PRG(port));
+               temp &= ~VID_MODE_FORMAT_MASK;
+               I915_WRITE(MIPI_DSI_FUNC_PRG(port), temp);
 
-       I915_WRITE(MIPI_DEVICE_READY(port), 0x1);
+               I915_WRITE(MIPI_DEVICE_READY(port), 0x1);
 
+               if (intel_dsi->dual_link)
+                       port = PORT_B;
+       }
        /* if disable packets are sent before sending shutdown packet then in
         * some next enable sequence send turn on packet error is observed */
        if (intel_dsi->dev.dev_ops->disable)
@@ -344,30 +348,40 @@ static void intel_dsi_clear_device_ready(struct 
intel_encoder *encoder)
 {
        struct drm_i915_private *dev_priv = encoder->base.dev->dev_private;
        struct intel_crtc *intel_crtc = to_intel_crtc(encoder->base.crtc);
+       struct intel_dsi *intel_dsi = enc_to_intel_dsi(&encoder->base);
        int port = PIPE_MAPPED_TO_MIPI_PORT(intel_crtc->pipe);
        u32 val;
+       int count;
 
        DRM_DEBUG_KMS("\n");
 
-       I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY | ULPS_STATE_ENTER);
-       usleep_range(2000, 2500);
+       for (count = 0; count < (intel_dsi->dual_link ? 2 : 1); count++) {
+               I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY |
+                                                       ULPS_STATE_ENTER);
+               usleep_range(2000, 2500);
 
-       I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY | ULPS_STATE_EXIT);
-       usleep_range(2000, 2500);
+               I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY |
+                                                       ULPS_STATE_EXIT);
+               usleep_range(2000, 2500);
 
-       I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY | ULPS_STATE_ENTER);
-       usleep_range(2000, 2500);
+               I915_WRITE(MIPI_DEVICE_READY(port), DEVICE_READY |
+                                                       ULPS_STATE_ENTER);
+               usleep_range(2000, 2500);
 
-       if (wait_for(((I915_READ(MIPI_PORT_CTRL(port)) & AFE_LATCHOUT)
-                     == 0x00000), 30))
-               DRM_ERROR("DSI LP not going Low\n");
+               if (wait_for(((I915_READ(MIPI_PORT_CTRL(port)) & AFE_LATCHOUT)
+                             == 0x00000), 30))
+                       DRM_ERROR("DSI LP not going Low\n");
 
-       val = I915_READ(MIPI_PORT_CTRL(port));
-       I915_WRITE(MIPI_PORT_CTRL(port), val & ~LP_OUTPUT_HOLD);
-       usleep_range(1000, 1500);
+               val = I915_READ(MIPI_PORT_CTRL(port));
+               I915_WRITE(MIPI_PORT_CTRL(port), val & ~LP_OUTPUT_HOLD);
+               usleep_range(1000, 1500);
+
+               I915_WRITE(MIPI_DEVICE_READY(port), 0x00);
+               usleep_range(2000, 2500);
 
-       I915_WRITE(MIPI_DEVICE_READY(port), 0x00);
-       usleep_range(2000, 2500);
+               if (intel_dsi->dual_link)
+                       port = PORT_B;
+       }
 
        vlv_disable_dsi_pll(encoder);
 }
-- 
1.7.9.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to