It looks fine to me, but I need a DT maintainer's ack. Stephen, could you help on this. I will merge this patch via my tree. Thanks, -Bryan On Mon, Nov 12, 2012 at 1:25 PM, Marek Belisko <marek.belisko@xxxxxxxxxxxxxxx> wrote: > Signed-off-by: Marek Belisko <marek.belisko@xxxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/leds/tca6507.txt | 40 ++++++++++++++++++++ > 1 file changed, 40 insertions(+) > create mode 100644 Documentation/devicetree/bindings/leds/tca6507.txt > > diff --git a/Documentation/devicetree/bindings/leds/tca6507.txt b/Documentation/devicetree/bindings/leds/tca6507.txt > new file mode 100644 > index 0000000..a49eb0c > --- /dev/null > +++ b/Documentation/devicetree/bindings/leds/tca6507.txt > @@ -0,0 +1,40 @@ > +LEDs conected to tca6507 > + > +Required properties: > +- compatible : should be : "ti,tca6507". > + > +Each led is represented as a sub-node of the ti,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 : (optional) This parameter, if present, is a > + string defining the trigger assigned to the LED. Current triggers are: > + "backlight" - LED will act as a back-light, controlled by the framebuffer > + system > + "default-on" - LED will turn on > + "heartbeat" - LED "double" flashes at a load average based rate > + "ide-disk" - LED indicates disk activity > + "timer" - LED flashes at a fixed, configurable rate > + > +Examples: > + > +tca6507@45 { > + compatible = "ti,tca6507"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x45>; > + > + led0: red_aux@0 { > + label = "red:aux"; > + reg = <0x0>; > + }; > + > + led1: green_aux@1 { > + label = "green:aux"; > + reg = <0x5>; > + linux,default-trigger = "default-on"; > + }; > + > +}; > + > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html