From 1867564b9080d096445f1c40b3fc360364735bb9 Mon Sep 17 00:00:00 2001 From: Imre Deak Date: Mon, 15 Dec 2025 21:23:45 +0200 Subject: [PATCH] drm/i915/dp: Use a mode's crtc_clock vs. clock during state computation The encoder state computation should use the drm_display_mode::crtc_clock member, instead of the clock member, the former one possibly having a necessary adjustment wrt. to the latter due to driver specific constraints. In practice the two values should not differ at spots changed in this patch, since only MSO and 3D modes would make them different, neither MSO or 3D relevant here, but still use the expected crtc_clock version for consistency. Reviewed-by: Luca Coelho Signed-off-by: Imre Deak Link: https://patch.msgid.link/20251215192357.172201-6-imre.deak@intel.com --- drivers/gpu/drm/i915/display/intel_dp.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 052cc0df0121..e687d61ff49c 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -2051,7 +2051,8 @@ static int dsc_compute_link_config(struct intel_dp *intel_dp, continue; } else { if (!is_bw_sufficient_for_dsc_config(dsc_bpp_x16, link_rate, - lane_count, adjusted_mode->clock, + lane_count, + adjusted_mode->crtc_clock, pipe_config->output_format, timeslots)) continue; @@ -2212,7 +2213,7 @@ static int dsc_compute_compressed_bpp(struct intel_dp *intel_dp, int bpp_x16; int ret; - dsc_joiner_max_bpp = get_max_compressed_bpp_with_joiner(display, adjusted_mode->clock, + dsc_joiner_max_bpp = get_max_compressed_bpp_with_joiner(display, adjusted_mode->crtc_clock, adjusted_mode->hdisplay, num_joined_pipes); max_bpp_x16 = min(fxp_q4_from_int(dsc_joiner_max_bpp), limits->link.max_bpp_x16); -- 2.47.3