Commit 90c49a09 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915/mst: Wait for ACT sent before enabling the pipe

The correct sequence according to bspec is to wait for the ACT sent
status before we turn on the pipe. Make it so.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200507144125.2458-1-ville.syrjala@linux.intel.comReviewed-by: default avatarImre Deak <imre.deak@intel.com>
parent a80d7367
...@@ -512,10 +512,6 @@ static void intel_mst_enable_dp(struct intel_atomic_state *state, ...@@ -512,10 +512,6 @@ static void intel_mst_enable_dp(struct intel_atomic_state *state,
intel_ddi_enable_transcoder_func(encoder, pipe_config); intel_ddi_enable_transcoder_func(encoder, pipe_config);
intel_enable_pipe(pipe_config);
intel_crtc_vblank_on(pipe_config);
drm_dbg_kms(&dev_priv->drm, "active links %d\n", drm_dbg_kms(&dev_priv->drm, "active links %d\n",
intel_dp->active_mst_links); intel_dp->active_mst_links);
...@@ -526,6 +522,11 @@ static void intel_mst_enable_dp(struct intel_atomic_state *state, ...@@ -526,6 +522,11 @@ static void intel_mst_enable_dp(struct intel_atomic_state *state,
drm_dp_check_act_status(&intel_dp->mst_mgr); drm_dp_check_act_status(&intel_dp->mst_mgr);
drm_dp_update_payload_part2(&intel_dp->mst_mgr); drm_dp_update_payload_part2(&intel_dp->mst_mgr);
intel_enable_pipe(pipe_config);
intel_crtc_vblank_on(pipe_config);
if (pipe_config->has_audio) if (pipe_config->has_audio)
intel_audio_codec_enable(encoder, pipe_config, conn_state); intel_audio_codec_enable(encoder, pipe_config, conn_state);
} }
......
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