Commit 322fbf06 authored Mar 25, 2015 by Ujwal Patel Committed by Gerrit - the friendly Code Review server Mar 27, 2015

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 <ujwalp@codeaurora.org>

parent 3350499c

drivers/video/msm/mdss/mdss_mdp_ctl.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -2043,6 +2043,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");
		@@ -2359,6 +2360,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);