Minor build fixes.

I made these two tiny changes to SeaBIOS when I was playing with the
HEAD version.

The Makefile change prevents the test command from failing if CC
contains a space.

The coreboot change prevents a warning on our gcc 4.5.2 in certain
circumstances when sizeof seems to be unsigned long int instead of
unsigned int.

They're both unintrusive, shouldn't have potential for breaking
anything.

Signed-off-by: Stefan Reinauer <stepan@coreboot.org>

diff --git a/Makefile b/Makefile
index 1543e0d7b39df354b82d7bafa0d8e3fef1e7a7e2..9805f857c47be4d458989fc4136b45db8a8a810b 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -67,7 +67,7 @@ vpath %.S src vgasrc
 ################ Build rules
 
 # Verify the gcc configuration and test if -fwhole-program works.
-TESTGCC:=$(shell CC=$(CC) tools/test-gcc.sh)
+TESTGCC:=$(shell CC="$(CC)" tools/test-gcc.sh)
 ifeq "$(TESTGCC)" "-1"
 $(error "Please upgrade GCC")
 endif

diff --git a/src/coreboot.c b/src/coreboot.c
index db0063beecaa51fce8a1fcfbc0c0e7eb367f17ed..554ea814565a383cdcd9f54b91de69c0e4a788c0 100644 (file)
--- a/src/coreboot.c
+++ b/src/coreboot.c
@@ -326,7 +326,7 @@ ulzma(u8 *dst, u32 maxlen, const u8 *src, u32 srclen)
     u8 scratch[15980];
     int need = (LzmaGetNumProbs(&state.Properties) * sizeof(CProb));
     if (need > sizeof(scratch)) {
-        dprintf(1, "LzmaDecode need %d have %d\n", need, sizeof(scratch));
+        dprintf(1, "LzmaDecode need %d have %d\n", need, (unsigned int)sizeof(scratch));
         return -1;
     }
     state.Probs = (CProb *)scratch;