On mer., sept. 27, 2023 at 08:22, Simon Glass sjg@chromium.org wrote:
This uses ARRAY_SIZE() but does not include the header file which declares it. Fix this, so that command.h can be included without common.h
Signed-off-by: Simon Glass sjg@chromium.org
Reviewed-by: Mattijs Korpershoek mkorpershoek@baylibre.com
Changes in v5:
- Adjust so this builds on azure
boot/bootm.c | 2 +- include/command.h | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/boot/bootm.c b/boot/bootm.c index b1c3afe0a3a..8f96a80d425 100644 --- a/boot/bootm.c +++ b/boot/bootm.c @@ -8,6 +8,7 @@ #include <common.h> #include <bootstage.h> #include <cli.h> +#include <command.h> #include <cpu_func.h> #include <env.h> #include <errno.h> @@ -29,7 +30,6 @@ #include "mkimage.h" #endif
-#include <command.h> #include <bootm.h> #include <image.h>
diff --git a/include/command.h b/include/command.h index ae7bb4a30b0..34ea989b39b 100644 --- a/include/command.h +++ b/include/command.h @@ -25,6 +25,10 @@ #endif
#ifndef __ASSEMBLY__
+/* For ARRAY_SIZE() */ +#include <linux/kernel.h>
/*
- Monitor Command Table
*/
2.42.0.515.g380fc7ccd1-goog