[Intel-gfx] [PATCH 1/2] drm/i915/ats-m: add ATS-M platform info

[Balasubramani Vivekanandan]

Looks good to me.

Reviewed-by: Balasubramani Vivekanandan <balasubramani.vivekanandan at intel.com>

On 28.03.2022 17:08, Matt Roper wrote:
> ATS-M is a server platform based on Xe_HPG and Xe_HPM, but without
> display support.  From a driver point of view, it's easiest to just
> handle it as DG2 (including identifying as PLATFORM_DG2), but with the
> display disabled in the device info.
> 
> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
> ---
>  drivers/gpu/drm/i915/i915_pci.c | 40 ++++++++++++++++++++-------------
>  1 file changed, 25 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
> index 67b89769f577..2025e1114927 100644
> --- a/drivers/gpu/drm/i915/i915_pci.c
> +++ b/drivers/gpu/drm/i915/i915_pci.c
> @@ -1040,25 +1040,35 @@ static const struct intel_device_info xehpsdv_info = {
>  	.require_force_probe = 1,
>  };
>  
> +#define DG2_FEATURES \
> +	XE_HP_FEATURES, \
> +	XE_HPM_FEATURES, \
> +	DGFX_FEATURES, \
> +	.graphics.rel = 55, \
> +	.media.rel = 55, \
> +	PLATFORM(INTEL_DG2), \
> +	.has_4tile = 1, \
> +	.has_64k_pages = 1, \
> +	.has_guc_deprivilege = 1, \
> +	.needs_compact_pt = 1, \
> +	.platform_engine_mask = \
> +		BIT(RCS0) | BIT(BCS0) | \
> +		BIT(VECS0) | BIT(VECS1) | \
> +		BIT(VCS0) | BIT(VCS2)
> +
>  static const struct intel_device_info dg2_info = {
> -	XE_HP_FEATURES,
> -	XE_HPM_FEATURES,
> +	DG2_FEATURES,
>  	XE_LPD_FEATURES,
> -	DGFX_FEATURES,
> -	.graphics.rel = 55,
> -	.media.rel = 55,
> -	.has_4tile = 1,
> -	PLATFORM(INTEL_DG2),
> -	.has_guc_deprivilege = 1,
> -	.has_64k_pages = 1,
> -	.needs_compact_pt = 1,
> -	.platform_engine_mask =
> -		BIT(RCS0) | BIT(BCS0) |
> -		BIT(VECS0) | BIT(VECS1) |
> -		BIT(VCS0) | BIT(VCS2),
> -	.require_force_probe = 1,
>  	.display.cpu_transcoder_mask = BIT(TRANSCODER_A) | BIT(TRANSCODER_B) |
>  			       BIT(TRANSCODER_C) | BIT(TRANSCODER_D),
> +	.require_force_probe = 1,
> +};
> +
> +__maybe_unused
> +static const struct intel_device_info ats_m_info = {
> +	DG2_FEATURES,
> +	.display = { 0 },
> +	.require_force_probe = 1,
>  };
>  
>  #undef PLATFORM
> -- 
> 2.34.1
>