Dirk Behme wrote:
Scott Wood wrote:
Dirk Behme wrote:
What I still miss: If this warning indicates that something *is* wrong, why not run MAKEALL and enable it in config for all boards that show this warning? What's the advantage of having all maintainers sending patches for their boards?
- That requires having all toolchains installed, and
Hmm? In nand_util.c we have
#if !defined(CONFIG_SYS_64BIT_VSPRINTF) #warning Please define CONFIG_SYS_64BIT_VSPRINTF for correct output! #endif
and in Makefile we have
ifdef CONFIG_CMD_NAND ifndef CONFIG_NAND_LEGACY ... COBJS-y += nand_util.o endif
So I would assume that this warning appears for all boards having CONFIG_CMD_NAND enabled and no CONFIG_NAND_LEGACY and no CONFIG_SYS_64BIT_VSPRINTF? Independent of tool chain?
I don't mean different versions, I mean toolchains that cover all architectures. If I don't have a superh or blackfin compiler installed, I'm not going to see any warnings on those boards because they won't build at all.
I agree though that it would be nice to grep for existing NAND users and try to cover the obvious ones.
-Scott