[coreboot-gerrit] Patch set updated for coreboot: northbridge/amd/amdmct/mct_ddr3: Force retraining on every boot

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sun Oct 18 02:53:46 CEST 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12034

-gerrit

commit a80d6d31d7f50a7ae97691672fd1f916202626aa
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Fri Aug 7 23:59:33 2015 -0500

    northbridge/amd/amdmct/mct_ddr3: Force retraining on every boot
    
    Change-Id: I017e0dd5120110124d5b5d5276befef6f7740614
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
index 6bb4778..3caadde 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -1733,6 +1733,16 @@ restartinit:
 					allow_config_restore = 0;
 		}
 
+		/* FIXME
+		 * Stability issues have arisen on multiple Family 15h systems
+		 * when configuration restoration is enabled.  In all cases these
+		 * stability issues resolved by allowing the RAM to go through a
+		 * full training cycle.
+		 *
+		 * Debug and reenable this!
+		 */
+		allow_config_restore = 0;
+
 		for (Node = 0; Node < MAX_NODES_SUPPORTED; Node++) {
 			struct DCTStatStruc *pDCTstat;
 			pDCTstat = pDCTstatA + Node;



More information about the coreboot-gerrit mailing list