leds: lt3593: Remove work queue
authorAndrew Lunn <andrew@lunn.ch>
Thu, 20 Aug 2015 10:50:05 +0000 (12:50 +0200)
committerJacek Anaszewski <j.anaszewski@samsung.com>
Mon, 4 Jan 2016 08:57:34 +0000 (09:57 +0100)
Now the core implements the work queue, remove it from the drivers,
and switch to using brightness_set_blocking op.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Cc: Daniel Mack <daniel@caiaq.de>
Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
drivers/leds/leds-lt3593.c

index 9f41124765cc5503044fc1af4fd5cba4a992d73c..a7ff510cbdd06e113ee5e744ea0c1c9eb8c4618e 100644 (file)
@@ -19,7 +19,6 @@
 #include <linux/kernel.h>
 #include <linux/platform_device.h>
 #include <linux/leds.h>
-#include <linux/workqueue.h>
 #include <linux/delay.h>
 #include <linux/gpio.h>
 #include <linux/slab.h>
 struct lt3593_led_data {
        struct led_classdev cdev;
        unsigned gpio;
-       struct work_struct work;
-       u8 new_level;
 };
 
-static void lt3593_led_work(struct work_struct *work)
+static int lt3593_led_set(struct led_classdev *led_cdev,
+                          enum led_brightness value)
 {
-       int pulses;
        struct lt3593_led_data *led_dat =
-               container_of(work, struct lt3593_led_data, work);
+               container_of(led_cdev, struct lt3593_led_data, cdev);
+       int pulses;
 
        /*
         * The LT3593 resets its internal current level register to the maximum
@@ -47,18 +45,18 @@ static void lt3593_led_work(struct work_struct *work)
         * applied is to the output driver.
         */
 
-       if (led_dat->new_level == 0) {
+       if (value == 0) {
                gpio_set_value_cansleep(led_dat->gpio, 0);
-               return;
+               return 0;
        }
 
-       pulses = 32 - (led_dat->new_level * 32) / 255;
+       pulses = 32 - (value * 32) / 255;
 
        if (pulses == 0) {
                gpio_set_value_cansleep(led_dat->gpio, 0);
                mdelay(1);
                gpio_set_value_cansleep(led_dat->gpio, 1);
-               return;
+               return 0;
        }
 
        gpio_set_value_cansleep(led_dat->gpio, 1);
@@ -69,16 +67,8 @@ static void lt3593_led_work(struct work_struct *work)
                gpio_set_value_cansleep(led_dat->gpio, 1);
                udelay(1);
        }
-}
 
-static void lt3593_led_set(struct led_classdev *led_cdev,
-       enum led_brightness value)
-{
-       struct lt3593_led_data *led_dat =
-               container_of(led_cdev, struct lt3593_led_data, cdev);
-
-       led_dat->new_level = value;
-       schedule_work(&led_dat->work);
+       return 0;
 }
 
 static int create_lt3593_led(const struct gpio_led *template,
@@ -97,7 +87,7 @@ static int create_lt3593_led(const struct gpio_led *template,
        led_dat->cdev.default_trigger = template->default_trigger;
        led_dat->gpio = template->gpio;
 
-       led_dat->cdev.brightness_set = lt3593_led_set;
+       led_dat->cdev.brightness_set_blocking = lt3593_led_set;
 
        state = (template->default_state == LEDS_GPIO_DEFSTATE_ON);
        led_dat->cdev.brightness = state ? LED_FULL : LED_OFF;
@@ -111,8 +101,6 @@ static int create_lt3593_led(const struct gpio_led *template,
        if (ret < 0)
                return ret;
 
-       INIT_WORK(&led_dat->work, lt3593_led_work);
-
        ret = led_classdev_register(parent, &led_dat->cdev);
        if (ret < 0)
                return ret;
@@ -129,7 +117,6 @@ static void delete_lt3593_led(struct lt3593_led_data *led)
                return;
 
        led_classdev_unregister(&led->cdev);
-       cancel_work_sync(&led->work);
 }
 
 static int lt3593_led_probe(struct platform_device *pdev)