servo: skip enabling the default device on servo v4
active_v4_device checks if cr50 knows servo is connected to determine if
ccd is enabled and interfering with servo micro. This doesn't work on
sarien, because there's no ec and servo detection doesn't work. There's
a change to fix this making active_v4_device check if ccd has TX
enabledd. This check should work on sarien, but it doesn't work in the
current labstation image. Skip enabling the main device until the
labstation gets updated.
BUG=chromium:1016842
BRANCH=none
TEST=run on sarien
Change-Id: Ie2867bc300fb1bfa7379e38d29fc1a265ab9a33b
Signed-off-by: Mary Ruthven <mruthven@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1875071
Reviewed-by: Namyoon Woo <namyoon@chromium.org>
diff --git a/server/cros/servo/servo.py b/server/cros/servo/servo.py
index f194d65..f37595f 100644
--- a/server/cros/servo/servo.py
+++ b/server/cros/servo/servo.py
@@ -977,6 +977,13 @@
def enable_main_servo_device(self):
"""Make sure the main device has control of the dut."""
+ # Cr50 detects servo using the EC uart. It doesn't work well if the
+ # board doesn't use EC uart. The lab active_v4_device doesn't handle
+ # this correctly. Check ec_uart_pty before trying to change the active
+ # device.
+ # TODO(crbug.com/1016842): reenable the setting the main device when
+ # active device works on labstations.
+ return
if not self.has_control('active_v4_device'):
return
self.set('active_v4_device', self.get_main_servo_device())