[U-Boot] [PATCH v3 6/7] kconfig: remove unneeded dependency on !SPL_BUILD

Simon Glass
2015-02-20T17:06:42Z

+Stephen

Hi Masahiro,

On 19 February 2015 at 22:25, Masahiro Yamada <yamada.m@jp.panasonic.com> wrote:
> Now CONFIG_SPL_BUILD is not defined in Kconfig, so
> "!depends on SPL_BUILD" and "if !SPL_BUILD" are redundant.
>
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> ---
>
> Changes in v3: None
> Changes in v2:
> - Fix a typo in commit description
>
> Kconfig | 12 ++----------
> arch/arm/Kconfig | 7 +++----
> arch/arm/cpu/arm1176/bcm2835/Kconfig | 6 +++---
> arch/arm/cpu/armv7/exynos/Kconfig | 26 +++++++++++++-------------
> arch/arm/cpu/armv7/omap3/Kconfig | 8 ++++----
> arch/arm/cpu/armv7/s5pc1xx/Kconfig | 4 ++--
> arch/arm/cpu/armv7/tegra-common/Kconfig | 12 ++++++------
> arch/arm/cpu/armv7/uniphier/Kconfig | 1 -
> board/compulab/cm_t335/Kconfig | 6 +++---
> board/gumstix/pepper/Kconfig | 6 +++---
> board/isee/igep0033/Kconfig | 6 +++---
> board/phytec/pcm051/Kconfig | 6 +++---
> board/samsung/goni/Kconfig | 6 +++---
> board/samsung/smdkc100/Kconfig | 6 +++---
> board/silica/pengwyn/Kconfig | 6 +++---
> board/ti/am335x/Kconfig | 8 ++++----
> common/Kconfig | 1 -
> drivers/core/Kconfig | 16 ++++++----------
> drivers/mtd/nand/Kconfig | 4 ----
> dts/Kconfig | 1 -
> include/config_uncmd_spl.h | 4 ++++
> scripts/Makefile.uncmd_spl | 4 +++-
> 22 files changed, 71 insertions(+), 85 deletions(-)
>
[snip]
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 41f3220..700e2a8 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -739,9 +739,8 @@ config TEGRA
> bool "NVIDIA Tegra"
> select SUPPORT_SPL
> select SPL
> - select OF_CONTROL if !SPL_BUILD
> - select CPU_ARM720T if SPL_BUILD
> - select CPU_V7 if !SPL_BUILD
> + select OF_CONTROL
> + select CPU_V7

Sorry if I have missed something here. On Tegra most unfortunately the
SPL uses ARMv4t and U-Boot proper uses ARMv7. In fact that is the only
reason that Tegra has SPL. Doesn't this change with this commit?

Regards,
Simon