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

Simon Glass sjg at chromium.org
Fri Feb 20 18:06:42 CET 2015


+Stephen

Hi Masahiro,

On 19 February 2015 at 22:25, Masahiro Yamada <yamada.m at 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 at 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


More information about the U-Boot mailing list