Hi Masahiro,
On 22 June 2017 at 01:50, Masahiro Yamada yamada.masahiro@socionext.com wrote:
Fix sparse warnings "... was not declared. Should it be static?"
Also, fix redefinition of dm_warn/dm_dbg.
Signed-off-by: Masahiro Yamada yamada.masahiro@socionext.com
drivers/core/util.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/drivers/core/util.c b/drivers/core/util.c index 5ceac8bbb15b..2e232d57a14f 100644 --- a/drivers/core/util.c +++ b/drivers/core/util.c @@ -5,9 +5,11 @@ */
#include <common.h> +#include <dm/util.h> #include <libfdt.h> #include <vsprintf.h>
+#ifdef CONFIG_DM_WARN void dm_warn(const char *fmt, ...) { va_list args; @@ -16,7 +18,9 @@ void dm_warn(const char *fmt, ...) vprintf(fmt, args); va_end(args); } +#endif
+#ifdef DEBUG
I don't think you can add this #ifdef, since DEBUG can be defined for any one, not necessarily this one. If someone adds "#define DEBUG' to (say) llists.c then this will cause a compile error.
void dm_dbg(const char *fmt, ...) { va_list args; @@ -25,6 +29,7 @@ void dm_dbg(const char *fmt, ...) vprintf(fmt, args); va_end(args); } +#endif
int list_count_items(struct list_head *head) { -- 2.7.4
Regards, Simon