faft-cr50: skip tests that put the ec into hibernate with c2d2
EC hibernate powers down cr50 with c2d2, so we can't verify cr50
behavior with the EC in hibernate. Skip tests that put the EC into
hiberante with c2d2
Revert this once the bug is fixed.
BUG=b:186535695
TEST=run on dedede
Change-Id: I4cb3848325a2ed340d6cfd374f080a134f533b01
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2853775
Tested-by: Mary Ruthven <mruthven@chromium.org>
Auto-Submit: Mary Ruthven <mruthven@chromium.org>
Reviewed-by: Aadit Maheshkumar Modi <modia@chromium.org>
Commit-Queue: Aadit Maheshkumar Modi <modia@chromium.org>
diff --git a/server/site_tests/firmware_Cr50ECReset/firmware_Cr50ECReset.py b/server/site_tests/firmware_Cr50ECReset/firmware_Cr50ECReset.py
index 2c2f11e..ff019e5 100644
--- a/server/site_tests/firmware_Cr50ECReset/firmware_Cr50ECReset.py
+++ b/server/site_tests/firmware_Cr50ECReset/firmware_Cr50ECReset.py
@@ -34,6 +34,12 @@
raise error.TestNAError("This DUT has a hardware limitation that "
"prevents cr50 from waking the EC with "
"EC_RST_L.")
+
+ # TODO(b/186535695): EC hibernate puts cr50 into reset, so the test
+ # can't verify cr50 behavior while the EC is hibernate.
+ if 'c2d2' in self.servo.get_servo_type():
+ raise error.TestNAError('Cannot run test with c2d2')
+
# Don't bother if there is no Chrome EC or if EC hibernate doesn't work.
if not self.check_ec_capability():
raise error.TestNAError("Nothing needs to be tested on this device")
diff --git a/server/site_tests/firmware_Cr50RddG3/firmware_Cr50RddG3.py b/server/site_tests/firmware_Cr50RddG3/firmware_Cr50RddG3.py
index c953478..fb8b9d5 100644
--- a/server/site_tests/firmware_Cr50RddG3/firmware_Cr50RddG3.py
+++ b/server/site_tests/firmware_Cr50RddG3/firmware_Cr50RddG3.py
@@ -17,6 +17,16 @@
# Cr50 debounces disconnects. We need to wait before checking Rdd state
RDD_DEBOUNCE = 3
+ def initialize(self, host, cmdline_args, full_args):
+ """Initialize the test"""
+ super(firmware_Cr50RddG3, self).initialize(host, cmdline_args,
+ full_args)
+
+ # TODO(b/186535695): EC hibernate puts cr50 into reset, so the test
+ # can't verify cr50 behavior while the EC is hibernate.
+ if 'c2d2' in self.servo.get_servo_type():
+ raise error.TestNAError('Cannot run test with c2d2')
+
def rdd_is_connected(self):
"""Return True if Cr50 detects Rdd."""
time.sleep(2)
