GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Elyes HAOUAS 2019-04-23 22:15:57 +02:00 committed by Patrick Georgi
parent d5d433e07f
commit 05c0455699
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/device/dram/ddr3.c
View File

@ -136,7 +136,7 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd)
u8 reg8; u8 reg8;
u32 mtb; /* medium time base */ u32 mtb; /* medium time base */
u32 ftb; /* fine time base */ u32 ftb; /* fine time base */
unsigned int val, param; unsigned int val;
ret = SPD_STATUS_OK; 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"); printram(" Invalid number of memory banks\n");
ret = SPD_STATUS_INVALID_FIELD; ret = SPD_STATUS_INVALID_FIELD;
} }
param = 1 << (val + 3); printram(" Banks : %u\n", 1 << (val + 3));
printram(" Banks : %u\n", param);
/* SDRAM capacity */ /* SDRAM capacity */
capacity_shift = reg8 & 0x0f; capacity_shift = reg8 & 0x0f;
if (capacity_shift > 0x06) { if (capacity_shift > 0x06) {