[RESEND PATCH v3 07/26] video: Drop unnecessary #ifdef around vid_console_color()

Simon Glass sjg at chromium.org
Fri Jul 3 05:12:18 CEST 2020


All of the functions in this file only apply if DM_VIDEO is enabled. Drop
the #ifdef as it just clutters things up. Add the needed forward
declaration.

Signed-off-by: Simon Glass <sjg at chromium.org>
Reviewed-by: Anatolij Gustschin <agust at denx.de>
---

(no changes since v1)

 include/video_console.h | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/include/video_console.h b/include/video_console.h
index 0936ceaaf1..d3bc063165 100644
--- a/include/video_console.h
+++ b/include/video_console.h
@@ -8,6 +8,8 @@
 
 #include <video.h>
 
+struct video_priv;
+
 #define VID_FRAC_DIV	256
 
 #define VID_TO_PIXEL(x)	((x) / VID_FRAC_DIV)
@@ -241,8 +243,6 @@ int vidconsole_put_string(struct udevice *dev, const char *str);
 void vidconsole_position_cursor(struct udevice *dev, unsigned col,
 				unsigned row);
 
-#ifdef CONFIG_DM_VIDEO
-
 /**
  * vid_console_color() - convert a color code to a pixel's internal
  * representation
@@ -257,5 +257,3 @@ void vidconsole_position_cursor(struct udevice *dev, unsigned col,
 u32 vid_console_color(struct video_priv *priv, unsigned int idx);
 
 #endif
-
-#endif
-- 
2.27.0.212.ge8ba1cc988-goog



More information about the U-Boot mailing list