Re: [PATCH v2 2/2] Add documentation for tca6507 devicetree bindings.

On Thu, Nov 8, 2012 at 12:14 AM, Stephen Warren [off-list ref]wrote:

On 11/07/2012 01:55 PM, Marek Belisko wrote:

quoted

diff --git a/Documentation/devicetree/bindings/leds/tca6507.txt

b/Documentation/devicetree/bindings/leds/tca6507.txt

quoted

+LEDs conected to tca6507
+
+Required properties:
+- compatible : should be : "leds-tca6507".

"ti,tca6507" would be more typical.

OK will post update for that

quoted

+Each led is represented as a sub-node of the leds-tca6507 device.
+
+LED sub-node properties:
+- label : label for this LED
+- reg : number of LED line (could be from 0 to 6)
+- linux,default-trigger : led default trigger

You probably want to mention which binding document defines the valid
values for that property; ../gpio/led.txt I think.

Will add that also. Just an idea this is common for most of the leds cannot
this be put to some common place as reference
to avoid copying same text again?

I wonder if we need to think about representing the blinking/fading/...
capabilities of the chip too? Should that be encoded into the DT, or
should it be controlled at run-time via some LED subsystem?

I think this is handled internally and current platform data doesn't have
support for that.

Thanks,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com