summaryrefslogtreecommitdiff
path: root/Makefile.toolchain
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@google.com>2017-06-27 15:26:58 +0200
committerchrome-bot <chrome-bot@chromium.org>2017-07-19 05:01:02 -0700
commit1a67ac2ad6a6c0eedaa672f7856e38fb5abf1ede (patch)
tree1668e251f28b4820e89f7e96c5de68740932e232 /Makefile.toolchain
parentef73893a4b73e9181382530dd98dd28b1de52c38 (diff)
downloadchrome-ec-1a67ac2ad6a6c0eedaa672f7856e38fb5abf1ede.tar.gz
Allow core code to request its cross compiler
Instead of having to pass in both the board and the arch used there (which might be complicated in the light of different ECs with different arches), allow passing in the board and a set of cross compilers. The core/*/build.mk then pick the compiler that is responsible for them. The current method works just the same: If you've already set CROSS_COMPILE, no override happens. If you set neither CROSS_COMPILE nor CROSS_COMPILE_$arch, the same default as before this CL is set. BUG=none BRANCH=none TEST=emerge-{samus,kevin} chromeos-ec behave reasonably with adapted ebuild Change-Id: Icf1866f296412dd92ecfe134394224c49f7c3df5 Signed-off-by: Patrick Georgi <pgeorgi@google.com> Reviewed-on: https://chromium-review.googlesource.com/549344 Commit-Ready: Patrick Georgi <pgeorgi@chromium.org> Tested-by: Patrick Georgi <pgeorgi@chromium.org> Reviewed-by: Stefan Reinauer <reinauer@google.com>
Diffstat (limited to 'Makefile.toolchain')
-rw-r--r--Makefile.toolchain6
1 files changed, 6 insertions, 0 deletions
diff --git a/Makefile.toolchain b/Makefile.toolchain
index 8bbac56dcd..1d1b458e3e 100644
--- a/Makefile.toolchain
+++ b/Makefile.toolchain
@@ -88,3 +88,9 @@ LDFLAGS=-nostdlib -Wl,-X -Wl,--gc-sections -Wl,--build-id=none $(LDFLAGS_EXTRA)
BUILD_LDFLAGS=$(LIBFTDI_LDLIBS)
HOST_TEST_LDFLAGS=-T core/host/host_exe.lds -lrt -pthread -rdynamic -lm\
$(if $(TEST_COVERAGE),-fprofile-arcs,)
+
+# utility function to provide overridable defaults
+# $1: name of variable to set
+# $2: first default to use
+# $3: second default to use
+set-option = $(eval $1?=$$(if $(2),$(2),$(3)))