Hello @Taylor, Clinton A This patch can be dropped from here, We are handling this scenario with Ankit's VRR default timing generator patch series.
https://patchwork.freedesktop.org/series/134383/ -Mitul > -----Original Message----- > From: Intel-gfx <[email protected]> On Behalf Of > Golani, Mitulkumar Ajitkumar > Sent: 25 October 2024 13:37 > To: Taylor, Clinton A <[email protected]>; intel- > [email protected]; [email protected] > Cc: Kurmi, Suresh Kumar <[email protected]>; Nautiyal, Ankit K > <[email protected]> > Subject: RE: [PATCH v4 10/11] drm/i915/xe3lpd: Skip disabling VRR during > modeset disable > > > > > -----Original Message----- > > From: Intel-gfx <[email protected]> On Behalf Of > > Clint Taylor > > Sent: 25 October 2024 04:01 > > To: [email protected]; [email protected] > > Subject: [PATCH v4 10/11] drm/i915/xe3lpd: Skip disabling VRR during > > modeset disable > > > > From: Ravi Kumar Vodapalli <[email protected]> > > > > Spec does not request to disable VRR in the modeset disabling sequence > > for DP and HDMI for xe3_lpd. > > > > Bspec: 68848 > > Signed-off-by: Ravi Kumar Vodapalli <[email protected]> > > Signed-off-by: Matt Atwood <[email protected]> > > Signed-off-by: Clint Taylor <[email protected]> > > --- > > drivers/gpu/drm/i915/display/intel_display.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c > > b/drivers/gpu/drm/i915/display/intel_display.c > > index c904f529d0c5..7a6961432616 100644 > > --- a/drivers/gpu/drm/i915/display/intel_display.c > > +++ b/drivers/gpu/drm/i915/display/intel_display.c > > @@ -1310,9 +1310,11 @@ static void intel_pre_plane_update(struct > > intel_atomic_state *state, > > intel_atomic_get_new_crtc_state(state, crtc); > > enum pipe pipe = crtc->pipe; > > > > - if (intel_crtc_vrr_disabling(state, crtc)) { > > - intel_vrr_disable(old_crtc_state); > > - intel_crtc_update_active_timings(old_crtc_state, false); > > + if (DISPLAY_VER(dev_priv) < 30) { > > + if (intel_crtc_vrr_disabling(state, crtc)) { > > + intel_vrr_disable(old_crtc_state); > > + intel_crtc_update_active_timings(old_crtc_state, > > false); > > + } > > Hello @Clint Taylor > > Seems comments from revision 2 got missed ? > > https://patchwork.freedesktop.org/patch/621235/?series=140196&rev=2 > > > > } > > > > if (audio_disabling(old_crtc_state, new_crtc_state)) > > -- > > 2.25.1
