diff options
author | Tom Hughes <tomhughes@chromium.org> | 2021-10-21 23:20:10 +0000 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-10-25 18:03:14 +0000 |
commit | d4531ba075b951d19df3eb49074073b535be86d2 (patch) | |
tree | e9ae89be9ac1f118174ebf6c3e7729dec9f23879 /board/morphius | |
parent | 03874334eac1b2bf77e33ea60a6587e6801faf6a (diff) | |
download | chrome-ec-d4531ba075b951d19df3eb49074073b535be86d2.tar.gz |
baseboard/zork: Unify enum types
enum ec_ssfc_base_gyro_sensor and enum ec_cfg_base_gyro_sensor_type have
the same values. clang warns that we're trying to convert between these
two types:
board/morphius/board.c:257:10: error: implicit conversion from
enumeration type 'enum ec_ssfc_base_gyro_sensor' to different
enumeration type 'enum ec_cfg_base_gyro_sensor_type'
[-Werror,-Wenum-conversion]
return get_cbi_ssfc_base_sensor();
Rather than duplicate the type (and it accidentally getting out of
sync), just use the canonical base type.
BRANCH=none
BUG=b:172020503
TEST=./util/compare_builds.sh -b all -j 120
=> All match, except ezkinil due to the change of the
base_gyro_config variable from "int" to "enum"
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I8168518a87469004c33adebed89879225c470ace
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3238249
Reviewed-by: Keith Short <keithshort@chromium.org>
Diffstat (limited to 'board/morphius')
-rw-r--r-- | board/morphius/board.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/board/morphius/board.c b/board/morphius/board.c index 48712ffb0c..8e0577b133 100644 --- a/board/morphius/board.c +++ b/board/morphius/board.c @@ -246,9 +246,9 @@ const struct pi3hdx1204_tuning pi3hdx1204_tuning = { /***************************************************************************** * Base Gyro Sensor dynamic configuration */ -static enum ec_cfg_base_gyro_sensor_type base_gyro_config; +static enum ec_ssfc_base_gyro_sensor base_gyro_config = SSFC_BASE_GYRO_NONE; -enum ec_cfg_base_gyro_sensor_type get_base_gyro_sensor(void) +enum ec_ssfc_base_gyro_sensor get_base_gyro_sensor(void) { switch (get_cbi_ssfc_base_sensor()) { case SSFC_BASE_GYRO_NONE: @@ -263,10 +263,10 @@ static void setup_base_gyro_config(void) base_gyro_config = get_base_gyro_sensor(); switch (base_gyro_config) { - case BASE_GYRO_BMI160: + case SSFC_BASE_GYRO_BMI160: ccprints("BASE GYRO is BMI160"); break; - case BASE_GYRO_ICM426XX: + case SSFC_BASE_GYRO_ICM426XX: motion_sensors[BASE_ACCEL] = icm426xx_base_accel; motion_sensors[BASE_GYRO] = icm426xx_base_gyro; ccprints("BASE GYRO is ICM426XX"); @@ -279,10 +279,10 @@ static void setup_base_gyro_config(void) void motion_interrupt(enum gpio_signal signal) { switch (base_gyro_config) { - case BASE_GYRO_BMI160: + case SSFC_BASE_GYRO_BMI160: bmi160_interrupt(signal); break; - case BASE_GYRO_ICM426XX: + case SSFC_BASE_GYRO_ICM426XX: icm426xx_interrupt(signal); break; default: |