[igt-dev] [v3 i-g-t 14/14] tests/kms_vblank: Reset the state before exiting the test

[Modem, Bhanuprakash]

> From: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>
> Sent: Monday, April 26, 2021 11:52 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; Deak, Imre
> <imre.deak at intel.com>; Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>; Daniel
> Vetter <daniel.vetter at ffwll.ch>
> Subject: [v3 i-g-t 14/14] tests/kms_vblank: Reset the state before exiting the
> test
> 
> Before starting the next subtest, clean up the states to default
> values, those are assumed by other tests.
> 
> Cc: Imre Deak <imre.deak at intel.com>
> Cc: Ankit Nautiyal <ankit.k.nautiyal at intel.com>
> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>  tests/kms_vblank.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c
> index e561e41d7..9a9ce48ea 100644
> --- a/tests/kms_vblank.c
> +++ b/tests/kms_vblank.c
> @@ -96,6 +96,8 @@ static void prepare_crtc(data_t *data, int fd, igt_output_t
> *output)
> 
>  static void cleanup_crtc(data_t *data, int fd, igt_output_t *output)
>  {
> +	igt_output_set_pipe(output, data->pipe);
This should be PIPE_NONE. s/data->pipe/PIPE_NONE

> +	igt_display_commit(&data->display);
>  	igt_remove_fb(fd, &data->primary_fb);
>  }
> 
> --
> 2.20.1