On Thu, Dec 03, 2020 at 06:37:46PM +0200, Jani Nikula wrote: > 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* Yup, thats where I added HAS_VRR macro Manasi > > > -- > Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
