[U-Boot] [PATCH] CONFIG_NEEDS_MANUAL_RELOC: Fix warnings when not set

Tom Rini trini at konsulko.com
Mon Dec 7 14:35:15 CET 2015


Now that we may compile (but not link) code calling fixup_cmdtable when
this is not set, we need to always have the declaration available.  We
should also make sure that anyone calling the function includes
<command.h> as that's where the function declaration is.

Signed-off-by: Tom Rini <trini at konsulko.com>

---
Changes in v2:
- Move declaration up to be under the current __ASSEMBLY__ test.
---
 common/board_r.c  |    1 +
 include/command.h |    5 +----
 test/dm/cmd_dm.c  |    1 +
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/common/board_r.c b/common/board_r.c
index 5af32dd..75ee43e 100644
--- a/common/board_r.c
+++ b/common/board_r.c
@@ -15,6 +15,7 @@
 #if defined(CONFIG_CMD_BEDBUG)
 #include <bedbug/type.h>
 #endif
+#include <command.h>
 #include <console.h>
 #ifdef CONFIG_HAS_DATAFLASH
 #include <dataflash.h>
diff --git a/include/command.h b/include/command.h
index 6c04cd9..2ae9b6c 100644
--- a/include/command.h
+++ b/include/command.h
@@ -142,6 +142,7 @@ enum command_ret_t {
 int cmd_process(int flag, int argc, char * const argv[],
 			       int *repeatable, unsigned long *ticks);
 
+void fixup_cmdtable(cmd_tbl_t *cmdtp, int size);
 #endif	/* __ASSEMBLY__ */
 
 /*
@@ -179,8 +180,4 @@ int cmd_process(int flag, int argc, char * const argv[],
 #define U_BOOT_CMD(_name, _maxargs, _rep, _cmd, _usage, _help)		\
 	U_BOOT_CMD_COMPLETE(_name, _maxargs, _rep, _cmd, _usage, _help, NULL)
 
-#if defined(CONFIG_NEEDS_MANUAL_RELOC)
-void fixup_cmdtable(cmd_tbl_t *cmdtp, int size);
-#endif
-
 #endif	/* __COMMAND_H */
diff --git a/test/dm/cmd_dm.c b/test/dm/cmd_dm.c
index b6e7109..a3c5971 100644
--- a/test/dm/cmd_dm.c
+++ b/test/dm/cmd_dm.c
@@ -8,6 +8,7 @@
  */
 
 #include <common.h>
+#include <command.h>
 #include <dm.h>
 #include <malloc.h>
 #include <mapmem.h>
-- 
1.7.9.5



More information about the U-Boot mailing list