commit 47e26b1b "cmd_usage(): simplify return code handling" caused the following compile warnings:
bedbug_860.c: In function 'bedbug860_do_break': bedbug_860.c:73: warning: 'return' with a value, in function returning void bedbug_860.c:121: warning: 'return' with a value, in function returning void
Fix the return type.
Actually the while file could need some cleanup - commands should return proper error codes, and there are coding style issues. => To be fixed later.
Signed-off-by: Wolfgang Denk wd@denx.de --- arch/powerpc/cpu/mpc8xx/bedbug_860.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/cpu/mpc8xx/bedbug_860.c b/arch/powerpc/cpu/mpc8xx/bedbug_860.c index 83db035..1a9c0ba 100644 --- a/arch/powerpc/cpu/mpc8xx/bedbug_860.c +++ b/arch/powerpc/cpu/mpc8xx/bedbug_860.c @@ -69,8 +69,10 @@ void bedbug860_do_break (cmd_tbl_t *cmdtp, int flag, int argc, int which_bp; /* Breakpoint number */ /* -------------------------------------------------- */
- if (argc < 2) - return cmd_usage(cmdtp); + if (argc < 2) { + cmd_usage(cmdtp); + return; + }
/* Turn off a breakpoint */