Hi Dirk,
On Sun, Aug 21, 2011 at 1:16 PM, Dirk Behme dirk.behme@googlemail.com wrote:
On 20.08.2011 17:32, Albert ARIBAUD wrote:
Hi all,
Le 20/08/2011 13:12, Albert ARIBAUD a écrit :
...
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?
I too didn't get this warning. But I see a problem with omap4/clocks.c.
I have submitted a patch just now. http://patchwork.ozlabs.org/patch/110811/
BTW, thanks for looking into this.
br, Aneesh