[U-Boot] [PATCH MAKEALL coldfire] fix tools/kwbimage.c format warning

Wolfgang Denk wd at denx.de
Mon Sep 21 22:53:13 CEST 2009


Dear Philippe De Muyter,

In message <20090921202802.GB12703 at frolo.macqel> you wrote:
>
> > >  	if (len % sizeof(uint32_t)) {
> > > -		printf ("Error:%s[%d] - lenght is not in multiple of %ld\n",
> > > +		printf ("Error:%s[%d] - length is not in multiple of %u\n",
> > >  			__FUNCTION__, len, sizeof(uint32_t));
> > 
> > Seems you are using old code. This has long been fixed (using a %zu
> > format).
> 
> That's surprising.  I got a tarball last friday (3 days ago) using the
> snapshot button on the gitweb interface :
> 
> 	u-boot-3dc5e00454a58499d4f4c790b38036b0e873747e.tar.gz

You need this commit:

commit ceb2d57c2205db5bbd868577f756c74a2568160c
Author: Wolfgang Denk <wd at denx.de>
Date:   Tue Sep 15 21:13:27 2009 +0200

    kwbimage.c: Fix compile warning when building on 64 bit systems (again)
    
    Commit 51003b89 attempted to fix a build problem on 64 bit systems,
    but just turned it into a build problem on 32 bit systems (silly me).
    
    Now do the Right Thing (TM) and use a "%zu" printf format.
    
    Also fix spelling error.
    
    Signed-off-by: Wolfgang Denk <wd at denx.de>

It was merged into the master branch on 2009-09-18 23:20:12.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
What is mind?  No matter.  What is matter?  Never mind.
                                      -- Thomas Hewitt Key, 1799-1875


More information about the U-Boot mailing list