diff options
| author | Tom Hughes <tomhughes@chromium.org> | 2019-01-28 14:12:34 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2019-01-30 01:25:49 -0800 |
| commit | 0a9e45cdb4544c7a9da21e256dc87f8bcbd3f477 (patch) | |
| tree | ac48876a66e5cac09acee019b0679651730dd2e9 /util/ectool.c | |
| parent | 08114a147764a980374aa9288c1d5f0cc8c07b5d (diff) | |
| download | chrome-ec-0a9e45cdb4544c7a9da21e256dc87f8bcbd3f477.tar.gz | |
ectool: remove legacy "reboot_ec A" flag
The "A" flag was added in 2012 and a TODO was added in 2013.
"repo grep reboot_ec" doesn't show any obvious places that the "A" flag
is still used.
BRANCH=none
BUG=b:35507625,b:35518769
TEST=make buildall -j
TEST=ectool --name=cros_fp reboot_ec; sleep 0.5;
ectool --name=cros_fp rwsigaction abort;
ectool --name=cros_fp version
TEST=ectool --name=cros_fp reboot_ec RW; ectool --name=cros_fp version
TEST=ectool --name=cros_fp reboot_ec A; echo $?
Change-Id: I33c46a4a846b38743c121db08feb84c8b02747d3
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1441031
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'util/ectool.c')
| -rw-r--r-- | util/ectool.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/util/ectool.c b/util/ectool.c index d8d8bf4388..7da67806c5 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -954,13 +954,9 @@ int cmd_reboot_ec(int argc, char *argv[]) p.cmd = EC_REBOOT_CANCEL; else if (!strcmp(argv[1], "RO")) p.cmd = EC_REBOOT_JUMP_RO; - else if (!strcmp(argv[1], "RW") || !strcmp(argv[1], "A")) { - /* - * TODO(crosbug.com/p/11149): remove "A" once all scripts are - * updated to use "RW". - */ + else if (!strcmp(argv[1], "RW")) p.cmd = EC_REBOOT_JUMP_RW; - } else if (!strcmp(argv[1], "cold")) + else if (!strcmp(argv[1], "cold")) p.cmd = EC_REBOOT_COLD; else if (!strcmp(argv[1], "disable-jump")) p.cmd = EC_REBOOT_DISABLE_JUMP; |