HAOUAS Elyes has uploaded this change for review.

View Change

nb/sandybridge/raminit_common.c: Remove variable set but not used

Change-Id: I109353775fe4ecce6a05ef88781939243f398d17
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 9 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/33005/1

diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 5347c5c..b8ad3c0 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -1824,7 +1824,7 @@
 
 static void adjust_high_timB(ramctr_timing * ctrl)
 {
-	int channel, slotrank, lane, old;
+	int channel, slotrank, lane;
 	MCHBAR32(0x3400) = 0x200;
 	FOR_ALL_POPULATED_CHANNELS {
 		fill_pattern1(ctrl, channel);
@@ -1898,15 +1898,18 @@
 				0x100 * channel + 4);
 			res |= ((u64) MCHBAR32(lane_registers[lane] +
 				0x100 * channel + 8)) << 32;
-			old = ctrl->timings[channel][slotrank].lanes[lane].timB;
-			ctrl->timings[channel][slotrank].lanes[lane].timB +=
-				get_timB_high_adjust(res) * 64;
 
 			printram("High adjust %d:%016llx\n", lane, res);
-			printram("Bval+: %d, %d, %d, %x -> %x\n", channel,
-				slotrank, lane, old,
+			printram("Bval+: %d, %d, %d, %x", channel,
+				slotrank, lane,
 				ctrl->timings[channel][slotrank].lanes[lane].
 				timB);
+			ctrl->timings[channel][slotrank].lanes[lane].timB +=
+				get_timB_high_adjust(res) * 64;
+			printram(" -> %x\n",
+				ctrl->timings[channel][slotrank].lanes[lane].
+				timB);
+
 		}
 	}
 	MCHBAR32(0x3400) = 0;

To view, visit change 33005. To unsubscribe, or for help writing mail filters, visit settings.