[v30,04/16] leds: Add multicolor ID to the color ID list
diff mbox series

Message ID 20200713154544.1683-5-dmurphy@ti.com
State New
Headers show
Series
  • Multicolor Framework v30
Related show

Commit Message

Dan Murphy July 13, 2020, 3:45 p.m. UTC
Add a new color ID that is declared as MULTICOLOR as with the
multicolor framework declaring a definitive color is not accurate
as the node can contain multiple colors.

Signed-off-by: Dan Murphy <dmurphy@ti.com>
---
 drivers/leds/led-core.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Marek Behún July 15, 2020, 1:20 p.m. UTC | #1
On Mon, 13 Jul 2020 10:45:32 -0500
Dan Murphy <dmurphy@ti.com> wrote:

> Add a new color ID that is declared as MULTICOLOR as with the
> multicolor framework declaring a definitive color is not accurate
> as the node can contain multiple colors.
> 
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>  drivers/leds/led-core.c | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Marek Behún <marek.behun@nic.cz>
Pavel Machek July 15, 2020, 5:36 p.m. UTC | #2
On Wed 2020-07-15 15:20:56, Marek Behún wrote:
> On Mon, 13 Jul 2020 10:45:32 -0500
> Dan Murphy <dmurphy@ti.com> wrote:
> 
> > Add a new color ID that is declared as MULTICOLOR as with the
> > multicolor framework declaring a definitive color is not accurate
> > as the node can contain multiple colors.
> > 
> > Signed-off-by: Dan Murphy <dmurphy@ti.com>
> > ---
> >  drivers/leds/led-core.c | 1 +
> >  1 file changed, 1 insertion(+)
> 
> Reviewed-by: Marek Behún <marek.behun@nic.cz>

Thanks for patches, thanks for reviews, 1-4 applied.
									Pavel
Marek Behún July 15, 2020, 5:59 p.m. UTC | #3
On Wed, 15 Jul 2020 19:36:10 +0200
Pavel Machek <pavel@ucw.cz> wrote:

> Thanks for patches, thanks for reviews, 1-4 applied.
> 									Pavel

The most important one is 5th, though :D

Patch
diff mbox series

diff --git a/drivers/leds/led-core.c b/drivers/leds/led-core.c
index f1f718dbe0f8..846248a0693d 100644
--- a/drivers/leds/led-core.c
+++ b/drivers/leds/led-core.c
@@ -34,6 +34,7 @@  const char * const led_colors[LED_COLOR_ID_MAX] = {
 	[LED_COLOR_ID_VIOLET] = "violet",
 	[LED_COLOR_ID_YELLOW] = "yellow",
 	[LED_COLOR_ID_IR] = "ir",
+	[LED_COLOR_ID_MULTI] = "multicolor",
 };
 EXPORT_SYMBOL_GPL(led_colors);