On Mon, 30 Sep 2024, Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx> wrote: > Add macro to check if platform supports Ultrajoiner. > > v2: > -Use check for DISPLAY_VER >= 20, and add bmg as a special case. (Ville) > -Add check for HAS_DSC. (Ville) > > Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx> > Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/i915/display/intel_display_device.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h b/drivers/gpu/drm/i915/display/intel_display_device.h > index 6a5bee59e6aa..220cca6333ee 100644 > --- a/drivers/gpu/drm/i915/display/intel_display_device.h > +++ b/drivers/gpu/drm/i915/display/intel_display_device.h > @@ -154,6 +154,9 @@ enum intel_display_subplatform { > #define HAS_TRANSCODER(i915, trans) ((DISPLAY_RUNTIME_INFO(i915)->cpu_transcoder_mask & \ > BIT(trans)) != 0) > #define HAS_UNCOMPRESSED_JOINER(i915) (DISPLAY_VER(i915) >= 13) > +#define HAS_ULTRAJOINER(i915) ((DISPLAY_VER(i915) >= 20 || \ > + (IS_DGFX(i915) && DISPLAY_VER(i915) == 14)) && \ Do we need to depend on IS_DGFX() here? BR, Jani. > + HAS_DSC(i915)) > #define HAS_VRR(i915) (DISPLAY_VER(i915) >= 11) > #define HAS_AS_SDP(i915) (DISPLAY_VER(i915) >= 13) > #define HAS_CMRR(i915) (DISPLAY_VER(i915) >= 20) -- Jani Nikula, Intel