diff options
author | Simon Glass <sjg@chromium.org> | 2020-11-01 14:15:36 -0700 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2020-12-01 10:33:38 -0500 |
commit | 7526deec7e52da92dee25aaf3a47514e639c2bb0 (patch) | |
tree | 7d6d7ebf4b1e2421986cd85ba96bfba8636a3a90 /cmd | |
parent | 90a9901764d71308192c96ecd7d735531fe9cc30 (diff) | |
download | u-boot-7526deec7e52da92dee25aaf3a47514e639c2bb0.zip u-boot-7526deec7e52da92dee25aaf3a47514e639c2bb0.tar.gz u-boot-7526deec7e52da92dee25aaf3a47514e639c2bb0.tar.bz2 |
command: Add constants for cmd_get_data_size string / error
At present these values are open-coded in a few places. Add constants so
the meaning is clear.
Also add a comment to cmd_get_data_size()
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/itest.c | 4 | ||||
-rw-r--r-- | cmd/mem.c | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/cmd/itest.c b/cmd/itest.c index a0cf4be..9a441ce 100644 --- a/cmd/itest.c +++ b/cmd/itest.c @@ -197,10 +197,10 @@ static int do_itest(struct cmd_tbl *cmdtp, int flag, int argc, #endif value = binary_test (argv[2], argv[1], argv[3], w); break; - case -2: + case CMD_DATA_SIZE_STR: value = binary_test (argv[2], argv[1], argv[3], 0); break; - case -1: + case CMD_DATA_SIZE_ERR: default: puts("Invalid data width specifier\n"); value = 0; @@ -393,7 +393,7 @@ static int do_mem_search(struct cmd_tbl *cmdtp, int flag, int argc, * Defaults to long if no or incorrect specification. */ size = cmd_get_data_size(argv[0], 4); - if (size < 0 && size != -2 /* string */) + if (size < 0 && size != CMD_DATA_SIZE_STR) return 1; argc--; |