[U-Boot] Intermediate ARM status

[U-Boot] Intermediate ARM status

[Albert ARIBAUD]

All,

I have merged Tom Rini's ti/master branch, but not the tegra or
stagin/trini at ti.com ones yet.

At this point 314 boards were built, 9 failing due to dtc not being
available.

But when I rebase onto u-boot/master, I get:

--------------------- SUMMARY ----------------------------
Boards compiled: 314
Boards with errors: 9 ( trimslice tec ventana seaboard plutux harmony
medcom whistler paz00 )
Boards with warnings but no errors: 2 ( VCMA9 smdk2410 )
----------------------------------------------------------

These two boards actually already build with warnings on u-boot/master:

- multiple instances of:

ydirectenv.h:31:0: warning: "BUG" redefined [enabled by default]
/home/uboot/src/u-boot-arm/include/common.h:160:0: note: this is the
location of the previous definition
ydirectenv.h:50:0: warning: "hweight8" redefined [enabled by default]
/home/uboot/src/u-boot-arm/build/smdk2410/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]
/home/uboot/src/u-boot-arm/build/smdk2410/include2/asm/bitops.h:137:0:
note: this is the location of the previous definition

One instance of:

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]

Multiples instances of:

ydirectenv.h:66:0: warning: "kmalloc" redefined [enabled by default]
/home/uboot/src/u-boot-arm/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]
/home/uboot/src/u-boot-arm/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]
/home/uboot/src/u-boot-arm/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]
/home/uboot/src/u-boot-arm/include/linux/compat.h:24:0: note: this is
the location of the previous definition
In file included from yportenv.h:80:0,

I believe the errors / warnings have nothing to do with the arm
sub-tree ARM commits per se.

Therefore I have pushed the rebased u-bot-arm/master and next.

already present 

[U-Boot] Intermediate ARM status

[Albert ARIBAUD]

Hi Anatolij,

On Sat, 25 Aug 2012 17:47:57 +0200, Anatolij Gustschin <agust@denx.de>
wrote:

> Hi Albert,
> 
> On Sat, 25 Aug 2012 14:22:26 +0200
> Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
> ...
> > ydirectenv.h:69:0: warning: "vfree" redefined [enabled by default]
> > /home/uboot/src/u-boot-arm/include/linux/compat.h:24:0: note: this
> > is the location of the previous definition
> > In file included from yportenv.h:80:0,
> > 
> > I believe the errors / warnings have nothing to do with the arm
> > sub-tree ARM commits per se.
> 
> Yes, these appear with the current master branch, too. A patch fixing
> them is already pending, http://patchwork.ozlabs.org/patch/177537/

It does fix some, but not all warnings:

===========================================================================
uboot at lilith:~/src/u-boot-arm$ time LANG=C BUILD_NBUILDS=7
BUILD_NCPUS=1 ./MAKEALL VCMA9 smdk2410
Configuring for VCMA9 board...
Configuring for smdk2410 board...
   text	   data	    bss	    dec
hex	filename
 482924	  10292	 287516	 780732
be9bc	./build/smdk2410/u-boot
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]
yaffs_summary.c: In function 'yaffs_summary_read':
yaffs_summary.c:194:6: warning: variable 'sum_tags_bytes' set but not
used [-Wunused-but-set-variable]
yaffs_verify.c: In function 'yaffs_verify_file':
yaffs_verify.c:227:6: warning: variable 'actual_depth' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs1.c: In function 'yaffs1_scan':
yaffs_yaffs1.c:26:6: warning: variable 'result' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs2.c: In function 'yaffs2_scan_chunk':
yaffs_yaffs2.c:949:6: warning: variable 'result' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs2.c: In function 'yaffs2_scan_backwards':
yaffs_yaffs2.c:1352:6: warning: variable 'deleted' set but not used
[-Wunused-but-set-variable]
   text	   data	    bss	    dec
hex	filename
 508063	  10688	 287580	 806331
c4dbb	./build/VCMA9/u-boot
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]
yaffs_summary.c: In function 'yaffs_summary_read':
yaffs_summary.c:194:6: warning: variable 'sum_tags_bytes' set but not
used [-Wunused-but-set-variable]
yaffs_verify.c: In function 'yaffs_verify_file':
yaffs_verify.c:227:6: warning: variable 'actual_depth' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs1.c: In function 'yaffs1_scan':
yaffs_yaffs1.c:26:6: warning: variable 'result' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs2.c: In function 'yaffs2_scan_chunk':
yaffs_yaffs2.c:949:6: warning: variable 'result' set but not used
[-Wunused-but-set-variable]
yaffs_yaffs2.c: In function 'yaffs2_scan_backwards':
yaffs_yaffs2.c:1352:6: warning: variable 'deleted' set but not used
[-Wunused-but-set-variable]

--------------------- SUMMARY ----------------------------
Boards compiled: 2
Boards with warnings but no errors: 2 ( VCMA9 smdk2410 )
----------------------------------------------------------

real	0m29.001s
user	0m40.947s
sys	0m6.328s
===========================================================================

Although, it may be my toolchain being more sensitive than others.

> Thanks,
> 
> Anatolij

Amicalement,
-- 
Albert.

[U-Boot] Intermediate ARM status

[Marek Vasut]

Dear Albert ARIBAUD,

> Hi Anatolij,
> 
> On Sat, 25 Aug 2012 17:47:57 +0200, Anatolij Gustschin <agust@denx.de>
> 
> wrote:
> > Hi Albert,
> > 
> > On Sat, 25 Aug 2012 14:22:26 +0200
> > Albert ARIBAUD <albert.u.boot@aribaud.net> wrote:
> > ...
> > 
> > > ydirectenv.h:69:0: warning: "vfree" redefined [enabled by default]
> > > /home/uboot/src/u-boot-arm/include/linux/compat.h:24:0: note: this
> > > is the location of the previous definition
> > > In file included from yportenv.h:80:0,
> > > 
> > > I believe the errors / warnings have nothing to do with the arm
> > > sub-tree ARM commits per se.
> > 
> > Yes, these appear with the current master branch, too. A patch fixing
> > them is already pending, http://patchwork.ozlabs.org/patch/177537/
> 
[...]

Yaffs is borked indeed, I submitted similar patch that Charles did, but his is 
better indeed. Maybe we can ask Detlev to apply some of the pulls ?

Best regards,
Marek Vasut