[U-Boot] [PATCH v2 14/15] of: flip CONFIG_SPL_DISABLE_OF_CONTROL into CONFIG_SPL_OF_CONTROL

Simon Glass sjg at chromium.org
Mon Aug 3 16:06:46 CEST 2015


Hi Masahiro,

On 1 August 2015 at 07:14, Masahiro Yamada
<yamada.masahiro at socionext.com> wrote:
> 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 actually two boards enabling OF_CONTROL on SPL:
>  - socfpga_arria5_defconfig
>  - socfpga_cyclone5_defconfig
>
> But they were enabled by mistake.

Yes it looks like they converted in between me generating the patch
and applying it. I did not notice. This underlines that we should
avoid negative options.

In any case this option only exists to avoid breaking existing boards
and I was hoping to drop it. My only concern here is that the option
(for a baord to use device tree in U-Boot proper but not in SPL) will
now live on. Still, for a few boards this will be necessary.

>
> Commit 47a785a9dd97 ("dts: Disable device tree for SPL on all
> boards") missed to add CONFIG_SPL_DISABLE_OF_CONTROL on those
> two boards.  They were silently enabled with OF_CONTROL on SPL
> without notifying their maintainers.
>
> After all, there exist no boards to support it in a justified
> process.
>
> Before being too late, delete all the defines in defconfig files
> and 'select's in Kconfig, and invert the logic.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
> Changes in v2:
>   - Rebase on commit cc35734358540a1
>
>  arch/arm/Kconfig                         | 3 ---
>  arch/arm/cpu/armv7/exynos/Kconfig        | 8 --------
>  arch/arm/cpu/armv7/s5pc1xx/Kconfig       | 2 --
>  arch/arm/mach-tegra/Kconfig              | 1 -
>  common/spl/spl.c                         | 2 +-
>  configs/am335x_boneblack_vboot_defconfig | 1 -
>  configs/arches_defconfig                 | 1 -
>  configs/canyonlands_defconfig            | 1 -
>  configs/galileo_defconfig                | 1 -
>  configs/microblaze-generic_defconfig     | 1 -
>  configs/odroid_defconfig                 | 1 -
>  configs/origen_defconfig                 | 1 -
>  configs/s5pc210_universal_defconfig      | 1 -
>  configs/socfpga_socrates_defconfig       | 1 -
>  configs/trats2_defconfig                 | 1 -
>  configs/trats_defconfig                  | 1 -
>  dts/Kconfig                              | 6 +++---
>  include/config_uncmd_spl.h               | 2 +-
>  include/fdtdec.h                         | 2 +-
>  lib/Makefile                             | 8 +++++++-
>  scripts/Makefile.uncmd_spl               | 2 +-
>  21 files changed, 14 insertions(+), 33 deletions(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list