diff options
| author | Thomas Weber | 2010-11-25 08:05:29 +0100 |
|---|---|---|
| committer | Wolfgang Denk | 2010-11-27 23:35:14 +0100 |
| commit | fd2aa8e2e7cba861097befed7ebd268956211041 (patch) | |
| tree | 8dfe2f53e328d6aca606a45a638e113969d2ba90 /board/gdsys/common | |
| parent | 0d302af0bb7e0be40d960bd2d8ee0a91dc9e2bbf (diff) | |
gdsys: osd: Use return value of cmd_usage
Use the return value of cmd_usage instead of ignoring this
and returning a 1.
Signed-off-by: Thomas Weber <weber@corscience.de>
Acked-by: Mike Frysinger <vapier@gentoo.org>
Diffstat (limited to 'board/gdsys/common')
| -rw-r--r-- | board/gdsys/common/osd.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/board/gdsys/common/osd.c b/board/gdsys/common/osd.c index 05800ffba48..239c870eff7 100644 --- a/board/gdsys/common/osd.c +++ b/board/gdsys/common/osd.c @@ -142,8 +142,7 @@ static int osd_print(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) char *text; if (argc < 5) { - cmd_usage(cmdtp); - return 1; + return cmd_usage(cmdtp); } x = simple_strtoul(argv[1], NULL, 16); @@ -204,8 +203,7 @@ int osd_write(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) unsigned count = (argc > 4) ? simple_strtoul(argv[4], NULL, 16) : 1; if ((argc < 4) || (strlen(argv[3]) % 4)) { - cmd_usage(cmdtp); - return 1; + return cmd_usage(cmdtp); } x = simple_strtoul(argv[1], NULL, 16); |