[U-Boot] [RESEND PATCH 0/3] Fix command repeat
Thomas Betker
thomas.betker at freenet.de
Thu Jun 5 20:07:55 CEST 2014
I am resending the patches because they didn't make it to Patchwork the
first time, probably due to a configuration issue on my side.
The intent of these patches is to get command repeat to work again.
Currently, successful commands won't be repeated but failed commands
will -- neither is as expected.
The issue is that run_command() returns 0 on success, 1 on error. In
order to get command repeat to work, we need a variant which returns -1
on error and 0/1 (non-repeatable/repeatable) on succcess, the same way
as cli_simple_run_command() does.
Patch 2 adds the run_command_repeatable() function, and patch 3
replaces run_command() by run_command_repeatable() where necessary.
Patch 1 is a cleanup of places which call run_command(), but expect it
to return -1 on error. This is actually independent of the other two
patches -- it just came up when checking run_command() invocations in
general.
Best regards,
Thomas Betker
Thomas Betker (3):
Check run_command() return code properly
Add run_command_repeatable()
Use run_command_repeatable()
arch/arm/cpu/arm926ejs/kirkwood/cpu.c | 2 +-
board/gdsys/p1022/controlcenterd.c | 6 +-----
common/cli.c | 24 ++++++++++++++++++++++++
common/cli_simple.c | 2 +-
common/cmd_bedbug.c | 2 +-
common/cmd_bootm.c | 6 +-----
include/common.h | 1 +
7 files changed, 30 insertions(+), 13 deletions(-)
--
1.9.3
More information about the U-Boot
mailing list