diff options
| author | Vincent Palatin <vpalatin@chromium.org> | 2015-04-17 09:43:33 -0700 |
|---|---|---|
| committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-04-17 19:38:11 +0000 |
| commit | 0df89000a5bcfc8307a3f8bc85b0fdbe6653da6c (patch) | |
| tree | 2ed62b93a390a034ad24baaa99f760efa41a4db3 /util | |
| parent | 78de9268b7fe9d96c3324c22b3bf4f88901f2372 (diff) | |
| download | chrome-ec-0df89000a5bcfc8307a3f8bc85b0fdbe6653da6c.tar.gz | |
pd: add a connection flag to pd port info for host
Export what the PD protocol stack thinks about the port connection
state. This simplifies getting a meaningful data role/power role from the
host (eg we are not really a UFP if we are simply dual-role toggling but
not connected).
Do not increment the command version as this is mostly
backward-compatible and currently no client actually uses that field.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
BRANCH=samus
BUG=none
TEST=ectool --name=cros_pd usbpd 0
plug and unplug various accessories on the port and check the result.
Change-Id: Ief3e0d47b6a288bcfc5b8fbb8156f29fd09dd336
Reviewed-on: https://chromium-review.googlesource.com/266120
Trybot-Ready: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Alec Berg <alecaberg@chromium.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
Commit-Queue: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'util')
| -rw-r--r-- | util/ectool.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/util/ectool.c b/util/ectool.c index d7e3cf6907..b173d7ad19 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -3710,8 +3710,9 @@ int cmd_usb_pd(int argc, char *argv[]) r->role == PD_ROLE_SOURCE ? "SRC" : "SNK", r->polarity + 1, r->state); } else { - printf("Port C%d is %s, Role:%s %s Polarity:CC%d State:%s\n", - p.port, (r_v1->enabled) ? "enabled" : "disabled", + printf("Port C%d is %s,%s, Role:%s %s Polarity:CC%d State:%s\n", + p.port, (r_v1->enabled & 1) ? "enabled" : "disabled", + (r_v1->enabled & 2) ? "connected" : "disconnected", r_v1->role & PD_ROLE_SOURCE ? "SRC" : "SNK", r_v1->role & (PD_ROLE_DFP << 1) ? "DFP" : "UFP", r_v1->polarity + 1, r_v1->state); |