summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWolfgang Denk <wd@denx.de>2011-12-09 12:14:22 +0100
committerWolfgang Denk <wd@denx.de>2011-12-10 23:08:49 +0100
commit5c2db13a2aa4bdb2f335830f9ecbfc9ee673e53f (patch)
tree93f78e84f28c15e5dc735c31ddefefd2ba575234
parentf49620665d67e43d92dd4deac2ffc025a7049306 (diff)
downloadu-boot-imx-5c2db13a2aa4bdb2f335830f9ecbfc9ee673e53f.zip
u-boot-imx-5c2db13a2aa4bdb2f335830f9ecbfc9ee673e53f.tar.gz
u-boot-imx-5c2db13a2aa4bdb2f335830f9ecbfc9ee673e53f.tar.bz2
fs/yaffs2/yaffs_guts.c: Fix GCC 4.6 compile warning (and bug)
Fix: yaffs_guts.c: In function 'yaffs_GarbageCollectBlock': yaffs_guts.c:2761:6: warning: variable 'retVal' set but not used [-Wunused-but-set-variable] Here GCC actually detected a bug. The code was always returning OK instead of the previously set retrun code. Fix that. Signed-off-by: Wolfgang Denk <wd@denx.de> Cc: William Juul <wiljuul@cisco.com> Cc: Scott Wood <scottwood@freescale.com> Acked-by: William Juul <wiljuul@cisco.com>
-rw-r--r--fs/yaffs2/yaffs_guts.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/yaffs2/yaffs_guts.c b/fs/yaffs2/yaffs_guts.c
index f0ef8db..7390b40 100644
--- a/fs/yaffs2/yaffs_guts.c
+++ b/fs/yaffs2/yaffs_guts.c
@@ -2976,7 +2976,7 @@ static int yaffs_GarbageCollectBlock(yaffs_Device * dev, int block)
dev->isDoingGC = 0;
- return YAFFS_OK;
+ return retVal;
}
/* New garbage collector