Re: [PATCH] Add attributes neccessary for LED flashes to devicetree/bindings/leds/common.txt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jacek,

On Fri, Nov 21, 2014 at 08:38:33AM +0100, Jacek Anaszewski wrote:
> Hi Pavel, Sakari,
> 
> On 11/20/2014 04:03 PM, Sakari Ailus wrote:
> >Hi Pavel,
> >
> >On Thu, Nov 20, 2014 at 02:17:13PM +0100, Pavel Machek wrote:
> >>
> >>Add attributes neccessary for LED flashes to
> >>devicetree/bindings/leds/common.txt .
> >>
> >>This will allow me to add device tree support for adp1653 i2c flash
> >>LED driver, and allow Jacek Anaszewski to add support for more LED
> >>drivers..
> >>
> >>Signed-off-by: Pavel Machek <pavel@xxxxxx>
> >>
> >>diff --git a/Documentation/devicetree/bindings/leds/common.txt b/Documentation/devicetree/bindings/leds/common.txt
> >>index 2d88816..e9acbbc 100644
> >>--- a/Documentation/devicetree/bindings/leds/common.txt
> >>+++ b/Documentation/devicetree/bindings/leds/common.txt
> >>@@ -14,6 +14,15 @@ Optional properties for child nodes:
> >>       "ide-disk" - LED indicates disk activity
> >>       "timer" - LED flashes at a fixed, configurable rate
> >>
> >>+- max-microamp : maximum intensity in microamperes of the LED
> >>+  	         (torch LED for flash devices)
> >>+- flash-max-microamp : maximum intensity in microamperes of the
> >>+                       flash LED; it is mandatory if the LED should
> >>+		       support the flash mode
> >>+- flash-timeout-microsec : timeout in microseconds after which the flash
> >>+                           LED is turned off
> >
> >It might be good to add a note that these may be arrays, depending on the
> >device.
> >
> 
> This binding is for a sub-node representing a single led.
> Do you know devices which would require an array of max currents
> for a sub-led?

Oops, I missed to reply to this one.

No, if this is just a single LED then I think it's fine.

-- 
Sakari Ailus
e-mail: sakari.ailus@xxxxxx	XMPP: sailus@xxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux