diff options
author | Gwendal Grignou <gwendal@chromium.org> | 2021-01-07 11:44:27 -0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-18 07:06:41 +0000 |
commit | 41e3971aeb68038f78c7e5c3f2a1aaa66433599e (patch) | |
tree | 1d78481a841e2c4ce6bff962bacd167b9beafff4 /board/kakadu/board.h | |
parent | e80d8466d9b6096d4e6f50200cad755c86c5c5d8 (diff) | |
download | chrome-ec-41e3971aeb68038f78c7e5c3f2a1aaa66433599e.tar.gz |
common: motion: fix races at shutdown
Do not use collection_rate blindly after a function may have slept:
the HOOK task could have run suspend() or suspend() call and set it to
0.
Fixes 104f5257 ("motion: Control on which task sensor setting functions are running on")
[CL:2553347]
Conflicts:
common/motion_sense.c: Remove indentation that must have happen
at merge.
BUG=b:176918310, b:170703322, b:251069579
BRANCH=kukui
TEST=unit test.
Change-Id: I9ef13ceca195db4b48866f1e53f9408fb2bbf595
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2616137
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
(cherry picked from commit ea99e40f31445e67659c09b32cab6857cad8b83e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3961714
Commit-Queue: Eric Yilun Lin <yllin@google.com>
Tested-by: Eric Yilun Lin <yllin@google.com>
Reviewed-by: Eric Yilun Lin <yllin@google.com>
Diffstat (limited to 'board/kakadu/board.h')
0 files changed, 0 insertions, 0 deletions