diff options
author | Wolfgang Denk <wd@denx.de> | 2011-11-09 09:29:05 (GMT) |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2011-11-16 20:05:47 (GMT) |
commit | be51c5c98b8d705a1795d00668344cfade8d6545 (patch) | |
tree | b08574b6f180463f52c31ed0f75b62d6b9b70f3f /board/esd | |
parent | 8a2d1f246e14c6595642898568c54fbf1ef065ef (diff) | |
download | u-boot-fsl-qoriq-be51c5c98b8d705a1795d00668344cfade8d6545.tar.xz |
board/esd/cpci750/cpci750.c: Fix error handling
ThE code recorded error conditions but did not pass these on to the
higher level caller. Fixing this fixes also this build warning:
cpci750.c: In function 'do_loadpci':
cpci750.c:569:6: warning: variable 'status' set but not used
[-Wunused-but-set-variable]
Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Reinhard Arlt <reinhard.arlt@esd-electronics.com>
Diffstat (limited to 'board/esd')
-rw-r--r-- | board/esd/cpci750/cpci750.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/board/esd/cpci750/cpci750.c b/board/esd/cpci750/cpci750.c index f27d65e..08311c9 100644 --- a/board/esd/cpci750/cpci750.c +++ b/board/esd/cpci750/cpci750.c @@ -566,7 +566,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) volatile unsigned int *ptr; int count = 0; int count2 = 0; - int status; + int status = 0; char addr[16]; char str[] = "\\|/-"; char *local_args[2]; @@ -622,7 +622,7 @@ int do_loadpci(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) break; } - return 0; + return status; } U_BOOT_CMD( |