mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-27 19:29:37 +00:00
HID: thingm: remove workqueue
Defining workqueues in LED drivers isn't needed any longer as the LED core was extended with a generic workqueue recently. Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
This commit is contained in:
parent
c2848f2eef
commit
3de68ce92b
1 changed files with 15 additions and 24 deletions
|
@ -14,7 +14,6 @@
|
|||
#include <linux/leds.h>
|
||||
#include <linux/module.h>
|
||||
#include <linux/mutex.h>
|
||||
#include <linux/workqueue.h>
|
||||
|
||||
#include "hid-ids.h"
|
||||
|
||||
|
@ -56,7 +55,6 @@ struct thingm_rgb {
|
|||
struct thingm_led red;
|
||||
struct thingm_led green;
|
||||
struct thingm_led blue;
|
||||
struct work_struct work;
|
||||
u8 num;
|
||||
};
|
||||
|
||||
|
@ -131,25 +129,21 @@ static int thingm_write_color(struct thingm_rgb *rgb)
|
|||
return thingm_send(rgb->tdev, buf);
|
||||
}
|
||||
|
||||
static void thingm_work(struct work_struct *work)
|
||||
{
|
||||
struct thingm_rgb *rgb = container_of(work, struct thingm_rgb, work);
|
||||
|
||||
mutex_lock(&rgb->tdev->lock);
|
||||
|
||||
if (thingm_write_color(rgb))
|
||||
hid_err(rgb->tdev->hdev, "failed to write color\n");
|
||||
|
||||
mutex_unlock(&rgb->tdev->lock);
|
||||
}
|
||||
|
||||
static void thingm_led_set(struct led_classdev *ldev,
|
||||
enum led_brightness brightness)
|
||||
static int thingm_led_set(struct led_classdev *ldev,
|
||||
enum led_brightness brightness)
|
||||
{
|
||||
struct thingm_led *led = container_of(ldev, struct thingm_led, ldev);
|
||||
int ret;
|
||||
|
||||
/* the ledclass has already stored the brightness value */
|
||||
schedule_work(&led->rgb->work);
|
||||
mutex_lock(&led->rgb->tdev->lock);
|
||||
|
||||
ret = thingm_write_color(led->rgb);
|
||||
if (ret)
|
||||
hid_err(led->rgb->tdev->hdev, "failed to write color\n");
|
||||
|
||||
mutex_unlock(&led->rgb->tdev->lock);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int thingm_init_rgb(struct thingm_rgb *rgb)
|
||||
|
@ -162,7 +156,7 @@ static int thingm_init_rgb(struct thingm_rgb *rgb)
|
|||
"thingm%d:red:led%d", minor, rgb->num);
|
||||
rgb->red.ldev.name = rgb->red.name;
|
||||
rgb->red.ldev.max_brightness = 255;
|
||||
rgb->red.ldev.brightness_set = thingm_led_set;
|
||||
rgb->red.ldev.brightness_set_blocking = thingm_led_set;
|
||||
rgb->red.rgb = rgb;
|
||||
|
||||
err = led_classdev_register(&rgb->tdev->hdev->dev, &rgb->red.ldev);
|
||||
|
@ -174,7 +168,7 @@ static int thingm_init_rgb(struct thingm_rgb *rgb)
|
|||
"thingm%d:green:led%d", minor, rgb->num);
|
||||
rgb->green.ldev.name = rgb->green.name;
|
||||
rgb->green.ldev.max_brightness = 255;
|
||||
rgb->green.ldev.brightness_set = thingm_led_set;
|
||||
rgb->green.ldev.brightness_set_blocking = thingm_led_set;
|
||||
rgb->green.rgb = rgb;
|
||||
|
||||
err = led_classdev_register(&rgb->tdev->hdev->dev, &rgb->green.ldev);
|
||||
|
@ -186,15 +180,13 @@ static int thingm_init_rgb(struct thingm_rgb *rgb)
|
|||
"thingm%d:blue:led%d", minor, rgb->num);
|
||||
rgb->blue.ldev.name = rgb->blue.name;
|
||||
rgb->blue.ldev.max_brightness = 255;
|
||||
rgb->blue.ldev.brightness_set = thingm_led_set;
|
||||
rgb->blue.ldev.brightness_set_blocking = thingm_led_set;
|
||||
rgb->blue.rgb = rgb;
|
||||
|
||||
err = led_classdev_register(&rgb->tdev->hdev->dev, &rgb->blue.ldev);
|
||||
if (err)
|
||||
goto unregister_green;
|
||||
|
||||
INIT_WORK(&rgb->work, thingm_work);
|
||||
|
||||
return 0;
|
||||
|
||||
unregister_green:
|
||||
|
@ -211,7 +203,6 @@ static void thingm_remove_rgb(struct thingm_rgb *rgb)
|
|||
led_classdev_unregister(&rgb->red.ldev);
|
||||
led_classdev_unregister(&rgb->green.ldev);
|
||||
led_classdev_unregister(&rgb->blue.ldev);
|
||||
flush_work(&rgb->work);
|
||||
}
|
||||
|
||||
static int thingm_probe(struct hid_device *hdev, const struct hid_device_id *id)
|
||||
|
|
Loading…
Reference in a new issue