> -----Original Message-----
> From: Nautiyal, Ankit K <[email protected]>
> Sent: Monday, April 14, 2025 8:13 AM
> To: [email protected]; [email protected]
> Cc: Kandpal, Suraj <[email protected]>; [email protected];
> Nautiyal, Ankit K <[email protected]>
> Subject: [PATCH 1/2] drm/i915/display: Add macro for checking 3 DSC engines
>
> 3 DSC engines per pipe is currently supported only for BMG.
Would it be better to have a function here which can be modified later to add
more
platforms in future if they decide to add 3 DSC engine elsewhere too?
> Add a macro to check whether a platform supports 3 DSC engines per pipe.
>
> Bspec: 50175
> 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 368b0d3417c2..1a215791d0ba 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_device.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_device.h
> @@ -163,6 +163,7 @@ struct intel_display_platforms {
> #define HAS_DP_MST(__display) (DISPLAY_INFO(__display)-
> >has_dp_mst)
> #define HAS_DSB(__display) (DISPLAY_INFO(__display)->has_dsb)
> #define HAS_DSC(__display) (DISPLAY_RUNTIME_INFO(__display)-
> >has_dsc)
> +#define HAS_DSC_3ENGINES(__display) (DISPLAY_VERx100(display) == 1401
Shouldn't this be __display instead of just display
Regards,
Suraj Kandpal
> && HAS_DSC(__display))
> #define HAS_DSC_MST(__display) (DISPLAY_VER(__display) >= 12
> && HAS_DSC(__display))
> #define HAS_FBC(__display) (DISPLAY_RUNTIME_INFO(__display)-
> >fbc_mask != 0)
> #define HAS_FBC_DIRTY_RECT(__display) (DISPLAY_VER(__display) >=
> 30)
> --
> 2.34.1
