dd8478d
From jwboyer@linux.vnet.ibm.com Tue Jun 28 09:42:53 2011
dd8478d
Date: Tue, 28 Jun 2011 09:42:53 -0400
dd8478d
From: Josh Boyer <jwboyer@linux.vnet.ibm.com>
dd8478d
To: jdl@jdl.com, david@gibson.dropbear.id.au
dd8478d
Cc: linuxppc-dev@lists.ozlabs.org
dd8478d
Subject: [PATCH] dtc: Remove unused variable in flat_read_mem_reserve
dd8478d
Message-ID: <20110628134253.GD10237@zod.rchland.ibm.com>
dd8478d
MIME-Version: 1.0
dd8478d
Content-Type: text/plain; charset=us-ascii
dd8478d
Content-Disposition: inline
dd8478d
User-Agent: Mutt/1.5.21 (2010-09-15)
dd8478d
Status: RO
dd8478d
Content-Length: 1865
dd8478d
Lines: 68
dd8478d
dd8478d
The *p variable is declared and used to save inb->ptr, however p is
dd8478d
later never used.  This has been the case since commit 6c0f3676 and can
dd8478d
lead to build failures with -Werror=unused-but-set-variable:
dd8478d
dd8478d
	flattree.c: In function 'flat_read_mem_reserve':
dd8478d
	flattree.c:700:14: error: variable 'p' set but not used [-Werror=unused-but-set-variable]
dd8478d
	cc1: all warnings being treated as errors
dd8478d
	make: *** [flattree.o] Error 1
dd8478d
dd8478d
Remove the variable.
dd8478d
dd8478d
Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
dd8478d
dd8478d
---
dd8478d
dd8478d
diff --git a/flattree.c b/flattree.c
dd8478d
index ead0332..28d0b23 100644
dd8478d
--- a/flattree.c
dd8478d
+++ b/flattree.c
dd8478d
@@ -697,7 +697,6 @@ static struct reserve_info *flat_read_mem_reserve(struct inbuf *inb)
dd8478d
 {
dd8478d
 	struct reserve_info *reservelist = NULL;
dd8478d
 	struct reserve_info *new;
dd8478d
-	const char *p;
dd8478d
 	struct fdt_reserve_entry re;
dd8478d
 
dd8478d
 	/*
dd8478d
@@ -706,7 +705,6 @@ static struct reserve_info *flat_read_mem_reserve(struct inbuf *inb)
dd8478d
 	 *
dd8478d
 	 * First pass, count entries.
dd8478d
 	 */
dd8478d
-	p = inb->ptr;
dd8478d
 	while (1) {
dd8478d
 		flat_read_chunk(inb, &re, sizeof(re));
dd8478d
 		re.address  = fdt64_to_cpu(re.address);
dd8478d
dd8478d