diff options
author | Gwendal Grignou <gwendal@chromium.org> | 2018-04-06 14:44:31 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-05-30 05:47:50 +0000 |
commit | 0aee48bdaab94faf674daa69b327d9a6675090a1 (patch) | |
tree | 444017dd73575b8c4b2c222f0e1843546ece94df | |
parent | 10d9010689d7d627f919c5787b99965cc14b53a8 (diff) | |
download | chrome-ec-0aee48bdaab94faf674daa69b327d9a6675090a1.tar.gz |
FIXUP: driver: lsm6dsm/lsm6dsl: Add FIFO support
Returns an error if the ODR set is less than 13Hz instead of silently
put the sensor in suspend mode.
BUG=b:77601149
BRANCH=None
TEST=Check with accelrate we get an error instead of
"Data rate for sensor 0: 0"
(cherrypicked from a796f04b1 'FIXUP: driver: lsm6dsm/lsm6dsl: Add FIFO
support')
Reviewed-on: https://chromium-review.googlesource.com/1000399
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Change-Id: Iead740f4205bbce1cfbccf2407f2a3a0dcf0ddaf
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1621263
Reviewed-by: Enrico Granata <egranata@chromium.org>
-rw-r--r-- | driver/accelgyro_lsm6dsm.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/driver/accelgyro_lsm6dsm.c b/driver/accelgyro_lsm6dsm.c index e45b4a6fcc..237cd2cf9b 100644 --- a/driver/accelgyro_lsm6dsm.c +++ b/driver/accelgyro_lsm6dsm.c @@ -410,6 +410,8 @@ static int set_data_rate(const struct motion_sensor_t *s, int rate, int rnd) reg_val++; normalized_rate = LSM6DSM_REG_TO_ODR(reg_val); } + if (normalized_rate == 0) + return EC_RES_INVALID_PARAM; } else { reg_val = 0; normalized_rate = 0; |