[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 11:53:36 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 0867424f014593859773fb84004149f535fbb37e
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 2ca65ca..330f37f 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -1735,6 +1735,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