New patch to review for coreboot: nb/amd/mct_ddr3: Run fence training on each node after memory clock change

[Timothy Pearson]

Timothy Pearson ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14445

-gerrit

commit 98908b162be85d56342e3fdad364fd232f4b794d
Author: Timothy Pearson 
Date:   Thu Apr 21 01:08:01 2016 -0500

nb/amd/mct_ddr3: Run fence training on each node after memory clock change

The BKDG requires that phy fences to be re-trained after a memory clock chagne.
    Memory training on the ASUS KGPE-D16 and KCMA-D8 somehow "mostly" worked
     -- without actually following this requirement -- !

Fix the single typo that caused several weeks of delay putting
    servers with Kingston RAM (and others) into production...

Tested-On: ASUS KGPE-D16 w/ 4x Crucial 36KSF1G72PZ-1G6M1 and 1x Opteron 6262HE
    Change-Id: I197e6728d2b0ac8c1535740599459d080b17af33
    Signed-off-by: Timothy Pearson 
---
 src/northbridge/amd/amdmct/mct_ddr3/mctsrc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mctsrc.c b/src/northbridge/amd/amdmct/mct_ddr3/mctsrc.c
index 9aadb2c..c8c75e1 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mctsrc.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mctsrc.c
@@ -2,7 +2,7 @@
  * This file is part of the coreboot project.
  *
  * Copyright (C) 2010 Advanced Micro Devices, Inc.
- * Copyright (C) 2015 Timothy Pearson , Raptor Engineering
+ * Copyright (C) 2015 - 2016 Raptor Engineering, LLC
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -2092,7 +2092,7 @@ void phyAssistedMemFnceTraining(struct MCTStatStruc *pMCTstat,

if (single_node_number >= 0) {
    	start_node = single_node_number;
-		end_node = single_node_number;
+		end_node = single_node_number + 1;
    }

/* FIXME: skip for Ax */