From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from jdl.com (jdl.com [208.123.74.7]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id A8D55B6F76 for ; Sun, 17 Jul 2011 23:11:47 +1000 (EST) To: David Gibson Subject: Re: [PATCH v2] dtc: Remove unused variable in flat_read_mem_reserve In-reply-to: <20110715135755.GD4368@yookeroo.fritz.box> References: <20110628134711.GF10237@zod.rchland.ibm.com> <20110715135755.GD4368@yookeroo.fritz.box> Date: Sun, 17 Jul 2011 07:36:55 -0500 From: Jon Loeliger Message-Id: Cc: linuxppc-dev@lists.ozlabs.org, Josh Boyer List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , > On Tue, Jun 28, 2011 at 09:47:11AM -0400, Josh Boyer wrote: > > The *p variable is declared and used to save inb->ptr, however p is > > later never used. This has been the case since commit 6c0f3676 and can > > lead to build failures with -Werror=unused-but-set-variable: > > > > flattree.c: In function 'flat_read_mem_reserve': > > flattree.c:700:14: error: variable 'p' set but not used [-Werror=unused-but-set-variable] > > cc1: all warnings being treated as errors > > make: *** [flattree.o] Error 1 > > > > Remove the variable. > > > > Signed-off-by: Josh Boyer > > Acked-by: David Gibson Applied. jdl