diff options
author | Bhanu Prakash Maiya <bhanumaiya@google.com> | 2021-07-15 17:59:26 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-07-16 20:36:22 +0000 |
commit | 97fdc267e7600d76d2c93bd765336918266c2cab (patch) | |
tree | a83e7e7fb90ff7875e18cbb830d0c67c4051bf15 /driver | |
parent | 67dd1137256af06b9645326c65b41c6099bbf4f2 (diff) | |
download | chrome-ec-97fdc267e7600d76d2c93bd765336918266c2cab.tar.gz |
driver: bmi3xx: Return with error for unsupported MOTIONSENSE_TYPE
BRANCH=none
BUG=b:178398789
TEST=Accel implementation tested on Guybrush
Accel init works on Guybrush
Signed-off-by: Bhanu Prakash Maiya <bhanumaiya@chromium.org>
Signed-off-by: Latchiamaran Senram <latchiamaran.senram@bosch.corp-partner.google.com>
Change-Id: I326152fb99d31b33c6e594ddcb9b4e5ad9e8379c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3033778
Commit-Queue: Bhanu Prakash Maiya <bhanumaiya@google.com>
Tested-by: Bhanu Prakash Maiya <bhanumaiya@google.com>
Auto-Submit: Bhanu Prakash Maiya <bhanumaiya@google.com>
Reviewed-by: Diana Z <dzigterman@chromium.org>
Diffstat (limited to 'driver')
-rw-r--r-- | driver/accelgyro_bmi3xx.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/driver/accelgyro_bmi3xx.c b/driver/accelgyro_bmi3xx.c index a887093530..c9415a3f56 100644 --- a/driver/accelgyro_bmi3xx.c +++ b/driver/accelgyro_bmi3xx.c @@ -1127,6 +1127,14 @@ static int init(struct motion_sensor_t *s) /* This driver requires a mutex */ ASSERT(s->mutex); + /* + * BMI3xx driver only supports MOTIONSENSE_TYPE_ACCEL and + * MOTIONSENSE_TYPE_GYR0 + */ + if (s->type != MOTIONSENSE_TYPE_ACCEL + && s->type != MOTIONSENSE_TYPE_GYRO) + return EC_ERROR_UNIMPLEMENTED; + /* Reset bmi3 device */ reg_data[0] = (uint8_t)(BMI3_CMD_SOFT_RESET & BMI3_SET_LOW_BYTE); reg_data[1] = (uint8_t)((BMI3_CMD_SOFT_RESET & BMI3_SET_HIGH_BYTE) |