In the loop looking for planes on a pipe, we always want to free up
the drm_plane afterwards.

Fixes: 36656239ef96 lib/igt_kms: Implement dynamic plane count support
Signed-off-by: Brian Starkey <brian.star...@arm.com>
Reviewed-by: Robert Foss <robert.f...@collabora.com>
---
 lib/igt_kms.c |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index d6b7d677ed8d..9e59e35f2e2f 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1613,12 +1613,10 @@ void igt_display_init(igt_display_t *display, int 
drm_fd)
                                                    plane_resources->planes[j]);
                        igt_assert(drm_plane);
 
-                       if (!(drm_plane->possible_crtcs & (1 << i))) {
-                               drmModeFreePlane(drm_plane);
-                               continue;
-                       }
+                       if (drm_plane->possible_crtcs & (1 << i))
+                               n_planes++;
 
-                       n_planes++;
+                       drmModeFreePlane(drm_plane);
                }
 
                igt_assert_lte(0, n_planes);
-- 
1.7.9.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to