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

Commit 85a5f85e authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab
Browse files

[media] v4l2-tpg: ignore V4L2_DV_RGB_RANGE setting for YUV formats 



The V4L2_DV_RGB_RANGE_* settings are, as the name says, for RGB formats only.
So they should be ignored for non-RGB formats.

Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent c1023ba7

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

+2 −2
Original line number Original line Diff line number Diff line
@@ -777,7 +777,7 @@ static void precalculate_color(struct tpg_data *tpg, int k) 
	 * Remember that r, g and b are still in the 0 - 0xff0 range.

	 * Remember that r, g and b are still in the 0 - 0xff0 range.

	 */

	 */

	if (tpg->real_rgb_range == V4L2_DV_RGB_RANGE_LIMITED &&

	if (tpg->real_rgb_range == V4L2_DV_RGB_RANGE_LIMITED &&

	    tpg->rgb_range == V4L2_DV_RGB_RANGE_FULL) {

	    tpg->rgb_range == V4L2_DV_RGB_RANGE_FULL && !tpg->is_yuv) {

		/*

		/*

		 * Convert from full range (which is what r, g and b are)

		 * Convert from full range (which is what r, g and b are)

		 * to limited range (which is the 'real' RGB range), which

		 * to limited range (which is the 'real' RGB range), which
@@ -787,7 +787,7 @@ static void precalculate_color(struct tpg_data *tpg, int k) 
		g = (g * 219) / 255 + (16 << 4);

		g = (g * 219) / 255 + (16 << 4);

		b = (b * 219) / 255 + (16 << 4);

		b = (b * 219) / 255 + (16 << 4);

	} else if (tpg->real_rgb_range != V4L2_DV_RGB_RANGE_LIMITED &&

	} else if (tpg->real_rgb_range != V4L2_DV_RGB_RANGE_LIMITED &&

		   tpg->rgb_range == V4L2_DV_RGB_RANGE_LIMITED) {

		   tpg->rgb_range == V4L2_DV_RGB_RANGE_LIMITED && !tpg->is_yuv) {

		/*

		/*

		 * Clamp r, g and b to the limited range and convert to full

		 * Clamp r, g and b to the limited range and convert to full

		 * range since that's what we deliver.

		 * range since that's what we deliver.