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

[Masahiro Yamada]

Hi Marek,


2015-08-12 7:47 GMT+09:00 Marek Vasut <marex at denx.de>:
> On Wednesday, August 12, 2015 at 12:31:54 AM, Masahiro Yamada wrote:
>
> Hi!
>
>> 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>
>
> Wouldn't it make more sense for the SOCFPGA stuff to simply select
> CONFIG_OF_CONTROL and CONFIG_SPL_OF_CONTROL in arch/arm/mach-socfpga/Kconfig
> by default instead of adding it into each boards' config file ?
>

It will make sense if SOCFPGA only works with
CONFIG_OF_CONTROL and CONFIG_SPL_OF_CONTROL enabled.

It was not clear if they were forcible or not
because CONFIG_OF_CONTROL was defined in the defconfig, so I am following it.


-- 
Best Regards
Masahiro Yamada