Hi Simon,
2017-07-06 13:49 GMT+09:00 Simon Glass sjg@chromium.org:
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.
You are right, (but you picked up this)
So, what shall we do?
Drop this patch from u-boot-dm, and respin it?
For example, we can use #ifdef CONFIG_DM_DEBUG instead of #ifdef DEBUG.