diff options
author | Thomas Betker <thomas.betker@freenet.de> | 2014-06-05 20:07:56 +0200 |
---|---|---|
committer | Tom Rini <trini@ti.com> | 2014-06-11 16:25:47 -0400 |
commit | 73671dad49bf2368959b7bf0e30ba931ea95565c (patch) | |
tree | ab8bfdab55cc81119814c5eb4a6ff782e88b3705 /arch/arm/cpu | |
parent | 32e2c42a83e8676511e178e0285ea634bc4564fd (diff) | |
download | u-boot-73671dad49bf2368959b7bf0e30ba931ea95565c.tar.gz u-boot-73671dad49bf2368959b7bf0e30ba931ea95565c.tar.xz u-boot-73671dad49bf2368959b7bf0e30ba931ea95565c.zip |
Check run_command() return code properly
run_command() returns 0 for success, 1 for failure. Fix places which
assume that failure is indicated by a negative return code.
Signed-off-by: Thomas Betker <thomas.betker@rohde-schwarz.com>
Acked-by: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
Tested-by: Stefan Roese <sr@denx.de>
Diffstat (limited to 'arch/arm/cpu')
-rw-r--r-- | arch/arm/cpu/arm926ejs/kirkwood/cpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c index 0937506268..da80240052 100644 --- a/arch/arm/cpu/arm926ejs/kirkwood/cpu.c +++ b/arch/arm/cpu/arm926ejs/kirkwood/cpu.c @@ -210,7 +210,7 @@ static void kw_sysrst_action(void) debug("Starting %s process...\n", __FUNCTION__); ret = run_command(s, 0); - if (ret < 0) + if (ret != 0) debug("Error.. %s failed\n", __FUNCTION__); else debug("%s process finished\n", __FUNCTION__); |