Re: [Linaro-uefi] [Patch v4 11/56] Hisilicon: update memory init data structure to support D05

On Sat, Nov 19, 2016 at 04:37:26PM +0800, Heyi Guo wrote:

Memory initialization module has a lot of changes to support D05, so the data structure definition has several changes accordingly:

1. Type of nRCD and nRP is changed to UINT32, to hold timing value in pico seconds rather than in clock cycles, to be more accurate.
2. More parameters are added to hold additional timing values, training result, additional mode registers and maximum DDR device frequency.
3. NUMA information is exposed to DXE ACPI driver.

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Heyi Guo heyi.guo@linaro.org Reviewed-by: Leif Lindholm leif.lindholm@linaro.org

Pushed as 69940470a39d9635f32f885fb9ef15cf6f9248f3.

---

Chips/Hisilicon/Include/Library/HwMemInitLib.h | 48 ++++++++++++++++++++++++-- Chips/Hisilicon/Include/PlatformArch.h | 2 ++ 2 files changed, 48 insertions(+), 2 deletions(-)

diff --git a/Chips/Hisilicon/Include/Library/HwMemInitLib.h b/Chips/Hisilicon/Include/Library/HwMemInitLib.h index 4a690af..6bf323d 100644 --- a/Chips/Hisilicon/Include/Library/HwMemInitLib.h +++ b/Chips/Hisilicon/Include/Library/HwMemInitLib.h @@ -123,8 +123,8 @@ typedef struct _DDR_DIMM{ UINT8 MtbDividend; UINT8 MtbDivsor; UINT8 nCL;

• UINT8 nRCD;
• UINT8 nRP;

• UINT32 nRCD;
• UINT32 nRP; UINT8 SPDftb; UINT8 SpdMinTCK; UINT8 SpdMinTCKFtb;

@@ -173,8 +173,14 @@ typedef struct { UINT32 ddrcTiming5; UINT32 ddrcTiming6; UINT32 ddrcTiming7;

• UINT32 ddrcTiming8;

}DDRC_TIMING; +typedef struct _MARGIN_RESULT{

• UINT32 OptimalDramVref[12];
• UINT32 optimalPhyVref[18];

+}MARGIN_RESULT;

typedef struct _DDR_Channel{ BOOLEAN Status; UINT8 CurrentDimmNum; @@ -184,22 +190,42 @@ typedef struct _DDR_Channel{ UINT8 DramWidth; UINT8 ModuleType; UINT32 MemSize;

• UINT32 tck;
• UINT32 ratio; UINT32 CLSupport; UINT32 minTck;
• UINT32 taref; UINT32 nAA;
• UINT32 nAOND;
• UINT32 nCKE; UINT32 nCL; UINT32 nCCDL;
• UINT32 nCKSRX;
• UINT32 nCKSRE;
• UINT32 nCCDNSW;
• UINT32 nCCDNSR; UINT32 nFAW;
• UINT32 nMRD;
• UINT32 nMOD; UINT32 nRCD; UINT32 nRRD; UINT32 nRRDL; UINT32 nRAS; UINT32 nRC; UINT32 nRFC;
• UINT32 nRFCAB; UINT32 nRTP;
• UINT32 nRTW; UINT32 nRP;
• UINT32 nSRE;
• UINT32 nWL; UINT32 nWR; UINT32 nWTR;
• UINT32 nWTRL;
• UINT32 nXARD;
• UINT32 nZQPRD;
• UINT32 nZQINIT;
• UINT32 nZQCS; UINT8 cwl; //tWL? UINT8 pl; //parity latency UINT8 wr_pre_2t_en;

@@ -232,11 +258,19 @@ typedef struct _DDR_Channel{ UINT32 ddrcCfgDfiLat0; UINT32 ddrcCfgDfiLat1; UINT32 parityLatency;

• UINT32 dimm_parity_en; DDRC_TIMING ddrcTiming; DDR_DIMM Dimm[MAX_DIMM];
• MARGIN_RESULT sMargin;

}DDR_CHANNEL; typedef struct _NVRAM_RANK{

• UINT16 MR0;
• UINT16 MR1;
• UINT16 MR2;
• UINT16 MR3;
• UINT16 MR4;
• UINT16 MR5; UINT16 MR6[9];

}NVRAM_RANK; @@ -306,6 +340,14 @@ typedef struct _MEMORY{ UINT32 Config2; }MEMORY; +typedef struct _NUMAINFO{

• UINT8 NodeId;
• UINT64 Base;
• UINT64 Length;
• UINT32 ScclInterleaveEn;

+}NUMAINFO;

typedef struct _GBL_DATA { DDR_CHANNEL Channel[MAX_SOCKET][MAX_CHANNEL]; @@ -319,6 +361,7 @@ typedef struct _GBL_DATA UINT32 SpdTck; UINT32 Tck; UINT32 DdrFreqIdx;

• UINT32 DevParaFreqIdx; //Maximum frequency of DDR device UINT32 MemSize; UINT32 EccEn;

@@ -365,6 +408,7 @@ typedef struct _GBL_DATA BOOLEAN chipIsEc; NVRAM nvram; MEMORY mem;

• NUMAINFO NumaInfo[MAX_SOCKET][MAX_NUM_PER_TYPE];

}GBL_DATA, *pGBL_DATA; diff --git a/Chips/Hisilicon/Include/PlatformArch.h b/Chips/Hisilicon/Include/PlatformArch.h index f1ccbb6..45995c5 100644 --- a/Chips/Hisilicon/Include/PlatformArch.h +++ b/Chips/Hisilicon/Include/PlatformArch.h @@ -26,6 +26,8 @@ #define MAX_DIMM 3 #define MAX_RANK_CH 12 #define MAX_RANK_DIMM 4 +// Max NUMA node number for each node type +#define MAX_NUM_PER_TYPE 8 #define S1_BASE 0x40000000000 -- 1.9.1