[coreboot-gerrit] New patch to review for coreboot: ddr3: Fix SPD CRC calculation

[Patrick Rudolph (siro@das-labor.org)]

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

-gerrit

commit 7400cd88379a158ede0581c181290dcdd5548a7e
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Mon Jun 22 19:32:53 2015 +0200

    ddr3: Fix SPD CRC calculation
    
    Use the correct SPD size for crc calculation. sizeof(*spd) returns 4
    while sizeof(spd_raw_data) returns the expected value of 256.
    
    Change-Id: Iba305c69debd64fa921e08e00ec0a3531c80f56f
    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 8170ae1..c10741b 100644
--- a/src/device/dram/ddr3.c
+++ b/src/device/dram/ddr3.c
@@ -128,7 +128,7 @@ int spd_decode_ddr3(dimm_attr * dimm, spd_raw_data spd)
 	dimm->dram_type = SPD_MEMORY_TYPE_SDRAM_DDR3;
 	dimm->dimm_type = spd[3] & 0xf;
 
-	crc = spd_ddr3_calc_crc(spd, sizeof(*spd));
+	crc = spd_ddr3_calc_crc(spd, sizeof(spd_raw_data));
 	/* Compare with the CRC in the SPD */
 	spd_crc = (spd[127] << 8) + spd[126];
 	/* Verify the CRC is correct */