On Tue, 01 Dec 2020, "Navare, Manasi" <manasi.d.navare@xxxxxxxxx> wrote: > On Tue, Nov 10, 2020 at 12:41:07PM +0200, Jani Nikula wrote: >> On Thu, 22 Oct 2020, Manasi Navare <manasi.d.navare@xxxxxxxxx> wrote: >> > + >> > + if (INTEL_GEN(dev_priv) >= 12) >> >> I wonder if we should just add a wrapper >> >> #define HAS_VRR(i915) (INTEL_GEN(i915) >= 12) >> >> to be more descriptive. And use it in the previous patch too. > > Yea I like the idea of adding this wrapper in intel_vrr.c and have the platform check in that > and then use that in intel_vrr_is_capable() ? Works for me, but might just throw that in i915_drv.h with all the other HAS_ helpers also. *shrug* -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
