[PATCH] drm/msm/mdp: full mode set if crtc enable state is changed

From: Jilai Wang
Date: Mon Jan 26 2015 - 14:09:11 EST


Crtc state may not reflect the current crtc state if drm_fb_helper_blank
is called to turn dpms off without turning crtc off. A full modeset in
this case to solve the problem.

Signed-off-by: Jilai Wang <jilaiw@xxxxxxxxxxxxxx>
---
drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c
index eea209e..60d0f58 100644
--- a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c
+++ b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c
@@ -310,6 +310,13 @@ static int mdp5_crtc_atomic_check(struct drm_crtc *crtc,
return -EINVAL;
}

+ /*
+ * check if the crtc state matches with the current state
+ * if not match, force a full mode set
+ */
+ if (state->enable != mdp5_crtc->enabled)
+ state->mode_changed = true;
+
/* verify that there are not too many planes attached to crtc
* and that we don't have conflicting mixer stages:
*/
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/