From: Michael Kubacki <michael.kuba...@microsoft.com>
Updates the Standalone MM module to have the necessary INF changes
to build with the following two recent commits made to rewrite the
the variable store header in the MM SPI FVB service.
- e95c798
- 88d44c5
Cc: Ashraf Ali S <ashraf.al...@intel.com>
Cc: Isaac Oram <isaac.w.o...@intel.com>
Cc: Rangasai V Chaganty <rangasai.v.chaga...@intel.com>
Cc: Ray Ni <ray...@intel.com>
Cc: Chasel Chiu <chasel.c...@intel.com>
Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com>
---
Silicon/Intel/IntelSiliconPkg/Feature/Flash/SpiFvbService/SpiFvbServiceStandaloneMm.inf
| 4 ++++
Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
| 4 +++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git
a/Silicon/Intel/IntelSiliconPkg/Feature/Flash/SpiFvbService/SpiFvbServiceStandaloneMm.inf
b/Silicon/Intel/IntelSiliconPkg/Feature/Flash/SpiFvbService/SpiFvbServiceStandaloneMm.inf
index 152cf0036fdb..717b6fc00cc6 100644
---
a/Silicon/Intel/IntelSiliconPkg/Feature/Flash/SpiFvbService/SpiFvbServiceStandaloneMm.inf
+++
b/Silicon/Intel/IntelSiliconPkg/Feature/Flash/SpiFvbService/SpiFvbServiceStandaloneMm.inf
@@ -44,6 +44,8 @@ [Packages]
[Pcd]
gIntelSiliconPkgTokenSpaceGuid.PcdFlashMicrocodeFvBase ## CONSUMES
gIntelSiliconPkgTokenSpaceGuid.PcdFlashMicrocodeFvSize ## CONSUMES
+ gIntelSiliconPkgTokenSpaceGuid.PcdFlashVariableStoreType ##
SOMETIMES_CONSUMES
+ gIntelSiliconPkgTokenSpaceGuid.PcdFlashNvStorageAdditionalSize ## CONSUMES
[Sources]
FvbInfo.c
@@ -60,6 +62,8 @@ [Protocols]
[Guids]
gEfiFirmwareFileSystem2Guid ## CONSUMES
gEfiSystemNvDataFvGuid ## CONSUMES
+ gEfiVariableGuid ## SOMETIMES_CONSUMES
+ gEfiAuthenticatedVariableGuid ## SOMETIMES_CONSUMES
[Depex]
TRUE
diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
index d73a51ca5235..cad22acda496 100644
--- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
+++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
@@ -188,7 +188,8 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic,
PcdsDynamicEx]
## Define Flash Variable Store type.<BR><BR>
# When Flash Variable Store corruption happened, the SpiFvbService will
recreate Variable Store
- # with valid header information provided by this PCD value.<BR>
+ # with valid header information provided by this PCD value.
+ # Note: This PCD must be FixedAtBuild when using Standalone MM.
# 0: Variable Store is gEfiVariableGuid type.<BR>
# 1: Variable Store is gEfiAuthenticatedVariableGuid type.<BR>
# Other value: reserved for future use.<BR>
@@ -199,6 +200,7 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic,
PcdsDynamicEx]
# Platform may implement a Regular variable region and an additional
region, which will require this PCD
# to tell SpiFvbService to include both regions.
# Note: This PCD is for compatible with legacy usages that should be
deprecated.
+ # Note: This PCD must be FixedAtBuild when using Standalone MM.
# The new usage model should define separate regions without implicit
connections to UEFI Variable or FTW regions.<BR>
# Example legacy usage is to set this PCD equal to platform
PcdFlashFvNvStorageEventLogSize.
# 0: No additional NVS region.<BR>
--
2.28.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99936): https://edk2.groups.io/g/devel/message/99936
Mute This Topic: https://groups.io/mt/96867025/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
