[igt-dev] [PATCH i-g-t] tests/kms_available_mode_crc: Print debug info on failure

[Dhinakaran Pandiyan]

On Tue, 2019-01-22 at 20:28 -0800, Dhinakaran Pandiyan wrote:

Do not review, missed the HAX subject prefix.

-DK

> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> ---
>  tests/kms_available_modes_crc.c | 14 ++++++++++++--
>  1 file changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/kms_available_modes_crc.c
> b/tests/kms_available_modes_crc.c
> index 7ff385f2..44b160bf 100644
> --- a/tests/kms_available_modes_crc.c
> +++ b/tests/kms_available_modes_crc.c
> @@ -414,7 +414,12 @@ test_one_mode(data_t* data, igt_output_t
> *output, igt_plane_t* plane,
>  					&data->fullscreen_crc)) {
>  					crccompare[0] =
> igt_crc_to_string(&current_crc);
>  					crccompare[1] =
> igt_crc_to_string(&data->fullscreen_crc);
> -					igt_warn("crc mismatch. target
> %.8s, result %.8s.\n", crccompare[0], crccompare[1]);
> +					igt_warn("%s PIPE %s plane %d
> format %.4s crc mismatch. target %.8s, result %.8s.\n",
> +						 igt_output_name(output
> ),
> +						 kmstest_pipe_name(plan
> e->pipe->pipe),
> +						 plane->index,
> +						 (char *)&data-
> >format.name,
> +						 crccompare[0],
> crccompare[1]);
>  					free(crccompare[0]);
>  					free(crccompare[1]);
>  					rVal++;
> @@ -424,7 +429,12 @@ test_one_mode(data_t* data, igt_output_t
> *output, igt_plane_t* plane,
>  					&data->cursor_crc)) {
>  					crccompare[0] =
> igt_crc_to_string(&current_crc);
>  					crccompare[1] =
> igt_crc_to_string(&data->cursor_crc);
> -					igt_warn("crc mismatch. target
> %.8s, result %.8s.\n", crccompare[0], crccompare[1]);
> +					igt_warn("%s PIPE %s plane %d
> format %.4s crc mismatch. target %.8s, result %.8s.\n",
> +						 igt_output_name(output
> ),
> +						 kmstest_pipe_name(plan
> e->pipe->pipe),
> +						 plane->index,
> +						 (char *)&data-
> >format.name,
> +						 crccompare[0],
> crccompare[1]);
>  					free(crccompare[0]);
>  					free(crccompare[1]);
>  					rVal++;