From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sun, 21 Aug 2011 10:14:22 +0200 Subject: [U-Boot] [STATUS] ARM In-Reply-To: References: <4E4D8DB4.6030800@aribaud.net> <4E4E2F1C.6080300@ti.com> <4E4EA91B.2050007@googlemail.com> <4E4F9692.8020502@aribaud.net> <4E4FD3A0.7060309@aribaud.net> Message-ID: <4E50BE5E.1080909@aribaud.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > 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. > best regards, > Aneesh Amicalement, -- Albert.