tree a2d7bb27995e39331ecb37f208e8c101542f3410
parent f3f63b5971a4c0eb4e88c96d2e1d9cec71a0f3f7
author Ruben Rodriguez Buchillon <coconutruben@chromium.org> 1569880165 -0700
committer Commit Bot <commit-bot@chromium.org> 1580966722 +0000

servo: abstract dts mode logic

This change adds a few helpers to handle dts mode

- dts_mode_is_valid: whether dts mode can be supported.
- dts_mode_is_safe: whether controlling dts mode will remove the
main device's console access

- get_dts_mode: helper to get the dts mode

BUG=chromium:1009616

TEST=test_that --autotest_dir . $DIP firmware_Cr50CCDServoCap

/tmp/test_that_results_b2EdTg/results-1-firmware_Cr50CCDServoCap
[  PASSED  ]

Note: all tests below are done by adding two logging statements into
initialize_dut() and subsequently asseting False to speed debug.

TEST=sudo servod -b atlas // v4 (type-c) + micro
     test_that --autotest_dir . $AIP power_Monitoring -b atlas

12:03:08 INFO | autoserv| TYPE: servo_v4_with_servo_micro
12:03:08 INFO | autoserv| DTS MODE: True

TEST=sudo servod -b atlas // v4 (type-c) + ccd
     test_that --autotest_dir . $AIP power_Monitoring -b atlas

12:05:20 INFO | autoserv| TYPE: servo_v4_with_ccd_cr50
12:05:20 INFO | autoserv| DTS MODE: True

TEST=sudo servod -b atlas // v4 (type-A) + micro
     test_that --autotest_dir . $AIP power_Monitoring -b atlas

12:06:49 INFO | autoserv| TYPE: servo_v4_with_servo_micro
12:06:50 INFO | autoserv| DTS controls require a type-c servo v4.
12:06:50 INFO | autoserv| DTS MODE: False

TEST=sudo servod -b atlas // v2
     test_that --autotest_dir . $AIP power_Monitoring -b atlas

12:08:19 INFO | autoserv| TYPE: servo_v2
12:08:19 INFO | autoserv| DTS MODE: False

Change-Id: I7f342691bf84896df5a779555bf33334ff1c17b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1835913
Tested-by: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Commit-Queue: Ruben Rodriguez Buchillon <coconutruben@chromium.org>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>