From 5147ef47114825b4f98742c2968f1a1051fdf7c4 Mon Sep 17 00:00:00 2001 From: Scott Chao Date: Fri, 20 Nov 2020 11:20:12 +0800 Subject: eldrid: ignore PROCHOT when system off BUG=b:173180788 BRANCH=firmware-volteer-13521.B-master TEST=make buildall Signed-off-by: Scott Chao Change-Id: I0896dc0e9dcb45a232d7109588b95bf4ee1e6bb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2550110 Reviewed-by: Keith Short --- common/throttle_ap.c | 10 ++++++++++ 1 file changed, 10 insertions(+) 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) { -- cgit v1.2.1