diff options
author | Gwendal Grignou <gwendal@chromium.org> | 2019-03-11 15:57:52 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-03-26 04:42:55 -0700 |
commit | bb266fc26fc05d4ab22de6ad7bce5b477c9f9140 (patch) | |
tree | f6ada087f62246c3a9547e649ac8846b0ed6d5ab /test/motion_lid.c | |
parent | 0bfc511527cf2aebfa163c63a1d028419ca0b0c3 (diff) | |
download | chrome-ec-bb266fc26fc05d4ab22de6ad7bce5b477c9f9140.tar.gz |
common: replace 1 << digits, with BIT(digits)
Requested for linux integration, use BIT instead of 1 <<
First step replace bit operation with operand containing only digits.
Fix an error in motion_lid try to set bit 31 of a signed integer.
BUG=None
BRANCH=None
TEST=compile
Change-Id: Ie843611f2f68e241f0f40d4067f7ade726951d29
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1518659
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'test/motion_lid.c')
-rw-r--r-- | test/motion_lid.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/motion_lid.c b/test/motion_lid.c index 3282812e86..c9c7570232 100644 --- a/test/motion_lid.c +++ b/test/motion_lid.c @@ -70,7 +70,7 @@ static int accel_set_data_rate(const struct motion_sensor_t *s, const int rate, const int rnd) { - test_data_rate[s - motion_sensors] = rate | (rnd ? ROUND_UP_FLAG : 0); + test_data_rate[s - motion_sensors] = rate; return EC_SUCCESS; } @@ -186,7 +186,7 @@ static int test_lid_angle(void) hook_notify(HOOK_CHIPSET_RESUME); msleep(1000); TEST_ASSERT(sensor_active == SENSOR_ACTIVE_S0); - TEST_ASSERT(accel_get_data_rate(lid) == (119000 | ROUND_UP_FLAG)); + TEST_ASSERT(accel_get_data_rate(lid) == 119000); TEST_ASSERT(motion_interval == TEST_LID_EC_RATE); /* |