Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a3d80366 authored by Benet Clark's avatar Benet Clark
Browse files

msm: mdss: Return error when Gamut table size is invalid 



Previously, the gamut caching code was not returning an error value when
an invalid table size was passed. This change will return error during
caching if the table size is invalid.

Change-Id: I212b345239543943670c2f03a0fc8376d31d1e86
Signed-off-by: default avatarBenet Clark <benetc@codeaurora.org>
parent 30b24a3d

drivers/video/msm/mdss/mdss_mdp_pp_cache_config.c

+6 −4
Original line number Diff line number Diff line
@@ -405,9 +405,10 @@ static int pp_gamut_cache_params_v1_7(struct mdp_gamut_cfg_data *config, 
	/* sanity check for sizes */

	for (i = 0; i < MDP_GAMUT_TABLE_NUM_V1_7; i++) {

		if (v17_usr_config.tbl_size[i] != tbl_sz) {

			pr_err("invalid tbl_sz %d exp %d for mode %d\n",

			       v17_usr_config.tbl_size[i], tbl_sz,

			pr_err("invalid tbl size %d exp %d tbl index %d mode %d\n",

			       v17_usr_config.tbl_size[i], tbl_sz, i,

			       v17_usr_config.mode);

			ret = -EINVAL;

			goto gamut_config_exit;

		}

		gamut_size += v17_usr_config.tbl_size[i];
@@ -415,10 +416,11 @@ static int pp_gamut_cache_params_v1_7(struct mdp_gamut_cfg_data *config, 
			continue;

		if (v17_usr_config.tbl_scale_off_sz[i] !=

		    MDP_GAMUT_SCALE_OFF_SZ) {

			pr_err("invalid scale_sz %d exp %d for mode %d\n",

			pr_err("invalid scale size %d exp %d scale index %d mode %d\n",

			       v17_usr_config.tbl_scale_off_sz[i],

			       MDP_GAMUT_SCALE_OFF_SZ,

			       MDP_GAMUT_SCALE_OFF_SZ, i,

			       v17_usr_config.mode);

			ret = -EINVAL;

			goto gamut_config_exit;

		}

		scal_coff_size += v17_usr_config.tbl_scale_off_sz[i];