Re: [PATCH v2 2/4] leds: lp50xx: add setting of default intensity from DT

From: Pavel Machek
Date: Thu Feb 04 2021 - 09:56:49 EST


On Thu 2021-02-04 14:37:38, Sven Schuchmann wrote:
> In order to use a multicolor-led together with a trigger
> the led needs to have an intensity set to see something.
> The trigger changes the brightness of the led but if there
> is no intensity we actually see nothing.
>
> This patch adds the ability to set the default intensity
> of each multi-led node so that it is turned on from DT.
> If no intensity is given the led will be initialized
> with full intensity.



> diff --git a/drivers/leds/leds-lp50xx.c b/drivers/leds/leds-lp50xx.c
> index f13117eed976..4b40bf66483c 100644
> --- a/drivers/leds/leds-lp50xx.c
> +++ b/drivers/leds/leds-lp50xx.c
> @@ -267,7 +267,6 @@ struct lp50xx_led {
> struct led_classdev_mc mc_cdev;
> struct lp50xx *priv;
> unsigned long bank_modules;
> - int led_intensity[LP50XX_LEDS_PER_MODULE];
> u8 ctrl_bank_enabled;
> int led_number;
> };

? Does not make sense and changelog does not help.
Pavel

--
http://www.livejournal.com/~pavelmachek

Attachment: signature.asc
Description: PGP signature