diff options
| author | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2016-04-06 22:19:43 -0500 |
|---|---|---|
| committer | Timothy Pearson <tpearson@raptorengineeringinc.com> | 2016-04-08 17:21:21 +0200 |
| commit | 5a577251263369d259bee033e41584d7c562ca6d (patch) | |
| tree | df3a4a96c14c4d3b43d7da0c5e08f4b0219c3f35 /src/northbridge/amd/amdmct/mct_ddr3 | |
| parent | ba817d0931eb9bbf23d6c09f56f9ea2ba22fdad6 (diff) | |
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
Reviewed-on: https://review.coreboot.org/14271
Tested-by: build bot (Jenkins)
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/northbridge/amd/amdmct/mct_ddr3')
| -rw-r--r-- | src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c index bb12746a5c..3582efa451 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; |