> -----Original Message-----
> From: Nikula, Jani <[email protected]>
> Sent: Wednesday, June 10, 2026 5:03 PM
> To: Manna, Animesh <[email protected]>; intel-
> [email protected]; [email protected]
> Cc: Shankar, Uma <[email protected]>; Dibin Moolakadan
> Subrahmanian <[email protected]>;
> [email protected]; Manna, Animesh <[email protected]>
> Subject: Re: [PATCH v8 08/20] drm/i915/vrr: Pass transcoder to
> intel_vrr_set_fixed_rr_timings()
>
> On Thu, 04 Jun 2026, Animesh Manna <[email protected]> wrote:
> > diff --git a/drivers/gpu/drm/i915/display/intel_vrr.h
> > b/drivers/gpu/drm/i915/display/intel_vrr.h
> > index 4f16ca4af91f..2daba0c16162 100644
> > --- a/drivers/gpu/drm/i915/display/intel_vrr.h
> > +++ b/drivers/gpu/drm/i915/display/intel_vrr.h
> > @@ -8,6 +8,8 @@
> >
> > #include <linux/types.h>
> >
> > +#include "intel_display_limits.h"
>
> Don't include headers from headers if it can be avoided.
Sure, taken care in latest version.
Regards,
Animesh
>
> > +
> > struct drm_connector_state;
> > struct intel_atomic_state;
> > struct intel_connector;
> > @@ -42,7 +44,8 @@ int intel_vrr_vmin_vblank_start(const struct
> > intel_crtc_state *crtc_state); bool intel_vrr_is_fixed_rr(const
> > struct intel_crtc_state *crtc_state); void
> > intel_vrr_transcoder_enable(const struct intel_crtc_state
> > *crtc_state); void intel_vrr_transcoder_disable(const struct
> > intel_crtc_state *crtc_state); -void
> > intel_vrr_set_fixed_rr_timings(const struct intel_crtc_state
> > *crtc_state);
> > +void intel_vrr_set_fixed_rr_timings(const struct intel_crtc_state
> *crtc_state,
> > + enum transcoder transcoder);
> > void intel_vrr_dcb_reset(const struct intel_crtc_state *old_crtc_state,
> > struct intel_crtc *crtc);
> > bool intel_vrr_always_use_vrr_tg(struct intel_display *display);
>
> --
> Jani Nikula, Intel
