Re: [PATCH v4 20/26] leds: aat1290: Use generic support for composing LED names

From: Pavel Machek
Date: Wed Apr 24 2019 - 09:57:24 EST


Hi!

> Switch to using generic LED support for composing LED class
> device name.
>
> Signed-off-by: Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx>

> @@ -257,9 +257,6 @@ static int aat1290_led_parse_dt(struct aat1290_led *led,
> return -EINVAL;
> }
>
> - led_cdev->name = of_get_property(child_node, "label", NULL) ? :
> - child_node->name;
> -
> ret = of_property_read_u32(child_node, "led-max-microamp",
> &cfg->max_mm_current);
> /*
> @@ -469,6 +466,7 @@ static int aat1290_led_probe(struct platform_device *pdev)
> struct aat1290_led *led;
> struct led_classdev *led_cdev;
> struct led_classdev_flash *fled_cdev;
> + struct led_init_data init_data = {};

You can do directly = { .devicename = ...; }, at least for the name.

Anyway, that's a detail.

8, 10, 12, 14, 16, 18, 20: Acked-by: Pavel Machek <pavel@xxxxxx>
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature