Re: [PATCH] leds: fix the menuconfig being wrongly displayed

From: Valdis . Kletnieks
Date: Thu Jun 02 2011 - 08:57:30 EST


On Thu, 02 Jun 2011 13:09:14 +0800, Eric Miao said:
> From: Eric Miao <eric.miao@xxxxxxxxxxxxx>
>
> Seems when a config option does not have a dependency of the menuconfig,
> it messes the display of the rest configs, even if it's a hidden one.
>
> Signed-off-by: Eric Miao <eric.y.miao@xxxxxxxxx>
> Cc: Richard Purdie <rpurdie@xxxxxxxxx>
> ---
> drivers/leds/Kconfig | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig
> index 23f0d5e..7066820 100644
> --- a/drivers/leds/Kconfig
> +++ b/drivers/leds/Kconfig
> @@ -7,6 +7,8 @@ menuconfig NEW_LEDS
> This is not related to standard keyboard LEDs which are controlled
> via the input system.
>
> +if NEW_LEDS
> +
> config LEDS_CLASS
> bool "LED Class Support"
> depends on NEW_LEDS

Having added that 'if', do we still need that 'depends'?

Attachment: pgp00000.pgp
Description: PGP signature