When I posted the RFC version of Kconfig series, I defined CONFIG_SYS_ARCH, CONFIG_SYS_CPU, CONFIG_SYS_SOC, CONFIG_SYS_VENDOR, CONFIG_SYS_BOARD in Kconfig.
For example, configs/harmony_defconfig was like this: CONFIG_SPL=y CONFIG_ARM=y CONFIG_SYS_CPU="armv7" CONFIG_SOC_DIR=y CONFIG_SYS_SOC="tegra20" CONFIG_SYS_BOARD="harmony" CONFIG_VENDOR_DIR=y CONFIG_SYS_VENDOR="nvidia" CONFIG_SYS_CONFIG_NAME="harmony" CONFIG_BOARD_MAINTAINER="Tom Warren twarren@nvidia.com"
But the most reviewers involved did not like that. (See http://patchwork.ozlabs.org/patch/330915/)
Instead, Daniel Schwierzeck suggested to select a board by using boolean macro: something like this, CONFIG_BOARD_HARMONY, CONFIG_VENDOR_NVIDIA, ...
I agree this is a good idea. But string macros are used everywhere in U-Boot.
$(CPU), $(SOC), $(VENDOR), $(BOARD) here and there in makefiles and CONFIG_SYS_{ARCH, CPU, SOC, VENDOR, BOARD} in C source files.
I took an action little by little to reduce the number of such variables in makefiles For example, these patches: http://patchwork.ozlabs.org/patch/335202/ http://patchwork.ozlabs.org/patch/335201/
I want to do something with CONFIG_SYS_{ARCH,...} in this series.
Masahiro Yamada (3): env: drop CONFIG_ENV_VARS_UBOOT_CONFIG support cmd_pxe: remove SoC, ARCH path from pxe_default_path mkconfig: Do not define CONFIG_SYS_{ARCH,CPU,SOC,VENDOR,BOARD} in config.h.
README | 14 -------------- arch/arm/include/asm/arch-tegra114/tegra.h | 1 + arch/arm/include/asm/arch-tegra124/tegra.h | 1 + arch/arm/include/asm/arch-tegra20/tegra.h | 1 + arch/arm/include/asm/arch-tegra30/tegra.h | 1 + arch/blackfin/lib/Makefile | 3 --- common/cmd_pxe.c | 4 ---- include/configs/am335x_igep0033.h | 1 - include/configs/apf27.h | 1 - include/configs/exynos4-dt.h | 2 +- include/configs/omap4_panda.h | 1 + include/configs/pcm051.h | 1 - include/configs/rpi_b.h | 1 - include/configs/s5p_goni.h | 1 - include/configs/s5pc210_universal.h | 3 ++- include/configs/siemens-am33x-common.h | 1 - include/configs/tegra-common.h | 1 - include/configs/ti814x_evm.h | 1 - include/configs/ti_armv7_common.h | 1 - include/configs/trats.h | 2 +- include/configs/trats2.h | 3 ++- include/env_default.h | 12 ------------ mkconfig | 8 -------- 23 files changed, 11 insertions(+), 54 deletions(-)