Hi Albert, On Sun, Aug 21, 2011 at 1:44 PM, Albert ARIBAUD <albert.u.b...@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.b...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
