[U-Boot] [PATCH] u-boot: Update yaffs2 file system

[Wolfgang Denk]

Dear Charles,

In message <20120810224140.0F1B9204042 at gemini.denx.de> I wrote:
> Dear Charles,
> 
> In message <20120809213931.755AD20401A at gemini.denx.de> I wrote:
> > 
> > In message <1336618517-2947-1-git-send-email-cdhmanning at gmail.com> you wrote:
> > > This patch updates the yaffs2 in u-boot to correspond to
> > > git://www.aleph1.co.uk/yaffs2
> > > commit id 9ee5d0643e559568dbe62215f76e0a7bd5a63d93
> ...
> > Applied, thanks.
> 
> Unfortunately, this code causes a TON of errors and warnings, like
> these:
> 
> In file included from yportenv.h:80:0,
>                  from yaffs_guts.h:19,
>                  from yaffs_allocator.h:19,
>                  from yaffs_allocator.c:14:
> ydirectenv.h:31:0: warning: "BUG" redefined [enabled by default]
> /work/wd/u-boot/include/common.h:160:0: note: this is the location of the previous definition
> ydirectenv.h:50:0: warning: "hweight8" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:139:0: note: this is the location of the previous definition
> ydirectenv.h:51:0: warning: "hweight32" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:137:0: note: this is the location of the previous definition
> 
> In file included from yportenv.h:80:0,
>                  from yaffscfg.h:26,
>                  from yaffs_uboot_glue.c:26:
> ydirectenv.h:31:0: warning: "BUG" redefined [enabled by default]
> /work/wd/u-boot/include/common.h:160:0: note: this is the location of the previous definition
> ydirectenv.h:50:0: warning: "hweight8" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:139:0: note: this is the location of the previous definition
> ydirectenv.h:51:0: warning: "hweight32" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:137:0: note: this is the location of the previous definition
> ydirectenv.h:66:0: warning: "kmalloc" redefined [enabled by default]
> /work/wd/u-boot/include/linux/compat.h:20:0: note: this is the location of the previous definition
> ydirectenv.h:67:0: warning: "kfree" redefined [enabled by default]
> /work/wd/u-boot/include/linux/compat.h:23:0: note: this is the location of the previous definition
> ydirectenv.h:68:0: warning: "vmalloc" redefined [enabled by default]
> /work/wd/u-boot/include/linux/compat.h:22:0: note: this is the location of the previous definition
> ydirectenv.h:69:0: warning: "vfree" redefined [enabled by default]
> /work/wd/u-boot/include/linux/compat.h:24:0: note: this is the location of the previous definition
> 
> In file included from yportenv.h:80:0,
>                  from yaffs_guts.c:14:
> ydirectenv.h:31:0: warning: "BUG" redefined [enabled by default]
> /work/wd/u-boot/include/common.h:160:0: note: this is the location of the previous definition
> ydirectenv.h:50:0: warning: "hweight8" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:139:0: note: this is the location of the previous definition
> ydirectenv.h:51:0: warning: "hweight32" redefined [enabled by default]
> /work/wd/tmp-arm/include2/asm/bitops.h:137:0: note: this is the location of the previous definition
> yaffs_guts.c: In function 'yaffs_check_chunk_erased':
> yaffs_guts.c:324:6: warning: variable 'result' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c: In function 'yaffs_verify_chunk_written':
> yaffs_guts.c:352:6: warning: variable 'result' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c: In function 'yaffs_grab_chunk_cache':
> yaffs_guts.c:1488:6: warning: variable 'pushout' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c: In function 'yaffs_check_obj_details_loaded':
> yaffs_guts.c:3180:6: warning: variable 'alloc_failed' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c:3179:6: warning: variable 'result' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c: In function 'yaffs_update_oh':
> yaffs_guts.c:3288:6: warning: variable 'result' set but not used [-Wunused-but-set-variable]
> yaffs_guts.c: In function 'yaffs_get_obj_name':
> yaffs_guts.c:4447:7: warning: variable 'result' set but not used [-Wunused-but-set-variable]
> 
> 
> and so on and on.
> 
> 
> Please provide a fix, ideally soon, as this breaking builds!  Thanks!


Ping...

This needs to be fixed, or we will have to revert this commit.

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: wd at denx.de
"There is no statute of limitations on stupidity."
- Randomly produced by a computer program called Markov3.