[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/sandybridge: Remove variable set but not used

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32938

Change subject: nb/intel/sandybridge: Remove variable set but not used
......................................................................

nb/intel/sandybridge: Remove variable set but not used

Change-Id: I75f5d821e018932d3f10d84b7ebed362777fb17d
Signed-off-by: Elyes HAOUAS 
---
M src/northbridge/intel/sandybridge/raminit_common.c
1 file changed, 6 insertions(+), 12 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/32938/1

diff --git a/src/northbridge/intel/sandybridge/raminit_common.c b/src/northbridge/intel/sandybridge/raminit_common.c
index 5347c5c..900633a 100644
--- a/src/northbridge/intel/sandybridge/raminit_common.c
+++ b/src/northbridge/intel/sandybridge/raminit_common.c
@@ -1452,9 +1452,8 @@
    int lane;

FOR_ALL_LANES {
-		volatile u32 tmp;
    	MCHBAR32(0x4340 + 0x400 * channel + 4 * lane) = 0;
-		tmp = MCHBAR32(0x4140 + 0x400 * channel + 4 * lane);
+		MCHBAR32(0x4140 + 0x400 * channel + 4 * lane);
    }

wait_428c(channel);
@@ -1997,9 +1996,8 @@
    MCHBAR32_OR(0x5030, 8);

FOR_ALL_POPULATED_CHANNELS {
-		volatile u32 tmp;
    	MCHBAR32_AND(0x4020 + 0x400 * channel, ~0x00200000);
-		tmp = MCHBAR32(0x428c + 0x400 * channel);
+		MCHBAR32(0x428c + 0x400 * channel);
    	wait_428c(channel);

/* DRAM command ZQCS */
@@ -2344,9 +2342,8 @@
    	program_timings(ctrl, channel);

FOR_ALL_LANES {
-			volatile u32 tmp;
    		MCHBAR32(0x4340 + 0x400 * channel + 4 * lane) = 0;
-			tmp = MCHBAR32(0x400 * channel + 4 * lane + 0x4140);
+			MCHBAR32(0x400 * channel + 4 * lane + 0x4140);
    	}

wait_428c(channel);
@@ -2425,8 +2422,7 @@
    	fill_pattern0(ctrl, channel, 0, 0);
    	MCHBAR32(0x4288 + (channel << 10)) = 0;
    	FOR_ALL_LANES {
-			volatile u32 tmp;
-			tmp = MCHBAR32(0x400 * channel + lane * 4 + 0x4140);
+			MCHBAR32(0x400 * channel + lane * 4 + 0x4140);
    	}

FOR_ALL_POPULATED_RANKS FOR_ALL_LANES {
@@ -2626,10 +2622,9 @@
    			program_timings(ctrl, channel);

FOR_ALL_LANES {
-					volatile u32 tmp;
    				MCHBAR32(0x4340 + 0x400 * channel +
    					4 * lane) = 0;
-					tmp = MCHBAR32(0x400 * channel +
+					MCHBAR32(0x400 * channel +
    					4 * lane + 0x4140);
    			}
    			wait_428c(channel);
@@ -2674,8 +2669,7 @@

wait_428c(channel);
    			FOR_ALL_LANES {
-					volatile u32 tmp;
-					tmp = MCHBAR32(0x4340 +
+					MCHBAR32(0x4340 +
    				       0x400 * channel + lane * 4);
    			}