[coreboot-gerrit] Patch merged into coreboot/master: 1ab2027 Intel: Add common header file for CAR setup
gerrit at coreboot.org
gerrit at coreboot.org
Thu Aug 14 15:12:28 CEST 2014
the following patch was just integrated into master:
commit 1ab202795532cdce155e42738d84e77a3e77ceb2
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date: Mon May 26 17:38:23 2014 +1000
Intel: Add common header file for CAR setup
When passing '-ffreestanding' the 'main' romstage.c may no longer
necessarily be considered the entry point.
From the C specification in 5.1.2.1 Freestanding environment;
"In a freestanding environment (in which C program execution may take
place without any benefit of an operating system), the name and type of
the function called at program startup are implementation-defined."
Clang complains about these being missing as Clang is somewhat more
strict about the spec than GNU/GCC is. An advantage here is that a
different entry-point type-signature shall now be warned about at
compile time.
Change-Id: I467001adabd47958c30c9a15e3248e42ed1151f3
Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Reviewed-on: http://review.coreboot.org/5872
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick at georgi-clan.de>
See http://review.coreboot.org/5872 for details.
-gerrit
More information about the coreboot-gerrit
mailing list