Skip to contentSkip to conversation

board/ti/omap2420h4/omap2420h4.c: Fix GC 4.6 build warning

PATCH New Wolfgang Denk 2 participants 3 messages Dec 12, 2011 – Dec 13, 2011

lore
Share & tools
lore

Wolfgang Denk

author
#0 lore raw

PATCH 1/1 board/ti/omap2420h4/omap2420h4.c: Fix GC 4.6 build warning

Fix:
omap2420h4.c: In function 'dram_init':
omap2420h4.c:196:25: warning: variable 'cpu' set but not used [-Wunused-but-set-variable]

Signed-off-by Wolfgang Denk <[email protected]>
Cc Richard Woodruff <[email protected]>
Cc Tom Rini <[email protected]>
---
 board/ti/omap2420h4/omap2420h4.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
board/ti/omap2420h4/omap2420h4.c +1 -2
index 1c98e1b..a3983e3 100644
@@ -193,7 +193,7 @@ void ether_init (void)
 int dram_init (void)
 {
 	unsigned int size0=0,size1=0;
-	u32 mtype, btype, rev, cpu;
+	u32 mtype, btype, rev;
 	u8 chg_on = 0x5; /* enable charge of back up battery */
 	u8 vmode_on = 0x8C;
 	#define NOT_EARLY 0

@@ -203,7 +203,6 @@ int dram_init (void)
 	btype = get_board_type();
 	mtype = get_mem_type();
 	rev = get_cpu_rev();
-	cpu = get_cpu_type();
 
 	display_board_info(btype);
 	if (btype == BOARD_H4_MENELAUS){
Signature · 1 line
1.7.6.4
author Applied
#1 lore raw

PATCH 1/1 board/ti/omap2420h4/omap2420h4.c: Fix GC 4.6 build warning

Dear Wolfgang Denk,

Quoted 11 lines · In message <[email protected]> you wrote:

Fix:
omap2420h4.c: In function 'dram_init':
omap2420h4.c:196:25: warning: variable 'cpu' set but not used [-Wunused-but-set-variable]

Signed-off-by Wolfgang Denk <[email protected]>
Cc Richard Woodruff <[email protected]>
Cc Tom Rini <[email protected]>
---
 board/ti/omap2420h4/omap2420h4.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

Signature · 9 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Absolute: Independent, irresponsible. An absolute monarchy is one  in
which  the  sovereign  does  as  he pleases so long as he pleases the
assassins. Not many absolute monarchies are left, most of them having
been replaced by limited monarchies, where the soverign's  power  for
evil (and for good) is greatly curtailed, and by republics, which are
governed by chance.                                  - Ambrose Bierce
#2 lore raw

PATCH 1/1 board/ti/omap2420h4/omap2420h4.c: Fix GC 4.6 build warning

Quoted 8 lines · Wolfgang Denk

Dear Wolfgang Denk,

Quoted 5 lines · In message <[email protected]> you wrote:

Fix:
omap2420h4.c: In function 'dram_init':
omap2420h4.c:196:25: warning: variable 'cpu' set but not used [-Wunused-but-set-variable]

Quoted 8 lines
Quoted 6 lines
Signed-off-by Wolfgang Denk <[email protected]>
Cc Richard Woodruff <[email protected]>
Cc Tom Rini <[email protected]>
---
 board/ti/omap2420h4/omap2420h4.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

Applied, thanks.

Too late then, but...

Acked-by Tom Rini <[email protected]>
Signature · 1 line
Tom