mirror of
https://github.com/Telecominfraproject/OpenCellular.git
synced 2026-01-12 10:55:21 +00:00
sensor: bmi160/bma2x2: Fix the timeout value of compensation.
CL:957872 introduced "get_time().val + 400" as the deadline to perform compensation and the comment in bma2x2.c mentioned the deadline should be 400ms. But the unit of val in timestamp_t is microsecond not milisecond so only 400us is defined not 400ms. BRANCH=none BUG=b:76234078 BUG=b:76202592 TEST=test manually on the dut by performing calibrate. Change-Id: I7a834ef6dcb0772569d2c8d6c507803deb5d2fc1 Signed-off-by: Marco Chen <marcochen@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/979512 Commit-Ready: Gwendal Grignou <gwendal@chromium.org> Tested-by: Gwendal Grignou <gwendal@chromium.org> Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
This commit is contained in:
@@ -235,7 +235,7 @@ static int perform_calib(const struct motion_sensor_t *s)
|
||||
* The sensor needs 16 samples. At 100Hz/10ms, it needs 160ms to
|
||||
* complete. Set 400ms to have some margin.
|
||||
*/
|
||||
deadline.val = get_time().val + 400;
|
||||
deadline.val = get_time().val + 400 * MSEC;
|
||||
do {
|
||||
if (timestamp_expired(deadline, NULL)) {
|
||||
ret = EC_RES_TIMEOUT;
|
||||
|
||||
@@ -636,7 +636,7 @@ static int perform_calib(const struct motion_sensor_t *s)
|
||||
ret = raw_write8(s->port, s->addr, BMI160_FOC_CONF, val);
|
||||
ret = raw_write8(s->port, s->addr, BMI160_CMD_REG,
|
||||
BMI160_CMD_START_FOC);
|
||||
deadline.val = get_time().val + 400;
|
||||
deadline.val = get_time().val + 400 * MSEC;
|
||||
do {
|
||||
if (timestamp_expired(deadline, NULL)) {
|
||||
ret = EC_RES_TIMEOUT;
|
||||
|
||||
Reference in New Issue
Block a user