New patch to review for coreboot: nb/amd/mct_ddr3: Scale lane delays for each DIMM after MEMCLK change

[Timothy Pearson]

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

-gerrit

commit 83d57fb5dee0de3520a4a3b441c7397d64c6b2d8
Author: Timothy Pearson 
Date:   Fri Apr 22 22:16:45 2016 -0500

nb/amd/mct_ddr3: Scale lane delays for each DIMM after MEMCLK change

When more that one DIMM was installed on a DCT only the first DIMM
    would have its delay values scaled to the new memory clock frequency
    after a memory clock change during write levelling.

Store the previous memory clock of each DIMM during write levelling
    to ensure that every DIMM has its delay values rescaled.

Change-Id: I56e816d3d3256925598219d92783246f5f4ab567
    Signed-off-by: Timothy Pearson 
---
 src/northbridge/amd/amdmct/mct_ddr3/mhwlc_d.c | 4 ++--
 src/northbridge/amd/amdmct/mct_ddr3/mwlc_d.h  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mhwlc_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mhwlc_d.c
index 9702126..6e1c850 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mhwlc_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mhwlc_d.c
@@ -1212,7 +1212,7 @@ void procConfig(struct MCTStatStruc *pMCTstat, struct DCTStatStruc *pDCTstat, ui
    					((pDCTData->WLGrossDelayPrevPass[lane_count*dimm+ByteLane] & 0x1f) << 5);
    				SeedTotalPreScaling[ByteLane] = (SeedTotal[ByteLane] - RegisterDelay - (0x20 * WrDqDqsEarly));
    				SeedTotal[ByteLane] = (int32_t) (RegisterDelay + ((((int64_t) SeedTotalPreScaling[ByteLane]) *
-						fam15h_freq_tab[MemClkFreq] * 100) / (fam15h_freq_tab[pDCTData->WLPrevMemclkFreq] * 100)));
+						fam15h_freq_tab[MemClkFreq] * 100) / (fam15h_freq_tab[pDCTData->WLPrevMemclkFreq[dimm]] * 100)));
    			}

/* Generate register values from seeds */
@@ -1326,7 +1326,7 @@ void procConfig(struct MCTStatStruc *pMCTstat, struct DCTStatStruc *pDCTstat, ui
    	}
    }

-	pDCTData->WLPrevMemclkFreq = MemClkFreq;
+	pDCTData->WLPrevMemclkFreq[dimm] = MemClkFreq;
    setWLByteDelay(pDCTstat, dct, ByteLane, dimm, 0, pass, lane_count);
 }

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mwlc_d.h b/src/northbridge/amd/amdmct/mct_ddr3/mwlc_d.h
index 28359a1..ca04d28 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mwlc_d.h
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mwlc_d.h
@@ -145,7 +145,7 @@ typedef struct _sDCTStruct
    int32_t WLCriticalGrossDelayFirstPass;
    int32_t WLCriticalGrossDelayPrevPass;
    int32_t WLCriticalGrossDelayFinalPass;
-	uint16_t WLPrevMemclkFreq;
+	uint16_t WLPrevMemclkFreq[MAX_TOTAL_DIMMS];
    u16 RegMan1Present;
    u8 DimmPresent[MAX_TOTAL_DIMMS];/* Indicates which DIMMs are present */
    				/* from Total Number of DIMMs(per Node)*/