@@ -1324,8 +1324,7 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915,
#define HAS_HECI_PXP(dev_priv) \
(INTEL_INFO(dev_priv)->has_heci_pxp)
-#define HAS_HECI_GSCFI(dev_priv) \
- (INTEL_INFO(dev_priv)->has_heci_gscfi)
+#define HAS_HECI_GSCFI(dev_priv) (IS_DGFX(dev_priv))
#define HAS_HECI_GSC(dev_priv) (HAS_HECI_PXP(dev_priv) || HAS_HECI_GSCFI(dev_priv))
@@ -859,8 +859,7 @@ static const struct intel_device_info rkl_info = {
#define DGFX_FEATURES \
.memory_regions = REGION_SMEM | REGION_LMEM | REGION_STOLEN_LMEM, \
.has_snoop = 1, \
- .is_dgfx = 1, \
- .has_heci_gscfi = 1
+ .is_dgfx = 1
static const struct intel_device_info dg1_info = {
GEN12_FEATURES,
@@ -143,7 +143,6 @@ enum intel_ppgtt_type {
func(gpu_reset_clobbers_display); \
func(has_flat_ccs); \
func(has_heci_pxp); \
- func(has_heci_gscfi); \
func(has_l3_dpf); \
func(has_logical_ring_contexts); \
func(has_mslices); \
No need to have this parameter in intel_device_info struct as all discrete platforms supports this feature. 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. Signed-off-by: José Roberto de Souza <jose.souza@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 3 +-- drivers/gpu/drm/i915/i915_pci.c | 3 +-- drivers/gpu/drm/i915/intel_device_info.h | 1 - 3 files changed, 2 insertions(+), 5 deletions(-)