[coreboot-gerrit] New patch to review for coreboot: ddr3: add missing newline

Patrick Rudolph (siro@das-labor.org) gerrit at coreboot.org
Mon Jun 22 20:06:40 CEST 2015


Patrick Rudolph (siro at das-labor.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10636

-gerrit

commit 98d2b0efe57a63e64bda027edf177d4f3976c0de
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Mon Jun 22 19:46:34 2015 +0200

    ddr3: add missing newline
    
    Add missing newline to SPD CRC verification error message.
    Verified by testing this code on Intel IvyBridge and Gigabyte GA-B75M-D3H.
    
    Change-Id: Id1a0a2329507975c3f66ab884f6e26d99003318e
    Signed-off-by: Patrick Rudolph <siro at das-labor.org>
---
 src/device/dram/ddr3.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/device/dram/ddr3.c b/src/device/dram/ddr3.c
index c10741b..e1bb873 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -133,7 +133,7 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd)
 	spd_crc = (spd[127] << 8) + spd[126];
 	/* Verify the CRC is correct */
 	if (crc != spd_crc) {
-		printram("ERROR: SPD CRC failed!!!");
+		printram("ERROR: SPD CRC failed!!!\n");
 		ret = SPD_STATUS_CRC_ERROR;
 	};
 



More information about the coreboot-gerrit mailing list