linux-kernel - Re: [PATCH v26 03/15] leds: multicolor: Introduce a multicolor class definition

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <02cf192f-1948-74a5-f2ef-6c2146422ecb@ti.com>
Date:   Mon, 8 Jun 2020 09:34:33 -0500
From:   Dan Murphy <dmurphy@...com>
To:     Jacek Anaszewski <jacek.anaszewski@...il.com>,
        Pavel Machek <pavel@....cz>
CC:     <devicetree@...r.kernel.org>, <linux-leds@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v26 03/15] leds: multicolor: Introduce a multicolor class
 definition

Jacek

On 6/6/20 2:59 PM, Jacek Anaszewski wrote:
> Dan,
>
> On 6/6/20 6:39 PM, Dan Murphy wrote:
>> Pavek
>>
>> Thanks for the review
>>
>> On 6/6/20 10:53 AM, Pavel Machek wrote:
>>> Hi!
>>>
>>>> Introduce a multicolor class that groups colored LEDs
>>>> within a LED node.
>>>>
>>>> The multi color class groups monochrome LEDs and allows controlling 
>>>> two
>>>> aspects of the final combined color: hue and lightness. The former is
>>>> controlled via the intensity file and the latter is controlled
>>>> via brightness file.
>>>>
>>>> Acked-by: Jacek Anaszewski <jacek.anaszewski@...il.com>
>>>> Signed-off-by: Dan Murphy <dmurphy@...com>
>>>> diff --git a/Documentation/ABI/testing/sysfs-class-led-multicolor 
>>>> b/Documentation/ABI/testing/sysfs-class-led-multicolor
>>>> new file mode 100644
> [...]
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -9533,6 +9533,14 @@ F: Documentation/devicetree/bindings/leds/
>>>>   F:    drivers/leds/
>>>>   F:    include/linux/leds.h
>>>> +LED MULTICOLOR FRAMEWORK
>>>> +M:    Dan Murphy <dmurphy@...com>
>>>> +L:    linux-leds@...r.kernel.org
>>> I'd like to be mentioned here, too. "M: Pavel Machek
>>> <pavel@....cz>". And I'm not sure if I should be taking MAINTAINER
>>> file update through a LED tree. Should definitely go to separate
>>> patch.
>>
>> Oh definitely.  I thought it was implied that you and Jacek are both 
>> Maintainers as well.
>>
>> I will add you but will wait to see if Jacek wants to be added.
>
> Actually I don't think that we need to add this separate entry
> for LED multicolor class. This is still under LED subsystem,
> and I didn't add anything for LED class flash.

We only need this because I am not a maintainer of the LED flash class 
or the LED class.

But since I authored the code it only made sense to add me as a 
maintainer for this specific class.

You are one of the maintainers of the LED subsystem and wrote the Flash 
class so your maintainer ship is implied and you will be CC'd for all 
patches.

This will not be the case for the multi color class

Dan