[U-Boot] [STATUS] ARM

[Dirk Behme]

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)