@@ -1241,7 +1241,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
IS_HASWELL(dev_priv) || \
IS_IVYBRIDGE(dev_priv) || \
IS_SANDYBRIDGE(dev_priv)))
-#define HAS_4TILE(dev_priv) (INTEL_INFO(dev_priv)->has_4tile)
+#define HAS_4TILE(dev_priv) (GRAPHICS_VER_FULL(dev_priv) >= IP_VER(12, 50))
#define HAS_SNOOP(dev_priv) (INTEL_INFO(dev_priv)->has_snoop)
#define HAS_EDRAM(dev_priv) ((dev_priv)->edram_size_mb)
#define HAS_SECURE_BATCHES(dev_priv) (GRAPHICS_VER(dev_priv) < 6)
@@ -1001,7 +1001,6 @@ static const struct intel_device_info xehpsdv_info = {
.graphics.rel = 55, \
.media.rel = 55, \
PLATFORM(INTEL_DG2), \
- .has_4tile = 1, \
.has_64k_pages = 1, \
.has_guc_deprivilege = 1, \
.has_heci_pxp = 1, \
@@ -142,7 +142,6 @@ enum intel_ppgtt_type {
func(has_64k_pages); \
func(needs_compact_pt); \
func(gpu_reset_clobbers_display); \
- func(has_4tile); \
func(has_flat_ccs); \
func(has_global_mocs); \
func(has_heci_pxp); \
No need to have this parameter in intel_device_info struct as this tiling format is supported in platforms with graphics 12.50 or newer what was missed in xehpsdv. As a side effect of the of removal this flag, it will not be printed in dmesg during driver load anymore and developers will have to rely on to check the macro and compare with platform being used and IP versions of it. BSpec: 44917 Signed-off-by: José Roberto de Souza <jose.souza@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 2 +- drivers/gpu/drm/i915/i915_pci.c | 1 - drivers/gpu/drm/i915/intel_device_info.h | 1 - 3 files changed, 1 insertion(+), 3 deletions(-)