There's no need for this check, always use set_fb(NULL) on the plane.

Signed-off-by: Damien Lespiau <damien.lesp...@intel.com>
---
 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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to