There's no need for this check, always use set_fb(NULL) on the plane. Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> --- tests/kms_rotation_crc.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index 0edc6c3..30eb227 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -185,11 +185,9 @@ static void cleanup_crtc(data_t *data, igt_output_t *output) data->pipe_crc = NULL; igt_remove_fb(data->gfx_fd, &data->fb); - - if (data->plane != NULL) - igt_plane_set_fb(data->plane, NULL); - + igt_plane_set_fb(data->plane, NULL); igt_output_set_pipe(output, PIPE_ANY); + igt_display_commit(display); } -- 1.8.3.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
