commit 8e88a74b74b7335d26791e79e9ef48f96a42a647
author Otabek Kasimov <otabek@google.com> Mon Jan 11 18:03:13 2021 -0800
committer Commit Bot <commit-bot@chromium.org> Fri Jan 15 06:46:48 2021 +0000
tree 50384ee66e12667866160e33b7604344140793a1
parent db48601acf9d646e8f4b4c223e67da2d40cf02e6

autotest: Add check if CR50 fail detected due SBU voltage issue

BUG=b:173151794
TEST=run local

--GOOD DUT
test_that --args="servo_host=chromeos6-row18-rack3-labstation4 servo_port=9997 servo_serial=G1911051929" --board=eve --autotest_dir=.  chromeos6-row18-rack3-host3 servo_USBInstall

./server/autoserv -s --host-info-subdir host_info_store -m  chromeos6-row18-rack3-host3 --lab True --local-only-host-info True -R -r /tr/

--BAD DUT
test_that --args="servo_host=chromeos6-row9-rack20-labstation servo_port=9998 servo_serial=C1706311408" --board=eve --autotest_dir=. chromeos6-row9-rack20-host3 servo_USBInstall

High sbu voltage
./server/autoserv -s --host-info-subdir host_info_store -m  chromeos6-row12-rack17-host1 --lab True --local-only-host-info True -v -r /tr/

Low sbu voltage
./server/autoserv -s --host-info-subdir host_info_store -m  chromeos6-row2-rack5-host3 --lab True --local-only-host-info True -v -r /tr/

Change-Id: I1962b6e4e02b2e1e0769ed08acce3b535b475f9d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2602604
Commit-Queue: Otabek Kasimov <otabek@google.com>
Tested-by: Otabek Kasimov <otabek@google.com>
Reviewed-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-by: Garry Wang <xianuowang@chromium.org>

server/hosts/servo_host.py

diff --git a/server/hosts/servo_host.py b/server/hosts/servo_host.py
index a813055..13f202d 100644
--- a/server/hosts/servo_host.py
+++ b/server/hosts/servo_host.py
@@ -1371,6 +1371,8 @@
         start_servod = self.get_verifier_state('servod_job')
         create_servo = self.get_verifier_state('servod_connection')
         init_servo = self.get_verifier_state('servod_control')
+        cr50_low_sbu = self.get_verifier_state('cr50_low_sbu')
+        cr50_off = self.get_verifier_state('cr50_off')
         servo_topology = self.get_verifier_state('servo_topology')
         dut_connected = self.get_verifier_state('dut_connected')
         hub_connected = self.get_verifier_state('hub_connected')
@@ -1401,6 +1403,10 @@
         if hub_connected == hosts.VERIFY_FAILED:
             logging.info('Servo HUB not connected')
             return servo_constants.SERVO_STATE_DUT_NOT_CONNECTED
+        if cr50_low_sbu == hosts.VERIFY_FAILED:
+            return servo_constants.SERVO_STATE_SBU_LOW_VOLTAGE
+        if cr50_off == hosts.VERIFY_FAILED:
+            return servo_constants.SERVO_STATE_CR50_NOT_ENUMERATED
         if servo_topology == hosts.VERIFY_FAILED:
             return servo_constants.SERVO_STATE_TOPOLOGY_ISSUE