diff options
| author | Wolfgang Denk <wd@denx.de> | 2011-12-09 12:14:22 +0100 |
|---|---|---|
| committer | Wolfgang Denk <wd@denx.de> | 2011-12-10 23:08:49 +0100 |
| commit | 5c2db13a2aa4bdb2f335830f9ecbfc9ee673e53f (patch) | |
| tree | 93f78e84f28c15e5dc735c31ddefefd2ba575234 /fs | |
| parent | f49620665d67e43d92dd4deac2ffc025a7049306 (diff) | |
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>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/yaffs2/yaffs_guts.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/yaffs2/yaffs_guts.c b/fs/yaffs2/yaffs_guts.c index f0ef8db9ee9..7390b40e011 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 |