[U-Boot] [PATCH v4 11/15] dm: Fix printf() strings in the 'dm' command

Andre Renaud andre at bluewatersys.com
Mon Jun 9 00:46:58 CEST 2014


On 7 June 2014 07:13, Simon Glass <sjg at chromium.org> wrote:
> The values here are int, but the map_to_sysmem() call can return a long.
> Add a cast to deal with this.
...
> -       printf("%s- %s @ %08x", buf, in->name, map_to_sysmem(in));
> +       printf("%s- %s @ %08x", buf, in->name, (uint)map_to_sysmem(in));

If the argument is a long, shouldn't it be printed as long, rather
than possibly truncated to an int?
 printf("%s- %s @ %08lx", buf, in->name, map_to_sysmem(in));

Regards,
Andre


More information about the U-Boot mailing list