From e03dabe86b3bcb5be9e6d325f10747e3115cf4d1 Mon Sep 17 00:00:00 2001 From: Thomas Weber Date: Thu, 25 Nov 2010 08:05:30 +0100 Subject: [PATCH] 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 Acked-by: Mike Frysinger --- arch/powerpc/cpu/ppc4xx/cmd_ecctest.c | 6 ++---- 1 file 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", -- 2.30.2