diff options
| author | David Lin <dtwlin@google.com> | 2017-06-01 21:33:46 -0700 |
|---|---|---|
| committer | Michael Bestas <mkbestas@lineageos.org> | 2019-12-23 23:43:31 +0200 |
| commit | 6e2242375fd7a131375deccd6b244ee33ad172fc (patch) | |
| tree | 8e93aaec81f3d18927c6c4f5b5ca64bc5570c5b5 /drivers/video/fbdev | |
| parent | 89e754b075e98365aff0fb5c4683a8da98170339 (diff) | |
msm: mdss: fix inconsistent mutex_lock
drivers/video/fbdev/msm/mdss_mdp_pp.c:4093 mdss_mdp_igc_lut_config()
warn: inconsistent returns 'mutex:&mdss_pp_mutex'
Change-Id: I956017162cde27fbaa3da5aeb28b76ec7308530e
Signed-off-by: David Lin <dtwlin@google.com>
Diffstat (limited to 'drivers/video/fbdev')
| -rw-r--r-- | drivers/video/fbdev/msm/mdss_mdp_pp.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/video/fbdev/msm/mdss_mdp_pp.c b/drivers/video/fbdev/msm/mdss_mdp_pp.c index 5b9798e2c24e..1ab39479d038 100644 --- a/drivers/video/fbdev/msm/mdss_mdp_pp.c +++ b/drivers/video/fbdev/msm/mdss_mdp_pp.c @@ -4081,7 +4081,8 @@ int mdss_mdp_igc_lut_config(struct msm_fb_data_type *mfd, if (config->len != IGC_LUT_ENTRIES) { pr_err("invalid len for IGC table for read %d\n", config->len); - return -EINVAL; + ret = -EINVAL; + goto igc_config_exit; } ret = pp_get_dspp_num(disp_num, &dspp_num); if (ret) { @@ -4147,7 +4148,8 @@ clock_off: if (config->len != IGC_LUT_ENTRIES) { pr_err("invalid len for IGC table for write %d\n", config->len); - return -EINVAL; + ret = -EINVAL; + goto igc_config_exit; } if (copy_from_kernel) { memcpy(&mdss_pp_res->igc_lut_c0c1[disp_num][0], |