diff options
| author | Fabio Baltieri <fabiobaltieri@google.com> | 2021-09-20 14:33:24 +0000 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-09-21 09:33:00 +0000 |
| commit | ad3c72d45cbfea3f5c8150a1a39625c99d82f784 (patch) | |
| tree | c7a8f09c9e6360912edbe76b7276c3a6c5e3777d /board/volteer | |
| parent | d8e0677d61d059e6c80db0e526de495b7d94a22b (diff) | |
| download | chrome-ec-ad3c72d45cbfea3f5c8150a1a39625c99d82f784.tar.gz | |
led_pwm: split color map on a separate structure
Currently color map is reusing the same structure as the channel and
callback selection, which is a bit wasteful and somewhat confusing.
Split it to its own struct with just three uint8_t fields.
BRANCH=none
BUG=none
TEST=build only
Signed-off-by: Fabio Baltieri <fabiobaltieri@google.com>
Change-Id: I56992fa9525db46980e450eb6569ba8291987b9b
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3168864
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Yuval Peress <peress@google.com>
Diffstat (limited to 'board/volteer')
| -rw-r--r-- | board/volteer/led.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/board/volteer/led.c b/board/volteer/led.c index 35bede89c5..357cdd038a 100644 --- a/board/volteer/led.c +++ b/board/volteer/led.c @@ -18,7 +18,7 @@ const enum ec_led_id supported_led_ids[] = { }; const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); -struct pwm_led led_color_map[] = { +struct pwm_led_color_map led_color_map[EC_LED_COLOR_COUNT] = { /* Red, Green, Blue */ [EC_LED_COLOR_RED] = { 100, 0, 0 }, [EC_LED_COLOR_GREEN] = { 0, 100, 0 }, @@ -112,4 +112,3 @@ static void board_led_init(void) pwm_set_duty(PWM_CH_LED4_SIDESEL, 50); } DECLARE_HOOK(HOOK_INIT, board_led_init, HOOK_PRIO_DEFAULT); - |