Re: [PATCH RFC leds-next] leds: initial support for Turris Omnia LEDs

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

 



On 4/2/20 4:29 PM, Marek Behun wrote:
> On Sun, 29 Mar 2020 14:53:57 +0200
> Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx> wrote:
> 
>>> in the case you mentioned there is a one "global" brightness setting per
>>> each RGB LED. On Omnia, all 12 RGB LEDs have just one "global"
>>> brightness property. Ie. I press the button, and all 12 LEDs glow get
>>> dimmer. So there could be a 13th LED device with color LUMA, but what
>>> function should it be given in DTS?  
>>
>> Then, in this particular case, adding devicename prefix for the whole
>> family of LEDs would be justified. The question is whether it should be
>> hardware related name or rather something different.
> 
> Hi Jacek,
> 
> so now we have
>   device:color:function
> what if we made it so that there was a 4th part of LED name, ie
>   group:device:color:function
> ?
> 
> Would this be a problem?

Indeed, the device alone would not be a sufficient differentiator
since it is possible to have more than one LED controller of the
same type on the board.

Nonetheless, I'd rather avoid the addition of a new generic section.
Probably we would have to make it specific to this device.

-- 
Best regards,
Jacek Anaszewski



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux