[RFC] [LEDS] reorganise Kconfig

[RFC] [LEDS] reorganise Kconfig

[Ben Dooks]

[-- Attachment #1: Type: text/plain, Size: 248 bytes --]

This patch reorganises the drivers/leds Kconfig
file to have the LED trigger enable with the
triggers themselves.

The patch also adds comments to divide up the
sections into the drivers and triggers

Signed-off-by: Ben Dooks <ben-linux@fluff.org>

[-- Attachment #2: 2616-leds-kconfig-reorg.patch --]
[-- Type: text/plain, Size: 1344 bytes --]

diff -urpN -X ../dontdiff linux-2.6.16-git20-bjd3/drivers/leds/Kconfig linux-2.6.16-git20-bjd4/drivers/leds/Kconfig
--- linux-2.6.16-git20-bjd3/drivers/leds/Kconfig	2006-04-01 14:49:22.000000000 +0100
+++ linux-2.6.16-git20-bjd4/drivers/leds/Kconfig	2006-04-01 16:37:11.000000000 +0100
@@ -14,13 +14,7 @@ config LEDS_CLASS
 	  This option enables the led sysfs class in /sys/class/leds.  You'll
 	  need this to do anything useful with LEDs.  If unsure, say N.
 
-config LEDS_TRIGGERS
-	bool "LED Trigger support"
-	depends NEW_LEDS
-	help
-	  This option enables trigger support for the leds class.
-	  These triggers allow kernel events to drive the LEDs and can
-	  be configured via sysfs. If unsure, say Y.
+comment "LED drivers"
 
 config LEDS_CORGI
 	tristate "LED Support for the Sharp SL-C7x0 series"
@@ -66,6 +60,16 @@ config LEDS_S3C24XX
 	  This option enables support for LEDs connected to GPIO lines
 	  on Samsung S3C24XX series CPUs, such as the S3C2410 and S3C2440.
 
+comment "LED Triggers"
+
+config LEDS_TRIGGERS
+	bool "LED Trigger support"
+	depends NEW_LEDS
+	help
+	  This option enables trigger support for the leds class.
+	  These triggers allow kernel events to drive the LEDs and can
+	  be configured via sysfs. If unsure, say Y.
+
 config LEDS_TRIGGER_TIMER
 	tristate "LED Timer Trigger"
 	depends LEDS_TRIGGERS

Re: [RFC] [LEDS] reorganise Kconfig

[Richard Purdie]

On Sat, 2006-04-01 at 16:42 +0100, Ben Dooks wrote:
> This patch reorganises the drivers/leds Kconfig
> file to have the LED trigger enable with the
> triggers themselves.
> 
> The patch also adds comments to divide up the
> sections into the drivers and triggers
> 
> Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Acked-by: Richard Purdie <rpurdie@rpsys.net>