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

Groups > linux.kernel > #1688521

Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface

Show all headers | View raw

Hi,

On 07/06/2017 05:18 AM, Pavel Machek wrote:
> Hi!
> 
>> Some LED controllers have support for autonomously controlling
>> brightness over time, according to some preprogrammed pattern or
>> function.
>>
>> This adds a new optional operator that LED class drivers can implement
>> if they support such functionality as well as a new device attribute to
>> configure the pattern for a given LED.
> 
>> @@ -61,3 +61,23 @@ Description:
>>  		gpio and backlight triggers. In case of the backlight trigger,
>>  		it is useful when driving a LED which is intended to indicate
>>  		a device in a standby like state.
>> +
>> +What:		/sys/class/leds/<led>/pattern
>> +Date:		July 2017
>> +KernelVersion:	4.14
>> +Description:
>> +		Specify a pattern for the LED, for LED hardware that support
>> +		altering the brightness as a function of time.
>> +
>> +		The pattern is given by a series of tuples, of brightness and
>> +		duration (ms). The LED is expected to traverse the series and
>> +		each brightness value for the specified duration.
>> +
>> +		Additionally a repeat marker ":|" can be appended to the
>> +		series, which should cause the pattern to be repeated
>> +		endlessly.
>> +
>> +		As LED hardware might have different capabilities and precision
>> +		the requested pattern might be slighly adjusted by the driver
>> +		and the resulting pattern of such operation should be returned
>> +		when this file is read.
> 
> Well. I believe this is mostly useful for RGB LEDs. Unfortunately, having patterns
> per-LED will present opportunity for different channels becoming de-synchronized
> from each other, which will not look nice.

Hmm, they are only [brightness duration] tuples, and no definition of
R, G and B LED device is covered here, so how it can be useful for RGB
LEDs?

I've been working on addition of RGB LED support to the LED core for
some time now, in the way as we agreed upon at [0], but it turns out to
be less trivial if we want to do it in an elegant way.

Less elegant way would be duplicating led-core functions and changing
single enum led_brightness argument with the three ones (or a struct
containing three brightness components)

I chose to go the elegant way and tried to introduce led_classdev_base
type that would be customizable with the set of ops, that would allow
for making the number of brightness components to be set at once
customizable. This of course entails significant amount of changes in
the LED core and some changes in LED Trigger core.

Unfortunately I can't predict how much time it will take
to submit an RFC due to limited amount of time I can spend working
on it.

[0] https://www.spinics.net/lists/linux-leds/msg07959.html

-- 
Best regards,
Jacek Anaszewski

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

Thread

[PATCH v2 0/3] Qualcomm Light Pulse Generator Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-15 00:50 +0200
  [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator binding Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-15 00:50 +0200
    Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Pavel Machek <pavel@ucw.cz> - 2017-07-15 11:20 +0200
      Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-16 07:40 +0200
    Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Jacek Anaszewski <jacek.anaszewski@gmail.com> - 2017-07-16 21:00 +0200
      Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-17 06:50 +0200
        Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Jacek Anaszewski <jacek.anaszewski@gmail.com> - 2017-07-17 23:10 +0200
          Re: [PATCH v2 3/3] DT: leds: Add Qualcomm Light Pulse Generator  binding Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-18 02:10 +0200
  [PATCH v2 1/3] leds: core: Introduce generic pattern interface Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-15 00:50 +0200
    Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Pavel Machek <pavel@ucw.cz> - 2017-07-16 09:40 +0200
      Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Jacek Anaszewski <jacek.anaszewski@gmail.com> - 2017-07-16 21:00 +0200
        Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-16 23:20 +0200
          Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Jacek Anaszewski <jacek.anaszewski@gmail.com> - 2017-07-17 23:10 +0200
            Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-18 01:40 +0200
      Re: [PATCH v2 1/3] leds: core: Introduce generic pattern interface Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-16 22:00 +0200
  [PATCH v2 2/3] leds: Add driver for Qualcomm LPG Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-15 00:50 +0200
  Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator Pavel Machek <pavel@ucw.cz> - 2017-07-15 11:20 +0200
    Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-16 07:40 +0200
      Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator Pavel Machek <pavel@ucw.cz> - 2017-07-16 09:40 +0200
        Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-07-16 23:00 +0200
          Re: [PATCH v2 0/3] Qualcomm Light Pulse Generator Pavel Machek <pavel@ucw.cz> - 2017-07-16 23:20 +0200

csiph-web