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

Commit f1eb926d authored by Ricardo Ribalda Delgado's avatar Ricardo Ribalda Delgado Committed by Mauro Carvalho Chehab
Browse files

[media] vivid: Local optimization 



Avoid duplicated clamps when possible.

Suggested-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: default avatarRicardo Ribalda Delgado <ricardo.ribalda@gmail.com>
Acked-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent 7a20f398

drivers/media/common/v4l2-tpg/v4l2-tpg-core.c

+10 −6
Original line number Original line Diff line number Diff line
@@ -916,14 +916,18 @@ static void precalculate_color(struct tpg_data *tpg, int k) 
		if (!ycbcr_valid)

		if (!ycbcr_valid)

			color_to_ycbcr(tpg, r, g, b, &y, &cb, &cr);

			color_to_ycbcr(tpg, r, g, b, &y, &cb, &cr);





		y >>= 4;

		cb >>= 4;

		cr >>= 4;

		if (tpg->real_quantization == V4L2_QUANTIZATION_LIM_RANGE) {

		if (tpg->real_quantization == V4L2_QUANTIZATION_LIM_RANGE) {

			y = clamp(y, 16 << 4, 235 << 4);

			y = clamp(y, 16, 235);

			cb = clamp(cb, 16 << 4, 240 << 4);

			cb = clamp(cb, 16, 240);

			cr = clamp(cr, 16 << 4, 240 << 4);

			cr = clamp(cr, 16, 240);

		} else {

			y = clamp(y, 1, 254);

			cb = clamp(cb, 1, 254);

			cr = clamp(cr, 1, 254);

		}

		}

		y = clamp(y >> 4, 1, 254);

		cb = clamp(cb >> 4, 1, 254);

		cr = clamp(cr >> 4, 1, 254);

		switch (tpg->fourcc) {

		switch (tpg->fourcc) {

		case V4L2_PIX_FMT_YUV444:

		case V4L2_PIX_FMT_YUV444:

			y >>= 4;

			y >>= 4;