From: Ankit Nautiyal Date: Thu, 30 Apr 2026 13:12:19 +0000 (+0530) Subject: drm/i915/intel_sdvo: Drop call to intel_panel_fixed_mode() from compute_config X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=90d8acc6d8b780822aa3fa7ff6f1614d0454fd58;p=thirdparty%2Fkernel%2Flinux.git drm/i915/intel_sdvo: Drop call to intel_panel_fixed_mode() from compute_config Currently, in intel_sdvo_compute_config(), intel_panel_fixed_mode() is called to get the fixed mode. However, since the call is made after intel_panel_compute_config, that copies the selected fixed mode to the adjusted mode, we can directly use the crtc_state->hw.adjusted_mode. So remove the extra call to intel_panel_fixed_mode() and use adjusted mode instead. Signed-off-by: Ankit Nautiyal Reviewed-by: Suraj Kandpal Link: https://patch.msgid.link/20260430131220.3891497-5-ankit.k.nautiyal@intel.com --- diff --git a/drivers/gpu/drm/i915/display/intel_sdvo.c b/drivers/gpu/drm/i915/display/intel_sdvo.c index 7843eb13f2e37..c18e7d5e9c343 100644 --- a/drivers/gpu/drm/i915/display/intel_sdvo.c +++ b/drivers/gpu/drm/i915/display/intel_sdvo.c @@ -1395,8 +1395,7 @@ static int intel_sdvo_compute_config(struct intel_encoder *encoder, adjusted_mode); pipe_config->sdvo_tv_clock = true; } else if (IS_LVDS(intel_sdvo_connector)) { - const struct drm_display_mode *fixed_mode = - intel_panel_fixed_mode(&intel_sdvo_connector->base, mode); + const struct drm_display_mode *fixed_mode; int ret; ret = intel_panel_compute_config(&intel_sdvo_connector->base, @@ -1404,6 +1403,8 @@ static int intel_sdvo_compute_config(struct intel_encoder *encoder, if (ret) return ret; + fixed_mode = &pipe_config->hw.adjusted_mode; + if (!intel_sdvo_set_output_timings_from_mode(intel_sdvo, intel_sdvo_connector, fixed_mode))