Andrew Klossner wrote: > The compiler will help find mismatches between printf formats and > arguments if you let it. This patch adds the necessary attributes to > declarations in include/common.h, then corrects the resulting compiler > warnings in several files. Some of these were bugs, e.g., "$d" > instead of "%d" and incorrect arguments in common/cmd_fdt.c. Others > were just annoying, like int-long mismatches on a system where both > are 32 bits. It's worth fixing the annoying errors to catch the real > ones. > > I can only compile for ppc architecture so have not tested much of the > source code for further warnings. > > common/cmd_fdt.c
cmd_fdt.c portion... Acked-by: Gerald Van Baren <[EMAIL PROTECTED]> I'm embarrassed by the number of bugs I wrote into that function and apologize for being so sloppy. My thanks to Andrew and others for finding and fixing my screwups. :-( Best regards, gvb ------------------------------------------------------------------------- Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy diet, reduces your potential for chronic lameness and boredom. Vote Now at http://www.sourceforge.net/community/cca08 _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
