diff options
author | Tom Hughes <tomhughes@chromium.org> | 2021-10-25 17:35:46 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-10-27 21:28:22 +0000 |
commit | a28c1cd01a01639a2adc5388838a16afbed06eae (patch) | |
tree | b8cfefeb99887b6f9c53dda4e96bf27d83402424 /board/dirinboz | |
parent | 09382e18c22f9788bba39300d4d1b1dcfc173332 (diff) | |
download | chrome-ec-a28c1cd01a01639a2adc5388838a16afbed06eae.tar.gz |
board/dirinboz,board/gumboz: Fix variable type
When building with clang, it warns:
board/dirinboz/led.c:47:6: error: implicit conversion from enumeration
type 'enum ioex_signal' to different enumeration type 'enum gpio_signal'
[-Werror,-Wenum-conversion]
IOEX_C1_CHARGER_LED_AMBER_DB);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
board/dirinboz/led.c:49:6: error: implicit conversion from enumeration
type 'enum ioex_signal' to different enumeration type 'enum gpio_signal'
[-Werror,-Wenum-conversion]
IOEX_C1_CHARGER_LED_WHITE_DB);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~
The led variables are used as parameters to gpio_or_ioex_set_level,
which takes an int, so just use int so we don't have to cast between
types.
BRANCH=none
BUG=b:172020503
TEST=none
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I16428fbc5ef49f497a3514ac56049e8bd0e40d53
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3243005
Reviewed-by: Diana Z <dzigterman@chromium.org>
Diffstat (limited to 'board/dirinboz')
-rw-r--r-- | board/dirinboz/led.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/dirinboz/led.c b/board/dirinboz/led.c index 3343d0f924..b05ade3bcc 100644 --- a/board/dirinboz/led.c +++ b/board/dirinboz/led.c @@ -38,7 +38,7 @@ enum led_port { static void led_set_color_battery(int port, enum led_color color) { - enum gpio_signal amber_led, white_led; + int amber_led, white_led; uint32_t board_ver = 0; int led_batt_on_lvl, led_batt_off_lvl; |