[igt-dev] [PATCH i-g-t v3 3/8] lib/intel_device_info: Add tiling information for early gens

[Zbigniew Kempczyński]

On Tue, Mar 14, 2023 at 03:44:35PM +0100, Karolina Stolarek wrote:
> Update relevant intel_device_info entries to store information
> on supported tiling formats.
> 
> Signed-off-by: Karolina Stolarek <karolina.stolarek at intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> ---
>  lib/intel_device_info.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/lib/intel_device_info.c b/lib/intel_device_info.c
> index 0baad721..bda4fb63 100644
> --- a/lib/intel_device_info.c
> +++ b/lib/intel_device_info.c
> @@ -27,12 +27,14 @@ static const struct intel_device_info intel_i830_info = {
>  	.graphics_ver = 2,
>  	.display_ver = 2,
>  	.is_almador = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "almador"
>  };
>  static const struct intel_device_info intel_i845_info = {
>  	.graphics_ver = 2,
>  	.display_ver = 2,
>  	.is_brookdale = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "brookdale"
>  };
>  static const struct intel_device_info intel_i855_info = {
> @@ -40,12 +42,14 @@ static const struct intel_device_info intel_i855_info = {
>  	.display_ver = 2,
>  	.is_mobile = true,
>  	.is_montara = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "montara"
>  };
>  static const struct intel_device_info intel_i865_info = {
>  	.graphics_ver = 2,
>  	.display_ver = 2,
>  	.is_springdale = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "spingdale"
>  };
>  
> @@ -53,6 +57,7 @@ static const struct intel_device_info intel_i915_info = {
>  	.graphics_ver = 3,
>  	.display_ver = 3,
>  	.is_grantsdale = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "grantsdale"
>  };
>  static const struct intel_device_info intel_i915m_info = {
> @@ -60,12 +65,14 @@ static const struct intel_device_info intel_i915m_info = {
>  	.display_ver = 3,
>  	.is_mobile = true,
>  	.is_alviso = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "alviso"
>  };
>  static const struct intel_device_info intel_i945_info = {
>  	.graphics_ver = 3,
>  	.display_ver = 3,
>  	.is_lakeport = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "lakeport"
>  };
>  static const struct intel_device_info intel_i945m_info = {
> @@ -73,6 +80,7 @@ static const struct intel_device_info intel_i945m_info = {
>  	.display_ver = 3,
>  	.is_mobile = true,
>  	.is_calistoga = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "calistoga"
>  };
>  
> @@ -80,6 +88,7 @@ static const struct intel_device_info intel_g33_info = {
>  	.graphics_ver = 3,
>  	.display_ver = 3,
>  	.is_bearlake = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "bearlake"
>  };
>  
> @@ -87,6 +96,7 @@ static const struct intel_device_info intel_pineview_g_info = {
>  	.graphics_ver = 3,
>  	.display_ver = 3,
>  	.is_pineview = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "pineview"
>  };
>  
> @@ -95,6 +105,7 @@ static const struct intel_device_info intel_pineview_m_info = {
>  	.display_ver = 3,
>  	.is_mobile = true,
>  	.is_pineview = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "pineview"
>  };
>  
> @@ -102,6 +113,7 @@ static const struct intel_device_info intel_i965_info = {
>  	.graphics_ver = 4,
>  	.display_ver = 4,
>  	.is_broadwater = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "broadwater"
>  };
>  
> @@ -110,6 +122,7 @@ static const struct intel_device_info intel_i965m_info = {
>  	.display_ver = 4,
>  	.is_mobile = true,
>  	.is_crestline = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "crestline"
>  };
>  
> @@ -117,6 +130,7 @@ static const struct intel_device_info intel_g45_info = {
>  	.graphics_ver = 4,
>  	.display_ver = 4,
>  	.is_eaglelake = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "eaglelake"
>  };
>  static const struct intel_device_info intel_gm45_info = {
> @@ -124,6 +138,7 @@ static const struct intel_device_info intel_gm45_info = {
>  	.display_ver = 4,
>  	.is_mobile = true,
>  	.is_cantiga = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "cantiga"
>  };
>  
> @@ -131,6 +146,7 @@ static const struct intel_device_info intel_ironlake_info = {
>  	.graphics_ver = 5,
>  	.display_ver = 5,
>  	.is_ironlake = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "ironlake" /* clarkdale? */
>  };
>  static const struct intel_device_info intel_ironlake_m_info = {
> @@ -138,6 +154,7 @@ static const struct intel_device_info intel_ironlake_m_info = {
>  	.display_ver = 5,
>  	.is_mobile = true,
>  	.is_arrandale = true,
> +	.cmds_info = &pre_gen6_cmds_info,
>  	.codename = "arrandale"
>  };
>  
> -- 
> 2.25.1
>

Looks ok,

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew