diff options
| author | Tom Hughes <tomhughes@chromium.org> | 2021-10-04 22:14:20 +0000 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-10-05 16:48:18 +0000 |
| commit | 036cda45bba9fca0137add1d8693a264ba686d71 (patch) | |
| tree | b74cff146a7d0bb744933858976be0e7b5108518 /board/lantis | |
| parent | d7995163073a06e63aa1e48ed7f9a5cb8e1d6fd3 (diff) | |
| download | chrome-ec-036cda45bba9fca0137add1d8693a264ba686d71.tar.gz | |
tree: Fix LED enum type
"enum ec_led_colors" was used here instead of "enum led_color".
Fixes the following clang warning:
error: implicit conversion from enumeration type
'enum led_color' to different enumeration type 'enum ec_led_colors'
[-Werror,-Wenum-conversion]
This is the same fix as commit 533640b6d2ce1189b8c0559ec69e1d64202d2c8b,
but this should fix all instances. These usages were found with:
git grep --name-only 'enum led_color' | xargs grep ec_led_colors
As indicated by the TEST line, all boards match with this change, except
for bloog since it uses the incorrect enum in two places.
BRANCH=none
BUG=b:172020503
TEST=CC=arm-none-eabi-clang BOARD=<affected board>
TEST=./util/compare_build.sh -b all -j 120
=> All match except bloog
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I75cff3769d5b3b82b9218bfb8584f449fb6a781f
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3203936
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'board/lantis')
| -rw-r--r-- | board/lantis/led.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/lantis/led.c b/board/lantis/led.c index c4868de740..97afef2f02 100644 --- a/board/lantis/led.c +++ b/board/lantis/led.c @@ -66,7 +66,7 @@ static int led_set_color_battery(int port, enum led_color color) return EC_SUCCESS; } -static int led_set_color_power(enum ec_led_colors color) +static int led_set_color_power(enum led_color color) { switch (color) { case LED_OFF: |