diff options
| author | Jerome Forissier <jerome.forissier@linaro.org> | 2015-10-23 17:30:26 +0200 |
|---|---|---|
| committer | Jerome Forissier <jerome.forissier@linaro.org> | 2015-10-27 13:25:27 +0100 |
| commit | dffb004951dfff24858d6d4d09a834fa57687b94 (patch) | |
| tree | e8179fe6b9686f47d4aa89be538aaf718cd20358 /core/core.mk | |
| parent | 92ea2867c506ef27a56706cdfed36d3fb33c7024 (diff) | |
plat-*/conf.mk cleanup
- Do not set CFG_ values that do not change the default
- Remove a few useless -D<flag>
- Always use '?=' as opposed to ':=' for consistent behavior with
'CFG_FOO=y make' and 'make CFG_FOO=y'
=> Use this form when both enabled and disabled are valid options
- Add macro: $(call force,CFG_FOO,y) to set a variable and make
sure it does not conflict with external values that may be given
on the command line or in the environment
=> Use this form when the variable can only have the specified value
(any other value would be invalid). 'override CFG_FOO := y' would
be correct, too, but would not detect conflicting values.
- Always include mk/conf.mk last so that the platform definitions
always take precedence over the global configuration
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Reviewed-by: Pascal Brand <pascal.brand@linaro.org>
Diffstat (limited to 'core/core.mk')
| -rw-r--r-- | core/core.mk | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/core.mk b/core/core.mk index 01fecd6d..1326000f 100644 --- a/core/core.mk +++ b/core/core.mk @@ -6,6 +6,7 @@ sm-$(sm) := y arch-dir := core/arch/$(ARCH) platform-dir := $(arch-dir)/plat-$(PLATFORM) +include mk/checkconf.mk include $(platform-dir)/conf.mk include core/arch/$(ARCH)/$(ARCH).mk @@ -39,7 +40,6 @@ $(conf-file): $(conf-mk-file) cleanfiles += $(conf-file) cleanfiles += $(conf-mk-file) -include mk/checkconf.mk $(conf-file): FORCE $(call check-conf-h) |