kms_rotation_crc: Always disable the plane in cleanup
authorDamien Lespiau <damien.lespiau@intel.com>
Tue, 8 Jul 2014 17:20:51 +0000 (18:20 +0100)
committerDamien Lespiau <damien.lespiau@intel.com>
Fri, 11 Jul 2014 14:12:22 +0000 (15:12 +0100)
There's no need for this check, always use set_fb(NULL) on the plane.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
tests/kms_rotation_crc.c

index 0edc6c3..30eb227 100644 (file)
@@ -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);
 }