On 10/1/2024 5:32 PM, Jani Nikula wrote:
On Mon, 30 Sep 2024, Ankit Nautiyal <ankit.k.nauti...@intel.com> 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.nauti...@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
  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?

For Display 14, yes as its supported on Battlemage only. I think Stan had used IS_DGFX() in original series, I went ahead with it.

We can replace with IS_BATTLEMAGE() if required.

Regards,

Ankit


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)

Reply via email to