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

Tom Rini trini at konsulko.com
Mon Dec 7 14:25:11 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>
---
 common/board_r.c  |    1 +
 include/command.h |    2 --
 test/dm/cmd_dm.c  |    1 +
 3 files changed, 2 insertions(+), 2 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..07ae525 100644
--- a/include/command.h
+++ b/include/command.h
@@ -179,8 +179,6 @@ 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