[U-Boot] [PATCH 1/2] tools/socfpgaimage.c: fix build on darwin

Pavel Machek pavel at denx.de
Sat Oct 25 11:25:51 CEST 2014


On Fri 2014-10-24 23:40:13, Marek Vasut wrote:
> On Friday, October 24, 2014 at 11:39:10 PM, andreas.devel at googlemail.com wrote:
> > From: Andreas Bießmann <andreas.devel at googlemail.com>
> > 
> > socfpgaimage utilizes htole32 and friends, unfortunately these functions
> > are not available on darwin. Fix it by using the cpu_to_le32 and friends
> > defined in compiler.h as other parts in mkimage do.
> > 
> > This patch fixes the following error:
> > ---8<---
> >   HOSTCC  tools/socfpgaimage.o
> > tools/socfpgaimage.c:77:22: warning: implicit declaration of function
> > 'htole32' is invalid in C99 [-Wimplicit-function-declaration]
> > header.validation = htole32(VALIDATION_WORD);
> >                             ^
> > tools/socfpgaimage.c:80:22: warning: implicit declaration of function
> > 'htole16' is invalid in C99 [-Wimplicit-function-declaration]
> > header.length_u32 = htole16(length_bytes/4);
> >                             ^
> > tools/socfpgaimage.c:95:6: warning: implicit declaration of function
> > 'le32toh' is invalid in C99 [-Wimplicit-function-declaration] if
> > (le32toh(header.validation) != VALIDATION_WORD)
> >             ^
> > tools/socfpgaimage.c:97:6: warning: implicit declaration of function
> > 'le16toh' is invalid in C99 [-Wimplicit-function-declaration] if
> > (le16toh(header.checksum) != hdr_checksum(&header))
> >             ^
> > 4 warnings generated.
> > ...
> >   HOSTLD  tools/dumpimage
> > Undefined symbols for architecture x86_64:
> >   "_htole16", referenced from:
> >       _socfpgaimage_set_header in socfpgaimage.o
> >   "_htole32", referenced from:
> >       _socfpgaimage_set_header in socfpgaimage.o
> >   "_le16toh", referenced from:
> >       _verify_buffer in socfpgaimage.o
> >   "_le32toh", referenced from:
> >       _verify_buffer in socfpgaimage.o
> > ld: symbol(s) not found for architecture x86_64
> > --->8---
> > 
> > Signed-off-by: Andreas Bießmann <andreas.devel at googlemail.com>
> 
> Acked-by: Marek Vasut <marex at denx.de>

Acked-by: Pavel Machek <pavel at denx.de>

Marek, do you want to take it to your tree? Tom was not cc-ed on the
mail, so I'm not sure if you he'll apply it...
								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html


More information about the U-Boot mailing list