[PATCH 20/24] drm/msm/dp: use drm_display_info in mode_valid callbacks
From: Dmitry Baryshkov
Date: Wed Jul 22 2026 - 02:49:59 EST
The .mode_valid bridge callback is passed the sink's drm_display_info
directly, so msm_dp_bridge_mode_valid() and msm_edp_bridge_mode_valid()
have no need to reach into the stored connector for it. Use the supplied
info pointer and drop the now-pointless !dp->connector guard.
Assisted-by: Claude:claude-opus-4-8
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
---
drivers/gpu/drm/msm/dp/dp_display.c | 6 +++---
drivers/gpu/drm/msm/dp/dp_drm.c | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
index 461420a72820..3397a3a47ef3 100644
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c
@@ -731,7 +731,7 @@ enum drm_mode_status msm_dp_bridge_mode_valid(struct drm_bridge *bridge,
dp = to_dp_bridge(bridge)->msm_dp_display;
- if (!dp || !mode_pclk_khz || !dp->connector) {
+ if (!dp || !mode_pclk_khz) {
DRM_ERROR("invalid params\n");
return -EINVAL;
}
@@ -739,7 +739,7 @@ enum drm_mode_status msm_dp_bridge_mode_valid(struct drm_bridge *bridge,
msm_dp_display = container_of(dp, struct msm_dp_display_private, msm_dp_display);
link_info = &msm_dp_display->panel->link_info;
- is_yuv_420 = drm_mode_is_420_only(&dp->connector->display_info, mode);
+ is_yuv_420 = drm_mode_is_420_only(info, mode);
/*
* YUV 420 is carried over DP by signalling the colorimetry through a
@@ -754,7 +754,7 @@ enum drm_mode_status msm_dp_bridge_mode_valid(struct drm_bridge *bridge,
if (mode_pclk_khz > DP_MAX_PIXEL_CLK_KHZ)
return MODE_CLOCK_HIGH;
- mode_bpp = dp->connector->display_info.bpc * num_components;
+ mode_bpp = info->bpc * num_components;
if (!mode_bpp)
mode_bpp = default_bpp;
diff --git a/drivers/gpu/drm/msm/dp/dp_drm.c b/drivers/gpu/drm/msm/dp/dp_drm.c
index 8dbe7d682ddf..6b55753da0f7 100644
--- a/drivers/gpu/drm/msm/dp/dp_drm.c
+++ b/drivers/gpu/drm/msm/dp/dp_drm.c
@@ -203,7 +203,7 @@ static enum drm_mode_status msm_edp_bridge_mode_valid(struct drm_bridge *bridge,
dp = to_dp_bridge(bridge)->msm_dp_display;
- if (!dp || !mode_pclk_khz || !dp->connector) {
+ if (!dp || !mode_pclk_khz) {
DRM_ERROR("invalid params\n");
return -EINVAL;
}
--
2.47.3