Reviewed-by: Kathappan Esakkithevar <kathappan.esakkithe...@intel.com>
> -----Original Message----- > From: Chiu, Chasel <chasel.c...@intel.com> > Sent: Friday, October 8, 2021 12:13 PM > To: devel@edk2.groups.io > Cc: Chiu, Chasel <chasel.c...@intel.com>; Desimone, Nathaniel L > <nathaniel.l.desim...@intel.com>; Chaganty, Rangasai V > <rangasai.v.chaga...@intel.com>; Kethi Reddy, Deepika > <deepika.kethi.re...@intel.com>; Esakkithevar, Kathappan > <kathappan.esakkithe...@intel.com> > Subject: [edk2-platforms: PATCH v3 2/9] CometlakeOpenBoardPkg: Use > same variable name for FspNvsHob. > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3678 > > To simplify the implementation the variable Name/GUID has been changed > to "FspNvsBuffer" and gFspNvsBufferVariableGuid regardless it stores the > data from FSP_NON_VOLATILE_STORAGE_HOB2 or > FSP_NON_VOLATILE_STORAGE_HOB. > > Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> > Cc: Rangasai V Chaganty <rangasai.v.chaga...@intel.com> > Cc: Deepika Kethi Reddy <deepika.kethi.re...@intel.com> > Cc: Kathappan Esakkithevar <kathappan.esakkithe...@intel.com> > Signed-off-by: Chasel Chiu <chasel.c...@intel.com> > --- > > Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconPolic > yUpdateLibFsp/PeiFspMiscUpdUpdateLib.c | 63 ++++++++++++++-------- > ----------------------------------------- > > Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg.d > sc | 3 +++ > > Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconPolic > yUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf | 5 ++--- > 3 files changed, 19 insertions(+), 52 deletions(-) > > diff --git > a/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiFspMiscUpdUpdateLib.c > b/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiFspMiscUpdUpdateLib.c > index 7dbdfa7c16..9e8606ada9 100644 > --- > a/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiFspMiscUpdUpdateLib.c > +++ > b/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSilicon > +++ PolicyUpdateLibFsp/PeiFspMiscUpdUpdateLib.c > @@ -2,7 +2,7 @@ > Implementation of Fsp Misc UPD Initialization. > > > > > > - Copyright (c) 2020, Intel Corporation. All rights reserved.<BR> > > + Copyright (c) 2020 - 2021, Intel Corporation. All rights > + reserved.<BR> > > SPDX-License-Identifier: BSD-2-Clause-Patent > > **/ > > > > @@ -17,7 +17,6 @@ > #include <FspmUpd.h> > > #include <FspsUpd.h> > > > > -#include <Library/MemoryAllocationLib.h> > > #include <Library/DebugLib.h> > > #include <Library/DebugPrintErrorLevelLib.h> > > #include <Library/PciLib.h> > > @@ -44,55 +43,21 @@ PeiFspMiscUpdUpdatePreMem ( > ) > > { > > EFI_STATUS Status; > > - EFI_PEI_READ_ONLY_VARIABLE2_PPI *VariableServices; > > - UINTN VariableSize; > > - VOID *MemorySavedData; > > - > > - Status = PeiServicesLocatePpi ( > > - &gEfiPeiReadOnlyVariable2PpiGuid, > > - 0, > > - NULL, > > - (VOID **) &VariableServices > > - ); > > - if (EFI_ERROR (Status)) { > > - ASSERT_EFI_ERROR (Status); > > - return Status; > > + UINTN FspNvsBufferSize; > > + VOID *FspNvsBufferPtr; > > + > > + // > > + // Initialize S3 Data variable (S3DataPtr). It may be used for warm and > fast > boot paths. > > + // > > + FspNvsBufferPtr = NULL; > > + FspNvsBufferSize = 0; > > + Status = PeiGetLargeVariable (L"FspNvsBuffer", > + &gFspNvsBufferVariableGuid, &FspNvsBufferPtr, &FspNvsBufferSize); > > + if (Status == EFI_SUCCESS) { > > + DEBUG ((DEBUG_INFO, "Get L\"FspNvsBuffer\" > + gFspNvsBufferVariableGuid - %r\n", Status)); > > + DEBUG ((DEBUG_INFO, "FspNvsBuffer Size - 0x%x\n", > + FspNvsBufferSize)); > > + FspmUpd->FspmArchUpd.NvsBufferPtr = FspNvsBufferPtr; > > } > > > > - VariableSize = 0; > > - MemorySavedData = NULL; > > - Status = VariableServices->GetVariable ( > > - VariableServices, > > - L"MemoryConfig", > > - &gFspNonVolatileStorageHobGuid, > > - NULL, > > - &VariableSize, > > - MemorySavedData > > - ); > > - if (Status == EFI_BUFFER_TOO_SMALL) { > > - MemorySavedData = AllocatePool (VariableSize); > > - if (MemorySavedData == NULL) { > > - ASSERT (MemorySavedData != NULL); > > - return EFI_OUT_OF_RESOURCES; > > - } > > - > > - DEBUG ((DEBUG_INFO, "VariableSize is 0x%x\n", VariableSize)); > > - Status = VariableServices->GetVariable ( > > - VariableServices, > > - L"MemoryConfig", > > - &gFspNonVolatileStorageHobGuid, > > - NULL, > > - &VariableSize, > > - MemorySavedData > > - ); > > - if (Status == EFI_SUCCESS) { > > - FspmUpd->FspmArchUpd.NvsBufferPtr = MemorySavedData; > > - } else { > > - DEBUG ((DEBUG_ERROR, "Fail to retrieve Variable:\"MemoryConfig\" > gMemoryConfigVariableGuid, Status = %r\n", Status)); > > - ASSERT_EFI_ERROR (Status); > > - } > > - } > > - FspmUpd->FspmArchUpd.NvsBufferPtr = MemorySavedData; > > > > return EFI_SUCCESS; > > } > > diff --git > a/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg > .dsc > b/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg > .dsc > index 44a1bd54d6..d63ff263c1 100644 > --- > a/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg > .dsc > +++ > b/Platform/Intel/CometlakeOpenBoardPkg/CometlakeURvp/OpenBoardPkg > .ds > +++ c > @@ -142,6 +142,9 @@ > ####################################### > > > PlatformHookLib|$(PROJECT)/Library/BasePlatformHookLib/BasePlatformHo > okLib.inf > > > > +[LibraryClasses.Common.SEC] > > + > + > VariableReadLib|MinPlatformPkg/Library/BaseVariableReadLibNull/BaseVar > + iableReadLibNull.inf > > + > > [LibraryClasses.IA32.SEC] > > ####################################### > > # Platform Package > > diff --git > a/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf > b/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf > index c842b7eef6..94776603c4 100644 > --- > a/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSiliconP > olicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf > +++ > b/Platform/Intel/CometlakeOpenBoardPkg/FspWrapper/Library/PeiSilicon > +++ PolicyUpdateLibFsp/PeiSiliconPolicyUpdateLibFsp.inf > @@ -2,7 +2,7 @@ > # Provide FSP wrapper platform related function. > > # > > # > > -# Copyright (c) 2020, Intel Corporation. All rights reserved.<BR> > > +# Copyright (c) 2020 - 2021, Intel Corporation. All rights > +reserved.<BR> > > # > > # SPDX-License-Identifier: BSD-2-Clause-Patent > > # > > @@ -82,7 +82,6 @@ > PchInfoLib > > PchHsioLib > > PchPcieRpLib > > - MemoryAllocationLib > > DebugPrintErrorLevelLib > > SiPolicyLib > > PchGbeLib > > @@ -132,7 +131,7 @@ > gCometlakeOpenBoardPkgTokenSpaceGuid.PcdMrcSpdAddressTable3 > > > > [Guids] > > - gFspNonVolatileStorageHobGuid ## CONSUMES > > + gFspNvsBufferVariableGuid ## CONSUMES > > gTianoLogoGuid ## CONSUMES > > gEfiMemoryOverwriteControlDataGuid > > > > -- > 2.28.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81671): https://edk2.groups.io/g/devel/message/81671 Mute This Topic: https://groups.io/mt/86164706/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-