[U-Boot] [PATCH] delete Kbuild "select" of long-dead SPL_DISABLE_OF_CONTROL

Masahiro Yamada yamada.masahiro at socionext.com
Mon Apr 15 02:32:09 UTC 2019


On Sun, Apr 14, 2019 at 7:21 PM Robert P. J. Day <rpjday at crashcourse.ca> wrote:
>
>
> From way back in 2015:
>
>   commit dffb86e468c8e02ba77283989aefef214d904dc5
>   Author: Masahiro Yamada <yamada.masahiro at socionext.com>
>   Date:   Wed Aug 12 07:31:54 2015 +0900
>
>     of: flip CONFIG_SPL_DISABLE_OF_CONTROL into CONFIG_SPL_OF_CONTROL
>
>     As we discussed a couple of times, negative CONFIG options make our
>     life difficult; CONFIG_SYS_NO_FLASH, CONFIG_SYS_DCACHE_OFF, ...
>     and here is another one.
>
>     Now, there are three boards enabling OF_CONTROL on SPL:
>      - socfpga_arria5_defconfig
>      - socfpga_cyclone5_defconfig
>      - socfpga_socrates_defconfig
>
>     This commit adds CONFIG_SPL_OF_CONTROL for them and deletes
>     CONFIG_SPL_DISABLE_OF_CONTROL from the other boards to invert
>     the logic.
>
>     Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>     Reviewed-by: Tom Rini <trini at konsulko.com>
>     Reviewed-by: Simon Glass <sjg at chromium.org>


Thanks for catching this.

Reviewed-by: Masahiro Yamada <yamada.masahiro at socionext.com>



> ---
>
>   AFAICT, simple deletion should be sufficient but i'm willing to be
> convinced otherwise.
>
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index 3807770362..14347e7c7d 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -116,7 +116,6 @@ config TARGET_SNOW
>  config TARGET_SPRING
>         bool "Spring board"
>         select OF_CONTROL
> -       select SPL_DISABLE_OF_CONTROL
>         select SUPPORT_SPL
>
>  config TARGET_SMDK5420
> @@ -150,7 +149,6 @@ config  TARGET_ESPRESSO7420
>         select OF_CONTROL
>         select PINCTRL
>         select PINCTRL_EXYNOS7420
> -       select SPL_DISABLE_OF_CONTROL
>         select SUPPORT_SPL
>
>  endchoice
>
> --
>
> ========================================================================
> Robert P. J. Day                                 Ottawa, Ontario, CANADA
>                          http://crashcourse.ca
>
> Twitter:                                       http://twitter.com/rpjday
> LinkedIn:                               http://ca.linkedin.com/in/rpjday
> ========================================================================
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot



-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list