vendorcode/intel/fsp: Update MemInfoHob header for Jasper Lake

Jasper Lake has been using the incorrect MemInfoHob header. Updating
the header to align it with Jasper Lake MRC code.

BUG=b:158722318
TEST=Verify memory info is populated for channnel 0 and 1 on wadddledoo.

Change-Id: Icca3e3b4cda9ca257f3b725823facf52ceec37b7
Signed-off-by: Aamir Bohra <aamir.bohra@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42420
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Reviewed-by: EricR Lai <ericr_lai@compal.corp-partner.google.com>
Reviewed-by: Ronak Kanabar <ronak.kanabar@intel.com>
This commit is contained in:
Aamir Bohra 2020-06-16 17:20:34 +05:30 committed by Patrick Georgi
parent dd48b176f3
commit 0be23eda13
1 changed files with 46 additions and 49 deletions

View File

@ -18,16 +18,18 @@
#ifndef _MEM_INFO_HOB_H_ #ifndef _MEM_INFO_HOB_H_
#define _MEM_INFO_HOB_H_ #define _MEM_INFO_HOB_H_
#pragma pack (push, 1)
#include <Uefi/UefiMultiPhase.h> #include <Uefi/UefiMultiPhase.h>
#include <Pi/PiBootMode.h> #include <Pi/PiBootMode.h>
#include <Pi/PiHob.h> #include <Pi/PiHob.h>
#pragma pack (push, 1)
extern EFI_GUID gSiMemoryS3DataGuid; extern EFI_GUID gSiMemoryS3DataGuid;
extern EFI_GUID gSiMemoryInfoDataGuid; extern EFI_GUID gSiMemoryInfoDataGuid;
extern EFI_GUID gSiMemoryPlatformDataGuid; extern EFI_GUID gSiMemoryPlatformDataGuid;
#define MAX_TRACE_CACHE_TYPE 3
#define MAX_NODE 1 #define MAX_NODE 1
#define MAX_CH 2 #define MAX_CH 2
#define MAX_DIMM 2 #define MAX_DIMM 2
@ -43,6 +45,7 @@ extern EFI_GUID gSiMemoryPlatformDataGuid;
#define B_RANK2_PRS BIT4 #define B_RANK2_PRS BIT4
#define B_RANK3_PRS BIT5 #define B_RANK3_PRS BIT5
/// ///
/// Defines taken from MRC so avoid having to include MrcInterface.h /// Defines taken from MRC so avoid having to include MrcInterface.h
/// ///
@ -64,8 +67,7 @@ typedef struct {
UINT8 Build; ///< Build number UINT8 Build; ///< Build number
} SiMrcVersion; } SiMrcVersion;
// //// Matches MrcChannelSts enum in MRC
// Matches MrcChannelSts enum in MRC
// //
#ifndef CHANNEL_NOT_PRESENT #ifndef CHANNEL_NOT_PRESENT
#define CHANNEL_NOT_PRESENT 0 // There is no channel present on the controller. #define CHANNEL_NOT_PRESENT 0 // There is no channel present on the controller.
@ -96,18 +98,20 @@ typedef struct {
// //
// Matches MrcBootMode enum in MRC // Matches MrcBootMode enum in MRC
// //
#ifndef bmCold #ifndef __MRC_BOOT_MODE__
#define bmCold 0 // Cold boot #define __MRC_BOOT_MODE__ //The below values are originated from MrcCommonTypes.h
#endif #ifndef INT32_MAX
#ifndef bmWarm #define INT32_MAX (0x7FFFFFFF)
#define bmWarm 1 // Warm boot #endif //INT32_MAX
#endif typedef enum {
#ifndef bmS3 bmCold, ///< Cold boot
#define bmS3 2 // S3 resume bmWarm, ///< Warm boot
#endif bmS3, ///< S3 resume
#ifndef bmFast bmFast, ///< Fast boot
#define bmFast 3 // Fast boot MrcBootModeMax, ///< MRC_BOOT_MODE enumeration maximum value.
#endif MrcBootModeDelim = INT32_MAX ///< This value ensures the enum size is consistent on both sides of the PPI.
} MRC_BOOT_MODE;
#endif //__MRC_BOOT_MODE__
// //
// Matches MrcDdrType enum in MRC // Matches MrcDdrType enum in MRC
@ -121,15 +125,15 @@ typedef struct {
#ifndef MRC_DDR_TYPE_LPDDR3 #ifndef MRC_DDR_TYPE_LPDDR3
#define MRC_DDR_TYPE_LPDDR3 2 #define MRC_DDR_TYPE_LPDDR3 2
#endif #endif
#ifndef CPU_CFL//CNL
#ifndef MRC_DDR_TYPE_LPDDR4 #ifndef MRC_DDR_TYPE_LPDDR4
#define MRC_DDR_TYPE_LPDDR4 3 #define MRC_DDR_TYPE_LPDDR4 3
#endif #endif
#else//CFL #ifndef MRC_DDR_TYPE_WIO2
#ifndef MRC_DDR_TYPE_UNKNOWN #define MRC_DDR_TYPE_WIO2 4
#define MRC_DDR_TYPE_UNKNOWN 3 #endif
#ifndef MRC_DDR_TYPE_UNKNOWN
#define MRC_DDR_TYPE_UNKNOWN 5
#endif #endif
#endif//CPU_CFL-endif
#define MAX_PROFILE_NUM 4 // number of memory profiles supported #define MAX_PROFILE_NUM 4 // number of memory profiles supported
#define MAX_XMP_PROFILE_NUM 2 // number of XMP profiles supported #define MAX_XMP_PROFILE_NUM 2 // number of XMP profiles supported
@ -162,13 +166,6 @@ typedef struct {
UINT16 tCCD_L; ///< Number of tCK cycles for the channel DIMM's minimum CAS-to-CAS delay for same bank group. UINT16 tCCD_L; ///< Number of tCK cycles for the channel DIMM's minimum CAS-to-CAS delay for same bank group.
} MRC_CH_TIMING; } MRC_CH_TIMING;
typedef struct {
UINT8 SG; ///< Number of tCK cycles between transactions in the same bank group.
UINT8 DG; ///< Number of tCK cycles between transactions when switching bank groups.
UINT8 DR; ///< Number of tCK cycles between transactions when switching between Ranks (in the same DIMM).
UINT8 DD; ///< Number of tCK cycles between transactions when switching between DIMMs.
} MRC_TA_TIMING;
/// ///
/// Memory SMBIOS & OC Memory Data Hob /// Memory SMBIOS & OC Memory Data Hob
/// ///
@ -183,7 +180,8 @@ typedef struct {
UINT8 SpdModuleType; ///< Save SPD ModuleType information needed for SMBIOS structure creation. UINT8 SpdModuleType; ///< Save SPD ModuleType information needed for SMBIOS structure creation.
UINT8 SpdModuleMemoryBusWidth; ///< Save SPD ModuleMemoryBusWidth information needed for SMBIOS structure creation. UINT8 SpdModuleMemoryBusWidth; ///< Save SPD ModuleMemoryBusWidth information needed for SMBIOS structure creation.
UINT8 SpdSave[MAX_SPD_SAVE]; ///< Save SPD Manufacturing information needed for SMBIOS structure creation. UINT8 SpdSave[MAX_SPD_SAVE]; ///< Save SPD Manufacturing information needed for SMBIOS structure creation.
UINT16 Speed; ///< The maximum capable speed of the device, in MHz. UINT16 Speed; ///< The maximum capable speed of the device, in MHz
UINT8 MdSocket; ///< MdSocket: 0 = Memory Down, 1 = Socketed. Needed for SMBIOS structure creation.
} DIMM_INFO; } DIMM_INFO;
typedef struct { typedef struct {
@ -192,10 +190,6 @@ typedef struct {
UINT8 DimmCount; ///< Number of valid DIMMs that exist in the channel. UINT8 DimmCount; ///< Number of valid DIMMs that exist in the channel.
MRC_CH_TIMING Timing[MAX_PROFILE_NUM]; ///< The channel timing values. MRC_CH_TIMING Timing[MAX_PROFILE_NUM]; ///< The channel timing values.
DIMM_INFO DimmInfo[MAX_DIMM]; ///< Save the DIMM output characteristics. DIMM_INFO DimmInfo[MAX_DIMM]; ///< Save the DIMM output characteristics.
MRC_TA_TIMING tRd2Rd; ///< Read-to-Read Turn Around Timings
MRC_TA_TIMING tRd2Wr; ///< Read-to-Write Turn Around Timings
MRC_TA_TIMING tWr2Rd; ///< Write-to-Read Turn Around Timings
MRC_TA_TIMING tWr2Wr; ///< Write-to-Write Turn Around Timings
} CHANNEL_INFO; } CHANNEL_INFO;
typedef struct { typedef struct {
@ -204,12 +198,16 @@ typedef struct {
UINT8 RevisionId; ///< The PCI revision id of this memory controller. UINT8 RevisionId; ///< The PCI revision id of this memory controller.
UINT8 ChannelCount; ///< Number of valid channels that exist on the controller. UINT8 ChannelCount; ///< Number of valid channels that exist on the controller.
CHANNEL_INFO ChannelInfo[MAX_CH]; ///< The following are channel level definitions. CHANNEL_INFO ChannelInfo[MAX_CH]; ///< The following are channel level definitions.
MRC_TA_TIMING tRd2Rd; ///< Deprecated and moved to CHANNEL_INFO. Read-to-Read Turn Around Timings
MRC_TA_TIMING tRd2Wr; ///< Deprecated and moved to CHANNEL_INFO. Read-to-Write Turn Around Timings
MRC_TA_TIMING tWr2Rd; ///< Deprecated and moved to CHANNEL_INFO. Write-to-Read Turn Around Timings
MRC_TA_TIMING tWr2Wr; ///< Deprecated and moved to CHANNEL_INFO. Write-to-Write Turn Around Timings
} CONTROLLER_INFO; } CONTROLLER_INFO;
typedef struct {
UINT64 BaseAddress; ///< Trace Base Address
UINT64 TotalSize; ///< Total Trace Region of Same Cache type
UINT8 CacheType; ///< Trace Cache Type
UINT8 ErrorCode; ///< Trace Region Allocation Fail Error code
UINT8 Rsvd[2];
} PSMI_MEM_INFO;
typedef struct { typedef struct {
UINT8 Revision; UINT8 Revision;
UINT16 DataWidth; ///< Data width, in bits, of this memory device UINT16 DataWidth; ///< Data width, in bits, of this memory device
@ -251,16 +249,15 @@ typedef struct {
UINT32 TsegSize; UINT32 TsegSize;
UINT32 TsegBase; UINT32 TsegBase;
UINT32 PrmrrSize; UINT32 PrmrrSize;
UINT32 PrmrrBase; UINT64 PrmrrBase;
UINT32 PramSize;
UINT64 PramBase;
UINT64 DismLimit;
UINT64 DismBase;
UINT32 GttBase; UINT32 GttBase;
UINT32 MmioSize; UINT32 MmioSize;
UINT32 PciEBaseAddress; UINT32 PciEBaseAddress;
#ifdef CPU_CFL PSMI_MEM_INFO PsmiInfo[MAX_TRACE_CACHE_TYPE];
UINT32 GdxcIotBase;
UINT32 GdxcIotSize;
UINT32 GdxcMotBase;
UINT32 GdxcMotSize;
#endif //CPU_CFL
} MEMORY_PLATFORM_DATA; } MEMORY_PLATFORM_DATA;
typedef struct { typedef struct {