[coreboot] [PATCH] fix CONFIG_FS_PAYLOAD=1

Marc Jones Marc.Jones at amd.com
Tue Nov 4 00:19:16 CET 2008


Jens Rottmann wrote:
> Fix compile errors if CONFIG_FS_PAYLOAD=1:
> 
> Compile error in filo.c if AUTOBOOT_DELAY=0. Replace
> #ifndef AUTOBOOT_DELAY
> with
> #if !AUTOBOOT_DELAY
> which should work for both the #undef and the =0 case.
> 
> In ext2fs.c, fat.c
> #if ARCH == 'i386'
> results in a compile warning: "multi-character character constant" and
> the condition ARCH == 'i386' is mis-evaluated as FALSE, eventually
> choking the assembler on a PPC instruction. Change it to
> #ifdef __i386
> 
> Signed-off-by: Jens Rottmann <JRottmann at LiPPERTEmbedded.de>

It looks like you are the firs/only x86 platform to use FS_PAYLOAD.
Acked-by: Marc Jones <marc.jones at amd.com>

And Stefan just beat me to the check in so I will stop now.....

:)

Marc



-- 
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors





More information about the coreboot mailing list