[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