Re: [U-Boot] [PATCH 3/7] fs/jffs2/jffs2_1pass.c: fix warning: variable ... set but not used

[Wolfgang Denk]

Dear Wolfgang Denk,

In message 1317849453-8216-3-git-send-email...@denx.de you wrote:
 Fix:
 jffs2_1pass.c: In function 'jffs2_1pass_read_inode':
 jffs2_1pass.c:699:7: warning: variable 'ret' set but not used 
 [-Wunused-but-set-variable]
 jffs2_1pass.c: In function 'jffs2_1pass_build_lists':
 jffs2_1pass.c:1578:14: warning: variable 'empty_start' set but not used 
 [-Wunused-but-set-variable]
 
 Signed-off-by: Wolfgang Denk w...@denx.de
 ---
  fs/jffs2/jffs2_1pass.c |   14 --
  1 files changed, 4 insertions(+), 10 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
If all you have is a hammer, everything looks like a nail.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 3/7] fs/jffs2/jffs2_1pass.c: fix warning: variable ... set but not used

[Wolfgang Denk]

Fix:
jffs2_1pass.c: In function 'jffs2_1pass_read_inode':
jffs2_1pass.c:699:7: warning: variable 'ret' set but not used 
[-Wunused-but-set-variable]
jffs2_1pass.c: In function 'jffs2_1pass_build_lists':
jffs2_1pass.c:1578:14: warning: variable 'empty_start' set but not used 
[-Wunused-but-set-variable]

Signed-off-by: Wolfgang Denk w...@denx.de
---
 fs/jffs2/jffs2_1pass.c |   14 --
 1 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/fs/jffs2/jffs2_1pass.c b/fs/jffs2/jffs2_1pass.c
index 5ddc2b9..a0b02e4 100644
--- a/fs/jffs2/jffs2_1pass.c
+++ b/fs/jffs2/jffs2_1pass.c
@@ -696,7 +696,6 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32 inode, char 
*dest)
u32 latestVersion = 0;
uchar *lDest;
uchar *src;
-   long ret;
int i;
u32 counter = 0;
 #ifdef CONFIG_SYS_JFFS2_SORT_FRAGMENTS
@@ -768,28 +767,25 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32 inode, 
char *dest)
 #endif
switch (jNode-compr) {
case JFFS2_COMPR_NONE:
-   ret = (unsigned long) ldr_memcpy(lDest, 
src, jNode-dsize);
+   ldr_memcpy(lDest, src, jNode-dsize);
break;
case JFFS2_COMPR_ZERO:
-   ret = 0;
for (i = 0; i  jNode-dsize; i++)
*(lDest++) = 0;
break;
case JFFS2_COMPR_RTIME:
-   ret = 0;
rtime_decompress(src, lDest, 
jNode-csize, jNode-dsize);
break;
case JFFS2_COMPR_DYNRUBIN:
/* this is slow but it works */
-   ret = 0;
dynrubin_decompress(src, lDest, 
jNode-csize, jNode-dsize);
break;
case JFFS2_COMPR_ZLIB:
-   ret = zlib_decompress(src, lDest, 
jNode-csize, jNode-dsize);
+   zlib_decompress(src, lDest, 
jNode-csize, jNode-dsize);
break;
 #if defined(CONFIG_JFFS2_LZO)
case JFFS2_COMPR_LZO:
-   ret = lzo_decompress(src, lDest, 
jNode-csize, jNode-dsize);
+   lzo_decompress(src, lDest, 
jNode-csize, jNode-dsize);
break;
 #endif
default:
@@ -803,7 +799,6 @@ jffs2_1pass_read_inode(struct b_lists *pL, u32 inode, char 
*dest)
 
 #if 0
putLabeledWord(read_inode: totalSize = , totalSize);
-   putLabeledWord(read_inode: compr ret = , ret);
 #endif
}
counter++;
@@ -1575,9 +1570,8 @@ jffs2_1pass_build_lists(struct part_info * part)
 
if (*(uint32_t *)(buf[ofs-buf_ofs]) == 0x) {
uint32_t inbuf_ofs;
-   uint32_t empty_start, scan_end;
+   uint32_t scan_end;
 
-   empty_start = ofs;
ofs += 4;
scan_end = min_t(uint32_t, EMPTY_SCAN_SIZE(
part-sector_size)/8,
-- 
1.7.6.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot