Re: [PATCH] nds32: add a Kconfig symbol for LOCKDEP_SUPPORT

From: Randy Dunlap
Date: Thu Feb 17 2022 - 14:01:42 EST


Add Alan and KC.

On 5/16/21 20:13, Randy Dunlap wrote:
> Each architecture (arch/) should define its own LOCKDEP_SUPPORT
> Kconfig symbol (if it is needed). arch/nds32/ is the only one
> that does "select LOCKDEP_SUPPORT", which is basically a no-op
> since the symbol is not defined. (It might be nice for kconfig
> to warn about that.)
>
> Add a "config LOCKDEP_SUPPORT" to arch/nds32/Kconfig to correct
> this issue.
>
> Since the current Kconfig file selects LOCKDEP_SUPPORT unconditionally,
> use "def_bool y" to set/enable it always.
>
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Cc: Nick Hu <nickhu@xxxxxxxxxxxxx>
> Cc: Greentime Hu <green.hu@xxxxxxxxx>
> Cc: Vincent Chen <deanbo422@xxxxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> ---
> arch/nds32/Kconfig | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> --- linux-next-20210514.orig/arch/nds32/Kconfig
> +++ linux-next-20210514/arch/nds32/Kconfig
> @@ -36,7 +36,6 @@ config NDS32
> select HAVE_REGS_AND_STACK_ACCESS_API
> select HAVE_PERF_EVENTS
> select IRQ_DOMAIN
> - select LOCKDEP_SUPPORT
> select MODULES_USE_ELF_RELA
> select OF
> select OF_EARLY_FLATTREE
> @@ -64,6 +63,9 @@ config GENERIC_LOCKBREAK
> def_bool y
> depends on PREEMPTION
>
> +config LOCKDEP_SUPPORT
> + def_bool y
> +
> config TRACE_IRQFLAGS_SUPPORT
> def_bool y
>

Anybody out there for Andes?

--
~Randy