On Mon, Jun 15, 2015 at 1:15 AM, Jacek Anaszewski <j.anaszewski@xxxxxxxxxxx> wrote: > Hi Matt, > > On 06/13/2015 06:17 AM, Matt Ranostay wrote: >> >> Some cap11xx devices have LEDs that can be controlled from userpace >> and via triggers. Document their use and functionality here. >> >> Signed-off-by: Matt Ranostay <mranostay@xxxxxxxxx> >> --- >> .../devicetree/bindings/input/cap11xx.txt | 23 >> ++++++++++++++++++++++ >> 1 file changed, 23 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/input/cap11xx.txt >> b/Documentation/devicetree/bindings/input/cap11xx.txt >> index 7d0a300..ef89c27 100644 >> --- a/Documentation/devicetree/bindings/input/cap11xx.txt >> +++ b/Documentation/devicetree/bindings/input/cap11xx.txt >> @@ -38,6 +38,11 @@ Optional properties: >> defaults. The array must have exactly six >> entries. >> >> + linux,led-brightness: Defines the ON brightness when the >> optional LED >> + functionality is used. Valid values are >> 0-15. >> + By default a value of 15 is set. > > > The ON brightness is controlled with the 'brightness' sysfs property. > The levels start from 1 and 0 turns the LED off. I think that this > property is useless then. > Yes but this defines the brightness for the ON state.. since it can be from 0-100% duty cycle. > >> + >> Example: >> >> i2c_controller { >> @@ -55,5 +60,23 @@ i2c_controller { >> <105>, /* KEY_LEFT */ >> <109>, /* KEY_PAGEDOWN */ >> <104>; /* KEY_PAGEUP */ >> + >> + linux,led-brightness = <15>; >> + >> + usr@0 { >> + label = "cap11xx:green:usr0"; >> + reg = <0>; >> + }; >> + >> + usr@1 { >> + label = "cap11xx:green:usr1"; >> + reg = <1>; >> + }; >> + >> + alive@2 { >> + label = "cap11xx:green:alive"; >> + reg = <2>; >> + linux,default_trigger = "heartbeat"; >> + }; >> }; >> } >> > > -- > Best Regards, > Jacek Anaszewski -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html