Unverified Commit 44663022 authored by Ankit Nautiyal's avatar Ankit Nautiyal Committed by Rodrigo Vivi
Browse files

drm/i915/dp: fix the Adaptive sync Operation mode for SDP 



Currently we support Adaptive sync operation mode with dynamic frame
rate, but instead the operation mode with fixed rate is set.
This was initially set correctly in the earlier version of changes but
later got changed, while defining a macro for the same.

Fixes: a5bd5991 ("drm/i915/display: Compute AS SDP parameters")
Cc: Mitul Golani <mitulkumar.ajitkumar.golani@intel.com>
Cc: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Reviewed-by: default avatarMitul Golani <mitulkumar.ajitkumar.golani@intel.com>
Signed-off-by: default avatarAnkit Nautiyal <ankit.k.nautiyal@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250130051609.1796524-4-mitulkumar.ajitkumar.golani@intel.com


(cherry picked from commit c5806862)
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 57965269

drivers/gpu/drm/i915/display/intel_dp.c

+1 −2
Original line number Diff line number Diff line
@@ -2829,7 +2829,6 @@ static void intel_dp_compute_as_sdp(struct intel_dp *intel_dp, 


	crtc_state->infoframes.enable |= intel_hdmi_infoframe_enable(DP_SDP_ADAPTIVE_SYNC);



	/* Currently only DP_AS_SDP_AVT_FIXED_VTOTAL mode supported */

	as_sdp->sdp_type = DP_SDP_ADAPTIVE_SYNC;

	as_sdp->length = 0x9;

	as_sdp->duration_incr_ms = 0;
@@ -2840,7 +2839,7 @@ static void intel_dp_compute_as_sdp(struct intel_dp *intel_dp, 
		as_sdp->target_rr = drm_mode_vrefresh(adjusted_mode);

		as_sdp->target_rr_divider = true;

	} else {

		as_sdp->mode = DP_AS_SDP_AVT_FIXED_VTOTAL;

		as_sdp->mode = DP_AS_SDP_AVT_DYNAMIC_VTOTAL;

		as_sdp->vtotal = adjusted_mode->vtotal;

		as_sdp->target_rr = 0;

	}