From 671ef8c9660d1d1d5fcf1a4cb8c32c30d6efea24 Mon Sep 17 00:00:00 2001 From: Ujwal Patel Date: Wed, 25 Mar 2015 16:15:43 -0700 Subject: [PATCH] msm: mdss: set valid_roi as true when ctl paths are initialized Mixer setup can be called from multiple contexts and not all may call mixer set roi which sets valid_roi flag. However with partial update there is a requirement that for a given ctl path, roi is not valid because there is no update on that mixer. In such cases, valid_roi will be false and we should ignore setting up the mixer. So to cover all cases, initialize valid_roi to true when ctl path is setup first time. Change-Id: I85ae4ce8711f0fcdf771d18b4896a8f61398ead5 Signed-off-by: Ujwal Patel --- drivers/video/fbdev/msm/mdss_mdp_ctl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/video/fbdev/msm/mdss_mdp_ctl.c b/drivers/video/fbdev/msm/mdss_mdp_ctl.c index 7f86a98d1b35..e73cca03989d 100644 --- a/drivers/video/fbdev/msm/mdss_mdp_ctl.c +++ b/drivers/video/fbdev/msm/mdss_mdp_ctl.c @@ -2167,6 +2167,7 @@ int mdss_mdp_ctl_setup(struct mdss_mdp_ctl *ctl) ctl->mixer_left->width = width; ctl->mixer_left->height = height; ctl->mixer_left->roi = (struct mdss_rect) {0, 0, width, height}; + ctl->valid_roi = true; if (ctl->mfd->split_mode == MDP_DUAL_LM_DUAL_DISPLAY) { pr_debug("dual display detected\n"); @@ -2433,6 +2434,7 @@ int mdss_mdp_ctl_split_display_setup(struct mdss_mdp_ctl *ctl, mixer->height = sctl->height; mixer->roi = (struct mdss_rect) {0, 0, mixer->width, mixer->height}; + sctl->valid_roi = true; sctl->mixer_left = mixer; return mdss_mdp_set_split_ctl(ctl, sctl);