summaryrefslogtreecommitdiff
path: root/baseboard/octopus/baseboard.c
diff options
context:
space:
mode:
authorDiana Z <dzigterman@chromium.org>2019-04-18 14:10:58 -0600
committerchrome-bot <chrome-bot@chromium.org>2019-05-22 20:56:57 -0700
commitd3f26175e58a1686def18af02239b4693674bc8f (patch)
tree62a92e24c4f2f2d145b1053cd1ff0eac5104297b /baseboard/octopus/baseboard.c
parent55f12ab58518fb079a99d1ce4d9717a430d7b9ce (diff)
downloadchrome-ec-d3f26175e58a1686def18af02239b4693674bc8f.tar.gz
Casta: Increase board_hibernate() wait to avoid hard resets
When hibernating with AC plugged in, casta will try negotiating down to 5V. This negotiation may sometimes take longer than the currently allocated sleep, causing the port partner to go into error recovery (re-sending PS_RDY, followed by soft resets, followed by a hard reset). This hard reset will wake the system back up from hibernation. Increasing this sleep to 300 ms appears to give enough time for a GoodCRC to be send to the PS_RDY message from the charger, avoiding the hard reset. Note that hibernating on AC is not a customer or factory requirement, as the typical EC hibernate scenario is when there is no external power and the battery needs to be conserved. BRANCH=octopus BUG=b:130687403 TEST=flashed to casta EVT, ran hibernate key sequence several times with AC plugged in and verified system hibernated Change-Id: Ia40ee63f3cf9a244ba35b4cc700e2b41eea1dde5 Signed-off-by: Diana Z <dzigterman@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/1584768 Tested-by: YongBeum Ha <ybha@samsung.com> Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org> Reviewed-by: Jett Rink <jettrink@chromium.org> Reviewed-by: YongBeum Ha <ybha@samsung.com>
Diffstat (limited to 'baseboard/octopus/baseboard.c')
-rw-r--r--baseboard/octopus/baseboard.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/baseboard/octopus/baseboard.c b/baseboard/octopus/baseboard.c
index 68e52ffcf7..1f5abc55f4 100644
--- a/baseboard/octopus/baseboard.c
+++ b/baseboard/octopus/baseboard.c
@@ -382,5 +382,5 @@ void board_hibernate(void)
* with any PD contract renegotiation, and tcpm to put TCPC into low
* power mode if required.
*/
- msleep(200);
+ msleep(300);
}