[U-Boot] [PATCH v2] common: fix compiler warning on wrong printf format

Alexey Brodkin Alexey.Brodkin at synopsys.com
Thu Aug 28 16:33:59 CEST 2014


Hi Vasili,

On Thu, 2014-08-28 at 17:17 +0300, Vasili Galka wrote:
> On Thu, Aug 28, 2014 at 4:15 PM, Alexey Brodkin
> <Alexey.Brodkin at synopsys.com> wrote:
>         From: Alexey Brodkin <Alexey.Brodkin at synopsys.com>
>         
>         bounce_buffer->len_aligned is of type "size_t" so use "%zd".
>         
>         this fixes following warning:
>         --->---
>         common/bouncebuf.c: In function "addr_aligned":
>         common/bouncebuf.c:26:3: warning: format "%d" expects argument
>         of type "int", but argument 2 has type "size_t" [-Wformat=]
>            debug("Unaligned buffer length %d\n", /*(int)*/state->len);
>            ^
>         --->---
> 
> Hmm, why not just use my original patch which included exactly this?
> http://patchwork.ozlabs.org/patch/383041/
> 

Well, I'm not following all the messages sent to this mailing list so
didn't know about your patch.

So as soon as I observed a problem I created a patch and sent it.

I see your patch fixes more things so definitely it should be applied
instead.

Sorry for noise.

-Alexey


More information about the U-Boot mailing list