[U-Boot] [STATUS] ARM

V, Aneesh
2011-08-21T08:56:18Z

Hi Albert,

On Sun, Aug 21, 2011 at 1:44 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Aneesh,
>
> Le 21/08/2011 09:51, V, Aneesh a ?crit :
>>
>> Hi Albert,
>>
>> On Sat, Aug 20, 2011 at 9:02 PM, Albert ARIBAUD
>> <albert.u.boot@aribaud.net> ?wrote:
>>>
>>> Hi all,
>>>
>>> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>>>
>>>>>> 1. http://patchwork.ozlabs.org/patch/107678/
>>>
>>> This one lowers bad board builds from 69 to 62, and has been applied.
>>>
>>>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>>>
>>> This one kind of half-fixes the two boards it touches: they now build
>>> but with warnings, all the same, of which here is one:
>>>
>>> clocks.c:274: warning: comparison is always true due to limited range of
>>> data type
>>
>> My tool-chain (Sourcery G++ Lite 2010q1 - GCC 4.4.1) never reported this
>> issue.
>> But looks like there is indeed a problem. Hopefully the following patch
>> should
>> solve it:
>>
>> http://patchwork.ozlabs.org/patch/110811/
>
> This indeed fixes it. Strange that the warning is not emitted by your
> toolchain.

Yes, that's indeed strange. Even with your tool-chain it should've emitted
more warnings. Many of those unsigned variables were assigned -1 in
elsewhere. That was also never reported.

br,
Aneesh