[igt-dev] [PATCH i-g-t] tests/kms_available_modes_crc: Reset output->pipe after testing it.

Juha-Pekka Heikkilä juhapekka.heikkila at gmail.com
Fri Jan 18 07:20:32 UTC 2019


Hi,

I actually had this fix included here 
https://patchwork.freedesktop.org/patch/268867/

my patch fixes also few other issues but it still need some work with 
gamma tables, I have there some mismatch.

Your patch can go in on my behalf, I don't yet know how soon I get those 
gamma tables working so all crc's will start to pass.

Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>

Dhinakaran Pandiyan kirjoitti 17.1.2019 klo 21.34:
> Not clearing the pipe results in a test failure when the same pipe is
> assigned to the next output.
> 
> Cc: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> Suggested-by: James Ausmus <james.ausmus at intel.com>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> ---
>   tests/kms_available_modes_crc.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/kms_available_modes_crc.c b/tests/kms_available_modes_crc.c
> index 45a47750..0c0282f8 100644
> --- a/tests/kms_available_modes_crc.c
> +++ b/tests/kms_available_modes_crc.c
> @@ -484,6 +484,7 @@ test_available_modes(data_t* data)
>   		igt_pipe_crc_stop(data->pipe_crc);
>   		igt_pipe_crc_free(data->pipe_crc);
>   		igt_display_commit2(&data->display, data->commit);
> +		igt_output_set_pipe(output, PIPE_NONE);
>   	}
>   	igt_assert(invalids == 0);
>   }
> 


More information about the igt-dev mailing list