From 2d6ee92c6af06eeb7f7410180d5faa8f5e840bbb Mon Sep 17 00:00:00 2001
From: Simon Glass <sjg@chromium.org>
Date: Thu, 1 Jun 2023 10:22:48 -0600
Subject: [PATCH] video: Use enum with video_index_to_colour()
Use the provided enum with this function, so it is clearer what should be
passed to it.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
drivers/video/video-uclass.c | 2 +-
include/video.h | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/video/video-uclass.c b/drivers/video/video-uclass.c
index d304e92c24..95e874b770 100644
--- a/drivers/video/video-uclass.c
+++ b/drivers/video/video-uclass.c
@@ -260,7 +260,7 @@ static const struct vid_rgb colours[VID_COLOUR_COUNT] = {
{ 0xff, 0xff, 0xff }, /* white */
};
-u32 video_index_to_colour(struct video_priv *priv, unsigned int idx)
+u32 video_index_to_colour(struct video_priv *priv, enum colour_idx idx)
{
switch (priv->bpix) {
case VIDEO_BPP16:
diff --git a/include/video.h b/include/video.h
index 55e9f32f59..e98d0f9c89 100644
--- a/include/video.h
+++ b/include/video.h
@@ -164,10 +164,10 @@ enum colour_idx {
* VID_COLOR_COUNT.
*
* @priv private data of the video device (UCLASS_VIDEO)
- * @idx color index
+ * @idx color index (e.g. VID_YELLOW)
* Return: color value
*/
-u32 video_index_to_colour(struct video_priv *priv, unsigned int idx);
+u32 video_index_to_colour(struct video_priv *priv, enum colour_idx idx);
/**
* video_reserve() - Reserve frame-buffer memory for video devices
--
2.39.5