[coreboot-gerrit] Patch set updated for coreboot: nb/intel/sandybridge/raminit: Fix disable_channel

Patrick Rudolph (siro@das-labor.org) gerrit at coreboot.org
Sat Nov 19 16:16:58 CET 2016


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

-gerrit

commit 60d6e529480a39144d7011769c5d5a5f5f49f742
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Thu Nov 17 20:02:43 2016 +0100

    nb/intel/sandybridge/raminit: Fix disable_channel
    
    Also memset info.dimm as it contains decoded SPD timings
    used to calculate common timings.
    
    Tested manually on Lenovo T420.
    
    Change-Id: I659e5bc2a6cbadd9539931ee00ddea0a5253295f
    Signed-off-by: Patrick Rudolph <siro at das-labor.org>
---
 src/northbridge/intel/sandybridge/raminit.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/northbridge/intel/sandybridge/raminit.c b/src/northbridge/intel/sandybridge/raminit.c
index 40a4d27..f631ad2 100644
--- a/src/northbridge/intel/sandybridge/raminit.c
+++ b/src/northbridge/intel/sandybridge/raminit.c
@@ -255,6 +255,7 @@ static void disable_channel(ramctr_timing *ctrl, int channel) {
 	ctrl->cmd_stretch[channel] = 0;
 	ctrl->mad_dimm[channel] = 0;
 	memset(&ctrl->timings[channel][0], 0, sizeof(ctrl->timings[0]));
+	memset(&ctrl->info.dimm[channel][0], 0, sizeof(ctrl->info.dimm[0]));
 }
 
 /*



More information about the coreboot-gerrit mailing list