Commit 9c06c6a2 authored by Rui Miguel Silva's avatar Rui Miguel Silva Committed by Greg Kroah-Hartman

greybus: lights: remove sync operation and work queue

In kernel v4.5 there is a change in LED api, which remove the need for
individual work queue and rename the set_sync operation to set_blocking.
This patch add the handling of this case and avoid compilation failure
for this kernel versions.
Signed-off-by: default avatarRui Miguel Silva <rui.silva@linaro.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@google.com>
parent b827e113
...@@ -30,7 +30,9 @@ struct gb_channel { ...@@ -30,7 +30,9 @@ struct gb_channel {
struct attribute **attrs; struct attribute **attrs;
struct attribute_group *attr_group; struct attribute_group *attr_group;
const struct attribute_group **attr_groups; const struct attribute_group **attr_groups;
#ifndef LED_HAVE_SET_BLOCKING
struct work_struct work_brightness_set; struct work_struct work_brightness_set;
#endif
struct led_classdev *led; struct led_classdev *led;
#ifdef LED_HAVE_FLASH #ifdef LED_HAVE_FLASH
struct led_classdev_flash fled; struct led_classdev_flash fled;
...@@ -381,6 +383,7 @@ static int __gb_lights_brightness_set(struct gb_channel *channel) ...@@ -381,6 +383,7 @@ static int __gb_lights_brightness_set(struct gb_channel *channel)
return ret; return ret;
} }
#ifndef LED_HAVE_SET_BLOCKING
static void gb_brightness_set_work(struct work_struct *work) static void gb_brightness_set_work(struct work_struct *work)
{ {
struct gb_channel *channel = container_of(work, struct gb_channel, struct gb_channel *channel = container_of(work, struct gb_channel,
...@@ -412,6 +415,17 @@ static void gb_brightness_set(struct led_classdev *cdev, ...@@ -412,6 +415,17 @@ static void gb_brightness_set(struct led_classdev *cdev,
cdev->brightness = value; cdev->brightness = value;
schedule_work(&channel->work_brightness_set); schedule_work(&channel->work_brightness_set);
} }
#else /* LED_HAVE_SET_BLOCKING */
static int gb_brightness_set(struct led_classdev *cdev,
enum led_brightness value)
{
struct gb_channel *channel = get_channel_from_cdev(cdev);
channel->led->brightness = value;
return __gb_lights_brightness_set(channel);
}
#endif
static enum led_brightness gb_brightness_get(struct led_classdev *cdev) static enum led_brightness gb_brightness_get(struct led_classdev *cdev)
...@@ -443,12 +457,17 @@ static int gb_blink_set(struct led_classdev *cdev, unsigned long *delay_on, ...@@ -443,12 +457,17 @@ static int gb_blink_set(struct led_classdev *cdev, unsigned long *delay_on,
static void gb_lights_led_operations_set(struct gb_channel *channel, static void gb_lights_led_operations_set(struct gb_channel *channel,
struct led_classdev *cdev) struct led_classdev *cdev)
{ {
cdev->brightness_set = gb_brightness_set;
cdev->brightness_get = gb_brightness_get; cdev->brightness_get = gb_brightness_get;
#ifdef LED_HAVE_SET_SYNC #ifdef LED_HAVE_SET_SYNC
cdev->brightness_set_sync = gb_brightness_set_sync; cdev->brightness_set_sync = gb_brightness_set_sync;
#endif #endif
#ifdef LED_HAVE_SET_BLOCKING
cdev->brightness_set_blocking = gb_brightness_set;
#endif
#ifndef LED_HAVE_SET_BLOCKING
cdev->brightness_set = gb_brightness_set;
INIT_WORK(&channel->work_brightness_set, gb_brightness_set_work); INIT_WORK(&channel->work_brightness_set, gb_brightness_set_work);
#endif
if (channel->flags & GB_LIGHT_CHANNEL_BLINK) if (channel->flags & GB_LIGHT_CHANNEL_BLINK)
cdev->blink_set = gb_blink_set; cdev->blink_set = gb_blink_set;
...@@ -986,8 +1005,10 @@ static int gb_lights_light_config(struct gb_lights *glights, u8 id) ...@@ -986,8 +1005,10 @@ static int gb_lights_light_config(struct gb_lights *glights, u8 id)
static void gb_lights_channel_free(struct gb_channel *channel) static void gb_lights_channel_free(struct gb_channel *channel)
{ {
#ifndef LED_HAVE_SET_BLOCKING
if (&channel->work_brightness_set) if (&channel->work_brightness_set)
flush_work(&channel->work_brightness_set); flush_work(&channel->work_brightness_set);
#endif
kfree(channel->attrs); kfree(channel->attrs);
kfree(channel->attr_group); kfree(channel->attr_group);
kfree(channel->attr_groups); kfree(channel->attr_groups);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment