[coreboot-gerrit] New patch to review for coreboot: Revert "nb/amd/mct_ddr3: Disable MCE framework during DRAM training"

[Timothy Pearson (tpearson@raptorengineeringinc.com)]

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

-gerrit

commit 540e40e2820a3a5caec213dac15a41475e3c6e80
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Thu Apr 21 01:18:36 2016 -0500

    Revert "nb/amd/mct_ddr3: Disable MCE framework during DRAM training"
    
    After substantial testing it has been determined that it is neither
    required nor safe to disable the DRAM MCA during initial startup.
    
    This (mostly) reverts commit c094d9961144871c472698c41ce634e58abb6a32.
    
    Tested-On: ASUS KGPE-D16 w/ 1x Kingston 9965516-483.A00LF and 1x Opteron 6262HE
    Tested-On: ASUS KGPE-D16 w/ 4x Crucial 36KSF1G72PZ-1G6M1 and 1x Opteron 6262HE
    Change-Id: I58fcc296b8c45ecaedf540951c365e4ce52baaf5
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdmct/mct_ddr3/mct_d.c    | 13 -------------
 src/northbridge/amd/amdmct/mct_ddr3/mct_d.h    |  6 ------
 src/northbridge/amd/amdmct/mct_ddr3/mctecc_d.c | 20 +++++++++++---------
 3 files changed, 11 insertions(+), 28 deletions(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
index 3582efa..284e890 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -8049,19 +8049,6 @@ void mct_SetDramConfigHi_D(struct MCTStatStruc *pMCTstat,
 
 	printk(BIOS_DEBUG, "mct_SetDramConfigHi_D: DramConfigHi:    %08x\n", DramConfigHi);
 
-	/* Prevent lockups on parity errors during initial DCT startup */
-	if (!pDCTstat->mca_config_backed_up) {
-		dword = Get_NB32(pDCTstat->dev_nbmisc, 0x44);
-		pDCTstat->sync_flood_on_dram_err = (dword >> 30) & 0x1;
-		pDCTstat->sync_flood_on_any_uc_err = (dword >> 21) & 0x1;
-		pDCTstat->sync_flood_on_uc_dram_ecc_err = (dword >> 2) & 0x1;
-		dword &= ~(0x1 << 30);
-		dword &= ~(0x1 << 21);
-		dword &= ~(0x1 << 2);
-		Set_NB32(pDCTstat->dev_nbmisc, 0x44, dword);
-		pDCTstat->mca_config_backed_up = 1;
-	}
-
 	/* Program the DRAM Configuration High register */
 	Set_NB32_DCT(dev, dct, 0x94, DramConfigHi);
 
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.h b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.h
index aba39c1..a5a3d88 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.h
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.h
@@ -581,12 +581,6 @@ struct DCTStatStruc {		/* A per Node structure*/
 	uint8_t NbPstateThreshold;
 	uint8_t NbPstateHi;
 
-	/* MCA backup variables */
-	uint8_t mca_config_backed_up;
-	uint8_t sync_flood_on_dram_err;
-	uint8_t sync_flood_on_any_uc_err;
-	uint8_t sync_flood_on_uc_dram_ecc_err;
-
 	/* New for LB Support */
 	u8 NodePresent;
 	u32 dev_host;
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mctecc_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mctecc_d.c
index 203d112..cef28f4 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mctecc_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mctecc_d.c
@@ -116,6 +116,17 @@ u8 ECCInit_D(struct MCTStatStruc *pMCTstat, struct DCTStatStruc *pDCTstatA)
 		pDCTstat = pDCTstatA + Node;
 
 		if (NodePresent_D(Node)) {
+			dword = Get_NB32(pDCTstat->dev_nbmisc, 0x44);
+			sync_flood_on_dram_err[Node] = (dword >> 30) & 0x1;
+			sync_flood_on_any_uc_err[Node] = (dword >> 21) & 0x1;
+			dword &= ~(0x1 << 30);
+			dword &= ~(0x1 << 21);
+			Set_NB32(pDCTstat->dev_nbmisc, 0x44, dword);
+
+			/* Clear MC4 error status */
+			pci_write_config32(pDCTstat->dev_nbmisc, 0x48, 0x0);
+			pci_write_config32(pDCTstat->dev_nbmisc, 0x4c, 0x0);
+
 			/* Clear the RAM before enabling ECC to prevent MCE-related lockups */
 			DCTMemClr_Init_D(pMCTstat, pDCTstat);
 			DCTMemClr_Sync_D(pMCTstat, pDCTstat);
@@ -264,15 +275,6 @@ u8 ECCInit_D(struct MCTStatStruc *pMCTstat, struct DCTStatStruc *pDCTstatA)
 				/* Clear MC4 error status */
 				pci_write_config32(pDCTstat->dev_nbmisc, 0x48, 0x0);
 				pci_write_config32(pDCTstat->dev_nbmisc, 0x4c, 0x0);
-
-				/* Restore MCA settings */
-				if (pDCTstat->mca_config_backed_up) {
-					val = pci_read_config32(pDCTstat->dev_nbmisc, 0x44);
-					val |= (pDCTstat->sync_flood_on_dram_err & 0x1) << 30;
-					val |= (pDCTstat->sync_flood_on_any_uc_err & 0x1) << 21;
-					val |= (pDCTstat->sync_flood_on_uc_dram_ecc_err & 0x1) << 2;
-					pci_write_config32(pDCTstat->dev_nbmisc, 0x44, val);
-				}
 			}
 		}
 	}