msm_dp_ctrl_configure_source_params() should only handle stream-related
configuration. Move the link setup out of it so MST can program link and
stream settings separately.

Signed-off-by: Yongxing Mou <[email protected]>
Reviewed-by: Dmitry Baryshkov <[email protected]>
---
 drivers/gpu/drm/msm/dp/dp_ctrl.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_ctrl.c b/drivers/gpu/drm/msm/dp/dp_ctrl.c
index 71d45b2c4daf..1c2eccec6ec6 100644
--- a/drivers/gpu/drm/msm/dp/dp_ctrl.c
+++ b/drivers/gpu/drm/msm/dp/dp_ctrl.c
@@ -482,10 +482,6 @@ static void msm_dp_ctrl_config_misc1_misc0(struct 
msm_dp_ctrl_private *ctrl,
 
 static void msm_dp_ctrl_configure_source_params(struct msm_dp_ctrl_private 
*ctrl)
 {
-       msm_dp_ctrl_lane_mapping(ctrl);
-       msm_dp_setup_peripheral_flush(ctrl);
-
-       msm_dp_ctrl_config_ctrl_link(ctrl);
        msm_dp_ctrl_config_ctrl_streams(ctrl, ctrl->panel);
 
        msm_dp_ctrl_config_misc1_misc0(ctrl, ctrl->panel);
@@ -2551,6 +2547,10 @@ int msm_dp_ctrl_on_stream(struct msm_dp_ctrl 
*msm_dp_ctrl, bool force_link_train
         */
        reinit_completion(&ctrl->video_comp);
 
+       msm_dp_ctrl_lane_mapping(ctrl);
+       msm_dp_setup_peripheral_flush(ctrl);
+       msm_dp_ctrl_config_ctrl_link(ctrl);
+
        msm_dp_ctrl_configure_source_params(ctrl);
 
        msm_dp_ctrl_config_msa(ctrl,

-- 
2.43.0

Reply via email to