Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1490725

Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding documentation

Show all headers | View raw

On 09/24/2016 01:58 PM, Pavel Machek wrote:
> Hi!
>
>> +Example
>> +=======
>> +
>> +led1: ncp5623@38 {
>> +	#address-cells = <1>;
>> +	#size-cells = <0>;
>> +	compatible = "onnn,ncp5623";
>> +	reg = <0x38>;
>> +	onnn,led-iref-microamp = <10>;
>> +
>> +	led1r@0 {
>> +		label = "ncp:power:red";
>> +		linux,default-trigger = "default-on";
> ...
>> +	led1b@1 {
>> +		label = "ncp:power:blue";
>> +		reg = <1>;
>
> Actually... the three LEDs are packaged such as this is one colorful
> light to the user, right? Some day we'll need to group them, so that
> kernel can automatically tell this is one led, and probably add extra
> attributes, such as values that produce white light.

We could try out the trigger approach we discussed few months ago.
Unfortunately I currently don't have enough time to propose the
implementation. Probably this work could be done on the occasion of
addition of RGB LED class driver like this, if the author had free
bandwidth for that.

-- 
Best regards,
Jacek Anaszewski

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding  documentation Pavel Machek <pavel@ucw.cz> - 2016-09-24 14:00 +0200
  Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding  documentation Jacek Anaszewski <jacek.anaszewski@gmail.com> - 2016-09-24 21:10 +0200
    Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding  documentation Florian Vaussard <florian.vaussard@gmail.com> - 2016-09-28 12:10 +0200
  Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding  documentation Florian Vaussard <florian.vaussard@gmail.com> - 2016-09-28 12:10 +0200
    Re: [PATCH v3 1/2] leds: ncp5623: Add device tree binding  documentation Pavel Machek <pavel@ucw.cz> - 2016-09-28 13:00 +0200

csiph-web