[head tinderbox] failure on arm/arm

[Warner Losh]

Thanks Garrett!

I switched from a switch to an if due to changes at the last minute, and didn't test somehow (too many kernels to compile for complete coverage, but I'm working to fix that :).  Sorry for the hassle.

Warner

On Jun 6, 2012, at 7:39 AM, Garrett Cooper wrote:

> ...
> 
> cc -mlittle-endian -c -O2 -pipe -fno-strict-aliasing  -std=c99  -Wall
> -Wredundant-decls -Wnested-externs -Wstrict-prototypes
> -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef
> -Wno-pointer-sign -fformat-extensions  -Wmissing-include-dirs
> -fdiagnostics-show-option   -nostdinc  -I. -I/src/sys
> -I/src/sys/contrib/altq -I/src/sys/contrib/libfdt -D_KERNEL
> -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common
> -finline-limit=8000 --param inline-unit-growth=100 --param
> large-function-growth=1000 -mcpu=arm9 -mno-apcs-frame -ffreestanding
> -Werror  /src/sys/arm/at91/at91sam9260.c
> /src/sys/arm/at91/at91sam9260.c: In function 'at91_identify':
> /src/sys/arm/at91/at91sam9260.c:203: error: break statement not within
> loop or switch
> *** Error code 1
> 
> Stop in /obj/arm.arm/src/sys/ETHERNUT5.
> *** Error code 1
> 
> Stop in /src.
> *** Error code 1
> 
> Stop in /src.
> TB --- 2012-06-06 13:21:10 - WARNING: /usr/bin/make returned exit code  1
> TB --- 2012-06-06 13:21:10 - ERROR: failed to build ETHERNUT5 kernel
> TB --- 2012-06-06 13:21:10 - 3927.58 user 801.22 system 6069.07 real
> 
>    This simple patch will unbreak arm tinderbox.
> Thanks!
> -Garrett
> 
> Index: sys/arm/at91/at91sam9260.c
> ===================================================================
> --- sys/arm/at91/at91sam9260.c   (revision 236680)
> +++ sys/arm/at91/at91sam9260.c   (working copy)
> @@ -200,7 +200,6 @@
>        if (soc_data.type == AT91_CPU_SAM9260) {
>                at91_add_child(parent, 0, "at91sam9260", 0, 0, 0, -1, 0, 0);
>                at91_cpu_add_builtin_children(parent);
> -               break;
>        }
> }
> 
>