[coreboot-gerrit] Change in ...coreboot[master]: device/dram/ddr3: Remove unused variable

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32428 )

Change subject: device/dram/ddr3: Remove unused variable
......................................................................

device/dram/ddr3: Remove unused variable

'param' variable is unused because 'printram' function only expands to
something in debug builds (not default ones).

Change-Id: I0cdf34cbb9aaed5045db5294eeefeaac642aeb1a
Signed-off-by: Elyes HAOUAS 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/32428
Reviewed-by: Angel Pons 
Reviewed-by: Jacob Garber 
Reviewed-by: Patrick Rudolph 
Tested-by: build bot (Jenkins) 
---
M src/device/dram/ddr3.c
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Rudolph: Looks good to me, approved
  Angel Pons: Looks good to me, approved
  Jacob Garber: Looks good to me, but someone else must approve

diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c
index f27fdcb..e545b4d 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -136,7 +136,7 @@
    u8 reg8;
    u32 mtb;		/* medium time base */
    u32 ftb;		/* fine time base */
-	unsigned int val, param;
+	unsigned int val;

ret = SPD_STATUS_OK;

@@ -173,8 +173,7 @@
    	printram("  Invalid number of memory banks\n");
    	ret = SPD_STATUS_INVALID_FIELD;
    }
-	param = 1 << (val + 3);
-	printram("  Banks              : %u\n", param);
+	printram("  Banks              : %u\n", 1 << (val + 3));
    /* SDRAM capacity */
    capacity_shift = reg8 & 0x0f;
    if (capacity_shift > 0x06) {