Re: [PATCH 1/2] leds: add DT support for LEDs wired to supply and default brightness

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

 



On Thu, Mar 27, 2014 at 3:57 PM, Uwe Kleine-König
<u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
> On Thu, Mar 27, 2014 at 03:28:32PM -0700, Bryan Wu wrote:
>> On Fri, Feb 28, 2014 at 1:25 PM, Russell King
>> <rmk+kernel@xxxxxxxxxxxxxxxx> wrote:
>> > Some PWM outputs are wired such that the LED they're controlling is
>> > connected to supply rather than ground.  Therefore, the duty cycle
>> > needs to be inverted to make the LED behave as it should do.
>> >
>> > We also provide a way to specify the default brightness when a
>> > trigger is not specified.
>> >
>> > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>
>> > ---
>> >  Documentation/devicetree/bindings/leds/leds-pwm.txt |  3 +++
>> >  drivers/leds/leds-pwm.c                             | 15 +++++++++++----
>> >  2 files changed, 14 insertions(+), 4 deletions(-)
>> >
>> > diff --git a/Documentation/devicetree/bindings/leds/leds-pwm.txt b/Documentation/devicetree/bindings/leds/leds-pwm.txt
>> > index 7297107cf832..ffcb74ab61fd 100644
>> > --- a/Documentation/devicetree/bindings/leds/leds-pwm.txt
>> > +++ b/Documentation/devicetree/bindings/leds/leds-pwm.txt
>> > @@ -13,6 +13,9 @@ node's name represents the name of the corresponding LED.
>> >    For the pwms and pwm-names property please refer to:
>> >    Documentation/devicetree/bindings/pwm/pwm.txt
>> >  - max-brightness : Maximum brightness possible for the LED
>> > +- default-brightness : (optional) Default brightness for the LED
>> > +- active-low : (optional) For PWMs where the LED is wired to supply
>> > +  rather than ground.
>> >  - label :  (optional)
>> >    see Documentation/devicetree/bindings/leds/common.txt
>> >  - linux,default-trigger :  (optional)
>>
>> Why remove "max-brightness", "label" and "linux,default-trigger" here?
> They are not removed, they are just unchanged list items after literal
> '-'s in the context of the patch.
>

Sorry, my bad. please ignore my question then.

Thanks,
-Bryan
--
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