diff options
author | Karthikeyan Ramasubramanian <kramasub@chromium.org> | 2019-06-12 11:30:49 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-07-20 00:49:26 +0000 |
commit | 5936367096413abf294aa4584df53ec80bf8ef4c (patch) | |
tree | 22b1ef0ea7e81953d5a4f32ebc3c9520d15e5572 /util | |
parent | 1251719cc5fd3ca23ace14fe627b010e670b3cf9 (diff) | |
download | chrome-ec-5936367096413abf294aa4584df53ec80bf8ef4c.tar.gz |
ec_commands: Rename 'renew' to 'live' in EC_CMD_USB_PD_CHIP_INFO
Semantics of renew field in EC_CMD_USB_PD_CHIP_INFO is changing as
follows:
0 -> Return hard-coded info for Vendor ID/Product ID and cached info for
the Firmware Version
1 -> Return the live chip info for Vendor ID/Product ID/Firmware Version
Also rename the 'renew' field to 'live' to match the new semantics.
BUG=b:128820536,b:119046668
BRANCH=None
TEST=make -j buildall; Boot to ChromeOS.
Change-Id: Ie3dd022336b0be5c9728bb0ebabef32b7a6b5d57
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1617893
Tested-by: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Diana Z <dzigterman@chromium.org>
Commit-Queue: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Auto-Submit: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Diffstat (limited to 'util')
-rw-r--r-- | util/ectool.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/util/ectool.c b/util/ectool.c index fa08cb7917..46f98be760 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -8259,7 +8259,12 @@ int cmd_pd_chip_info(int argc, char *argv[]) int cmdver = 1; if (argc < 2 || 3 < argc) { - fprintf(stderr, "Usage: %s <port> [renew(on/off)]\n", argv[0]); + fprintf(stderr, "Usage: %s <port> [<live>]\n" + "live parameter can take values 0 or 1\n" + "0 -> Return hard-coded value for VID/PID and\n" + " cached value for Firmware Version\n" + "1 -> Return live chip value for VID/PID/FW Version\n", + argv[0]); return -1; } @@ -8269,14 +8274,13 @@ int cmd_pd_chip_info(int argc, char *argv[]) return -1; } - p.renew = 0; + p.live = 0; if (argc == 3) { - int val; - if (!parse_bool(argv[2], &val)) { + p.live = strtol(argv[2], &e, 0); + if (e && *e) { fprintf(stderr, "invalid arg \"%s\"\n", argv[2]); return -1; } - p.renew = val; } if (!ec_cmd_version_supported(EC_CMD_PD_CHIP_INFO, cmdver)) |