Timothy Pearson (tpearson@raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14271
-gerrit
commit d00634f8346f09afc673f880eb75c0c258744522 Author: Timothy Pearson tpearson@raptorengineeringinc.com Date: Wed Apr 6 22:19:43 2016 -0500
Revert "nb/amd/mct_ddr3: Enable DIMM parity when RDIMMs installed"
This reverts commit f961becc433bf23fc8744fdfd757f0cdb75c2c62.
On studying the BKDG more closely this is not the correct place to enable DIMM parity. Further patches to clarify the parity setup process on Family 15h are forthcoming.
Change-Id: I5a3a4f1621e3048f9dfc159709410be9de6ebecd --- src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c index bb12746..3582efa 100644 --- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c +++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c @@ -5152,7 +5152,9 @@ static u8 AutoConfig_D(struct MCTStatStruc *pMCTstat,
if (Status & (1 << SB_Registered)) { /* Registered DIMMs */ - DramConfigLo |= 1 << ParEn; + if (!is_fam15h()) { + DramConfigLo |= 1 << ParEn; + } } else { /* Unbuffered DIMMs */ DramConfigLo |= 1 << UnBuffDimm;