Commit 4690985e authored by Vandita Kulkarni's avatar Vandita Kulkarni Committed by Rodrigo Vivi

drm/i915/icl: Fix port disable sequence for mipi-dsi

Re-enable clock gating of DDI clocks.

v2: Fix the default ddi clk state for mipi-dsi (Imre)

Fixes: 1026bea0 ("drm/i915/icl: Ungate DSI clocks")
Signed-off-by: default avatarVandita Kulkarni <vandita.kulkarni@intel.com>
Reviewed-by: default avatarUma Shankar <uma.shankar@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/1553513202-13863-2-git-send-email-vandita.kulkarni@intel.com
(cherry picked from commit 942d1cf4)
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 20eea462
...@@ -1132,7 +1132,7 @@ static void gen11_dsi_disable_port(struct intel_encoder *encoder) ...@@ -1132,7 +1132,7 @@ static void gen11_dsi_disable_port(struct intel_encoder *encoder)
DRM_ERROR("DDI port:%c buffer not idle\n", DRM_ERROR("DDI port:%c buffer not idle\n",
port_name(port)); port_name(port));
} }
gen11_dsi_ungate_clocks(encoder); gen11_dsi_gate_clocks(encoder);
} }
static void gen11_dsi_disable_io_power(struct intel_encoder *encoder) static void gen11_dsi_disable_io_power(struct intel_encoder *encoder)
......
...@@ -2824,10 +2824,10 @@ void icl_sanitize_encoder_pll_mapping(struct intel_encoder *encoder) ...@@ -2824,10 +2824,10 @@ void icl_sanitize_encoder_pll_mapping(struct intel_encoder *encoder)
return; return;
} }
/* /*
* DSI ports should have their DDI clock ungated when disabled * For DSI we keep the ddi clocks gated
* and gated when enabled. * except during enable/disable sequence.
*/ */
ddi_clk_needed = !encoder->base.crtc; ddi_clk_needed = false;
} }
val = I915_READ(DPCLKA_CFGCR0_ICL); val = I915_READ(DPCLKA_CFGCR0_ICL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment