diff mbox series

[libdrm] util: Fix grey in YUV SMPTE patterns

Message ID d6baedf436195f6cfd6d3e81ddea5219e09d861d.1657554831.git.geert@linux-m68k.org (mailing list archive)
State New, archived
Headers show
Series [libdrm] util: Fix grey in YUV SMPTE patterns | expand

Commit Message

Geert Uytterhoeven July 11, 2022, 3:54 p.m. UTC
The YUV SMPTE patterns use RGB 191/192/192 instead of 192/192/192 for
the grey color in the top color bar.

Change it to 192/192/192, to match the RGB SMPTE patterns.

Fixes: a94ee624292bff96 ("modetest: Add SMPTE test pattern")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
 tests/util/pattern.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Laurent Pinchart July 11, 2022, 4:25 p.m. UTC | #1
Hi Geert,

Thank you for the patch.

On Mon, Jul 11, 2022 at 05:54:46PM +0200, Geert Uytterhoeven wrote:
> The YUV SMPTE patterns use RGB 191/192/192 instead of 192/192/192 for
> the grey color in the top color bar.
> 
> Change it to 192/192/192, to match the RGB SMPTE patterns.
> 
> Fixes: a94ee624292bff96 ("modetest: Add SMPTE test pattern")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  tests/util/pattern.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/util/pattern.c b/tests/util/pattern.c
> index 158c0b160a2ee870..f28fad311ec3de11 100644
> --- a/tests/util/pattern.c
> +++ b/tests/util/pattern.c
> @@ -162,7 +162,7 @@ static void fill_smpte_yuv_planar(const struct util_yuv_info *yuv,
>  				  unsigned int height, unsigned int stride)
>  {
>  	const struct color_yuv colors_top[] = {
> -		MAKE_YUV_601(191, 192, 192),	/* grey */
> +		MAKE_YUV_601(192, 192, 192),	/* grey */
>  		MAKE_YUV_601(192, 192, 0),	/* yellow */
>  		MAKE_YUV_601(0, 192, 192),	/* cyan */
>  		MAKE_YUV_601(0, 192, 0),	/* green */
> @@ -265,7 +265,7 @@ static void fill_smpte_yuv_packed(const struct util_yuv_info *yuv, void *mem,
>  				  unsigned int stride)
>  {
>  	const struct color_yuv colors_top[] = {
> -		MAKE_YUV_601(191, 192, 192),	/* grey */
> +		MAKE_YUV_601(192, 192, 192),	/* grey */
>  		MAKE_YUV_601(192, 192, 0),	/* yellow */
>  		MAKE_YUV_601(0, 192, 192),	/* cyan */
>  		MAKE_YUV_601(0, 192, 0),	/* green */
diff mbox series

Patch

diff --git a/tests/util/pattern.c b/tests/util/pattern.c
index 158c0b160a2ee870..f28fad311ec3de11 100644
--- a/tests/util/pattern.c
+++ b/tests/util/pattern.c
@@ -162,7 +162,7 @@  static void fill_smpte_yuv_planar(const struct util_yuv_info *yuv,
 				  unsigned int height, unsigned int stride)
 {
 	const struct color_yuv colors_top[] = {
-		MAKE_YUV_601(191, 192, 192),	/* grey */
+		MAKE_YUV_601(192, 192, 192),	/* grey */
 		MAKE_YUV_601(192, 192, 0),	/* yellow */
 		MAKE_YUV_601(0, 192, 192),	/* cyan */
 		MAKE_YUV_601(0, 192, 0),	/* green */
@@ -265,7 +265,7 @@  static void fill_smpte_yuv_packed(const struct util_yuv_info *yuv, void *mem,
 				  unsigned int stride)
 {
 	const struct color_yuv colors_top[] = {
-		MAKE_YUV_601(191, 192, 192),	/* grey */
+		MAKE_YUV_601(192, 192, 192),	/* grey */
 		MAKE_YUV_601(192, 192, 0),	/* yellow */
 		MAKE_YUV_601(0, 192, 192),	/* cyan */
 		MAKE_YUV_601(0, 192, 0),	/* green */