> -----Original Message-----
> From: Intel-xe <intel-xe-boun...@lists.freedesktop.org> On Behalf Of Ankit
> Nautiyal
> Sent: Sunday, September 7, 2025 1:02 PM
> To: intel-gfx@lists.freedesktop.org; intel...@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Nautiyal, Ankit K
> <ankit.k.nauti...@intel.com>
> Subject: [PATCH 01/14] drm/i915/vrr: Use crtc_vsync_start/end for computing
> vrr.vsync_start/end
>
> Use adjusted_mode->crtc_vsync_start/end instead of adjusted_mode-
> >vsync_start while computing vrr.vsync_start/end.
> For most modes, these are same but for 3D/stereo modes the crtc_vsync_start is
> different than vsync_start.
Looks Good to me.
Reviewed-by: Uma Shankar <uma.shan...@intel.com>
> Signed-off-by: Ankit Nautiyal <ankit.k.nauti...@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_vrr.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_vrr.c
> b/drivers/gpu/drm/i915/display/intel_vrr.c
> index 3eed37f271b0..266cf5e1859d 100644
> --- a/drivers/gpu/drm/i915/display/intel_vrr.c
> +++ b/drivers/gpu/drm/i915/display/intel_vrr.c
> @@ -406,10 +406,10 @@ intel_vrr_compute_config(struct intel_crtc_state
> *crtc_state,
> if (HAS_AS_SDP(display)) {
> crtc_state->vrr.vsync_start =
> (crtc_state->hw.adjusted_mode.crtc_vtotal -
> - crtc_state->hw.adjusted_mode.vsync_start);
> + crtc_state->hw.adjusted_mode.crtc_vsync_start);
> crtc_state->vrr.vsync_end =
> (crtc_state->hw.adjusted_mode.crtc_vtotal -
> - crtc_state->hw.adjusted_mode.vsync_end);
> + crtc_state->hw.adjusted_mode.crtc_vsync_end);
> }
> }
>
> --
> 2.45.2
