summaryrefslogtreecommitdiff
path: root/src/device
diff options
context:
space:
mode:
authorElyes HAOUAS <ehaouas@noos.fr>2019-04-23 22:15:57 +0200
committerPatrick Georgi <pgeorgi@google.com>2019-04-25 15:55:02 +0000
commit05c04556990c74f7d4f3303c33be2c43c3051e20 (patch)
tree6dbadf4920b98feb5014ec7987376c063bde90f6 /src/device
parentd5d433e07f8adf29ef02a2a99596299cc2cf473d (diff)
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 <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/c/coreboot/+/32428 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Jacob Garber <jgarber1@ualberta.ca> Reviewed-by: Patrick Rudolph <siro@das-labor.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/device')
-rw-r--r--src/device/dram/ddr3.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c
index f27fdcb1e0..e545b4d036 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -136,7 +136,7 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd)
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 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd)
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) {