Re: [PATCH] make RUNTIME_TESTS a menuconfig to ease disabling it all

[Randy Dunlap]

On 01/06/18 09:24, Vincent Legoll wrote:
> No need to get into the submenu to disable all related
> config entries.
> 
> This makes it easier to disable all RUNTIME_TESTS config options
> without entering the submenu. It will also enable one to see that
> en/dis-abled state from the outside menu.
> 
> This is only intended to change menuconfig UI, not change
> the config dependencies.
> 
> v2: added "default y" to avoid breaking existing configs
> 
> Signed-off-by: Vincent Legoll <vincent.legoll@xxxxxxxxx>

Acked-and-tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>

Thanks.

> ---
>  lib/Kconfig.debug | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index 9d5b78aad4c5..732f7b09ad6f 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -1627,7 +1627,11 @@ config DMA_API_DEBUG
>  
>  	  If unsure, say N.
>  
> -menu "Runtime Testing"
> +menuconfig RUNTIME_TESTING_MENU
> +	bool "Runtime Testing"
> +	default y
> +
> +if RUNTIME_TESTING_MENU
>  
>  config LKDTM
>  	tristate "Linux Kernel Dump Test Tool Module"
> @@ -1915,7 +1919,7 @@ config TEST_DEBUG_VIRTUAL
>  
>  	  If unsure, say N.
>  
> -endmenu # runtime tests
> +endif # RUNTIME_TESTING_MENU
>  
>  config MEMTEST
>  	bool "Memtest"
> 


-- 
~Randy