Add a 1.0 second delay after all long power presses.
On Alex, after a long power button press, the EC seems to need a
short grace period before it recognizes a subsequent power button
press. The delay is required whenever DUT software fails to shut
down within the 8 seconds, causing the EC to turn off power instead.
For simplicity, the change simply adds a delay after every long
power press.
BUG=chromium-os:32494
TEST=platform_InstallTestImage
Change-Id: I8244e60c3365fdd2db6245735cadfeb1e0a2132e
Reviewed-on: https://gerrit.chromium.org/gerrit/27822
Reviewed-by: Chris Masone <cmasone@chromium.org>
Tested-by: Richard Barnette <jrbarnette@chromium.org>
Commit-Ready: Richard Barnette <jrbarnette@chromium.org>
diff --git a/server/cros/servo.py b/server/cros/servo.py
index 5969321..bf73bb9 100644
--- a/server/cros/servo.py
+++ b/server/cros/servo.py
@@ -114,7 +114,12 @@
def power_long_press(self):
"""Simulate a long power button press."""
+ # After a long power press, the EC may ignore the next power
+ # button press (at least on Alex). To guarantee that this
+ # won't happen, we need to allow the EC one second to
+ # collect itself.
self.power_key(Servo.LONG_DELAY)
+ time.sleep(1.0)
def power_normal_press(self):