[coreboot-gerrit] Patch merged into coreboot/master: rules.h: Add ENV_ macros to detect current architecture
gerrit at coreboot.org
gerrit at coreboot.org
Tue Nov 17 21:31:09 CET 2015
the following patch was just integrated into master:
commit d3634c108d63d07ce004a66e3abb05e8da57d65b
Author: Julius Werner <jwerner at chromium.org>
Date: Fri Nov 13 13:28:41 2015 -0800
rules.h: Add ENV_ macros to detect current architecture
This patch expands the existing ENV_<stage> macros in <rules.h> with a
set of ENV_<arch> macros which can be used to detect which architecture
the current compilation unit is built for. These are more consistent
than compiler-defined macros (like '#ifdef __arm__') and will make it
easier to write small, architecture-dependent differences in common code
(where we currently often use IS_ENABLED(CONFIG_ARCH_...), which is
technically incorrect in a world where every stage can run on a
different architecture, and merely kinda happened to work out for now).
Also remove a vestigal <arch/rules.h> from ARM64 which was no longer
used, and genericise ARM subarchitecture Makefiles a little to make
things like __COREBOOT_ARM_ARCH__ available from all file types
(including .ld).
BUG=None
TEST=Compiled Falco, Blaze, Jerry and Smaug.
Change-Id: Id51aeb290b5c215c653e42a51919d0838e28621f
Signed-off-by: Julius Werner <jwerner at chromium.org>
Reviewed-on: http://review.coreboot.org/12433
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin at chromium.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
Tested-by: Raptor Engineering Automated Test Stand <noreply at raptorengineeringinc.com>
See http://review.coreboot.org/12433 for details.
-gerrit
More information about the coreboot-gerrit
mailing list