diff options
author | Scott Chao <scott_chao@wistron.corp-partner.google.com> | 2020-11-20 11:20:12 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-12-11 17:40:51 +0000 |
commit | 5147ef47114825b4f98742c2968f1a1051fdf7c4 (patch) | |
tree | ba990ad3c6ec298f572e4137600dc69bebc60a7f /common/throttle_ap.c | |
parent | d7bf71b42b78fa09da12f764bd3571d0cc446765 (diff) | |
download | chrome-ec-5147ef47114825b4f98742c2968f1a1051fdf7c4.tar.gz |
eldrid: ignore PROCHOT when system off
BUG=b:173180788
BRANCH=firmware-volteer-13521.B-master
TEST=make buildall
Signed-off-by: Scott Chao <scott_chao@wistron.corp-partner.google.com>
Change-Id: I0896dc0e9dcb45a232d7109588b95bf4ee1e6bb8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2550110
Reviewed-by: Keith Short <keithshort@chromium.org>
Diffstat (limited to 'common/throttle_ap.c')
-rw-r--r-- | common/throttle_ap.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/common/throttle_ap.c b/common/throttle_ap.c index 8a4314db4d..e07fa42357 100644 --- a/common/throttle_ap.c +++ b/common/throttle_ap.c @@ -95,6 +95,16 @@ static void prochot_input_deferred(void) if (prochot_in == debounced_prochot_in) return; + /* + * b/173180788 Confirmed from Intel internal that SLP_S3# asserts low + * about 10us before PROCHOT# asserts low, which means that + * the CPU is already in reset and therefore the PROCHOT# + * asserting low is normal behavior and not a concern + * for PROCHOT# event. Ignore all PROCHOT changes while the AP is off + */ + if (chipset_in_state(CHIPSET_STATE_ANY_OFF)) + return; + debounced_prochot_in = prochot_in; if (debounced_prochot_in) { |