[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171215210042.GC19442@amd>
Date: Fri, 15 Dec 2017 22:00:42 +0100
From: Pavel Machek <pavel@....cz>
To: Dan Murphy <dmurphy@...com>
Cc: robh+dt@...nel.org, mark.rutland@....com, rpurdie@...ys.net,
jacek.anaszewski@...il.com, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-leds@...r.kernel.org
Subject: Re: [PATCH v4 4/6] dt: bindings: lp8860: Add trigger binding to the
lp8860
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
Download attachment "signature.asc" of type "application/pgp-signature" (182 bytes)
Powered by blists - more mailing lists