diff options
author | Gwendal Grignou <gwendal@chromium.org> | 2015-11-20 10:25:19 -0800 |
---|---|---|
committer | ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> | 2015-11-25 00:45:43 +0000 |
commit | 3e7aba4e78a2edf6f41d51b97c3ca98a142666e9 (patch) | |
tree | 4fbfd40fa54b703da5cd611fd867f029c4e8812b | |
parent | 461f2cc2d05f0b2d7a1ca543d3fb0db9e13c603e (diff) | |
download | chrome-ec-3e7aba4e78a2edf6f41d51b97c3ca98a142666e9.tar.gz |
motion: fix ec_rate to be more accurate
In case the actual ODR rate is way higher that the AP asked for,
we don't have to settle to a slower EC rate if
EC rate == AP requested ODR rate.
BRANCH=smaug
BUG=none
TEST=Run android.hardware.cts.SingleSensorTests
Change-Id: I437f47bd942a16694c7efcdbc00201352f0480a6
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/313641
Reviewed-by: Alec Berg <alecaberg@chromium.org>
(cherry picked from commit 68502864c7db47b0dae250600dec5531c5f35619)
Reviewed-on: https://chromium-review.googlesource.com/313648
-rw-r--r-- | common/motion_sense.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/motion_sense.c b/common/motion_sense.c index 7be3340293..0c6e461995 100644 --- a/common/motion_sense.c +++ b/common/motion_sense.c @@ -248,7 +248,7 @@ static int motion_sense_set_ec_rate_from_ap( const struct motion_sensor_t *sensor, unsigned int new_rate_us) { - int ap_odr_mhz = BASE_ODR(sensor->config[SENSOR_CONFIG_AP].odr); + int odr_mhz = sensor->drv->get_data_rate(sensor); if (new_rate_us == 0) return 0; @@ -256,7 +256,7 @@ static int motion_sense_set_ec_rate_from_ap( if (CONFIG_ACCEL_FORCE_MODE_MASK & (1 << (sensor - motion_sensors))) goto end_set_ec_rate_from_ap; #endif - if (ap_odr_mhz == 0) + if (odr_mhz == 0) goto end_set_ec_rate_from_ap; /* @@ -270,7 +270,7 @@ static int motion_sense_set_ec_rate_from_ap( * We wll apply that correction only if the ec rate is within 10% of * the data rate. */ - if (SECOND * 1100 / ap_odr_mhz > new_rate_us) + if (SECOND * 1100 / odr_mhz > new_rate_us) new_rate_us = new_rate_us / 100 * 105; end_set_ec_rate_from_ap: |