[PATCH i-g-t 1/2] tests/kms_plane: Print out tested formats

[Kahola, Mika]

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of Juha-Pekka Heikkila
> Sent: Wednesday, 21 May 2025 15.17
> To: igt-dev at lists.freedesktop.org
> Cc: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> Subject: [PATCH i-g-t 1/2] tests/kms_plane: Print out tested formats
> 
> Earlier tested pixel formats were moved from igt_info to igt_debug, let's put igt_info back in place so it's easier for debug time to
> see what's going on.
> 

Reviewed-by: Mika Kahola <mika.kahola at intel.com>

> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> ---
>  tests/kms_plane.c | 38 +++++++++++++++++++-------------------
>  1 file changed, 19 insertions(+), 19 deletions(-)
> 
> diff --git a/tests/kms_plane.c b/tests/kms_plane.c index d08ec81e4..2831b2d18 100644
> --- a/tests/kms_plane.c
> +++ b/tests/kms_plane.c
> @@ -887,9 +887,9 @@ static bool test_format_plane_rgb(data_t *data, enum pipe pipe,
>  				  igt_crc_t ref_crc[],
>  				  struct igt_fb *fb)
>  {
> -	igt_debug("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n",
> -		  IGT_FORMAT_ARGS(format), IGT_MODIFIER_ARGS(modifier),
> -		  kmstest_pipe_name(pipe), plane->index);
> +	igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n",
> +		 IGT_FORMAT_ARGS(format), IGT_MODIFIER_ARGS(modifier),
> +		 kmstest_pipe_name(pipe), plane->index);
> 
>  	return test_format_plane_colors(data, pipe, plane,
>  					format, modifier,
> @@ -925,13 +925,13 @@ static bool test_format_plane_yuv(data_t *data, enum pipe pipe,
>  						     igt_color_range_to_str(r)))
>  				continue;
> 
> -			igt_debug("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT
> -				" (%s, %s) on %s.%u\n", IGT_FORMAT_ARGS(format),
> -				IGT_MODIFIER_ARGS(modifier),
> -				igt_color_encoding_to_str(e),
> -				igt_color_range_to_str(r),
> -				kmstest_pipe_name(pipe),
> -				plane->index);
> +			igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT
> +				 " (%s, %s) on %s.%u\n", IGT_FORMAT_ARGS(format),
> +				 IGT_MODIFIER_ARGS(modifier),
> +				 igt_color_encoding_to_str(e),
> +				 igt_color_range_to_str(r),
> +				 kmstest_pipe_name(pipe),
> +				 plane->index);
> 
>  			result &= test_format_plane_colors(data, pipe, plane,
>  							   format, modifier,
> @@ -1073,9 +1073,9 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
> 
>  	igt_pipe_crc_start(data->pipe_crc);
> 
> -	igt_debug("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n",
> -		  IGT_FORMAT_ARGS(ref.format), IGT_MODIFIER_ARGS(ref.modifier),
> -		  kmstest_pipe_name(pipe), plane->index);
> +	igt_info("Testing format " IGT_FORMAT_FMT " / modifier " IGT_MODIFIER_FMT " on %s.%u\n",
> +		 IGT_FORMAT_ARGS(ref.format), IGT_MODIFIER_ARGS(ref.modifier),
> +		 kmstest_pipe_name(pipe), plane->index);
> 
>  	check_allowed_plane_size_64x64(data, plane, &width, &height, ref.format);
> 
> @@ -1107,12 +1107,12 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
>  			continue;
> 
>  		if (skip_format_mod(data, f.format, f.modifier, &tested_formats)) {
> -			igt_debug("Skipping format " IGT_FORMAT_FMT " / modifier "
> -				  IGT_MODIFIER_FMT " on %s.%u\n",
> -				  IGT_FORMAT_ARGS(f.format),
> -				  IGT_MODIFIER_ARGS(f.modifier),
> -				  kmstest_pipe_name(pipe),
> -				  plane->index);
> +			igt_info("Skipping format " IGT_FORMAT_FMT " / modifier "
> +				 IGT_MODIFIER_FMT " on %s.%u\n",
> +				 IGT_FORMAT_ARGS(f.format),
> +				 IGT_MODIFIER_ARGS(f.modifier),
> +				 kmstest_pipe_name(pipe),
> +				 plane->index);
>  			continue;
>  		}
> 
> --
> 2.45.2