[PATCH 03/20] display_options.h: Correct includes

Tom Rini trini at konsulko.com
Wed Nov 1 17:28:07 CET 2023


First, a header should never include itself so remove that. Second, this
header needs <linux/types.h> to be included as the function prototypes
use types that we get via that header.

Signed-off-by: Tom Rini <trini at konsulko.com>
---
 include/display_options.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/display_options.h b/include/display_options.h
index 85dacbc75902..66e596077370 100644
--- a/include/display_options.h
+++ b/include/display_options.h
@@ -9,6 +9,8 @@
 #ifndef __DISPLAY_OPTIONS_H
 #define __DISPLAY_OPTIONS_H
 
+#include <linux/types.h>
+
 /**
  * print_size() - Print a size with a suffix
  *
@@ -19,7 +21,6 @@
  * @size:	Size to print
  * @suffix	String to print after the size
  */
-#include <display_options.h>
 void print_size(uint64_t size, const char *suffix);
 
 /**
-- 
2.34.1



More information about the U-Boot mailing list