On 4/24/19 3:57 PM, Pavel Machek wrote:
Hi!
Switch to using generic LED support for composing LED class
device name.
Signed-off-by: Jacek Anaszewski <[email protected]>
@@ -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.
In previous versions I applied similar approach in few places but
later figured out it will be more consistent if all assignments are
next to each other.
Anyway, that's a detail.
8, 10, 12, 14, 16, 18, 20: Acked-by: Pavel Machek <[email protected]>
Thanks!
--
Best regards,
Jacek Anaszewski
