[U-Boot] [STATUS] ARM

Dirk Behme
2011-08-21T07:46:50Z

On 20.08.2011 17:32, Albert ARIBAUD wrote:
> Hi all,
>
> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
...
>>>> 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
>
> I'll put this one on hold unless somebody can tell me what's going on.

First, the patch 108865 above fixes a linking issue with the SPL for
these boards. So I think that this is independent of the clocks.c
warning you report above. That would mean it would be worth to apply
108865 to fix at least the linking issue.

Second, applying 108865 to the recent u-boot-arm.git doesn't show any
clocks.c warning for me [1]. Could this be tool chain dependent? I
tried it with [2] and [3], without any warning.

Btw, about which clocks.c (directory) do we talk?

Best regards

Dirk

[1]

> cat include/configs/omap4_sdp4430.h | grep CONFIG_SPL_MAX_SIZE
#define CONFIG_SPL_MAX_SIZE (38 * 1024)

> ./MAKEALL omap4_sdp4430
Configuring for omap4_sdp4430 board...
text data bss dec hex filename
191043 4468 202816 398327 613f7 ./u-boot

--------------------- SUMMARY ----------------------------
Boards compiled: 1
----------------------------------------------------------

> cat include/configs/omap4_panda.h | grep CONFIG_SPL_MAX_SIZE
#define CONFIG_SPL_MAX_SIZE (38 * 1024)

> ./MAKEALL omap4_panda
Configuring for omap4_panda board...
text data bss dec hex filename
189478 4380 202820 396678 60d86 ./u-boot

--------------------- SUMMARY ----------------------------
Boards compiled: 1
----------------------------------------------------------

[2] gcc version 4.3.3 (Sourcery G++ Lite 2009q1-203)

[3] gcc version 4.5.1 (Sourcery G++ Lite 2010.09-50)