On Thu, Nov 8, 2012 at 12:14 AM, Stephen Warren <span dir="ltr"><<a href="mailto:swarren@wwwdotorg.org" target="_blank">swarren@wwwdotorg.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 11/07/2012 01:55 PM, Marek Belisko wrote:<br>
<br>
> diff --git a/Documentation/devicetree/bindings/leds/tca6507.txt b/Documentation/devicetree/bindings/leds/tca6507.txt<br>
<div class="im"><br>
> +LEDs conected to tca6507<br>
> +<br>
> +Required properties:<br>
> +- compatible : should be : "leds-tca6507".<br>
<br>
</div>"ti,tca6507" would be more typical.<br></blockquote><div>OK will post update for that </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> +Each led is represented as a sub-node of the leds-tca6507 device.<br>
> +<br>
> +LED sub-node properties:<br>
> +- label : label for this LED<br>
> +- reg : number of LED line (could be from 0 to 6)<br>
> +- linux,default-trigger : led default trigger<br>
<br>
</div>You probably want to mention which binding document defines the valid<br>
values for that property; ../gpio/led.txt I think.<br></blockquote><div>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</div><div>to avoid copying same text again? </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I wonder if we need to think about representing the blinking/fading/...<br>
capabilities of the chip too? Should that be encoded into the DT, or<br>
should it be controlled at run-time via some LED subsystem?<br></blockquote><div>I think this is handled internally and current platform data doesn't have support for that.</div></div><br>Thanks,</div><div class="gmail_extra">
<br></div><div class="gmail_extra">marek<br clear="all"><div><br></div>-- <br>as simple and primitive as possible<br>-------------------------------------------------<br>Marek Belisko - OPEN-NANDRA<br>Freelance Developer<br>
<br>Ruska Nova Ves 219 | Presov, 08005 Slovak Republic<br>Tel: +421 915 052 184<br>skype: marekwhite<br>twitter: #opennandra<br>web: <a href="http://open-nandra.com" target="_blank">http://open-nandra.com</a><br>
</div>