Skip to content

Commit

Permalink
leds: lt3593: Use generic support for composing LED names
Browse files Browse the repository at this point in the history
Switch to using generic LED support for composing LED class
device name.

Signed-off-by: Jacek Anaszewski <[email protected]>
Cc: Daniel Mack <[email protected]>
Acked-by: Pavel Machek <[email protected]>
  • Loading branch information
jacek-anaszewski committed Jul 25, 2019
1 parent e12e91a commit 68e1920
Showing 1 changed file with 8 additions and 12 deletions.
20 changes: 8 additions & 12 deletions drivers/leds/leds-lt3593.c
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@
#include <linux/slab.h>
#include <linux/module.h>
#include <linux/of.h>
#include <uapi/linux/uleds.h>

#define LED_LT3593_NAME "lt3593"

struct lt3593_led_data {
char name[LED_MAX_NAME_SIZE];
struct led_classdev cdev;
struct gpio_desc *gpiod;
};
Expand Down Expand Up @@ -66,6 +66,7 @@ static int lt3593_led_probe(struct platform_device *pdev)
struct lt3593_led_data *led_data;
struct fwnode_handle *child;
int ret, state = LEDS_GPIO_DEFSTATE_OFF;
struct led_init_data init_data = {};
const char *tmp;

if (!dev->of_node)
Expand All @@ -86,14 +87,6 @@ static int lt3593_led_probe(struct platform_device *pdev)

child = device_get_next_child_node(dev, NULL);

ret = fwnode_property_read_string(child, "label", &tmp);
if (ret < 0)
snprintf(led_data->name, sizeof(led_data->name),
"lt3593::");
else
snprintf(led_data->name, sizeof(led_data->name),
"lt3593:%s", tmp);

fwnode_property_read_string(child, "linux,default-trigger",
&led_data->cdev.default_trigger);

Expand All @@ -102,11 +95,14 @@ static int lt3593_led_probe(struct platform_device *pdev)
state = LEDS_GPIO_DEFSTATE_ON;
}

led_data->cdev.name = led_data->name;
led_data->cdev.brightness_set_blocking = lt3593_led_set;
led_data->cdev.brightness = state ? LED_FULL : LED_OFF;

ret = devm_led_classdev_register(dev, &led_data->cdev);
init_data.fwnode = child;
init_data.devicename = LED_LT3593_NAME;
init_data.default_label = ":";

ret = devm_led_classdev_register_ext(dev, &led_data->cdev, &init_data);
if (ret < 0) {
fwnode_handle_put(child);
return ret;
Expand Down

0 comments on commit 68e1920

Please sign in to comment.