On Wed, Sep 18, 2024 at 08:13:37PM +0530, Ankit Nautiyal wrote:
> Add macro to check if platform supports Ultrajoiner.
> 
> Signed-off-by: Ankit Nautiyal <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/intel_display_device.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display_device.h 
> b/drivers/gpu/drm/i915/display/intel_display_device.h
> index 5306bbd13e59..8dd7bef49133 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.h
> @@ -154,6 +154,7 @@ 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) == 14 && 
> IS_DGFX(i915)) || DISPLAY_VER(i915) > 14)

s/> 14/>= 20/ please

Since we generally prefer the new->old order, should probably start with
the >=20 check, and follow with the bmg special case.

>  #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)
> -- 
> 2.45.2

-- 
Ville Syrjälä
Intel

Reply via email to