[igt-dev] [v6 i-g-t 10/14] tests/kms_cursor_edge_walk: Reset the state before exiting the test

[Nautiyal, Ankit K]

LGTM.

Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>

On 5/12/2021 10:35 PM, Bhanuprakash Modem wrote:
> 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_cursor_edge_walk.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/tests/kms_cursor_edge_walk.c b/tests/kms_cursor_edge_walk.c
> index 69662afec..d1d8a9143 100644
> --- a/tests/kms_cursor_edge_walk.c
> +++ b/tests/kms_cursor_edge_walk.c
> @@ -225,6 +225,7 @@ static void cleanup_crtc(data_t *data)
>   
>   	igt_remove_fb(data->drm_fd, &data->primary_fb);
>   	igt_remove_fb(data->drm_fd, &data->fb);
> +	igt_display_commit2(display, display->is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
>   }
>   
>   static void prepare_crtc(data_t *data)
> @@ -268,6 +269,8 @@ static void test_crtc(data_t *data, unsigned int edges)
>   	create_cursor_fb(data, data->curw, data->curh);
>   
>   	test_edges(data, edges);
> +
> +	cleanup_crtc(data);
>   }
>   
>   static int opt_handler(int opt, int opt_index, void *_data)