[coreboot-gerrit] Patch set updated for coreboot: nb/amd/amdmct/mct_ddr3: Fix a minor RDIMM CS select error

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Thu Nov 26 22:14:53 CET 2015


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

-gerrit

commit d53ab4f110939890b69fd4799f37a6b09352408e
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Thu Aug 27 13:18:53 2015 -0500

    nb/amd/amdmct/mct_ddr3: Fix a minor RDIMM CS select error
    
    Change-Id: I4cdfeec887813c17edcdee8858222414fb19b72c
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdmct/mct_ddr3/mctrci.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mctrci.c b/src/northbridge/amd/amdmct/mct_ddr3/mctrci.c
index 01061a7..a63fe2e 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mctrci.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mctrci.c
@@ -232,7 +232,7 @@ void mct_DramControlReg_Init_D(struct MCTStatStruc *pMCTstat,
 	for (MrsChipSel = 0; MrsChipSel < 8; MrsChipSel ++, MrsChipSel ++) {
 		if (pDCTstat->CSPresent & (1 << MrsChipSel)) {
 			val = Get_NB32_DCT(dev, dct, 0xa8);
-			val &= ~(0xf << 8);
+			val &= ~(0xff << 8);
 
 			switch (MrsChipSel) {
 				case 0:
@@ -279,7 +279,7 @@ void FreqChgCtrlWrd(struct MCTStatStruc *pMCTstat,
 			/* 2. Program F2x[1, 0]A8[CtrlWordCS]=bit mask for target chip selects. */
 			val = Get_NB32_DCT(dev, dct, 0xa8);
 			val &= ~(0xff << 8);
-			val |= (0x3 << (MrsChipSel & 0xfe)) << 8;
+			val |= (0x3 << (MrsChipSel & ~0x1)) << 8;
 			Set_NB32_DCT(dev, dct, 0xa8, val);
 
 			/* Resend control word 10 */



More information about the coreboot-gerrit mailing list