drm/msm/dp: store mode YUV420 information to be used by rest of DP

Wide bus is not supported when the mode is YUV420 in DP. In preparation
for changing the DPU programming to reflect this, the value and
assignment location of wide_bus_en for the DP submodules must be
changed. Move it from boot time in dp_init_sub_modules() to run time in
dp_display_mode_set.

Signed-off-by: Paloma Arellano <quic_parellan@quicinc.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Patchwork: https://patchwork.freedesktop.org/patch/579614/
Link: https://lore.kernel.org/r/20240222194025.25329-8-quic_parellan@quicinc.com
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
index 873fa228a967d55037348116ae758160a48a03b1..4877050bd8b56f48bea88ea261671e303ac12732 100644 (file)
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c
@@ -766,10 +766,6 @@ static int dp_init_sub_modules(struct dp_display_private *dp)
                goto error_ctrl;
        }
 
-       /* populate wide_bus_supported to different layers */
-       dp->ctrl->wide_bus_en = dp->wide_bus_supported;
-       dp->catalog->wide_bus_en = dp->wide_bus_supported;
-
        return rc;
 
 error_ctrl:
@@ -790,6 +786,7 @@ static int dp_display_set_mode(struct msm_dp *dp_display,
        drm_mode_copy(&dp->panel->dp_mode.drm_mode, &mode->drm_mode);
        dp->panel->dp_mode.bpp = mode->bpp;
        dp->panel->dp_mode.capabilities = mode->capabilities;
+       dp->panel->dp_mode.out_fmt_is_yuv_420 = mode->out_fmt_is_yuv_420;
        dp_panel_init_panel_info(dp->panel);
        return 0;
 }
@@ -1381,6 +1378,9 @@ bool msm_dp_wide_bus_available(const struct msm_dp *dp_display)
 
        dp = container_of(dp_display, struct dp_display_private, dp_display);
 
+       if (dp->dp_mode.out_fmt_is_yuv_420)
+               return false;
+
        return dp->wide_bus_supported;
 }
 
@@ -1570,6 +1570,15 @@ void dp_bridge_mode_set(struct drm_bridge *drm_bridge,
 
        dp_display->dp_mode.h_active_low =
                !!(dp_display->dp_mode.drm_mode.flags & DRM_MODE_FLAG_NHSYNC);
+
+       dp_display->dp_mode.out_fmt_is_yuv_420 =
+               drm_mode_is_420_only(&dp->connector->display_info, adjusted_mode);
+
+       /* populate wide_bus_support to different layers */
+       dp_display->ctrl->wide_bus_en =
+               dp_display->dp_mode.out_fmt_is_yuv_420 ? false : dp_display->wide_bus_supported;
+       dp_display->catalog->wide_bus_en =
+               dp_display->dp_mode.out_fmt_is_yuv_420 ? false : dp_display->wide_bus_supported;
 }
 
 void dp_bridge_hpd_enable(struct drm_bridge *bridge)

diff --git a/drivers/gpu/drm/msm/dp/dp_panel.h b/drivers/gpu/drm/msm/dp/dp_panel.h
index a0dfc579c5f9f4475c0553c16cacef760a4f5571..6ec68be9f2366ba168a6fcfd49fb2cbfc5f82fec 100644 (file)
--- a/drivers/gpu/drm/msm/dp/dp_panel.h
+++ b/drivers/gpu/drm/msm/dp/dp_panel.h
@@ -19,6 +19,7 @@ struct dp_display_mode {
        u32 bpp;
        u32 h_active_low;
        u32 v_active_low;
+       bool out_fmt_is_yuv_420;
 };
 
 struct dp_panel_in {