From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Loeliger Subject: Re: dtc: Enable more warnings Date: Fri, 05 Mar 2010 15:46:38 -0600 Message-ID: References: <20100303052630.GQ23435@yookeroo> <20100304001218.GC4488@yookeroo> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5467137138587515709==" Return-path: In-reply-to: <20100304001218.GC4488@yookeroo> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: David Gibson Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org List-Id: devicetree@vger.kernel.org --===============5467137138587515709== Content-Type: text/plain > This patch turns on a bunch of extra gcc warnings, most of which are > probably a good idea. Of the new warnings -Wnested-externs and > -Wstrict-prototypes need no code changes, we're already warning-clean. > The remaining one, -Wmissing-prototypes requires trivial changes in > some of the tests (making functions local). > > This patch also rearranges the warnings flags into a separate make > variable for convenience, and turns on -Werror, to really encourage > people to keep the code warning-clean. > > Signed-off-by: David Gibson Well, I'd like to apply this patch. But doing so appears to be very effective. So much so that the "make check" now fails: $ gcc --version gcc (Debian 4.3.2-1.1) 4.3.2 $ make check CHK version_gen.h CC tests/testutils.o cc1: warnings being treated as errors tests/testutils.c: In function ‘sigint_handler’: tests/testutils.c:44: error: unused parameter ‘si’ tests/testutils.c:44: error: unused parameter ‘uc’ tests/testutils.c: In function ‘test_init’: tests/testutils.c:52: error: unused parameter ‘argc’ tests/testutils.c: In function ‘check_property’: tests/testutils.c:113: error: comparison between signed and unsigned tests/testutils.c:117: error: comparison between signed and unsigned make: *** [tests/testutils.o] Error 1 --===============5467137138587515709== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ devicetree-discuss mailing list devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org https://lists.ozlabs.org/listinfo/devicetree-discuss --===============5467137138587515709==--