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

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@synopsys.com wrote: From: Alexey Brodkin Alexey.Brodkin@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