[PATCH] leds: add exit function to cobalt-raq led.

[Thiago Farina-2]

Signed-off-by: Thiago Farina <tfransosi@...>
---
 drivers/leds/leds-cobalt-raq.c |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/drivers/leds/leds-cobalt-raq.c b/drivers/leds/leds-cobalt-raq.c
index defc212..e43fcb2 100644
--- a/drivers/leds/leds-cobalt-raq.c
+++ b/drivers/leds/leds-cobalt-raq.c
@@ -135,4 +135,15 @@ static int __init cobalt_raq_led_init(void)
  return platform_driver_register(&cobalt_raq_led_driver);
 }
 
+static void __exit cobalt_raq_led_exit(void)
+{
+ platform_driver_unregister(&cobalt_raq_led_driver);
+}
+
 module_init(cobalt_raq_led_init);
+module_exit(cobalt_raq_led_exit);
+
+MODULE_AUTHOR("Yoichi Yuasa <yuasa@...>");
+MODULE_DESCRIPTION("LEDs driver for the Cobalt Raq series.");
+MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:cobalt-raq-leds");
--
1.6.5.2.150.g1b52a

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Ralf Baechle DL5RB]

On Sat, Nov 07, 2009 at 12:50:35PM -0500, Thiago Farina wrote:

This driver has no exit function because it cannot be built as a module.
I've not reviewed the driver in detail but at the very least the
LEDS_COBALT_RAQ Kconfig statement for the driver would need to be changed
to tristate before this patch makes any sense.

  Ralf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Thiago Farina-2]

On Sat, Nov 7, 2009 at 9:45 PM, Ralf Baechle <ralf@...> wrote:
> On Sat, Nov 07, 2009 at 12:50:35PM -0500, Thiago Farina wrote:
>
> This driver has no exit function because it cannot be built as a module.
> I've not reviewed the driver in detail but at the very least the
> LEDS_COBALT_RAQ Kconfig statement for the driver would need to be changed
> to tristate before this patch makes any sense.

According to this thread:
"Therefore, this driver is not tristate."
http://lkml.org/lkml/2007/9/13/377
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/