On Jul 14, 2008, at 10:09 AM, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 09:42 Mon 14 Jul , Kumar Gala wrote: >> >> Signed-off-by: Kumar Gala <[EMAIL PROTECTED]> >> --- >> common/cmd_mp.c | 2 +- >> cpu/mpc85xx/mp.c | 4 ++-- >> 2 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/common/cmd_mp.c b/common/cmd_mp.c >> index 26a57c5..59d0d15 100644 >> --- a/common/cmd_mp.c >> +++ b/common/cmd_mp.c >> @@ -35,7 +35,7 @@ cpu_cmd(cmd_tbl_t *cmdtp, int flag, int argc, >> char *argv[]) >> >> cpuid = simple_strtoul(argv[1], NULL, 10); >> if (cpuid >= CONFIG_NR_CPUS) { >> - printf ("Core num: %d is out of range[0..%d]\n", >> + printf ("Core num: %ld is out of range[0..%d]\n", > is it not %lu? cpuid is a unsigned long.
probably. > >> cpuid, CONFIG_NR_CPUS - 1); >> return 1; >> } >> diff --git a/cpu/mpc85xx/mp.c b/cpu/mpc85xx/mp.c >> index a527cf3..75ceba9 100644 >> --- a/cpu/mpc85xx/mp.c >> +++ b/cpu/mpc85xx/mp.c >> @@ -50,12 +50,12 @@ int cpu_status(int nr) >> >> if (nr == id) { >> table = (u32 *)get_spin_addr(); >> - printf("table base @ 0x%08x\n", table); >> + printf("table base @ 0x%08x\n", (u32)table); > why not use %p? it's a pointer. do we support %p in u-boot? - k ------------------------------------------------------------------------- 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