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>
This commit is contained in:
parent
470f82b668
commit
671ef8c966
1 changed files with 2 additions and 0 deletions
|
@ -2167,6 +2167,7 @@ int mdss_mdp_ctl_setup(struct mdss_mdp_ctl *ctl)
|
||||||
ctl->mixer_left->width = width;
|
ctl->mixer_left->width = width;
|
||||||
ctl->mixer_left->height = height;
|
ctl->mixer_left->height = height;
|
||||||
ctl->mixer_left->roi = (struct mdss_rect) {0, 0, width, 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) {
|
if (ctl->mfd->split_mode == MDP_DUAL_LM_DUAL_DISPLAY) {
|
||||||
pr_debug("dual display detected\n");
|
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->height = sctl->height;
|
||||||
mixer->roi = (struct mdss_rect)
|
mixer->roi = (struct mdss_rect)
|
||||||
{0, 0, mixer->width, mixer->height};
|
{0, 0, mixer->width, mixer->height};
|
||||||
|
sctl->valid_roi = true;
|
||||||
sctl->mixer_left = mixer;
|
sctl->mixer_left = mixer;
|
||||||
|
|
||||||
return mdss_mdp_set_split_ctl(ctl, sctl);
|
return mdss_mdp_set_split_ctl(ctl, sctl);
|
||||||
|
|
Loading…
Add table
Reference in a new issue