[U-Boot] [RFC PATCH 0/3] Remove CONFIG_SYS_{ARCH, CPU, SOC, VENDOR, BOARD}
Masahiro Yamada
yamada.m at jp.panasonic.com
Tue Apr 22 11:43:43 CEST 2014
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 at 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(-)
--
1.8.3.2
More information about the U-Boot
mailing list