[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