leds: add more LED color and function definitions

[PATCH v2 0/2] leds: add more LED color and function definitions

Posted by Shiji Yang 2 years, 2 months ago

Hi!

This series patches add the missing LED color definitions and
introduces two widely used LED functions "internet" and "rssi".

Regards,
Shiji Yang


Changes in v2:
* Remove the LED name sorting patch as it changes the ABI


Shiji Yang (2):
  leds: core: add missing LED color strings
  dt-bindings: leds: add "internet" and "rssi" function definitions

 drivers/leds/led-core.c           | 5 +++++
 include/dt-bindings/leds/common.h | 2 ++
 2 files changed, 7 insertions(+)

-- 
2.39.2

Re: [PATCH v2 0/2] leds: add more LED color and function definitions

Posted by Lee Jones 2 years, 1 month ago

On Tue, 31 Oct 2023, Shiji Yang wrote:

> Hi!
> 
> This series patches add the missing LED color definitions and
> introduces two widely used LED functions "internet" and "rssi".
> 
> Regards,
> Shiji Yang
> 
> 
> Changes in v2:
> * Remove the LED name sorting patch as it changes the ABI
> 
> 
> Shiji Yang (2):
>   leds: core: add missing LED color strings
>   dt-bindings: leds: add "internet" and "rssi" function definitions

I don't see these patches attached to this cover letter.

Did you send them all threaded? 

-- 
Lee Jones [李琼斯]

Re: [PATCH v2 0/2] leds: add more LED color and function definitions

Posted by Shiji Yang 2 years, 1 month ago

On Fri, 17 Nov 2023 12:52:19 +0000, Lee Jones wrote:

>On Tue, 31 Oct 2023, Shiji Yang wrote:
>
>> Hi!
>> 
>> This series patches add the missing LED color definitions and
>> introduces two widely used LED functions "internet" and "rssi".
>> 
>> Regards,
>> Shiji Yang
>> 
>> 
>> Changes in v2:
>> * Remove the LED name sorting patch as it changes the ABI
>> 
>> 
>> Shiji Yang (2):
>>   leds: core: add missing LED color strings
>>   dt-bindings: leds: add "internet" and "rssi" function definitions
>
>I don't see these patches attached to this cover letter.
>
>Did you send them all threaded? 
>
>-- 
>Lee Jones [李琼斯]

Hi!
Thanks for your review. Please ignore this patch. My git-email
tool has some issues, so they have been separated[1]. The final
v4 patch[2] has been sent some days ago.

[1] https://lore.kernel.org/all/TYAP286MB03159A83A77E6FD59F271D9BBCA0A@TYAP286MB0315.JPNP286.PROD.OUTLOOK.COM/

[2] https://lore.kernel.org/all/TYAP286MB0315F4D71698370875F58F6EBCAAA@TYAP286MB0315.JPNP286.PROD.OUTLOOK.COM/

Regards,
Shiji Yang