Skip to contentSkip to conversation

leds: led-bd2802: bd2802_unregister_led_classdev should unregister all registered leds

PATCH RESEND In review Axel Lin 2 participants 2 messages Aug 23, 2011 – Aug 24, 2011 LED

lore
Share & tools
lore

Axel Lin

author
#0 lore raw

PATCH RESEND 1/1 leds: led-bd2802: bd2802_unregister_led_classdev should unregister all registered leds

bd2802_unregister_led_classdev() should unregister all registered instances of led_classdev class that had registered by bd2802_register_led_classdev().

Signed-off-by Axel Lin <[email protected]>
---
 drivers/leds/leds-bd2802.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
drivers/leds/leds-bd2802.c +5 -0
index 3ebe382..ea21855 100644
@@ -662,6 +662,11 @@ failed_unregister_led1_R:
 static void bd2802_unregister_led_classdev(struct bd2802_led *led)
 {
 	cancel_work_sync(&led->work);
+	led_classdev_unregister(&led->cdev_led2b);
+	led_classdev_unregister(&led->cdev_led2g);
+	led_classdev_unregister(&led->cdev_led2r);
+	led_classdev_unregister(&led->cdev_led1b);
+	led_classdev_unregister(&led->cdev_led1g);
 	led_classdev_unregister(&led->cdev_led1r);
 }
Signature · 1 line
1.7.4.1
Acked-by on v1 1/1
#1 lore raw
Acked-by Kim Kyuwon <[email protected]>

-----Original Message-----
From: Axel Lin [mailto:[email protected]]
Sent: Wednesday, August 24, 2011 8:32 AM
To: [email protected]
Cc: Kim Kyuwon; Richard Purdie; Andrew Morton
Subject: [PATCH RESEND] leds: led-bd2802: bd2802_unregister_led_classdev should unregister all registered leds

bd2802_unregister_led_classdev() should unregister all registered instances of led_classdev class that had registered by bd2802_register_led_classdev().

Signed-off-by Axel Lin <[email protected]>
---
 drivers/leds/leds-bd2802.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)
drivers/leds/leds-bd2802.c +5 -0
index 3ebe382..ea21855 100644
@@ -662,6 +662,11 @@ failed_unregister_led1_R:
 static void bd2802_unregister_led_classdev(struct bd2802_led *led)
 {
 	cancel_work_sync(&led->work);
+	led_classdev_unregister(&led->cdev_led2b);
+	led_classdev_unregister(&led->cdev_led2g);
+	led_classdev_unregister(&led->cdev_led2r);
+	led_classdev_unregister(&led->cdev_led1b);
+	led_classdev_unregister(&led->cdev_led1g);
 	led_classdev_unregister(&led->cdev_led1r);
 }
Signature · 1 line
1.7.4.1