From: Dirk Eibach <dirk.eibach@gdsys.cc>
Date: Thu, 13 Nov 2014 18:21:17 +0000 (+0100)
Subject: board: iocon: Fix uninitialized access
X-Git-Tag: v2015.01-rc2~48
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=a8cb9d0b17af9d7b3ad205cf83fbfd47528fa9d0;p=u-boot

board: iocon: Fix uninitialized access

Wolfgang Denk found this issue using cppcheck:
(error) Uninitialized variable: fpga_features

Signed-off-by: Dirk Eibach <dirk.eibach@gdsys.cc>
Signed-off-by: Stefan Roese <sr@denx.de>
---

diff --git a/board/gdsys/405ep/iocon.c b/board/gdsys/405ep/iocon.c
index fffed7390b..1d7e14119d 100644
--- a/board/gdsys/405ep/iocon.c
+++ b/board/gdsys/405ep/iocon.c
@@ -369,10 +369,11 @@ int last_stage_init(void)
 	unsigned char mclink_controllers[] = { 0x24, 0x25, 0x26 };
 	int legacy = get_fpga_state(0) & FPGA_STATE_PLATFORM;
 	u16 fpga_features;
-	int feature_carrier_speed = fpga_features & (1<<4);
+	int feature_carrier_speed;
 	bool ch0_rgmii2_present = false;
 
 	FPGA_GET_REG(0, fpga_features, &fpga_features);
+	feature_carrier_speed = fpga_features & (1<<4);
 
 	if (!legacy) {
 		/* Turn on Parade DP501 */