Re: [PATCH v4 4/6] dt: bindings: lp8860: Add trigger binding to the lp8860

From: Pavel Machek
Date: Fri Dec 15 2017 - 16:00:49 EST


On Fri 2017-12-15 11:22:24, Dan Murphy wrote:
> Pavel
>
> On 12/15/2017 03:09 AM, Pavel Machek wrote:
> > Hi!
> >
> >> @@ -35,6 +37,7 @@ led-controller@2d {
> >> led@0 {
> >> reg = <0>;
> >> label = "white:display_cluster";
> >> + linux,default-trigger = "backlight";
> >> };
> >> }
> >
> > What is "display_cluster"? Is it display backlight?
> >
>
> This device can be configured to drive all 4 LED strings in display mode
> so that all 4 strings have the same brightness controlled either through PWM or
> through the brightness register
>
> or
>
> As cluster string where the brightness of individual strings or clustered strings
> can be controlled via independent brightness control registers. I am currently working
> on adding this support to the driver.

Ok; but I guess I'd like to keep this implementation detail away from
kernel, and just expose as display:white:backlight or something like that.

--
(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