* [edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update SA_MISC_PEI_PREMEM_CONFIG
@ 2021-08-10 4:36 Nate DeSimone
2021-08-11 1:55 ` Chiu, Chasel
0 siblings, 1 reply; 2+ messages in thread
From: Nate DeSimone @ 2021-08-10 4:36 UTC (permalink / raw)
To: devel; +Cc: Chasel Chiu, Michael Kubacki, Benjamin Doron
Updates SA_MISC_PEI_PREMEM_CONFIG from revision 1
to revision 3. Add initialization of the policy values.
Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Michael Kubacki <Michael.Kubacki@microsoft.com>
Cc: Benjamin Doron <benjamin.doron00@gmail.com>
Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
---
.../KabylakeRvp3/OpenBoardPkg.dsc | 24 +++---
.../PeiSiliconPolicyUpdateLib.c | 39 +++++++++-
.../PeiSiliconPolicyUpdateLib.inf | 9 ++-
.../ConfigBlock/SaMiscPeiPreMemConfig.h | 77 ++++++++++++++++++-
.../Library/PeiSaPolicyLib/PeiSaPolicyLib.c | 37 ++++++++-
5 files changed, 169 insertions(+), 17 deletions(-)
diff --git a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
index 8523ab3f4f..f64555e391 100644
--- a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
+++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
@@ -182,17 +182,6 @@
# Board-specific
#######################################
PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHookLib.inf
-!if gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection == 1
- #
- # FSP API mode
- #
- SiliconPolicyUpdateLib|$(PROJECT)/FspWrapper/Library/PeiSiliconPolicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf
-!else
- #
- # FSP Dispatch mode and non-FSP build (EDK2 build)
- #
- SiliconPolicyUpdateLib|$(PROJECT)/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
-!endif
[LibraryClasses.IA32.SEC]
#######################################
@@ -200,6 +189,7 @@
#######################################
TestPointCheckLib|$(PLATFORM_PACKAGE)/Test/Library/TestPointCheckLib/SecTestPointCheckLib.inf
SecBoardInitLib|$(PLATFORM_PACKAGE)/PlatformInit/Library/SecBoardInitLibNull/SecBoardInitLibNull.inf
+ SiliconPolicyUpdateLib|MinPlatformPkg/PlatformInit/Library/SiliconPolicyUpdateLibNull/SiliconPolicyUpdateLibNull.inf
[LibraryClasses.common.PEIM]
#######################################
@@ -222,6 +212,18 @@
#######################################
# Board Package
#######################################
+!if gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection == 1
+ #
+ # FSP API mode
+ #
+ SiliconPolicyUpdateLib|$(PROJECT)/FspWrapper/Library/PeiSiliconPolicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf
+!else
+ #
+ # FSP Dispatch mode and non-FSP build (EDK2 build)
+ #
+ SiliconPolicyUpdateLib|$(PROJECT)/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
+!endif
+
# Thunderbolt
!if gKabylakeOpenBoardPkgTokenSpaceGuid.PcdTbtEnable == TRUE
PeiDTbtInitLib|$(PLATFORM_BOARD_PACKAGE)/Features/Tbt/Library/Private/PeiDTbtInitLib/PeiDTbtInitLib.inf
diff --git a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
index 5cc7c03c61..2dce9be63c 100644
--- a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
+++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
@@ -1,7 +1,7 @@
/** @file
Provides silicon policy update library functions.
-Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
**/
@@ -398,6 +398,8 @@ SiliconPolicyUpdatePreMem (
SA_MISC_PEI_PREMEM_CONFIG *MiscPeiPreMemConfig;
MEMORY_CONFIG_NO_CRC *MemConfigNoCrc;
VOID *Buffer;
+ UINTN VariableSize;
+ VOID *MemorySavedData;
UINT8 SpdAddressTable[4];
DEBUG((DEBUG_INFO, "\nUpdating Policy in Pre-Mem\n"));
@@ -417,6 +419,41 @@ SiliconPolicyUpdatePreMem (
// Pass board specific SpdAddressTable to policy
//
CopyMem ((VOID *) MiscPeiPreMemConfig->SpdAddressTable, (VOID *) SpdAddressTable, (sizeof (UINT8) * 4));
+
+ //
+ // Set size of SMRAM
+ //
+ MiscPeiPreMemConfig->TsegSize = PcdGet32 (PcdTsegSize);
+
+ //
+ // Initialize S3 Data variable (S3DataPtr). It may be used for warm and fast boot paths.
+ // Note: AmberLake FSP does not implement the FSPM_ARCH_CONFIG_PPI added in FSP 2.1, hence
+ // the platform specific S3DataPtr must be used instead.
+ //
+ VariableSize = 0;
+ MemorySavedData = NULL;
+ Status = PeiGetVariable (
+ L"MemoryConfig",
+ &gFspNonVolatileStorageHobGuid,
+ &MemorySavedData,
+ &VariableSize
+ );
+ DEBUG ((DEBUG_INFO, "Get L\"MemoryConfig\" gFspNonVolatileStorageHobGuid - %r\n", Status));
+ DEBUG ((DEBUG_INFO, "MemoryConfig Size - 0x%x\n", VariableSize));
+ if (!EFI_ERROR (Status)) {
+ MiscPeiPreMemConfig->S3DataPtr = MemorySavedData;
+ }
+
+ //
+ // In FSP Dispatch Mode these BAR values are initialized by SiliconPolicyInitPreMem() in
+ // KabylakeSiliconPkg/Library/PeiSiliconPolicyInitLib/PeiPolicyInitPreMem.c; this function calls
+ // PEI_PREMEM_SI_DEFAULT_POLICY_INIT_PPI->PeiPreMemPolicyInit() to initialize all Config Blocks
+ // with default policy values (including these BAR values.) PEI_PREMEM_SI_DEFAULT_POLICY_INIT_PPI
+ // is implemented in the FSP. Make sure the value that FSP is using matches the value we are using.
+ //
+ ASSERT (PcdGet64 (PcdMchBaseAddress) <= 0xFFFFFFFF);
+ ASSERT (MiscPeiPreMemConfig->MchBar == (UINT32) PcdGet64 (PcdMchBaseAddress));
+ ASSERT (MiscPeiPreMemConfig->SmbusBar == PcdGet16 (PcdSmbusBaseAddress));
}
MemConfigNoCrc = NULL;
Status = GetConfigBlock (Policy, &gMemoryConfigNoCrcGuid, (VOID *) &MemConfigNoCrc);
diff --git a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
index 97ec70f611..5c2da68bf9 100644
--- a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
+++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
@@ -1,7 +1,7 @@
### @file
# Component information file for silicon policy update library
#
-# Copyright (c) 2019 - 2020 Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2019 - 2021 Intel Corporation. All rights reserved.<BR>
#
# SPDX-License-Identifier: BSD-2-Clause-Patent
#
@@ -11,7 +11,7 @@
INF_VERSION = 0x00010005
BASE_NAME = PeiSiliconPolicyUpdateLib
FILE_GUID = 14F5D83D-76A5-4241-BEC5-987E70E233D5
- MODULE_TYPE = BASE
+ MODULE_TYPE = PEIM
VERSION_STRING = 1.0
LIBRARY_CLASS = SiliconPolicyUpdateLib
@@ -33,6 +33,7 @@
[Packages]
MinPlatformPkg/MinPlatformPkg.dec
MdePkg/MdePkg.dec
+ IntelFsp2Pkg/IntelFsp2Pkg.dec
UefiCpuPkg/UefiCpuPkg.dec
KabylakeSiliconPkg/SiPkg.dec
KabylakeOpenBoardPkg/OpenBoardPkg.dec
@@ -49,11 +50,15 @@
gHsioPciePreMemConfigGuid ## CONSUMES
gHsioSataPreMemConfigGuid ## CONSUMES
gSaMiscPeiPreMemConfigGuid ## CONSUMES
+ gFspNonVolatileStorageHobGuid ## CONSUMES
[Pcd]
gSiPkgTokenSpaceGuid.PcdPeiMinMemorySize
gSiPkgTokenSpaceGuid.PcdFlashMicrocodeFvBase
gSiPkgTokenSpaceGuid.PcdFlashMicrocodeFvSize
+ gSiPkgTokenSpaceGuid.PcdMchBaseAddress
+ gSiPkgTokenSpaceGuid.PcdSmbusBaseAddress
+ gSiPkgTokenSpaceGuid.PcdTsegSize
gKabylakeOpenBoardPkgTokenSpaceGuid.PcdGraphicsVbtGuid
gKabylakeOpenBoardPkgTokenSpaceGuid.PcdMrcRcompResistor ## CONSUMES
gKabylakeOpenBoardPkgTokenSpaceGuid.PcdMrcRcompTarget ## CONSUMES
diff --git a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPeiPreMemConfig.h b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPeiPreMemConfig.h
index 4aa02e3142..2ed587f425 100644
--- a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPeiPreMemConfig.h
+++ b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPeiPreMemConfig.h
@@ -1,7 +1,7 @@
/** @file
Policy details for miscellaneous configuration in System Agent
-Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
**/
@@ -14,18 +14,91 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
#define SA_MC_MAX_SOCKETS 4
#endif
-#define SA_MISC_PEI_PREMEM_CONFIG_REVISION 1
+#define SA_MISC_PEI_PREMEM_CONFIG_REVISION 3
/**
This configuration block is to configure SA Miscellaneous variables during PEI Pre-Mem phase like programming
different System Agent BARs, TsegSize, IedSize, MmioSize required etc.
<b>Revision 1</b>:
- Initial version.
+ <b>Revision 2</b>:
+ - Added SgDelayAfterOffMethod, SgDelayAfterLinkEnable and SgGenSpeedChangeEnable.
+ <b>Revision 3</b>:
+ - Added BdatTestType and BdatSchema.
**/
typedef struct {
CONFIG_BLOCK_HEADER Header; ///< Offset 0-27 Config Block Header
UINT8 SpdAddressTable[SA_MC_MAX_SOCKETS];///< Offset 28 Memory DIMMs' SPD address for reading SPD data. <b>example: SpdAddressTable[0]=0xA2(C0D0), SpdAddressTable[1]=0xA0(C0D1), SpdAddressTable[2]=0xA2(C1D0), SpdAddressTable[3]=0xA0(C1D1)</b>
+ VOID *S3DataPtr; ///< Offset 32 Memory data save pointer for S3 resume. The memory space should be allocated and filled with proper S3 resume data on a resume path
UINT32 MchBar; ///< Offset 36 Address of System Agent MCHBAR: <b>0xFED10000</b>
+ UINT32 DmiBar; ///< Offset 40 Address of System Agent DMIBAR: <b>0xFED18000</b>
+ UINT32 EpBar; ///< Offset 44 Address of System Agent EPBAR: <b>0xFED19000</b>
+ UINT32 SmbusBar; ///< Offset 48 Address of System Agent SMBUS BAR: <b>0xEFA0</b>
+ UINT32 GdxcBar; ///< Offset 52 Address of System Agent GDXCBAR: <b>0xFED84000</b>
+ /**
+ Offset 56 Size of TSEG in bytes. (Must be power of 2)
+ <b>0x400000</b>: 4MB for Release build (When IED enabled, it will be 8MB)
+ 0x1000000 : 16MB for Debug build (Regardless IED enabled or disabled)
+ **/
+ UINT32 TsegSize;
+ UINT32 EdramBar; ///< Offset 60 Address of System Agent EDRAMBAR: <b>0xFED80000</b>
+ /**
+ Offset 64
+ <b>(Test)</b> Size of IED region in bytes.
+ <b>0</b> : IED Disabled (no memory occupied)
+ 0x400000 : 4MB SMM memory occupied by IED (Part of TSEG)
+ <b>Note: Enabling IED may also enlarge TsegSize together.</b>
+ **/
+ UINT32 IedSize;
+ UINT8 UserBd; ///< Offset 68 <b>0=Mobile/Mobile Halo</b>, 1=Desktop/DT Halo, 5=ULT/ULX/Mobile Halo, 7=UP Server
+ UINT8 SgMode; ///< Offset 69 SgMode: <b>0=Disabled</b>, 1=SG Muxed, 2=SG Muxless, 3=PEG
+ UINT16 SgSubSystemId; ///< Offset 70 Switchable Graphics Subsystem ID: <b>2212</b>
+ UINT16 SgDelayAfterPwrEn; ///< Offset 72 Dgpu Delay after Power enable using Setup option: 0=Minimal, 1000=Maximum, <b>300=300 microseconds</b>
+ UINT16 SgDelayAfterHoldReset; ///< Offset 74 Dgpu Delay after Hold Reset using Setup option: 0=Minimal, 1000=Maximum, <b>100=100 microseconds</b>
+ UINT32 SkipExtGfxScan:1; ///< <b>(Test)</b> OFfset 76:0 :1=Skip External Gfx Device Scan; <b>0=Scan for external graphics devices</b>. Set this policy to skip External Graphics card scanning if the platform uses Internal Graphics only.
+ UINT32 BdatEnable:1; ///< <b>(Test)</b> OFfset 76:1 :This field enables the generation of the BIOS DATA ACPI Tables: <b>0=FALSE</b>, 1=TRUE\n Please refer to the MRC documentation for more details
+ UINT32 TxtImplemented:1; ///< OFfset 76:2 :This field currently is used to tell MRC if it should run after TXT initializatoin completed: <b>0=Run without waiting for TXT</b>, 1=Run after TXT initialization by callback
+ /**
+ Offset 76:3 :
+ <b>(Test)</b> Scan External Discrete Graphics Devices for Legacy Only VGA OpROMs
+
+ When enabled, if the primary graphics device is an external discrete graphics device, Si will scan the
+ graphics device for legacy only VGA OpROMs. If the primary graphics device only implements legacy VBIOS, then the
+ LegacyOnlyVgaOpRomDetected field in the SA_DATA_HOB will be set to 1.
+
+ This is intended to ease the implementation of a BIOS feature to automatically enable CSM if the Primary Gfx device
+ only supports Legacy VBIOS (No UEFI GOP Present). Otherwise disabling CSM won't result in no video being displayed.
+ This is useful for platforms that implement PCIe slots that allow the end user to install an arbitrary Gfx device.
+
+ This setting will only take effect if SkipExtGfxScan == 0. It is ignored otherwise.
+
+ - Disabled (0x0) : Don't Scan for Legacy Only VGA OpROMs (Default)
+ - <b>Enabled</b> (0x1) : Scan External Gfx for Legacy Only VGA OpROM
+ **/
+ UINT32 ScanExtGfxForLegacyOpRom:1;
+ UINT32 RsvdBits0 :28; ///< OFfset 76:4 :Reserved for future use
+ UINT8 LockPTMregs; ///< <b>(Test)</b> Offset 80 Lock PCU Thermal Management registers: 0=FALSE, <b>1=TRUE</b>
+ UINT8 BdatTestType; ///< Offset 81 When BdatEnable is set to TRUE, this option selects the type of data which will be populated in the BIOS Data ACPI Tables: <b>0=RMT</b>, 1=RMT Per Bit, 2=Margin 2D.
+ UINT8 BdatSchema; ///< Offset 82 When BdatEnable is set to TRUE, this option selects the BDAT Schema version which will be used to format BDAT Test results: 0=Schema 2, <b>1=Schema 6B</b>
+ UINT8 Rsvd1; ///< Offset 83 Reserved for future use
+ /**
+ Offset 84 :
+ Size of reserved MMIO space for PCI devices\n
+ <b>0=AUTO</b>, 512=512MB, 768=768MB, 1024=1024MB, 1280=1280MB, 1536=1536MB, 1792=1792MB,
+ 2048=2048MB, 2304=2304MB, 2560=2560MB, 2816=2816MB, 3072=3072MB\n
+ When AUTO mode selected, the MMIO size will be calculated by required MMIO size from PCIe devices detected.
+ **/
+ UINT16 MmioSize;
+ INT16 MmioSizeAdjustment; ///< Offset 86 Increase (given positive value) or Decrease (given negative value) the Reserved MMIO size when Dynamic Tolud/AUTO mode enabled (in MBs): <b>0=no adjustment</b>
+ UINT64 AcpiReservedMemoryBase; ///< Offset 88 The Base address of a Reserved memory buffer allocated in previous boot for S3 resume used. Originally it is retrieved from AcpiVariableCompatibility variable.
+ UINT64 SystemMemoryLength; ///< Offset 96 Total system memory length from previous boot, this is required for S3 resume. Originally it is retrieved from AcpiVariableCompatibility variable.
+ UINT32 AcpiReservedMemorySize; ///< Offset 104 The Size of a Reserved memory buffer allocated in previous boot for S3 resume used. Originally it is retrieved from AcpiVariableCompatibility variable.
+ UINT32 OpRomScanTempMmioBar; ///< <b>(Test)</b> Offset 108 Temporary address to MMIO map OpROMs during VGA scanning. Used for ScanExtGfxForLegacyOpRom feature. MUST BE 16MB ALIGNED!
+ UINT32 OpRomScanTempMmioLimit; ///< <b>(Test)</b> Offset 112 Limit address for OpROM MMIO range. Used for ScanExtGfxForLegacyOpRom feature. (OpROMScanTempMmioLimit - OpRomScanTempMmioBar) MUST BE >= 16MB!
+ UINT16 SgDelayAfterOffMethod; ///< Offset 128 Dgpu Delay after off method is called using Setup option: 0=Minimal, 1000=Maximum, <b>300=300 microseconds</b>
+ UINT16 SgDelayAfterLinkEnable; ///< Offset 130 Delay after link enable method is called using Setup option: 0=Minimal, 1000=Maximum, <b>100=100 microseconds</b>
+ UINT8 SgGenSpeedChangeEnable; ///< Offset 132 Enable/Disable Gen speed changes using Setup option: 0=Disable, 1=Enable
+ UINT8 Rsvd3[3]; ///< Offset 133 Reserved for future use
} SA_MISC_PEI_PREMEM_CONFIG;
#pragma pack(pop)
diff --git a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPolicyLib.c b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPolicyLib.c
index eb18d993e7..5210856346 100644
--- a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPolicyLib.c
+++ b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPolicyLib.c
@@ -1,7 +1,7 @@
/** @file
This file provides services for PEI policy default initialization
-Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
**/
@@ -19,6 +19,9 @@ extern EFI_GUID gMemoryConfigNoCrcGuid;
extern EFI_GUID gGraphicsPeiConfigGuid;
extern EFI_GUID gVtdConfigGuid;
+#define DEFAULT_OPTION_ROM_TEMP_BAR 0x80000000
+#define DEFAULT_OPTION_ROM_TEMP_MEM_LIMIT 0xC0000000
+
//
// Function call to Load defaults for Individial IP Blocks
//
@@ -33,6 +36,38 @@ LoadSaMiscPeiPreMemDefault (
DEBUG ((DEBUG_INFO, "MiscPeiPreMemConfig->Header.GuidHob.Name = %g\n", &MiscPeiPreMemConfig->Header.GuidHob.Name));
DEBUG ((DEBUG_INFO, "MiscPeiPreMemConfig->Header.GuidHob.Header.HobLength = 0x%x\n", MiscPeiPreMemConfig->Header.GuidHob.Header.HobLength));
+
+ //
+ // Policy initialization commented out here is because it's the same with default 0 and no need to re-do again.
+ //
+ MiscPeiPreMemConfig->LockPTMregs = 1;
+
+ //
+ // Initialize the Platform Configuration
+ //
+ MiscPeiPreMemConfig->MchBar = (UINT32) PcdGet64 (PcdMchBaseAddress);
+ MiscPeiPreMemConfig->DmiBar = 0xFED18000;
+ MiscPeiPreMemConfig->EpBar = 0xFED19000;
+ MiscPeiPreMemConfig->EdramBar = 0xFED80000;
+ MiscPeiPreMemConfig->SmbusBar = PcdGet16 (PcdSmbusBaseAddress);
+ MiscPeiPreMemConfig->TsegSize = PcdGet32 (PcdTsegSize);
+ MiscPeiPreMemConfig->GdxcBar = 0xFED84000;
+
+ //
+ // Initialize the Switchable Graphics Default Configuration
+ //
+ MiscPeiPreMemConfig->SgDelayAfterHoldReset = 100; //100ms
+ MiscPeiPreMemConfig->SgDelayAfterPwrEn = 300; //300ms
+ MiscPeiPreMemConfig->SgDelayAfterOffMethod = 0;
+ MiscPeiPreMemConfig->SgDelayAfterLinkEnable = 0;
+ MiscPeiPreMemConfig->SgGenSpeedChangeEnable = 0;
+
+ ///
+ /// Initialize the DataPtr for S3 resume
+ ///
+ MiscPeiPreMemConfig->S3DataPtr = NULL;
+ MiscPeiPreMemConfig->OpRomScanTempMmioBar = DEFAULT_OPTION_ROM_TEMP_BAR;
+ MiscPeiPreMemConfig->OpRomScanTempMmioLimit = DEFAULT_OPTION_ROM_TEMP_MEM_LIMIT;
}
VOID
--
2.27.0.windows.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update SA_MISC_PEI_PREMEM_CONFIG
2021-08-10 4:36 [edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update SA_MISC_PEI_PREMEM_CONFIG Nate DeSimone
@ 2021-08-11 1:55 ` Chiu, Chasel
0 siblings, 0 replies; 2+ messages in thread
From: Chiu, Chasel @ 2021-08-11 1:55 UTC (permalink / raw)
To: Desimone, Nathaniel L, devel@edk2.groups.io
Cc: Michael Kubacki, Benjamin Doron
Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>
> -----Original Message-----
> From: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>
> Sent: Tuesday, August 10, 2021 12:37 PM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Michael Kubacki
> <Michael.Kubacki@microsoft.com>; Benjamin Doron
> <benjamin.doron00@gmail.com>
> Subject: [edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update
> SA_MISC_PEI_PREMEM_CONFIG
>
> Updates SA_MISC_PEI_PREMEM_CONFIG from revision 1 to revision 3. Add
> initialization of the policy values.
>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Michael Kubacki <Michael.Kubacki@microsoft.com>
> Cc: Benjamin Doron <benjamin.doron00@gmail.com>
> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
> ---
> .../KabylakeRvp3/OpenBoardPkg.dsc | 24 +++---
> .../PeiSiliconPolicyUpdateLib.c | 39 +++++++++-
> .../PeiSiliconPolicyUpdateLib.inf | 9 ++-
> .../ConfigBlock/SaMiscPeiPreMemConfig.h | 77 ++++++++++++++++++-
> .../Library/PeiSaPolicyLib/PeiSaPolicyLib.c | 37 ++++++++-
> 5 files changed, 169 insertions(+), 17 deletions(-)
>
> diff --git
> a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
> b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
> index 8523ab3f4f..f64555e391 100644
> --- a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
> +++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/OpenBoardPkg.dsc
> @@ -182,17 +182,6 @@
> # Board-specific
> #######################################
>
> PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHook
> Lib.inf
> -!if gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection == 1
> - #
> - # FSP API mode
> - #
> -
> SiliconPolicyUpdateLib|$(PROJECT)/FspWrapper/Library/PeiSiliconPolicyUpdate
> LibFsp/PeiSiliconPolicyUpdateLibFsp.inf
> -!else
> - #
> - # FSP Dispatch mode and non-FSP build (EDK2 build)
> - #
> -
> SiliconPolicyUpdateLib|$(PROJECT)/Policy/Library/PeiSiliconPolicyUpdateLib/Pei
> SiliconPolicyUpdateLib.inf
> -!endif
>
> [LibraryClasses.IA32.SEC]
> #######################################
> @@ -200,6 +189,7 @@
> #######################################
>
> TestPointCheckLib|$(PLATFORM_PACKAGE)/Test/Library/TestPointCheckLib/Sec
> TestPointCheckLib.inf
>
> SecBoardInitLib|$(PLATFORM_PACKAGE)/PlatformInit/Library/SecBoardInitLibN
> ull/SecBoardInitLibNull.inf
> +
> + SiliconPolicyUpdateLib|MinPlatformPkg/PlatformInit/Library/SiliconPoli
> + cyUpdateLibNull/SiliconPolicyUpdateLibNull.inf
>
> [LibraryClasses.common.PEIM]
> #######################################
> @@ -222,6 +212,18 @@
> #######################################
> # Board Package
> #######################################
> +!if gIntelFsp2WrapperTokenSpaceGuid.PcdFspModeSelection == 1
> + #
> + # FSP API mode
> + #
> +
> +SiliconPolicyUpdateLib|$(PROJECT)/FspWrapper/Library/PeiSiliconPolicyUp
> +dateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf
> +!else
> + #
> + # FSP Dispatch mode and non-FSP build (EDK2 build)
> + #
> +
> +SiliconPolicyUpdateLib|$(PROJECT)/Policy/Library/PeiSiliconPolicyUpdate
> +Lib/PeiSiliconPolicyUpdateLib.inf
> +!endif
> +
> # Thunderbolt
> !if gKabylakeOpenBoardPkgTokenSpaceGuid.PcdTbtEnable == TRUE
>
> PeiDTbtInitLib|$(PLATFORM_BOARD_PACKAGE)/Features/Tbt/Library/Private/P
> eiDTbtInitLib/PeiDTbtInitLib.inf
> diff --git
> a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
> b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
> index 5cc7c03c61..2dce9be63c 100644
> ---
> a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
> +++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/Pe
> +++ iSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.c
> @@ -1,7 +1,7 @@
> /** @file
> Provides silicon policy update library functions.
>
> -Copyright (c) 2019, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2019 - 2021, Intel Corporation. All rights reserved.<BR>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
> @@ -398,6 +398,8 @@ SiliconPolicyUpdatePreMem (
> SA_MISC_PEI_PREMEM_CONFIG *MiscPeiPreMemConfig;
> MEMORY_CONFIG_NO_CRC *MemConfigNoCrc;
> VOID *Buffer;
> + UINTN VariableSize;
> + VOID *MemorySavedData;
> UINT8 SpdAddressTable[4];
>
> DEBUG((DEBUG_INFO, "\nUpdating Policy in Pre-Mem\n")); @@ -417,6
> +419,41 @@ SiliconPolicyUpdatePreMem (
> // Pass board specific SpdAddressTable to policy
> //
> CopyMem ((VOID *) MiscPeiPreMemConfig->SpdAddressTable, (VOID *)
> SpdAddressTable, (sizeof (UINT8) * 4));
> +
> + //
> + // Set size of SMRAM
> + //
> + MiscPeiPreMemConfig->TsegSize = PcdGet32 (PcdTsegSize);
> +
> + //
> + // Initialize S3 Data variable (S3DataPtr). It may be used for warm and fast
> boot paths.
> + // Note: AmberLake FSP does not implement the FSPM_ARCH_CONFIG_PPI
> added in FSP 2.1, hence
> + // the platform specific S3DataPtr must be used instead.
> + //
> + VariableSize = 0;
> + MemorySavedData = NULL;
> + Status = PeiGetVariable (
> + L"MemoryConfig",
> + &gFspNonVolatileStorageHobGuid,
> + &MemorySavedData,
> + &VariableSize
> + );
> + DEBUG ((DEBUG_INFO, "Get L\"MemoryConfig\"
> gFspNonVolatileStorageHobGuid - %r\n", Status));
> + DEBUG ((DEBUG_INFO, "MemoryConfig Size - 0x%x\n", VariableSize));
> + if (!EFI_ERROR (Status)) {
> + MiscPeiPreMemConfig->S3DataPtr = MemorySavedData;
> + }
> +
> + //
> + // In FSP Dispatch Mode these BAR values are initialized by
> SiliconPolicyInitPreMem() in
> + // KabylakeSiliconPkg/Library/PeiSiliconPolicyInitLib/PeiPolicyInitPreMem.c;
> this function calls
> + // PEI_PREMEM_SI_DEFAULT_POLICY_INIT_PPI->PeiPreMemPolicyInit() to
> initialize all Config Blocks
> + // with default policy values (including these BAR values.)
> PEI_PREMEM_SI_DEFAULT_POLICY_INIT_PPI
> + // is implemented in the FSP. Make sure the value that FSP is using matches
> the value we are using.
> + //
> + ASSERT (PcdGet64 (PcdMchBaseAddress) <= 0xFFFFFFFF);
> + ASSERT (MiscPeiPreMemConfig->MchBar == (UINT32) PcdGet64
> (PcdMchBaseAddress));
> + ASSERT (MiscPeiPreMemConfig->SmbusBar == PcdGet16
> + (PcdSmbusBaseAddress));
> }
> MemConfigNoCrc = NULL;
> Status = GetConfigBlock (Policy, &gMemoryConfigNoCrcGuid, (VOID *)
> &MemConfigNoCrc); diff --git
> a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
> b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
> index 97ec70f611..5c2da68bf9 100644
> ---
> a/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/PeiSilico
> nPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
> +++ b/Platform/Intel/KabylakeOpenBoardPkg/KabylakeRvp3/Policy/Library/Pe
> +++ iSiliconPolicyUpdateLib/PeiSiliconPolicyUpdateLib.inf
> @@ -1,7 +1,7 @@
> ### @file
> # Component information file for silicon policy update library # -# Copyright (c)
> 2019 - 2020 Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2019 - 2021 Intel Corporation. All rights reserved.<BR>
> #
> # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -11,7 +11,7 @@
> INF_VERSION = 0x00010005
> BASE_NAME = PeiSiliconPolicyUpdateLib
> FILE_GUID = 14F5D83D-76A5-4241-BEC5-987E70E233D5
> - MODULE_TYPE = BASE
> + MODULE_TYPE = PEIM
> VERSION_STRING = 1.0
> LIBRARY_CLASS = SiliconPolicyUpdateLib
>
> @@ -33,6 +33,7 @@
> [Packages]
> MinPlatformPkg/MinPlatformPkg.dec
> MdePkg/MdePkg.dec
> + IntelFsp2Pkg/IntelFsp2Pkg.dec
> UefiCpuPkg/UefiCpuPkg.dec
> KabylakeSiliconPkg/SiPkg.dec
> KabylakeOpenBoardPkg/OpenBoardPkg.dec
> @@ -49,11 +50,15 @@
> gHsioPciePreMemConfigGuid ## CONSUMES
> gHsioSataPreMemConfigGuid ## CONSUMES
> gSaMiscPeiPreMemConfigGuid ## CONSUMES
> + gFspNonVolatileStorageHobGuid ## CONSUMES
>
> [Pcd]
> gSiPkgTokenSpaceGuid.PcdPeiMinMemorySize
> gSiPkgTokenSpaceGuid.PcdFlashMicrocodeFvBase
> gSiPkgTokenSpaceGuid.PcdFlashMicrocodeFvSize
> + gSiPkgTokenSpaceGuid.PcdMchBaseAddress
> + gSiPkgTokenSpaceGuid.PcdSmbusBaseAddress
> + gSiPkgTokenSpaceGuid.PcdTsegSize
> gKabylakeOpenBoardPkgTokenSpaceGuid.PcdGraphicsVbtGuid
> gKabylakeOpenBoardPkgTokenSpaceGuid.PcdMrcRcompResistor ##
> CONSUMES
> gKabylakeOpenBoardPkgTokenSpaceGuid.PcdMrcRcompTarget ##
> CONSUMES
> diff --git
> a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPei
> PreMemConfig.h
> b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPei
> PreMemConfig.h
> index 4aa02e3142..2ed587f425 100644
> ---
> a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/SaMiscPei
> PreMemConfig.h
> +++ b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Include/ConfigBlock/S
> +++ aMiscPeiPreMemConfig.h
> @@ -1,7 +1,7 @@
> /** @file
> Policy details for miscellaneous configuration in System Agent
>
> -Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
> @@ -14,18 +14,91 @@ SPDX-License-Identifier: BSD-2-Clause-Patent #define
> SA_MC_MAX_SOCKETS 4 #endif
>
> -#define SA_MISC_PEI_PREMEM_CONFIG_REVISION 1
> +#define SA_MISC_PEI_PREMEM_CONFIG_REVISION 3
>
> /**
> This configuration block is to configure SA Miscellaneous variables during PEI
> Pre-Mem phase like programming
> different System Agent BARs, TsegSize, IedSize, MmioSize required etc.
> <b>Revision 1</b>:
> - Initial version.
> + <b>Revision 2</b>:
> + - Added SgDelayAfterOffMethod, SgDelayAfterLinkEnable and
> SgGenSpeedChangeEnable.
> + <b>Revision 3</b>:
> + - Added BdatTestType and BdatSchema.
> **/
> typedef struct {
> CONFIG_BLOCK_HEADER Header; ///< Offset 0-27 Config Block
> Header
> UINT8 SpdAddressTable[SA_MC_MAX_SOCKETS];///< Offset 28 Memory
> DIMMs' SPD address for reading SPD data. <b>example:
> SpdAddressTable[0]=0xA2(C0D0), SpdAddressTable[1]=0xA0(C0D1),
> SpdAddressTable[2]=0xA2(C1D0), SpdAddressTable[3]=0xA0(C1D1)</b>
> + VOID *S3DataPtr; ///< Offset 32 Memory data save pointer for
> S3 resume. The memory space should be allocated and filled with proper S3
> resume data on a resume path
> UINT32 MchBar; ///< Offset 36 Address of System Agent
> MCHBAR: <b>0xFED10000</b>
> + UINT32 DmiBar; ///< Offset 40 Address of System Agent
> DMIBAR: <b>0xFED18000</b>
> + UINT32 EpBar; ///< Offset 44 Address of System Agent EPBAR:
> <b>0xFED19000</b>
> + UINT32 SmbusBar; ///< Offset 48 Address of System Agent
> SMBUS BAR: <b>0xEFA0</b>
> + UINT32 GdxcBar; ///< Offset 52 Address of System Agent
> GDXCBAR: <b>0xFED84000</b>
> + /**
> + Offset 56 Size of TSEG in bytes. (Must be power of 2)
> + <b>0x400000</b>: 4MB for Release build (When IED enabled, it will be 8MB)
> + 0x1000000 : 16MB for Debug build (Regardless IED enabled or disabled)
> + **/
> + UINT32 TsegSize;
> + UINT32 EdramBar; ///< Offset 60 Address of System Agent
> EDRAMBAR: <b>0xFED80000</b>
> + /**
> + Offset 64
> + <b>(Test)</b> Size of IED region in bytes.
> + <b>0</b> : IED Disabled (no memory occupied)
> + 0x400000 : 4MB SMM memory occupied by IED (Part of TSEG)
> + <b>Note: Enabling IED may also enlarge TsegSize together.</b> **/
> + UINT32 IedSize;
> + UINT8 UserBd; ///< Offset 68 <b>0=Mobile/Mobile Halo</b>,
> 1=Desktop/DT Halo, 5=ULT/ULX/Mobile Halo, 7=UP Server
> + UINT8 SgMode; ///< Offset 69 SgMode: <b>0=Disabled</b>,
> 1=SG Muxed, 2=SG Muxless, 3=PEG
> + UINT16 SgSubSystemId; ///< Offset 70 Switchable Graphics
> Subsystem ID: <b>2212</b>
> + UINT16 SgDelayAfterPwrEn; ///< Offset 72 Dgpu Delay after Power
> enable using Setup option: 0=Minimal, 1000=Maximum, <b>300=300
> microseconds</b>
> + UINT16 SgDelayAfterHoldReset; ///< Offset 74 Dgpu Delay after Hold
> Reset using Setup option: 0=Minimal, 1000=Maximum, <b>100=100
> microseconds</b>
> + UINT32 SkipExtGfxScan:1; ///< <b>(Test)</b> OFfset 76:0 :1=Skip
> External Gfx Device Scan; <b>0=Scan for external graphics devices</b>. Set this
> policy to skip External Graphics card scanning if the platform uses Internal
> Graphics only.
> + UINT32 BdatEnable:1; ///< <b>(Test)</b> OFfset 76:1 :This field
> enables the generation of the BIOS DATA ACPI Tables: <b>0=FALSE</b>,
> 1=TRUE\n Please refer to the MRC documentation for more details
> + UINT32 TxtImplemented:1; ///< OFfset 76:2 :This field currently is
> used to tell MRC if it should run after TXT initializatoin completed: <b>0=Run
> without waiting for TXT</b>, 1=Run after TXT initialization by callback
> + /**
> + Offset 76:3 :
> + <b>(Test)</b> Scan External Discrete Graphics Devices for Legacy
> + Only VGA OpROMs
> +
> + When enabled, if the primary graphics device is an external discrete graphics
> device, Si will scan the
> + graphics device for legacy only VGA OpROMs. If the primary graphics device
> only implements legacy VBIOS, then the
> + LegacyOnlyVgaOpRomDetected field in the SA_DATA_HOB will be set to 1.
> +
> + This is intended to ease the implementation of a BIOS feature to
> automatically enable CSM if the Primary Gfx device
> + only supports Legacy VBIOS (No UEFI GOP Present). Otherwise disabling CSM
> won't result in no video being displayed.
> + This is useful for platforms that implement PCIe slots that allow the end user
> to install an arbitrary Gfx device.
> +
> + This setting will only take effect if SkipExtGfxScan == 0. It is ignored
> otherwise.
> +
> + - Disabled (0x0) : Don't Scan for Legacy Only VGA OpROMs (Default)
> + - <b>Enabled</b> (0x1) : Scan External Gfx for Legacy Only VGA
> + OpROM **/
> + UINT32 ScanExtGfxForLegacyOpRom:1;
> + UINT32 RsvdBits0 :28; ///< OFfset 76:4 :Reserved for future use
> + UINT8 LockPTMregs; ///< <b>(Test)</b> Offset 80 Lock PCU
> Thermal Management registers: 0=FALSE, <b>1=TRUE</b>
> + UINT8 BdatTestType; ///< Offset 81 When BdatEnable is set to
> TRUE, this option selects the type of data which will be populated in the BIOS
> Data ACPI Tables: <b>0=RMT</b>, 1=RMT Per Bit, 2=Margin 2D.
> + UINT8 BdatSchema; ///< Offset 82 When BdatEnable is set to
> TRUE, this option selects the BDAT Schema version which will be used to format
> BDAT Test results: 0=Schema 2, <b>1=Schema 6B</b>
> + UINT8 Rsvd1; ///< Offset 83 Reserved for future use
> + /**
> + Offset 84 :
> + Size of reserved MMIO space for PCI devices\n
> + <b>0=AUTO</b>, 512=512MB, 768=768MB, 1024=1024MB, 1280=1280MB,
> 1536=1536MB, 1792=1792MB,
> + 2048=2048MB, 2304=2304MB, 2560=2560MB, 2816=2816MB,
> 3072=3072MB\n
> + When AUTO mode selected, the MMIO size will be calculated by required
> MMIO size from PCIe devices detected.
> + **/
> + UINT16 MmioSize;
> + INT16 MmioSizeAdjustment; ///< Offset 86 Increase (given positive
> value) or Decrease (given negative value) the Reserved MMIO size when
> Dynamic Tolud/AUTO mode enabled (in MBs): <b>0=no adjustment</b>
> + UINT64 AcpiReservedMemoryBase; ///< Offset 88 The Base address of
> a Reserved memory buffer allocated in previous boot for S3 resume used.
> Originally it is retrieved from AcpiVariableCompatibility variable.
> + UINT64 SystemMemoryLength; ///< Offset 96 Total system memory
> length from previous boot, this is required for S3 resume. Originally it is retrieved
> from AcpiVariableCompatibility variable.
> + UINT32 AcpiReservedMemorySize; ///< Offset 104 The Size of a
> Reserved memory buffer allocated in previous boot for S3 resume used.
> Originally it is retrieved from AcpiVariableCompatibility variable.
> + UINT32 OpRomScanTempMmioBar; ///< <b>(Test)</b> Offset 108
> Temporary address to MMIO map OpROMs during VGA scanning. Used for
> ScanExtGfxForLegacyOpRom feature. MUST BE 16MB ALIGNED!
> + UINT32 OpRomScanTempMmioLimit; ///< <b>(Test)</b> Offset 112
> Limit address for OpROM MMIO range. Used for ScanExtGfxForLegacyOpRom
> feature. (OpROMScanTempMmioLimit - OpRomScanTempMmioBar) MUST BE
> >= 16MB!
> + UINT16 SgDelayAfterOffMethod; ///< Offset 128 Dgpu Delay after off
> method is called using Setup option: 0=Minimal, 1000=Maximum, <b>300=300
> microseconds</b>
> + UINT16 SgDelayAfterLinkEnable; ///< Offset 130 Delay after link enable
> method is called using Setup option: 0=Minimal, 1000=Maximum, <b>100=100
> microseconds</b>
> + UINT8 SgGenSpeedChangeEnable; ///< Offset 132 Enable/Disable Gen
> speed changes using Setup option: 0=Disable, 1=Enable
> + UINT8 Rsvd3[3]; ///< Offset 133 Reserved for future use
> } SA_MISC_PEI_PREMEM_CONFIG;
> #pragma pack(pop)
>
> diff --git
> a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPol
> icyLib.c
> b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPol
> icyLib.c
> index eb18d993e7..5210856346 100644
> ---
> a/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLib/PeiSaPol
> icyLib.c
> +++ b/Silicon/Intel/KabylakeSiliconPkg/SystemAgent/Library/PeiSaPolicyLi
> +++ b/PeiSaPolicyLib.c
> @@ -1,7 +1,7 @@
> /** @file
> This file provides services for PEI policy default initialization
>
> -Copyright (c) 2017, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
> **/
> @@ -19,6 +19,9 @@ extern EFI_GUID gMemoryConfigNoCrcGuid; extern
> EFI_GUID gGraphicsPeiConfigGuid; extern EFI_GUID gVtdConfigGuid;
>
> +#define DEFAULT_OPTION_ROM_TEMP_BAR 0x80000000
> +#define DEFAULT_OPTION_ROM_TEMP_MEM_LIMIT 0xC0000000
> +
> //
> // Function call to Load defaults for Individial IP Blocks // @@ -33,6 +36,38
> @@ LoadSaMiscPeiPreMemDefault (
>
> DEBUG ((DEBUG_INFO, "MiscPeiPreMemConfig->Header.GuidHob.Name =
> %g\n", &MiscPeiPreMemConfig->Header.GuidHob.Name));
> DEBUG ((DEBUG_INFO, "MiscPeiPreMemConfig-
> >Header.GuidHob.Header.HobLength = 0x%x\n", MiscPeiPreMemConfig-
> >Header.GuidHob.Header.HobLength));
> +
> + //
> + // Policy initialization commented out here is because it's the same with
> default 0 and no need to re-do again.
> + //
> + MiscPeiPreMemConfig->LockPTMregs = 1;
> +
> + //
> + // Initialize the Platform Configuration //
> + MiscPeiPreMemConfig->MchBar = (UINT32) PcdGet64
> (PcdMchBaseAddress);
> + MiscPeiPreMemConfig->DmiBar = 0xFED18000;
> + MiscPeiPreMemConfig->EpBar = 0xFED19000;
> + MiscPeiPreMemConfig->EdramBar = 0xFED80000;
> + MiscPeiPreMemConfig->SmbusBar = PcdGet16 (PcdSmbusBaseAddress);
> + MiscPeiPreMemConfig->TsegSize = PcdGet32 (PcdTsegSize);
> + MiscPeiPreMemConfig->GdxcBar = 0xFED84000;
> +
> + //
> + // Initialize the Switchable Graphics Default Configuration //
> + MiscPeiPreMemConfig->SgDelayAfterHoldReset = 100; //100ms
> + MiscPeiPreMemConfig->SgDelayAfterPwrEn = 300; //300ms
> + MiscPeiPreMemConfig->SgDelayAfterOffMethod = 0;
> + MiscPeiPreMemConfig->SgDelayAfterLinkEnable = 0;
> + MiscPeiPreMemConfig->SgGenSpeedChangeEnable = 0;
> +
> + ///
> + /// Initialize the DataPtr for S3 resume ///
> + MiscPeiPreMemConfig->S3DataPtr = NULL;
> + MiscPeiPreMemConfig->OpRomScanTempMmioBar =
> DEFAULT_OPTION_ROM_TEMP_BAR;
> + MiscPeiPreMemConfig->OpRomScanTempMmioLimit =
> DEFAULT_OPTION_ROM_TEMP_MEM_LIMIT;
> }
>
> VOID
> --
> 2.27.0.windows.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-08-11 1:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-08-10 4:36 [edk2-platforms] [PATCH V1] KabylakeSiliconPkg: Update SA_MISC_PEI_PREMEM_CONFIG Nate DeSimone
2021-08-11 1:55 ` Chiu, Chasel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox