From a796f04b1acd6602266e210a4eb571b778c56721 Mon Sep 17 00:00:00 2001 From: Gwendal Grignou Date: Fri, 6 Apr 2018 14:44:31 -0700 Subject: 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" Change-Id: Iead740f4205bbce1cfbccf2407f2a3a0dcf0ddaf Signed-off-by: Gwendal Grignou Reviewed-on: https://chromium-review.googlesource.com/1000399 Reviewed-by: Aseda Aboagye --- driver/accelgyro_lsm6dsm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/driver/accelgyro_lsm6dsm.c b/driver/accelgyro_lsm6dsm.c index eaccda78ca..83b6f27021 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 *= 2; } + if (normalized_rate == 0) + return EC_RES_INVALID_PARAM; } else { reg_val = 0; normalized_rate = 0; -- cgit v1.2.1