[coreboot-gerrit] Change in ...coreboot[master]: src/device/dram/ddr3.c: Remove not used variable

HAOUAS Elyes (Code Review) gerrit at coreboot.org
Thu Nov 29 20:15:19 CET 2018


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


Change subject: src/device/dram/ddr3.c: Remove not used variable
......................................................................

src/device/dram/ddr3.c: Remove not used variable

Change-Id: Ib4947f1798da558c69d7ff7060cf7de1c171b031
Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
M src/device/dram/ddr3.c
1 file changed, 2 insertions(+), 3 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/29937/1

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) {

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29937
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib4947f1798da558c69d7ff7060cf7de1c171b031
Gerrit-Change-Number: 29937
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes <ehaouas at noos.fr>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181129/1734af57/attachment.html>


More information about the coreboot-gerrit mailing list