Re: [PATCH] Use CFLAGS for compiling bin2hex

2008-07-15 Thread Simon Horman
On Tue, Jul 15, 2008 at 10:48:03AM +0200, Bernhard Walle wrote: * Jeremy Kerr [2008-07-15 11:41]: $(BIN_TO_HEX): CC=$(BUILD_CC) -$(BIN_TO_HEX): CFLAGS=$(BUILD_CFLAGS) +$(BIN_TO_HEX): CFLAGS+=$(BUILD_CFLAGS) $(BIN_TO_HEX): LDFLAGS= Are you sure? CFLAGS will include stuff

Re: [PATCH] Use CFLAGS for compiling bin2hex

2008-07-15 Thread Bernhard Walle
* Simon Horman [2008-07-15 20:28]: On Tue, Jul 15, 2008 at 10:48:03AM +0200, Bernhard Walle wrote: * Jeremy Kerr [2008-07-15 11:41]: $(BIN_TO_HEX): CC=$(BUILD_CC) -$(BIN_TO_HEX): CFLAGS=$(BUILD_CFLAGS) +$(BIN_TO_HEX): CFLAGS+=$(BUILD_CFLAGS) $(BIN_TO_HEX): LDFLAGS=

Re: [PATCH] Use CFLAGS for compiling bin2hex

2008-07-15 Thread Jeremy Kerr
BUILD_CFLAGS=... ./configure didn't also work. That would be acceptable for me. OK, patch coming. However, note that the BUILD_* is only used to compile tools to run on the build machine, and hence during the build process. There's not much point adding optimisation flags here, as it's

[PATCH] Use CFLAGS for compiling bin2hex

2008-07-14 Thread Bernhard Walle
This is a minor correction in the Makefile to use the CFLAGS (from configure) also for bin2hex. Signed-off-by: Bernhard Walle [EMAIL PROTECTED] --- util/Makefile |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/util/Makefile b/util/Makefile index 11d82fd..a1a78ac

Re: [PATCH] Use CFLAGS for compiling bin2hex

2008-07-14 Thread Simon Horman
On Mon, Jul 14, 2008 at 04:33:32PM +0200, Bernhard Walle wrote: This is a minor correction in the Makefile to use the CFLAGS (from configure) also for bin2hex. Thanks, applied. -- Horms ___ kexec mailing list kexec@lists.infradead.org

Re: [PATCH] Use CFLAGS for compiling bin2hex

2008-07-14 Thread Jeremy Kerr
$(BIN_TO_HEX): CC=$(BUILD_CC) -$(BIN_TO_HEX): CFLAGS=$(BUILD_CFLAGS) +$(BIN_TO_HEX): CFLAGS+=$(BUILD_CFLAGS) $(BIN_TO_HEX): LDFLAGS= Are you sure? CFLAGS will include stuff for $host (eg, -m64 for a 64-bit host machine). Cheers, Jeremy ___