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

Commit 9a479999 authored by Alan Kwong's avatar Alan Kwong Committed by Steve Cohen
Browse files

msm: sde: update rotator OT to use total pixels 



Currently, OT calculation compares resolution and frame
rate separately, which may select inappropriate OT
according to overall pixels. Update rotator OT
calculation to combine resolution and frame rate into
total pixel before comparison. Also, apply the same
same OT calculation for RGB formats as well.

Change-Id: Id7e4db53c55b5af61ca45ade88ecd3318f6eef15
Signed-off-by: default avatarAlan Kwong <akwong@codeaurora.org>
parent 9fe06237

drivers/media/platform/msm/sde/rotator/sde_rotator_base.c

+9 −8
Original line number Diff line number Diff line
@@ -182,7 +182,7 @@ u32 sde_mdp_get_ot_limit(u32 width, u32 height, u32 pixfmt, u32 fps, u32 is_rd) 
	struct sde_mdp_format_params *fmt;

	u32 ot_lim;

	u32 is_yuv;

	u32 res;

	u64 res;



	ot_lim = (is_rd) ? mdata->default_ot_rd_limit :

				mdata->default_ot_wr_limit;
@@ -198,7 +198,11 @@ u32 sde_mdp_get_ot_limit(u32 width, u32 height, u32 pixfmt, u32 fps, u32 is_rd) 
	if (false == test_bit(SDE_QOS_OTLIM, mdata->sde_qos_map))

		goto exit;



	width = min_t(u32, width, SDE_ROT_MAX_IMG_WIDTH);

	height = min_t(u32, height, SDE_ROT_MAX_IMG_HEIGHT);



	res = width * height;

	res = res * fps;



	fmt = sde_get_format_params(pixfmt);
@@ -209,17 +213,14 @@ u32 sde_mdp_get_ot_limit(u32 width, u32 height, u32 pixfmt, u32 fps, u32 is_rd) 


	is_yuv = sde_mdp_is_yuv_format(fmt);



	SDEROT_DBG("w:%d h:%d fps:%d pixfmt:%8.8x yuv:%d res:%d rd:%d\n",

	SDEROT_DBG("w:%d h:%d fps:%d pixfmt:%8.8x yuv:%d res:%llu rd:%d\n",

		width, height, fps, pixfmt, is_yuv, res, is_rd);



	if (!is_yuv)

		goto exit;



	if ((res <= RES_1080p) && (fps <= 30))

	if (res <= (RES_1080p * 30))

		ot_lim = 2;

	else if ((res <= RES_1080p) && (fps <= 60))

	else if (res <= (RES_1080p * 60))

		ot_lim = 4;

	else if ((res <= RES_UHD) && (fps <= 30))

	else if (res <= (RES_UHD * 30))

		ot_lim = 8;



exit: