[U-Boot] U-Boot Digest, Vol 47, Issue 24

[U-Boot] U-Boot Digest, Vol 47, Issue 24

[Tom Warren]

> ------------------------------
> 
> Message: 54
> Date: Thu, 19 Apr 2012 09:13:38 +0200
> From: Albert ARIBAUD <albert.u.boot@aribaud.net>
> Subject: [U-Boot] [ARM] [STATUS] Increasingly high build failures
> To: "U-Boot at lists.denx.de" <U-Boot@lists.denx.de>
> Cc: Simon Guinot <simon.guinot@sequanux.org>
> Message-ID: <4F8FBB22.5000205@aribaud.net>
> Content-Type: text/plain; charset=ISO-8859-1; format=flowed
> 
> Folks,
> 
> I just did another round of ./MAKEALL arm with MG/CS 2011.09-69, and the
> failure rate is rising to say the least, with 277 boards failing out of
> 278. :(
> 
> Basically, the issues are, in order of impact:
> 
> - Now 264 'uses variable-size enums yet the output is to use 32-bit
> enums' errors. That's a raise! Probably caused by some commonly included
> .h, but as the issue is at link stage and only refers to library
> intrinsics, tracing the culprit may take time. However, I cannot keep on
> ignoring this warning now, and thus I will look for a general solution,
> probably along the lines of enabling variable-size enums throughout the
> U-Boot codebase at least for toolchains that complain about it.
> 
> - 6 warnings are 'ap20.c:44:18: warning: array subscript is above array
> bounds'. Last changed was in commit 03c609f6. Simon Glass, as you are
> listed as author of the commit, please have a look.

Albert,

Looks like you CC'd Simon Guinot instead of Simon Glass. I've CC'd the correct Simon. ;)

This change (ap20.c) went in a _long_ time ago, and hasn't produced any warnings until now?

As Wolfgang said, what toolset are you using? I tried TOT U-Boot master with gcc 4.4.1, 4.2.2 (ELDK4.2), and 4.6.1, and don't see any warnings at all for ap20.c in any build (Seaboard, Harmony, etc.).

Tom
> 
> - netspace_v2 board causing the only error case: Undefined or
> unsupported LaCie KW board variant. Simon Guinot, as you are listed as
> the board maintainer, can you have a look?
> 
> This pretty much covers the immediate actions, but extrapolating from
> the situation, I would advise any and all contributors whose first name
> is Simon and last name initial is G to watch out for issues. :)
> 
> Amicalement,
> --
> Albert.
> 
> 
-- 
nvpublic

[U-Boot] U-Boot Digest, Vol 47, Issue 24

[Albert ARIBAUD]

Le 19/04/2012 18:59, Tom Warren a ?crit :

> Albert,
>
> Looks like you CC'd Simon Guinot instead of Simon Glass. I've CC'd the correct Simon. ;)

I actually intended to Cc: both, as I though eache one had something to 
do. :)

> This change (ap20.c) went in a _long_ time ago, and hasn't produced any warnings until now?

As I said in the followup discussion with Wolfgang, I had a double 
issue, with the commit I was using and with the toolchain. The only real 
issue is the enums one, and I am looking into it.

Amicalement,
-- 
Albert.