[igt-dev] [PATCH i-g-t] lib/i915: Fix ATS-M definitions

Grzegorzek, Dominik dominik.grzegorzek at intel.com
Wed Jun 15 09:57:25 UTC 2022


On Wed, 2022-06-15 at 11:32 +0200, Karolina Drobnik wrote:
> Instead of using a separate "is_ats_m" flag, reuse "is_dg2" in ATS-M
> intel_device_info definition. Delete is_ats_m in intel_device_info struct
> definition and IS_ATS_M() macro, as they are no longer needed.
> 
> Signed-off-by: Karolina Drobnik <karolina.drobnik at intel.com>

This is what I would do too. intel_ats_m_info struct name clearly shows the intence.
Acked-by: Dominik Grzegorzek <dominik.grzegorzek at intel.com>
> ---
>  lib/intel_chipset.h     | 2 --
>  lib/intel_device_info.c | 2 +-
>  2 files changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/lib/intel_chipset.h b/lib/intel_chipset.h
> index de4749e1..06f73211 100644
> --- a/lib/intel_chipset.h
> +++ b/lib/intel_chipset.h
> @@ -84,7 +84,6 @@ struct intel_device_info {
>  	bool is_raptorlake_s : 1;
>  	bool is_alderlake_p : 1;
>  	bool is_alderlake_n : 1;
> -	bool is_ats_m : 1;
>  	const char *codename;
>  };
>  
> @@ -191,7 +190,6 @@ void intel_check_pch(void);
>  #define IS_RAPTORLAKE_S(devid)	(intel_get_device_info(devid)->is_raptorlake_s)
>  #define IS_ALDERLAKE_P(devid)	(intel_get_device_info(devid)->is_alderlake_p)
>  #define IS_ALDERLAKE_N(devid)	(intel_get_device_info(devid)->is_alderlake_n)
> -#define IS_ATS_M(devid)	(intel_get_device_info(devid)->is_ats_m)
>  
>  #define IS_GEN(devid, x)	(intel_get_device_info(devid)->graphics_ver == x)
>  #define AT_LEAST_GEN(devid, x)	(intel_get_device_info(devid)->graphics_ver >= x)
> diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> index 5b1da33f..bfdd9fa5 100644
> --- a/lib/intel_device_info.c
> +++ b/lib/intel_device_info.c
> @@ -430,7 +430,7 @@ static const struct intel_device_info intel_alderlake_n_info = {
>  static const struct intel_device_info intel_ats_m_info = {
>  	.graphics_ver = 12,
>  	.display_ver = 0, /* no display support */
> -	.is_ats_m = true,
> +	.is_dg2 = true,
>  	.has_4tile = true,
>  	.codename = "ats_m",
>  	.has_flatccs = true,


More information about the igt-dev mailing list