diff options
| author | Gwendal Grignou <gwendal@chromium.org> | 2021-01-28 13:46:25 -0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-01-29 19:44:52 +0000 |
| commit | aed20b2a58f9fc2eb29d9114b7622286dcb97a7b (patch) | |
| tree | 43d1022f23446aafd0d8ff240c7f781cec543415 | |
| parent | 1ba4f4c7dc0919b648f2079a00c3183fbb1d3c6f (diff) | |
| download | chrome-ec-aed20b2a58f9fc2eb29d9114b7622286dcb97a7b.tar.gz | |
driver: icm426xx: Use calibration unit to set/get offset
EC interface use constant (not range dependent) units to get/set
offsets. Use them in icm426xx driver.
BUG=b:177292639
BRANCH=hatch,nami,kukui,dedede,grunt,zork,octopus,volteer
TEST=compile
Change-Id: I6e6b1551464ea389db34646ba5b2bb553d683d7a
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2657955
Reviewed-by: Ching-Kang Yen <chingkang@chromium.org>
Reviewed-by: Jean-Baptiste Maneyrol <jmaneyrol@invensense.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2659407
| -rw-r--r-- | driver/accelgyro_icm426xx.c | 39 |
1 files changed, 15 insertions, 24 deletions
diff --git a/driver/accelgyro_icm426xx.c b/driver/accelgyro_icm426xx.c index 56d8fa5bf9..c4f6f42133 100644 --- a/driver/accelgyro_icm426xx.c +++ b/driver/accelgyro_icm426xx.c @@ -658,28 +658,24 @@ out_unlock: static int icm426xx_set_offset(const struct motion_sensor_t *s, const int16_t *offset, int16_t temp) { - struct accelgyro_saved_data_t *data = ICM_GET_SAVED_DATA(s); intv3_t v = { offset[X], offset[Y], offset[Z] }; - int range, div1, div2; + int div1, div2; int i; - /* unscale values and rotate back to chip frame */ - for (i = X; i <= Z; ++i) - v[i] = SENSOR_APPLY_DIV_SCALE(v[i], data->scale[i]); + /* rotate back to chip frame */ rotate_inv(v, *s->rot_standard_ref, v); /* convert raw data to hardware offset units */ - range = icm_get_range(s); switch (s->type) { case MOTIONSENSE_TYPE_ACCEL: - /* hardware offset is 1/2048g by LSB */ - div1 = range * 2048; - div2 = 32768; + /* hardware offset is 1/2048g /LSB, EC offset 1/1024g /LSB. */ + div1 = 2; + div2 = 1; break; case MOTIONSENSE_TYPE_GYRO: - /* hardware offset is 1/32dps by LSB */ - div1 = range * 32; - div2 = 32768; + /* hardware offset is 1/32dps /LSB, EC offset 1/1024dps /LSB. */ + div1 = 1; + div2 = 32; break; default: return EC_ERROR_INVAL; @@ -693,9 +689,8 @@ static int icm426xx_set_offset(const struct motion_sensor_t *s, static int icm426xx_get_offset(const struct motion_sensor_t *s, int16_t *offset, int16_t *temp) { - struct accelgyro_saved_data_t *data = ICM_GET_SAVED_DATA(s); intv3_t v; - int range, div1, div2; + int div1, div2; int i, ret; ret = icm426xx_get_hw_offset(s, v); @@ -703,17 +698,16 @@ static int icm426xx_get_offset(const struct motion_sensor_t *s, int16_t *offset, return ret; /* transform offset to raw data */ - range = icm_get_range(s); switch (s->type) { case MOTIONSENSE_TYPE_ACCEL: - /* hardware offset is 1/2048g by LSB */ - div1 = 32768; - div2 = 2048 * range; + /* hardware offset is 1/2048g /LSB, EC offset 1/1024g /LSB. */ + div1 = 1; + div2 = 2; break; case MOTIONSENSE_TYPE_GYRO: - /* hardware offset is 1/32dps by LSB */ - div1 = 32768; - div2 = 32 * range; + /* hardware offset is 1/32dps /LSB, EC offset 1/1024dps /LSB. */ + div1 = 32; + div2 = 1; break; default: return EC_ERROR_INVAL; @@ -722,9 +716,6 @@ static int icm426xx_get_offset(const struct motion_sensor_t *s, int16_t *offset, v[i] = round_divide(v[i] * div1, div2); rotate(v, *s->rot_standard_ref, v); - for (i = X; i <= Z; i++) - v[i] = SENSOR_APPLY_SCALE(v[i], data->scale[i]); - offset[X] = v[X]; offset[Y] = v[Y]; offset[Z] = v[Z]; |