AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT

2021-02-03 Thread Sven Schuchmann
Helo Pavel, > > > Yes, sounds reasonable. Could we get default intensity of 100% on all > > > channels if nothing else is specified? > > > > > > Or maybe simply "if intensity is not specified, start with 100%, and > > > use explicit =0 if other color is expected". > > > > > Mh, if someone is alrea

AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT

2021-02-03 Thread Sven Schuchmann
Hello Pavel, hello Marek > > Is the property default-intensity documented in DT bindings? I updated the documentation in leds-lp50xx.yaml. Is it this what you mean? > > Wouldn't it be better if the property was used in the multi-led node > > instead of the channel node? I.e. > > multi-led@3 {

AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT

2021-02-03 Thread Sven Schuchmann
Hello Pavel, > > Yes, sounds reasonable. Could we get default intensity of 100% on all > channels if nothing else is specified? > > Or maybe simply "if intensity is not specified, start with 100%, and > use explicit =0 if other color is expected". > Mh, if someone is already using the led drive

AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT

2021-02-03 Thread Sven Schuchmann
Hello Pavel, > > In order to use a multicolor-led together with a trigger > > from DT 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

AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT

2021-02-02 Thread Sven Schuchmann
Hi, sorry to ask but was someone able to look at this? Any thoughts? Best Regards, Sven > -Ursprüngliche Nachricht- > Von: Sven Schuchmann > Gesendet: Dienstag, 19. Januar 2021 11:53 > An: Sven Schuchmann > Cc: Pavel Machek ; Dan Murphy ; Rob Herring > ; linux- > l...@vger.kernel.o