Hi Tom,
On 08/26/2015 09:21 PM, Tom Rini wrote:
> Otherwise we get:
> tools/atmelimage.c:134:3: warning: format ‘%d’ expects argument of type
> ‘int’, but argument 2 has type ‘size_t’ [-Wformat=]
> debug("atmelimage: interrupt vector #%d is 0x%08X\n", pos+1,
> ^
>
> Signed-off-by: Tom Rini <tr...@konsulko.com>
> ---
> tools/atmelimage.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/atmelimage.c b/tools/atmelimage.c
> index 5b72ac5..0979eef 100644
> --- a/tools/atmelimage.c
> +++ b/tools/atmelimage.c
> @@ -131,7 +131,7 @@ static int atmel_verify_header(unsigned char *ptr, int
> image_size,
>
> /* check the seven interrupt vectors of binary */
> for (pos = 0; pos < 7; pos++) {
> - debug("atmelimage: interrupt vector #%d is 0x%08X\n", pos+1,
> + debug("atmelimage: interrupt vector #%ld is 0x%08X\n", pos+1,
I'd rather use the '%zu' modifier here. On 32 Bit systems the size_t
might be a simple int and not a long int which also leads to warnings.
Andreas
> ints[pos]);
> /*
> * all vectors except the 6'th one must contain valid
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
