Hi Dirk, On Sun, Aug 21, 2011 at 1:16 PM, Dirk Behme <dirk.be...@googlemail.com> wrote: > 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?
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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
