[PATCH 05/14] drm/crtc_helper: Disable and reenable primary plane in drm_helper_crtc_mode_set

Liu Ying gnuiyl at gmail.com
Tue May 24 10:10:44 UTC 2016


Since CRTC has already been disabled in crtc_funcs->prepare(), it doesn't hurt
to disable the primary plane in drm_helper_crtc_mode_set() before enabling it
in drm_helper_crtc_mode_set_base().  This makes those who reject active plane
update in plane_funcs->atomic_check() happy.

Signed-off-by: Liu Ying <gnuiyl at gmail.com>
---
 drivers/gpu/drm/drm_crtc_helper.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_crtc_helper.c
index 79555d2..7fabcd7 100644
--- a/drivers/gpu/drm/drm_crtc_helper.c
+++ b/drivers/gpu/drm/drm_crtc_helper.c
@@ -1013,6 +1013,15 @@ int drm_helper_crtc_mode_set(struct drm_crtc *crtc, struct drm_display_mode *mod
 			goto out;
 	}
 
+	/*
+	 * It doesn't hurt to disable primary plane here since crtc is off
+	 * and we'll enable it again in drm_helper_crtc_mode_set_base()
+	 * below soon.
+	 */
+	ret = drm_plane_helper_disable(crtc->primary);
+	if (ret)
+		goto out;
+
 	swap(crtc->state, crtc_state);
 
 	crtc_funcs->mode_set_nofb(crtc);
-- 
2.7.4



More information about the dri-devel mailing list