diff options
author | Randall Spangler <rspangler@chromium.org> | 2012-05-21 10:29:23 -0700 |
---|---|---|
committer | Randall Spangler <rspangler@chromium.org> | 2012-05-21 14:57:08 -0700 |
commit | b2c4ee6cf88baffa7f69b1551fcda0e468bae216 (patch) | |
tree | 57d84af558f9210220668dc159770b4571aa5710 /common/eoption.c | |
parent | 1ac197d566d74b5dfd2abbd57c191e464aa46d8b (diff) | |
download | chrome-ec-b2c4ee6cf88baffa7f69b1551fcda0e468bae216.tar.gz |
Even more debug command cleanup to save space
BUG=none
TEST=(run the commands)
Change-Id: Ibc414ffd594e06dbdce64c51859b6f247bb10d36
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'common/eoption.c')
-rw-r--r-- | common/eoption.c | 21 |
1 files changed, 6 insertions, 15 deletions
diff --git a/common/eoption.c b/common/eoption.c index f25efcd68d..83a94647fc 100644 --- a/common/eoption.c +++ b/common/eoption.c @@ -140,17 +140,14 @@ static int command_eoption_get(int argc, char **argv) /* If a signal is specified, print only that one */ if (argc == 2) { i = find_option_by_name(argv[1], bool_opts); - if (i == -1) { - ccputs("Unknown option.\n"); + if (i == -1) return EC_ERROR_INVAL; - } d = bool_opts + i; ccprintf(" %d %s\n", eoption_get_bool(i), d->name); return EC_SUCCESS; } /* Otherwise print them all */ - ccputs("Boolean options:\n"); for (i = 0, d = bool_opts; d->name; i++, d++) { ccprintf(" %d %s\n", eoption_get_bool(i), d->name); @@ -168,23 +165,17 @@ static int command_eoption_set(int argc, char **argv) char *e; int v, i; - if (argc < 3) { - ccputs("Usage: optset <option> <value>\n"); + if (argc < 3) return EC_ERROR_INVAL; - } v = strtoi(argv[2], &e, 0); - if (*e) { - ccputs("Invalid value.\n"); + if (*e) return EC_ERROR_INVAL; - } i = find_option_by_name(argv[1], bool_opts); - if (i != -1) { - return eoption_set_bool(i, v); - } else { - ccputs("Unknown option.\n"); + if (i == -1) return EC_ERROR_INVAL; - } + + return eoption_set_bool(i, v); } DECLARE_CONSOLE_COMMAND(optset, command_eoption_set); |