summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGwendal Grignou <gwendal@chromium.org>2018-04-06 14:44:31 -0700
committerCommit Bot <commit-bot@chromium.org>2019-05-30 05:47:50 +0000
commit0aee48bdaab94faf674daa69b327d9a6675090a1 (patch)
tree444017dd73575b8c4b2c222f0e1843546ece94df
parent10d9010689d7d627f919c5787b99965cc14b53a8 (diff)
downloadchrome-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.c2
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;