[Intel-gfx] [PATCH] lib: Reset 'position_changed' after a drmModeSetCrtc()

Damien Lespiau damien.lespiau at intel.com
Mon Jul 7 19:01:18 CEST 2014


So the next commit won't trigger a drmModeSetCrtc() if the primary plane
doesn't have any update needing it.

This shouldn't be a problem at the moment as we don't allow the primary
plane to be of a different size than the CRTC viewport, but it will most
likely change in the future and we don't want to have that bug there.

Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
---
 lib/igt_kms.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index d792008..82bdec5 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1025,6 +1025,7 @@ static int igt_primary_plane_commit_legacy(igt_plane_t *primary,
 
 	primary->pipe->enabled = (fb_id != 0);
 	primary->fb_changed = false;
+	primary->position_changed = false;
 
 	return 0;
 }
-- 
1.8.3.1




More information about the Intel-gfx mailing list