[PATCH] ARM: use "depends on" for SoC configs instead of "if" after prompt

Gregory CLEMENT gregory.clement at free-electrons.com
Mon Nov 23 10:10:04 PST 2015


Hi Masahiro,
 
 On lun., nov. 16 2015, Masahiro Yamada <yamada.masahiro at socionext.com> wrote:

> Many ARM sub-architectures use prompts followed by "if" conditional,
> but it is wrong.
>
> Please notice the difference between
>
>     config ARCH_FOO
>             bool "Foo SoCs" if ARCH_MULTI_V7
>
> and
>
>     config ARCH_FOO
>             bool "Foo SoCs"
>             depends on ARCH_MULTI_V7
>
> These two are *not* equivalent!
>
> In the former statement, it is not ARCH_FOO, but its prompt that
> depends on ARCH_MULTI_V7.  So, it is completely valid that ARCH_FOO
> is selected by another, but ARCH_MULTI_V7 is still disabled. As it is
> not unmet dependency, Kconfig never warns.  This is probably not what
> you want.
>
> The former should be used only when you need to do so, and you really
> understand what you are doing.  (In most cases, it should be wrong!)
>
> For enabling/disabling sub-architectures, the latter is always correct.
>
> As a good side effect, this commit fixes some entries over 80 columns
> (mach-imx, mach-integrator, mach-mbevu).
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
> ---
>
> I hope this patch is applied to ARM-SOC, but am CCing Kbuild ML
> because the correct understanding of Kconfig is required for this patch.
>

[...]
>  arch/arm/mach-mvebu/Kconfig      | 24 ++++++++++++++++--------
For mvebu part
Acked-by: Gregory CLEMENT <gregory.clement at free-electrons.com>

Thanks,

Gregory
-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com



More information about the Linux-mediatek mailing list