[PATCH 1/2] drm/i915/display: Add macro for checking 3 DSC engines

[Kandpal, Suraj]

> -----Original Message-----
> From: Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>
> Sent: Monday, April 14, 2025 9:36 AM
> To: Kandpal, Suraj <suraj.kandpal at intel.com>; intel-gfx at lists.freedesktop.org;
> intel-xe at lists.freedesktop.org
> Cc: stable at vger.kernel.org
> Subject: Re: [PATCH 1/2] drm/i915/display: Add macro for checking 3 DSC
> engines
> 
> 
> On 4/14/2025 9:18 AM, Kandpal, Suraj wrote:
> >
> >> -----Original Message-----
> >> From: Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>
> >> Sent: Monday, April 14, 2025 8:13 AM
> >> To: intel-gfx at lists.freedesktop.org; intel-xe at lists.freedesktop.org
> >> Cc: Kandpal, Suraj <suraj.kandpal at intel.com>; stable at vger.kernel.org;
> >> Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>
> >> 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?
> 
> I was thinking about it too but it seems this is not going to be commonly used
> and perhaps we can add specific platforms in the macro as well.

Ohkay if that’s the case then it seems okay to me

> 
> 
> >
> >> Add a macro to check whether a platform supports 3 DSC engines per pipe.
> >>
> >> Bspec: 50175
> >> Signed-off-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> >> ---
> >>   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
> 
> You are right, this should have been __display.
> 
> 
> Regards,
> 
> Ankit
> 
> >
> > 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