diff options
author | Thomas Weber <weber@corscience.de> | 2010-11-25 08:05:30 +0100 |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2010-11-27 23:35:14 +0100 |
commit | e03dabe86b3bcb5be9e6d325f10747e3115cf4d1 (patch) | |
tree | 520f5e9c67728070b82b90c2ae0272718152eae4 /arch | |
parent | fd2aa8e2e7cba861097befed7ebd268956211041 (diff) |
powerpc: cmd_ecctest: 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 'arch')
-rw-r--r-- | arch/powerpc/cpu/ppc4xx/cmd_ecctest.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/arch/powerpc/cpu/ppc4xx/cmd_ecctest.c b/arch/powerpc/cpu/ppc4xx/cmd_ecctest.c index fd81b70e7a..542ab69a13 100644 --- a/arch/powerpc/cpu/ppc4xx/cmd_ecctest.c +++ b/arch/powerpc/cpu/ppc4xx/cmd_ecctest.c @@ -190,15 +190,13 @@ static int do_ecctest(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) int error; if (argc < 3) { - cmd_usage(cmdtp); - return 1; + return cmd_usage(cmdtp); } ptr = (u32 *)simple_strtoul(argv[1], NULL, 16); error = simple_strtoul(argv[2], NULL, 16); if ((error < 1) || (error > 2)) { - cmd_usage(cmdtp); - return 1; + return cmd_usage(cmdtp); } printf("Using address %p for %d bit ECC error injection\n", |