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 <[email protected]> --- 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
