mirror of
https://github.com/u-boot/u-boot.git
synced 2024-11-25 13:14:19 +08:00
097ff01fb9
At present if an optional Kconfig value needs to be used it must be
bracketed by #ifdef. For example, with this Kconfig setup:
config WIBBLE
bool "Support wibbles, the world needs more wibbles"
config WIBBLE_ADDR
hex "Address of the wibble"
depends on WIBBLE
then the following code must be used:
#ifdef CONFIG_WIBBLE
static void handle_wibble(void)
{
int val = CONFIG_WIBBLE_ADDR;
...
}
#endif
static void init_machine()
{
...
#ifdef CONFIG_WIBBLE
handle_wibble();
#endif
}
Add a new IF_ENABLED_INT() to help with this. So now it is possible to
write, without #ifdefs:
static void handle_wibble(void)
{
int val = IF_ENABLED_INT(CONFIG_WIBBLE, CONFIG_WIBBLE_ADDR);
...
}
static void init_machine()
{
...
if (IS_ENABLED(CONFIG_WIBBLE))
handle_wibble();
}
The value will be CONFIG_WIBBLE_ADDR if CONFIG_WIBBLE is defined and will
produce a build error if not.. This allows us to reduce the use of #ifdef
in the code, ensuring that the compiler still checks the code even if it
is not ultimately used for a particular build.
Add a CONFIG_IF_ENABLED_INT() version as well.
If an attempt is made to use a value that does not exist (i.e. when the
conditional is not enabled), an error about a non-existing function is
generated, e.g.:
common/bloblist.c:447: undefined reference to `invalid_use_of_IF_ENABLED_INT'
Signed-off-by: Simon Glass <sjg@chromium.org>
147 lines
4.5 KiB
Makefile
147 lines
4.5 KiB
Makefile
# SPDX-License-Identifier: GPL-2.0
|
|
# This helper makefile is used for creating
|
|
# - symbolic links (arch/$ARCH/include/asm/arch
|
|
# - include/autoconf.mk, {spl,tpl}/include/autoconf.mk
|
|
# - include/config.h
|
|
#
|
|
# When our migration to Kconfig is done
|
|
# (= When we move all CONFIGs from header files to Kconfig)
|
|
# this makefile can be deleted.
|
|
|
|
__all: include/autoconf.mk include/autoconf.mk.dep
|
|
|
|
ifeq ($(shell grep -q '^CONFIG_SPL=y' include/config/auto.conf 2>/dev/null && echo y),y)
|
|
__all: spl/include/autoconf.mk
|
|
endif
|
|
|
|
ifeq ($(shell grep -q '^CONFIG_TPL=y' include/config/auto.conf 2>/dev/null && echo y),y)
|
|
__all: tpl/include/autoconf.mk
|
|
endif
|
|
|
|
include include/config/auto.conf
|
|
|
|
include scripts/Kbuild.include
|
|
|
|
# Need to define CC and CPP again here in case the top Makefile did not
|
|
# include config.mk. Some architectures expect CROSS_COMPILE to be defined
|
|
# in arch/$(ARCH)/config.mk
|
|
CC = $(CROSS_COMPILE)gcc
|
|
CPP = $(CC) -E
|
|
|
|
include config.mk
|
|
|
|
UBOOTINCLUDE := \
|
|
-Iinclude \
|
|
$(if $(KBUILD_SRC), -I$(srctree)/include) \
|
|
-I$(srctree)/arch/$(ARCH)/include \
|
|
-include $(srctree)/include/linux/kconfig.h
|
|
|
|
c_flags := $(KBUILD_CFLAGS) $(KBUILD_CPPFLAGS) $(PLATFORM_CPPFLAGS) \
|
|
$(UBOOTINCLUDE) $(NOSTDINC_FLAGS)
|
|
|
|
quiet_cmd_autoconf_dep = GEN $@
|
|
cmd_autoconf_dep = $(CC) -x c -DDO_DEPS_ONLY -M -MP $(c_flags) \
|
|
-MQ include/config/auto.conf $(srctree)/include/common.h > $@ || { \
|
|
rm $@; false; \
|
|
}
|
|
include/autoconf.mk.dep: include/config.h FORCE
|
|
$(call cmd,autoconf_dep)
|
|
|
|
# We are migrating from board headers to Kconfig little by little.
|
|
# In the interim, we use both of
|
|
# - include/config/auto.conf (generated by Kconfig)
|
|
# - include/autoconf.mk (used in the U-Boot conventional configuration)
|
|
# The following rule creates autoconf.mk
|
|
# include/config/auto.conf is grepped in order to avoid duplication of the
|
|
# same CONFIG macros
|
|
quiet_cmd_autoconf = GEN $@
|
|
cmd_autoconf = \
|
|
sed -n -f $(srctree)/tools/scripts/define2mk.sed $< | \
|
|
while read line; do \
|
|
if [ -n "${KCONFIG_IGNORE_DUPLICATES}" ] || \
|
|
! grep -q "$${line%=*}=" include/config/auto.conf; then \
|
|
echo "$$line"; \
|
|
fi; \
|
|
done > $@
|
|
|
|
quiet_cmd_u_boot_cfg = CFG $@
|
|
cmd_u_boot_cfg = \
|
|
$(CPP) $(c_flags) $2 -DDO_DEPS_ONLY -dM $(srctree)/include/common.h > $@.tmp && { \
|
|
grep 'define CONFIG_' $@.tmp | \
|
|
sed '/define CONFIG_IS_ENABLED(/d;/define CONFIG_IF_ENABLED_INT(/d;/define CONFIG_VAL(/d;' > $@; \
|
|
rm $@.tmp; \
|
|
} || { \
|
|
rm $@.tmp; false; \
|
|
}
|
|
|
|
u-boot.cfg: include/config.h FORCE
|
|
$(call cmd,u_boot_cfg)
|
|
|
|
spl/u-boot.cfg: include/config.h FORCE
|
|
$(Q)mkdir -p $(dir $@)
|
|
$(call cmd,u_boot_cfg,-DCONFIG_SPL_BUILD)
|
|
|
|
tpl/u-boot.cfg: include/config.h FORCE
|
|
$(Q)mkdir -p $(dir $@)
|
|
$(call cmd,u_boot_cfg,-DCONFIG_SPL_BUILD -DCONFIG_TPL_BUILD)
|
|
|
|
include/autoconf.mk: u-boot.cfg
|
|
$(call cmd,autoconf)
|
|
|
|
spl/include/autoconf.mk: spl/u-boot.cfg
|
|
$(Q)mkdir -p $(dir $@)
|
|
$(call cmd,autoconf)
|
|
|
|
tpl/include/autoconf.mk: tpl/u-boot.cfg
|
|
$(Q)mkdir -p $(dir $@)
|
|
$(call cmd,autoconf)
|
|
|
|
# include/config.h
|
|
# Prior to Kconfig, it was generated by mkconfig. Now it is created here.
|
|
define filechk_config_h
|
|
(echo "/* Automatically generated - do not edit */"; \
|
|
for i in $$(echo $(CONFIG_SYS_EXTRA_OPTIONS) | sed 's/,/ /g'); do \
|
|
echo \#define CONFIG_$$i \
|
|
| sed '/=/ {s/=/ /;q; } ; { s/$$/ 1/; }'; \
|
|
done; \
|
|
echo \#define CONFIG_BOARDDIR board/$(if $(VENDOR),$(VENDOR)/)$(BOARD);\
|
|
echo \#include \<config_uncmd_spl.h\>; \
|
|
echo \#include \<configs/$(CONFIG_SYS_CONFIG_NAME).h\>; \
|
|
echo \#include \<asm/config.h\>; \
|
|
echo \#include \<linux/kconfig.h\>; \
|
|
echo \#include \<config_fallbacks.h\>;)
|
|
endef
|
|
|
|
include/config.h: scripts/Makefile.autoconf create_symlink FORCE
|
|
$(call filechk,config_h)
|
|
|
|
# symbolic links
|
|
# If arch/$(ARCH)/mach-$(SOC)/include/mach exists,
|
|
# make a symbolic link to that directory.
|
|
# Otherwise, create a symbolic link to arch/$(ARCH)/include/asm/arch-$(SOC).
|
|
PHONY += create_symlink
|
|
create_symlink:
|
|
ifdef CONFIG_CREATE_ARCH_SYMLINK
|
|
ifneq ($(KBUILD_SRC),)
|
|
$(Q)mkdir -p include/asm
|
|
$(Q)if [ -d $(KBUILD_SRC)/arch/$(ARCH)/mach-$(SOC)/include/mach ]; then \
|
|
dest=arch/$(ARCH)/mach-$(SOC)/include/mach; \
|
|
else \
|
|
dest=arch/$(ARCH)/include/asm/arch-$(if $(SOC),$(SOC),$(CPU)); \
|
|
fi; \
|
|
ln -fsn $(KBUILD_SRC)/$$dest include/asm/arch
|
|
else
|
|
$(Q)if [ -d arch/$(ARCH)/mach-$(SOC)/include/mach ]; then \
|
|
dest=../../mach-$(SOC)/include/mach; \
|
|
else \
|
|
dest=arch-$(if $(SOC),$(SOC),$(CPU)); \
|
|
fi; \
|
|
ln -fsn $$dest arch/$(ARCH)/include/asm/arch
|
|
endif
|
|
endif
|
|
|
|
PHONY += FORCE
|
|
FORCE:
|
|
|
|
.PHONY: $(PHONY)
|