]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/PCD/Pei/Pcd.c
UefiCpuPkg: Move AsmRelocateApLoopStart from Mpfuncs.nasm to AmdSev.nasm
[mirror_edk2.git] / MdeModulePkg / Universal / PCD / Pei / Pcd.c
index e552e151760a9cf48c129d24dcbed361deaf3cc4..c51ac96a32e382a6909cb14de8e4a7796f76a3f0 100644 (file)
@@ -1,15 +1,9 @@
 /** @file\r
   All Pcd Ppi services are implemented here.\r
 \r
-Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2021, Intel Corporation. All rights reserved.<BR>\r
 (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>\r
-This program and the accompanying materials\r
-are licensed and made available under the terms and conditions of the BSD License\r
-which accompanies this distribution.  The full text of the license may be found at\r
-http://opensource.org/licenses/bsd-license.php\r
-\r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
@@ -19,7 +13,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 /// Instance of PCD_PPI protocol is EDKII native implementation.\r
 /// This protocol instance support dynamic and dynamicEx type PCDs.\r
 ///\r
-PCD_PPI mPcdPpiInstance = {\r
+PCD_PPI  mPcdPpiInstance = {\r
   PeiPcdSetSku,\r
 \r
   PeiPcdGet8,\r
@@ -78,8 +72,8 @@ EFI_PEI_PCD_PPI  mEfiPcdPpiInstance = {
   PeiPcdSet64Ex,\r
   PeiPcdSetPtrEx,\r
   PeiPcdSetBoolEx,\r
-  (EFI_PEI_PCD_PPI_CALLBACK_ON_SET) PeiRegisterCallBackOnSet,\r
-  (EFI_PEI_PCD_PPI_CANCEL_CALLBACK) PcdUnRegisterCallBackOnSet,\r
+  (EFI_PEI_PCD_PPI_CALLBACK_ON_SET)PeiRegisterCallBackOnSet,\r
+  (EFI_PEI_PCD_PPI_CANCEL_CALLBACK)PcdUnRegisterCallBackOnSet,\r
   PeiPcdGetNextToken,\r
   PeiPcdGetNextTokenSpace\r
 };\r
@@ -88,7 +82,7 @@ EFI_PEI_PCD_PPI  mEfiPcdPpiInstance = {
 /// Instance of GET_PCD_INFO_PPI protocol is EDKII native implementation.\r
 /// This protocol instance support dynamic and dynamicEx type PCDs.\r
 ///\r
-GET_PCD_INFO_PPI mGetPcdInfoInstance = {\r
+GET_PCD_INFO_PPI  mGetPcdInfoInstance = {\r
   PeiGetPcdInfoGetInfo,\r
   PeiGetPcdInfoGetInfoEx,\r
   PeiGetPcdInfoGetSku\r
@@ -144,85 +138,91 @@ EFI_PEI_PPI_DESCRIPTOR  mPpiList2[] = {
 VOID\r
 EFIAPI\r
 PcdSetNvStoreDefaultIdCallBack (\r
-  IN CONST EFI_GUID         *CallBackGuid, OPTIONAL\r
-  IN       UINTN            CallBackToken,\r
-  IN OUT   VOID             *TokenData,\r
-  IN       UINTN            TokenDataSize\r
+  IN CONST EFI_GUID  *CallBackGuid  OPTIONAL,\r
+  IN       UINTN     CallBackToken,\r
+  IN OUT   VOID      *TokenData,\r
+  IN       UINTN     TokenDataSize\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-  UINT16     DefaultId;\r
-  SKU_ID     SkuId;\r
-  UINTN      FullSize;\r
-  UINTN      Index;\r
-  UINT8      *DataBuffer;\r
-  UINT8      *VarStoreHobData;\r
-  UINT8      *BufferEnd;\r
-  BOOLEAN    IsFound;\r
-  VARIABLE_STORE_HEADER *NvStoreBuffer;\r
-  PCD_DEFAULT_DATA      *DataHeader;\r
-  PCD_DEFAULT_INFO      *DefaultInfo;\r
-  PCD_DATA_DELTA        *DeltaData;\r
-\r
-  DefaultId = *(UINT16 *) TokenData;\r
-  SkuId     = GetPcdDatabase()->SystemSkuId;\r
+  EFI_STATUS             Status;\r
+  UINT16                 DefaultId;\r
+  SKU_ID                 SkuId;\r
+  UINTN                  FullSize;\r
+  UINTN                  Index;\r
+  UINT8                  *DataBuffer;\r
+  UINT8                  *VarStoreHobData;\r
+  UINT8                  *BufferEnd;\r
+  BOOLEAN                IsFound;\r
+  VARIABLE_STORE_HEADER  *NvStoreBuffer;\r
+  PCD_DEFAULT_DATA       *DataHeader;\r
+  PCD_DEFAULT_INFO       *DefaultInfo;\r
+  PCD_DATA_DELTA         *DeltaData;\r
+\r
+  DefaultId = *(UINT16 *)TokenData;\r
+  SkuId     = GetPcdDatabase ()->SystemSkuId;\r
   IsFound   = FALSE;\r
 \r
   if (PeiPcdGetSizeEx (&gEfiMdeModulePkgTokenSpaceGuid, PcdToken (PcdNvStoreDefaultValueBuffer)) > sizeof (PCD_NV_STORE_DEFAULT_BUFFER_HEADER)) {\r
-    DataBuffer = (UINT8 *) PeiPcdGetPtrEx (&gEfiMdeModulePkgTokenSpaceGuid, PcdToken (PcdNvStoreDefaultValueBuffer));\r
-    FullSize   = ((PCD_NV_STORE_DEFAULT_BUFFER_HEADER *) DataBuffer)->Length;\r
-    DataHeader = (PCD_DEFAULT_DATA *) (DataBuffer + sizeof (PCD_NV_STORE_DEFAULT_BUFFER_HEADER));\r
+    DataBuffer = (UINT8 *)PeiPcdGetPtrEx (&gEfiMdeModulePkgTokenSpaceGuid, PcdToken (PcdNvStoreDefaultValueBuffer));\r
+    FullSize   = ((PCD_NV_STORE_DEFAULT_BUFFER_HEADER *)DataBuffer)->Length;\r
+    DataHeader = (PCD_DEFAULT_DATA *)(DataBuffer + sizeof (PCD_NV_STORE_DEFAULT_BUFFER_HEADER));\r
     //\r
     // The first section data includes NV storage default setting.\r
     //\r
-    NvStoreBuffer   = (VARIABLE_STORE_HEADER *) ((UINT8 *) DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize);\r
-    VarStoreHobData = (UINT8 *) BuildGuidHob (&NvStoreBuffer->Signature, NvStoreBuffer->Size);\r
+    NvStoreBuffer   = (VARIABLE_STORE_HEADER *)((UINT8 *)DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize);\r
+    VarStoreHobData = (UINT8 *)BuildGuidHob (&NvStoreBuffer->Signature, NvStoreBuffer->Size);\r
     ASSERT (VarStoreHobData != NULL);\r
     CopyMem (VarStoreHobData, NvStoreBuffer, NvStoreBuffer->Size);\r
     //\r
     // Find the matched SkuId and DefaultId in the first section\r
     //\r
-    DefaultInfo    = &(DataHeader->DefaultInfo[0]);\r
-    BufferEnd      = (UINT8 *) DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize;\r
-    while ((UINT8 *) DefaultInfo < BufferEnd) {\r
-      if (DefaultInfo->DefaultId == DefaultId && DefaultInfo->SkuId == SkuId) {\r
+    DefaultInfo = &(DataHeader->DefaultInfo[0]);\r
+    BufferEnd   = (UINT8 *)DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize;\r
+    while ((UINT8 *)DefaultInfo < BufferEnd) {\r
+      if ((DefaultInfo->DefaultId == DefaultId) && (DefaultInfo->SkuId == SkuId)) {\r
         IsFound = TRUE;\r
         break;\r
       }\r
-      DefaultInfo ++;\r
+\r
+      DefaultInfo++;\r
     }\r
+\r
     //\r
     // Find the matched SkuId and DefaultId in the remaining section\r
     //\r
     Index      = sizeof (PCD_NV_STORE_DEFAULT_BUFFER_HEADER) + ((DataHeader->DataSize + 7) & (~7));\r
-    DataHeader = (PCD_DEFAULT_DATA *) (DataBuffer + Index);\r
+    DataHeader = (PCD_DEFAULT_DATA *)(DataBuffer + Index);\r
     while (!IsFound && Index < FullSize && DataHeader->DataSize != 0xFFFFFFFF) {\r
       DefaultInfo = &(DataHeader->DefaultInfo[0]);\r
-      BufferEnd   = (UINT8 *) DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize;\r
-      while ((UINT8 *) DefaultInfo < BufferEnd) {\r
-        if (DefaultInfo->DefaultId == DefaultId && DefaultInfo->SkuId == SkuId) {\r
+      BufferEnd   = (UINT8 *)DataHeader + sizeof (DataHeader->DataSize) + DataHeader->HeaderSize;\r
+      while ((UINT8 *)DefaultInfo < BufferEnd) {\r
+        if ((DefaultInfo->DefaultId == DefaultId) && (DefaultInfo->SkuId == SkuId)) {\r
           IsFound = TRUE;\r
           break;\r
         }\r
-        DefaultInfo ++;\r
+\r
+        DefaultInfo++;\r
       }\r
+\r
       if (IsFound) {\r
-        DeltaData = (PCD_DATA_DELTA *) BufferEnd;\r
-        BufferEnd = (UINT8 *) DataHeader + DataHeader->DataSize;\r
-        while ((UINT8 *) DeltaData < BufferEnd) {\r
-          *(VarStoreHobData + DeltaData->Offset) = (UINT8) DeltaData->Value;\r
-          DeltaData ++;\r
+        DeltaData = (PCD_DATA_DELTA *)BufferEnd;\r
+        BufferEnd = (UINT8 *)DataHeader + DataHeader->DataSize;\r
+        while ((UINT8 *)DeltaData < BufferEnd) {\r
+          *(VarStoreHobData + DeltaData->Offset) = (UINT8)DeltaData->Value;\r
+          DeltaData++;\r
         }\r
+\r
         break;\r
       }\r
-      Index      = (Index + DataHeader->DataSize + 7) & (~7) ;\r
-      DataHeader = (PCD_DEFAULT_DATA *) (DataBuffer + Index);\r
+\r
+      Index      = (Index + DataHeader->DataSize + 7) & (~7);\r
+      DataHeader = (PCD_DEFAULT_DATA *)(DataBuffer + Index);\r
     }\r
   }\r
 \r
   Status = PcdUnRegisterCallBackOnSet (\r
              &gEfiMdeModulePkgTokenSpaceGuid,\r
-             PcdToken(PcdSetNvStoreDefaultId),\r
+             PcdToken (PcdSetNvStoreDefaultId),\r
              PcdSetNvStoreDefaultIdCallBack\r
              );\r
   ASSERT_EFI_ERROR (Status);\r
@@ -245,17 +245,17 @@ EndOfPeiSignalPpiNotifyCallback (
   IN VOID                       *Ppi\r
   )\r
 {\r
-  PEI_PCD_DATABASE       *Database;\r
-  EFI_BOOT_MODE          BootMode;\r
-  EFI_STATUS             Status;\r
-  UINTN                  Instance;\r
-  EFI_PEI_FV_HANDLE      VolumeHandle;\r
-  EFI_PEI_FILE_HANDLE    FileHandle;\r
-  VOID                   *PcdDb;\r
-  UINT32                 Length;\r
-  PEI_PCD_DATABASE       *PeiPcdDb;\r
-\r
-  Status = PeiServicesGetBootMode(&BootMode);\r
+  PEI_PCD_DATABASE     *Database;\r
+  EFI_BOOT_MODE        BootMode;\r
+  EFI_STATUS           Status;\r
+  UINTN                Instance;\r
+  EFI_PEI_FV_HANDLE    VolumeHandle;\r
+  EFI_PEI_FILE_HANDLE  FileHandle;\r
+  VOID                 *PcdDb;\r
+  UINT32               Length;\r
+  PEI_PCD_DATABASE     *PeiPcdDb;\r
+\r
+  Status = PeiServicesGetBootMode (&BootMode);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   //\r
@@ -265,8 +265,8 @@ EndOfPeiSignalPpiNotifyCallback (
     return EFI_SUCCESS;\r
   }\r
 \r
-  PeiPcdDb = GetPcdDatabase();\r
-  if (PeiPcdDb->SystemSkuId != (SKU_ID) 0) {\r
+  PeiPcdDb = GetPcdDatabase ();\r
+  if (PeiPcdDb->SystemSkuId != (SKU_ID)0) {\r
     //\r
     // SkuId has been set. Don't need to report it to DXE phase.\r
     //\r
@@ -276,8 +276,8 @@ EndOfPeiSignalPpiNotifyCallback (
   //\r
   // Get full PCD database from PcdPeim FileHandle\r
   //\r
-  Instance    = 0;\r
-  FileHandle  = NULL;\r
+  Instance   = 0;\r
+  FileHandle = NULL;\r
   while (TRUE) {\r
     //\r
     // Traverse all firmware volume instances\r
@@ -292,13 +292,14 @@ EndOfPeiSignalPpiNotifyCallback (
     // Find PcdDb file from the beginning in this firmware volume.\r
     //\r
     FileHandle = NULL;\r
-    Status = PeiServicesFfsFindFileByName (&gEfiCallerIdGuid, VolumeHandle, &FileHandle);\r
+    Status     = PeiServicesFfsFindFileByName (&gEfiCallerIdGuid, VolumeHandle, &FileHandle);\r
     if (!EFI_ERROR (Status)) {\r
       //\r
       // Find PcdPeim FileHandle in this volume\r
       //\r
       break;\r
     }\r
+\r
     //\r
     // We cannot find PcdPeim in this firmware volume, then search the next volume.\r
     //\r
@@ -310,14 +311,14 @@ EndOfPeiSignalPpiNotifyCallback (
   //\r
   Status = PeiServicesFfsFindSectionData (EFI_SECTION_RAW, FileHandle, &PcdDb);\r
   ASSERT_EFI_ERROR (Status);\r
-  Length = PeiPcdDb->LengthForAllSkus;\r
+  Length   = PeiPcdDb->LengthForAllSkus;\r
   Database = BuildGuidHob (&gPcdDataBaseHobGuid, Length);\r
   CopyMem (Database, PcdDb, Length);\r
 \r
   return EFI_SUCCESS;\r
 }\r
 \r
-EFI_PEI_NOTIFY_DESCRIPTOR mEndOfPeiSignalPpiNotifyList[] = {\r
+EFI_PEI_NOTIFY_DESCRIPTOR  mEndOfPeiSignalPpiNotifyList[] = {\r
   {\r
     (EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK | EFI_PEI_PPI_DESCRIPTOR_TERMINATE_LIST),\r
     &gEfiEndOfPeiSignalPpiGuid,\r
@@ -343,7 +344,76 @@ PcdPeimInit (
   IN CONST EFI_PEI_SERVICES     **PeiServices\r
   )\r
 {\r
-  EFI_STATUS Status;\r
+  EFI_STATUS  Status;\r
+\r
+  Status = PeiServicesRegisterForShadow (FileHandle);\r
+  if (Status == EFI_ALREADY_STARTED) {\r
+    //\r
+    // This is now starting in memory, the second time starting.\r
+    //\r
+    EFI_PEI_PPI_DESCRIPTOR  *OldPpiList;\r
+    EFI_PEI_PPI_DESCRIPTOR  *OldPpiList2;\r
+    VOID                    *Ppi;\r
+    VOID                    *Ppi2;\r
+\r
+    OldPpiList = NULL;\r
+    Status     = PeiServicesLocatePpi (\r
+                   &gPcdPpiGuid,\r
+                   0,\r
+                   &OldPpiList,\r
+                   &Ppi\r
+                   );\r
+    ASSERT_EFI_ERROR (Status);\r
+\r
+    if (OldPpiList != NULL) {\r
+      Status = PeiServicesReInstallPpi (OldPpiList, &mPpiList[0]);\r
+      ASSERT_EFI_ERROR (Status);\r
+    }\r
+\r
+    OldPpiList2 = NULL;\r
+    Status      = PeiServicesLocatePpi (\r
+                    &gGetPcdInfoPpiGuid,\r
+                    0,\r
+                    &OldPpiList2,\r
+                    &Ppi2\r
+                    );\r
+    ASSERT_EFI_ERROR (Status);\r
+\r
+    if (OldPpiList2 != NULL) {\r
+      Status = PeiServicesReInstallPpi (OldPpiList2, &mPpiList2[0]);\r
+      ASSERT_EFI_ERROR (Status);\r
+    }\r
+\r
+    OldPpiList = NULL;\r
+    Status     = PeiServicesLocatePpi (\r
+                   &gEfiPeiPcdPpiGuid,\r
+                   0,\r
+                   &OldPpiList,\r
+                   &Ppi\r
+                   );\r
+    ASSERT_EFI_ERROR (Status);\r
+\r
+    if (OldPpiList != NULL) {\r
+      Status = PeiServicesReInstallPpi (OldPpiList, &mPpiList[1]);\r
+      ASSERT_EFI_ERROR (Status);\r
+    }\r
+\r
+    OldPpiList2 = NULL;\r
+    Status      = PeiServicesLocatePpi (\r
+                    &gEfiGetPcdInfoPpiGuid,\r
+                    0,\r
+                    &OldPpiList2,\r
+                    &Ppi2\r
+                    );\r
+    ASSERT_EFI_ERROR (Status);\r
+\r
+    if (OldPpiList2 != NULL) {\r
+      Status = PeiServicesReInstallPpi (OldPpiList2, &mPpiList2[1]);\r
+      ASSERT_EFI_ERROR (Status);\r
+    }\r
+\r
+    return Status;\r
+  }\r
 \r
   BuildPcdDatabase (FileHandle);\r
 \r
@@ -364,7 +434,7 @@ PcdPeimInit (
 \r
   Status = PeiRegisterCallBackOnSet (\r
              &gEfiMdeModulePkgTokenSpaceGuid,\r
-             PcdToken(PcdSetNvStoreDefaultId),\r
+             PcdToken (PcdSetNvStoreDefaultId),\r
              PcdSetNvStoreDefaultIdCallBack\r
              );\r
   ASSERT_EFI_ERROR (Status);\r
@@ -388,8 +458,8 @@ PcdPeimInit (
 EFI_STATUS\r
 EFIAPI\r
 PeiGetPcdInfoGetInfo (\r
-  IN        UINTN           TokenNumber,\r
-  OUT       EFI_PCD_INFO    *PcdInfo\r
+  IN        UINTN         TokenNumber,\r
+  OUT       EFI_PCD_INFO  *PcdInfo\r
   )\r
 {\r
   return PeiGetPcdInfo (NULL, TokenNumber, PcdInfo);\r
@@ -412,9 +482,9 @@ PeiGetPcdInfoGetInfo (
 EFI_STATUS\r
 EFIAPI\r
 PeiGetPcdInfoGetInfoEx (\r
-  IN CONST  EFI_GUID        *Guid,\r
-  IN        UINTN           TokenNumber,\r
-  OUT       EFI_PCD_INFO    *PcdInfo\r
+  IN CONST  EFI_GUID      *Guid,\r
+  IN        UINTN         TokenNumber,\r
+  OUT       EFI_PCD_INFO  *PcdInfo\r
   )\r
 {\r
   return PeiGetPcdInfo (Guid, TokenNumber, PcdInfo);\r
@@ -433,7 +503,7 @@ PeiGetPcdInfoGetSku (
   VOID\r
   )\r
 {\r
-  return (UINTN) GetPcdDatabase()->SystemSkuId;\r
+  return (UINTN)GetPcdDatabase ()->SystemSkuId;\r
 }\r
 \r
 /**\r
@@ -460,24 +530,24 @@ PeiGetPcdInfoGetSku (
 VOID\r
 EFIAPI\r
 PeiPcdSetSku (\r
-  IN  UINTN                  SkuId\r
+  IN  UINTN  SkuId\r
   )\r
 {\r
-  PEI_PCD_DATABASE  *PeiPcdDb;\r
-  SKU_ID            *SkuIdTable;\r
-  UINTN             Index;\r
-  EFI_STATUS            Status;\r
-  UINTN                 Instance;\r
-  EFI_PEI_FV_HANDLE     VolumeHandle;\r
-  EFI_PEI_FILE_HANDLE   FileHandle;\r
-  VOID                  *PcdDb;\r
-  UINT32                Length;\r
-  PCD_DATABASE_SKU_DELTA *SkuDelta;\r
-  PCD_DATA_DELTA         *SkuDeltaData;\r
-\r
-  DEBUG ((DEBUG_INFO, "PcdPei - SkuId 0x%lx is to be set.\n", (SKU_ID) SkuId));\r
+  PEI_PCD_DATABASE        *PeiPcdDb;\r
+  SKU_ID                  *SkuIdTable;\r
+  UINTN                   Index;\r
+  EFI_STATUS              Status;\r
+  UINTN                   Instance;\r
+  EFI_PEI_FV_HANDLE       VolumeHandle;\r
+  EFI_PEI_FILE_HANDLE     FileHandle;\r
+  VOID                    *PcdDb;\r
+  UINT32                  Length;\r
+  PCD_DATABASE_SKU_DELTA  *SkuDelta;\r
+  PCD_DATA_DELTA          *SkuDeltaData;\r
+\r
+  DEBUG ((DEBUG_INFO, "PcdPei - SkuId 0x%lx is to be set.\n", (SKU_ID)SkuId));\r
 \r
-  PeiPcdDb = GetPcdDatabase();\r
+  PeiPcdDb = GetPcdDatabase ();\r
 \r
   if (SkuId == PeiPcdDb->SystemSkuId) {\r
     //\r
@@ -487,19 +557,19 @@ PeiPcdSetSku (
     return;\r
   }\r
 \r
-  if (PeiPcdDb->SystemSkuId != (SKU_ID) 0) {\r
+  if (PeiPcdDb->SystemSkuId != (SKU_ID)0) {\r
     DEBUG ((DEBUG_ERROR, "PcdPei - The SKU Id could be changed only once."));\r
     DEBUG ((\r
       DEBUG_ERROR,\r
       "PcdPei - The SKU Id was set to 0x%lx already, it could not be set to 0x%lx any more.",\r
       PeiPcdDb->SystemSkuId,\r
-      (SKU_ID) SkuId\r
+      (SKU_ID)SkuId\r
       ));\r
     ASSERT (FALSE);\r
     return;\r
   }\r
 \r
-  SkuIdTable = (SKU_ID *) ((UINT8 *) PeiPcdDb + PeiPcdDb->SkuIdTableOffset);\r
+  SkuIdTable = (SKU_ID *)((UINT8 *)PeiPcdDb + PeiPcdDb->SkuIdTableOffset);\r
   for (Index = 0; Index < SkuIdTable[0]; Index++) {\r
     if (SkuId == SkuIdTable[Index + 1]) {\r
       DEBUG ((DEBUG_INFO, "PcdPei - SkuId is found in SkuId table.\n"));\r
@@ -511,8 +581,8 @@ PeiPcdSetSku (
     //\r
     // Get full PCD database from PcdPeim FileHandle\r
     //\r
-    Instance    = 0;\r
-    FileHandle  = NULL;\r
+    Instance   = 0;\r
+    FileHandle = NULL;\r
     while (TRUE) {\r
       //\r
       // Traverse all firmware volume instances\r
@@ -527,13 +597,14 @@ PeiPcdSetSku (
       // Find PcdDb file from the beginning in this firmware volume.\r
       //\r
       FileHandle = NULL;\r
-      Status = PeiServicesFfsFindFileByName (&gEfiCallerIdGuid, VolumeHandle, &FileHandle);\r
+      Status     = PeiServicesFfsFindFileByName (&gEfiCallerIdGuid, VolumeHandle, &FileHandle);\r
       if (!EFI_ERROR (Status)) {\r
         //\r
         // Find PcdPeim FileHandle in this volume\r
         //\r
         break;\r
       }\r
+\r
       //\r
       // We cannot find PcdPeim in this firmware volume, then search the next volume.\r
       //\r
@@ -545,28 +616,30 @@ PeiPcdSetSku (
     //\r
     Status = PeiServicesFfsFindSectionData (EFI_SECTION_RAW, FileHandle, &PcdDb);\r
     ASSERT_EFI_ERROR (Status);\r
-    Length = PeiPcdDb->LengthForAllSkus;\r
-    Index  = (PeiPcdDb->Length + 7) & (~7);\r
+    Length   = PeiPcdDb->LengthForAllSkus;\r
+    Index    = (PeiPcdDb->Length + 7) & (~7);\r
     SkuDelta = NULL;\r
     while (Index < Length) {\r
-      SkuDelta = (PCD_DATABASE_SKU_DELTA *) ((UINT8 *) PcdDb + Index);\r
-      if (SkuDelta->SkuId == SkuId && SkuDelta->SkuIdCompared == 0) {\r
+      SkuDelta = (PCD_DATABASE_SKU_DELTA *)((UINT8 *)PcdDb + Index);\r
+      if ((SkuDelta->SkuId == SkuId) && (SkuDelta->SkuIdCompared == 0)) {\r
         break;\r
       }\r
+\r
       Index = (Index + SkuDelta->Length + 7) & (~7);\r
     }\r
 \r
     //\r
     // Patch the delta data into current PCD database\r
     //\r
-    if (Index < Length && SkuDelta != NULL) {\r
-      SkuDeltaData = (PCD_DATA_DELTA *) (SkuDelta + 1);\r
-      while ((UINT8 *) SkuDeltaData < (UINT8 *) SkuDelta + SkuDelta->Length) {\r
-        *((UINT8 *) PeiPcdDb + SkuDeltaData->Offset) = (UINT8) SkuDeltaData->Value;\r
-        SkuDeltaData ++;\r
+    if ((Index < Length) && (SkuDelta != NULL)) {\r
+      SkuDeltaData = (PCD_DATA_DELTA *)(SkuDelta + 1);\r
+      while ((UINT8 *)SkuDeltaData < (UINT8 *)SkuDelta + SkuDelta->Length) {\r
+        *((UINT8 *)PeiPcdDb + SkuDeltaData->Offset) = (UINT8)SkuDeltaData->Value;\r
+        SkuDeltaData++;\r
       }\r
-      PeiPcdDb->SystemSkuId = (SKU_ID) SkuId;\r
-      DEBUG ((DEBUG_INFO, "PcdPei - Set current SKU Id to 0x%lx.\n", (SKU_ID) SkuId));\r
+\r
+      PeiPcdDb->SystemSkuId = (SKU_ID)SkuId;\r
+      DEBUG ((DEBUG_INFO, "PcdPei - Set current SKU Id to 0x%lx.\n", (SKU_ID)SkuId));\r
       return;\r
     }\r
   }\r
@@ -593,10 +666,10 @@ PeiPcdSetSku (
 UINT8\r
 EFIAPI\r
 PeiPcdGet8 (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return *((UINT8 *) GetWorker (TokenNumber, sizeof (UINT8)));\r
+  return *((UINT8 *)GetWorker (TokenNumber, sizeof (UINT8)));\r
 }\r
 \r
 /**\r
@@ -613,7 +686,7 @@ PeiPcdGet8 (
 UINT16\r
 EFIAPI\r
 PeiPcdGet16 (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
   return ReadUnaligned16 (GetWorker (TokenNumber, sizeof (UINT16)));\r
@@ -633,7 +706,7 @@ PeiPcdGet16 (
 UINT32\r
 EFIAPI\r
 PeiPcdGet32 (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
   return ReadUnaligned32 (GetWorker (TokenNumber, sizeof (UINT32)));\r
@@ -653,7 +726,7 @@ PeiPcdGet32 (
 UINT64\r
 EFIAPI\r
 PeiPcdGet64 (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
   return ReadUnaligned64 (GetWorker (TokenNumber, sizeof (UINT64)));\r
@@ -675,7 +748,7 @@ PeiPcdGet64 (
 VOID *\r
 EFIAPI\r
 PeiPcdGetPtr (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
   return GetWorker (TokenNumber, 0);\r
@@ -697,10 +770,10 @@ PeiPcdGetPtr (
 BOOLEAN\r
 EFIAPI\r
 PeiPcdGetBool (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return *((BOOLEAN *) GetWorker (TokenNumber, sizeof (BOOLEAN)));\r
+  return *((BOOLEAN *)GetWorker (TokenNumber, sizeof (BOOLEAN)));\r
 }\r
 \r
 /**\r
@@ -717,13 +790,13 @@ PeiPcdGetBool (
 UINTN\r
 EFIAPI\r
 PeiPcdGetSize (\r
-  IN UINTN                    TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  PEI_PCD_DATABASE    *PeiPcdDb;\r
-  UINTN               Size;\r
-  UINTN               MaxSize;\r
-  UINT32              LocalTokenCount;\r
+  PEI_PCD_DATABASE  *PeiPcdDb;\r
+  UINTN             Size;\r
+  UINTN             MaxSize;\r
+  UINT32            LocalTokenCount;\r
 \r
   PeiPcdDb        = GetPcdDatabase ();\r
   LocalTokenCount = PeiPcdDb->LocalTokenCount;\r
@@ -749,7 +822,6 @@ PeiPcdGetSize (
   } else {\r
     return Size;\r
   }\r
-\r
 }\r
 \r
 /**\r
@@ -769,11 +841,11 @@ PeiPcdGetSize (
 UINT8\r
 EFIAPI\r
 PeiPcdGet8Ex (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber\r
   )\r
 {\r
-  return *((UINT8 *) ExGetWorker (Guid, ExTokenNumber, sizeof (UINT8)));\r
+  return *((UINT8 *)ExGetWorker (Guid, ExTokenNumber, sizeof (UINT8)));\r
 }\r
 \r
 /**\r
@@ -793,8 +865,8 @@ PeiPcdGet8Ex (
 UINT16\r
 EFIAPI\r
 PeiPcdGet16Ex (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber\r
   )\r
 {\r
   return ReadUnaligned16 (ExGetWorker (Guid, ExTokenNumber, sizeof (UINT16)));\r
@@ -817,8 +889,8 @@ PeiPcdGet16Ex (
 UINT32\r
 EFIAPI\r
 PeiPcdGet32Ex (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber\r
   )\r
 {\r
   return ReadUnaligned32 (ExGetWorker (Guid, ExTokenNumber, sizeof (UINT32)));\r
@@ -841,8 +913,8 @@ PeiPcdGet32Ex (
 UINT64\r
 EFIAPI\r
 PeiPcdGet64Ex (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber\r
   )\r
 {\r
   return ReadUnaligned64 (ExGetWorker (Guid, ExTokenNumber, sizeof (UINT64)));\r
@@ -865,8 +937,8 @@ PeiPcdGet64Ex (
 VOID *\r
 EFIAPI\r
 PeiPcdGetPtrEx (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber\r
   )\r
 {\r
   return ExGetWorker (Guid, ExTokenNumber, 0);\r
@@ -889,11 +961,11 @@ PeiPcdGetPtrEx (
 BOOLEAN\r
 EFIAPI\r
 PeiPcdGetBoolEx (\r
-  IN CONST  EFI_GUID              *Guid,\r
-  IN UINTN                        ExTokenNumber\r
+  IN CONST  EFI_GUID  *Guid,\r
+  IN UINTN            ExTokenNumber\r
   )\r
 {\r
-  return *((BOOLEAN *) ExGetWorker (Guid, ExTokenNumber, sizeof (BOOLEAN)));\r
+  return *((BOOLEAN *)ExGetWorker (Guid, ExTokenNumber, sizeof (BOOLEAN)));\r
 }\r
 \r
 /**\r
@@ -911,8 +983,8 @@ PeiPcdGetBoolEx (
 UINTN\r
 EFIAPI\r
 PeiPcdGetSizeEx (\r
-  IN CONST  EFI_GUID              *Guid,\r
-  IN UINTN                        ExTokenNumber\r
+  IN CONST  EFI_GUID  *Guid,\r
+  IN UINTN            ExTokenNumber\r
   )\r
 {\r
   return PeiPcdGetSize (GetExPcdTokenNumber (Guid, ExTokenNumber));\r
@@ -938,8 +1010,8 @@ PeiPcdGetSizeEx (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet8 (\r
-  IN UINTN                        TokenNumber,\r
-  IN UINT8                        Value\r
+  IN UINTN  TokenNumber,\r
+  IN UINT8  Value\r
   )\r
 {\r
   return SetValueWorker (TokenNumber, &Value, sizeof (Value));\r
@@ -965,8 +1037,8 @@ PeiPcdSet8 (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet16 (\r
-  IN UINTN                         TokenNumber,\r
-  IN UINT16                        Value\r
+  IN UINTN   TokenNumber,\r
+  IN UINT16  Value\r
   )\r
 {\r
   return SetValueWorker (TokenNumber, &Value, sizeof (Value));\r
@@ -992,8 +1064,8 @@ PeiPcdSet16 (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet32 (\r
-  IN UINTN                         TokenNumber,\r
-  IN UINT32                        Value\r
+  IN UINTN   TokenNumber,\r
+  IN UINT32  Value\r
   )\r
 {\r
   return SetValueWorker (TokenNumber, &Value, sizeof (Value));\r
@@ -1019,8 +1091,8 @@ PeiPcdSet32 (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet64 (\r
-  IN UINTN                         TokenNumber,\r
-  IN UINT64                        Value\r
+  IN UINTN   TokenNumber,\r
+  IN UINT64  Value\r
   )\r
 {\r
   return SetValueWorker (TokenNumber, &Value, sizeof (Value));\r
@@ -1050,9 +1122,9 @@ PeiPcdSet64 (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSetPtr (\r
-  IN      UINTN                    TokenNumber,\r
-  IN OUT  UINTN                    *SizeOfBuffer,\r
-  IN      VOID                     *Buffer\r
+  IN      UINTN  TokenNumber,\r
+  IN OUT  UINTN  *SizeOfBuffer,\r
+  IN      VOID   *Buffer\r
   )\r
 {\r
   return SetWorker (TokenNumber, Buffer, SizeOfBuffer, TRUE);\r
@@ -1078,8 +1150,8 @@ PeiPcdSetPtr (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSetBool (\r
-  IN UINTN                         TokenNumber,\r
-  IN BOOLEAN                       Value\r
+  IN UINTN    TokenNumber,\r
+  IN BOOLEAN  Value\r
   )\r
 {\r
   return SetValueWorker (TokenNumber, &Value, sizeof (Value));\r
@@ -1106,9 +1178,9 @@ PeiPcdSetBool (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet8Ex (\r
-  IN CONST EFI_GUID               *Guid,\r
-  IN UINTN                        ExTokenNumber,\r
-  IN UINT8                        Value\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber,\r
+  IN UINT8           Value\r
   )\r
 {\r
   return ExSetValueWorker (ExTokenNumber, Guid, &Value, sizeof (Value));\r
@@ -1135,9 +1207,9 @@ PeiPcdSet8Ex (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet16Ex (\r
-  IN CONST EFI_GUID               *Guid,\r
-  IN UINTN                        ExTokenNumber,\r
-  IN UINT16                       Value\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber,\r
+  IN UINT16          Value\r
   )\r
 {\r
   return ExSetValueWorker (ExTokenNumber, Guid, &Value, sizeof (Value));\r
@@ -1164,9 +1236,9 @@ PeiPcdSet16Ex (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet32Ex (\r
-  IN CONST EFI_GUID               *Guid,\r
-  IN UINTN                        ExTokenNumber,\r
-  IN UINT32                       Value\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber,\r
+  IN UINT32          Value\r
   )\r
 {\r
   return ExSetValueWorker (ExTokenNumber, Guid, &Value, sizeof (Value));\r
@@ -1193,9 +1265,9 @@ PeiPcdSet32Ex (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSet64Ex (\r
-  IN CONST EFI_GUID               *Guid,\r
-  IN UINTN                        ExTokenNumber,\r
-  IN UINT64                       Value\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber,\r
+  IN UINT64          Value\r
   )\r
 {\r
   return ExSetValueWorker (ExTokenNumber, Guid, &Value, sizeof (Value));\r
@@ -1226,10 +1298,10 @@ PeiPcdSet64Ex (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSetPtrEx (\r
-  IN     CONST EFI_GUID               *Guid,\r
-  IN     UINTN                        ExTokenNumber,\r
-  IN OUT UINTN                        *SizeOfBuffer,\r
-  IN     VOID                         *Value\r
+  IN     CONST EFI_GUID  *Guid,\r
+  IN     UINTN           ExTokenNumber,\r
+  IN OUT UINTN           *SizeOfBuffer,\r
+  IN     VOID            *Value\r
   )\r
 {\r
   return ExSetWorker (ExTokenNumber, Guid, Value, SizeOfBuffer, TRUE);\r
@@ -1256,9 +1328,9 @@ PeiPcdSetPtrEx (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdSetBoolEx (\r
-  IN CONST EFI_GUID             *Guid,\r
-  IN UINTN                      ExTokenNumber,\r
-  IN BOOLEAN                    Value\r
+  IN CONST EFI_GUID  *Guid,\r
+  IN UINTN           ExTokenNumber,\r
+  IN BOOLEAN         Value\r
   )\r
 {\r
   return ExSetValueWorker (ExTokenNumber, Guid, &Value, sizeof (Value));\r
@@ -1279,12 +1351,12 @@ PeiPcdSetBoolEx (
 EFI_STATUS\r
 EFIAPI\r
 PeiRegisterCallBackOnSet (\r
-  IN  CONST EFI_GUID              *Guid, OPTIONAL\r
-  IN  UINTN                       ExTokenNumber,\r
-  IN  PCD_PPI_CALLBACK            CallBackFunction\r
+  IN  CONST EFI_GUID    *Guid  OPTIONAL,\r
+  IN  UINTN             ExTokenNumber,\r
+  IN  PCD_PPI_CALLBACK  CallBackFunction\r
   )\r
 {\r
-  if (!FeaturePcdGet(PcdPeiFullPcdDatabaseEnable)) {\r
+  if (!FeaturePcdGet (PcdPeiFullPcdDatabaseEnable)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -1310,12 +1382,12 @@ PeiRegisterCallBackOnSet (
 EFI_STATUS\r
 EFIAPI\r
 PcdUnRegisterCallBackOnSet (\r
-  IN  CONST EFI_GUID              *Guid, OPTIONAL\r
-  IN  UINTN                       ExTokenNumber,\r
-  IN  PCD_PPI_CALLBACK            CallBackFunction\r
+  IN  CONST EFI_GUID    *Guid  OPTIONAL,\r
+  IN  UINTN             ExTokenNumber,\r
+  IN  PCD_PPI_CALLBACK  CallBackFunction\r
   )\r
 {\r
-  if (!FeaturePcdGet(PcdPeiFullPcdDatabaseEnable)) {\r
+  if (!FeaturePcdGet (PcdPeiFullPcdDatabaseEnable)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -1354,19 +1426,19 @@ PcdUnRegisterCallBackOnSet (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdGetNextToken (\r
-  IN CONST EFI_GUID               *Guid, OPTIONAL\r
-  IN OUT  UINTN                   *TokenNumber\r
+  IN CONST EFI_GUID  *Guid  OPTIONAL,\r
+  IN OUT  UINTN      *TokenNumber\r
   )\r
 {\r
-  UINTN               GuidTableIdx;\r
-  PEI_PCD_DATABASE    *PeiPcdDb;\r
-  EFI_GUID            *MatchGuid;\r
-  EFI_GUID            *GuidTable;\r
-  DYNAMICEX_MAPPING   *ExMapTable;\r
-  UINTN               Index;\r
-  BOOLEAN             Found;\r
-  BOOLEAN             PeiExMapTableEmpty;\r
-  UINTN               PeiNexTokenNumber;\r
+  UINTN              GuidTableIdx;\r
+  PEI_PCD_DATABASE   *PeiPcdDb;\r
+  EFI_GUID           *MatchGuid;\r
+  EFI_GUID           *GuidTable;\r
+  DYNAMICEX_MAPPING  *ExMapTable;\r
+  UINTN              Index;\r
+  BOOLEAN            Found;\r
+  BOOLEAN            PeiExMapTableEmpty;\r
+  UINTN              PeiNexTokenNumber;\r
 \r
   if (!FeaturePcdGet (PcdPeiFullPcdDatabaseEnable)) {\r
     return EFI_UNSUPPORTED;\r
@@ -1381,22 +1453,25 @@ PeiPcdGetNextToken (
   } else {\r
     PeiExMapTableEmpty = FALSE;\r
   }\r
+\r
   if (Guid == NULL) {\r
     if (*TokenNumber > PeiNexTokenNumber) {\r
       return EFI_NOT_FOUND;\r
     }\r
+\r
     (*TokenNumber)++;\r
     if (*TokenNumber > PeiNexTokenNumber) {\r
       *TokenNumber = PCD_INVALID_TOKEN_NUMBER;\r
       return EFI_NOT_FOUND;\r
     }\r
+\r
     return EFI_SUCCESS;\r
   } else {\r
     if (PeiExMapTableEmpty) {\r
       return EFI_NOT_FOUND;\r
     }\r
 \r
-    MatchGuid = ScanGuid (GuidTable, PeiPcdDb->GuidTableCount * sizeof(EFI_GUID), Guid);\r
+    MatchGuid = ScanGuid (GuidTable, PeiPcdDb->GuidTableCount * sizeof (EFI_GUID), Guid);\r
 \r
     if (MatchGuid == NULL) {\r
       return EFI_NOT_FOUND;\r
@@ -1424,7 +1499,7 @@ PeiPcdGetNextToken (
       //\r
       if (*TokenNumber == PCD_INVALID_TOKEN_NUMBER) {\r
         *TokenNumber = ExMapTable[Index].ExTokenNumber;\r
-         return EFI_SUCCESS;\r
+        return EFI_SUCCESS;\r
       }\r
 \r
       for ( ; Index < PeiPcdDb->ExTokenCount; Index++) {\r
@@ -1474,18 +1549,18 @@ PeiPcdGetNextToken (
 EFI_STATUS\r
 EFIAPI\r
 PeiPcdGetNextTokenSpace (\r
-  IN OUT CONST EFI_GUID          **Guid\r
+  IN OUT CONST EFI_GUID  **Guid\r
   )\r
 {\r
-  UINTN               GuidTableIdx;\r
-  EFI_GUID            *MatchGuid;\r
-  PEI_PCD_DATABASE    *PeiPcdDb;\r
-  DYNAMICEX_MAPPING   *ExMapTable;\r
-  UINTN               Index;\r
-  UINTN               Index2;\r
-  BOOLEAN             Found;\r
-  BOOLEAN             PeiExMapTableEmpty;\r
-  EFI_GUID            *GuidTable;\r
+  UINTN              GuidTableIdx;\r
+  EFI_GUID           *MatchGuid;\r
+  PEI_PCD_DATABASE   *PeiPcdDb;\r
+  DYNAMICEX_MAPPING  *ExMapTable;\r
+  UINTN              Index;\r
+  UINTN              Index2;\r
+  BOOLEAN            Found;\r
+  BOOLEAN            PeiExMapTableEmpty;\r
+  EFI_GUID           *GuidTable;\r
 \r
   if (!FeaturePcdGet (PcdPeiFullPcdDatabaseEnable)) {\r
     return EFI_UNSUPPORTED;\r
@@ -1516,7 +1591,7 @@ PeiPcdGetNextTokenSpace (
     return EFI_SUCCESS;\r
   }\r
 \r
-  MatchGuid = ScanGuid (GuidTable, PeiPcdDb->GuidTableCount * sizeof(GuidTable[0]), *Guid);\r
+  MatchGuid = ScanGuid (GuidTable, PeiPcdDb->GuidTableCount * sizeof (GuidTable[0]), *Guid);\r
 \r
   if (MatchGuid == NULL) {\r
     return EFI_NOT_FOUND;\r
@@ -1537,7 +1612,7 @@ PeiPcdGetNextTokenSpace (
     for ( ; Index < PeiPcdDb->ExTokenCount; Index++ ) {\r
       if (ExMapTable[Index].ExGuidIndex != GuidTableIdx) {\r
         Found = FALSE;\r
-        for (Index2 = 0 ; Index2 < Index; Index2++) {\r
+        for (Index2 = 0; Index2 < Index; Index2++) {\r
           if (ExMapTable[Index2].ExGuidIndex == ExMapTable[Index].ExGuidIndex) {\r
             //\r
             // This token namespace should have been found and output at preceding getting.\r
@@ -1546,17 +1621,18 @@ PeiPcdGetNextTokenSpace (
             break;\r
           }\r
         }\r
+\r
         if (!Found) {\r
           *Guid = (EFI_GUID *)((UINT8 *)PeiPcdDb + PeiPcdDb->GuidTableOffset) + ExMapTable[Index].ExGuidIndex;\r
           return EFI_SUCCESS;\r
         }\r
       }\r
     }\r
+\r
     *Guid = NULL;\r
   }\r
 \r
   return EFI_NOT_FOUND;\r
-\r
 }\r
 \r
 /**\r
@@ -1579,9 +1655,9 @@ GetPtrTypeSize (
   IN    PEI_PCD_DATABASE  *Database\r
   )\r
 {\r
-  INTN        SizeTableIdx;\r
-  UINTN       LocalTokenNumber;\r
-  SIZE_INFO   *SizeTable;\r
+  INTN       SizeTableIdx;\r
+  UINTN      LocalTokenNumber;\r
+  SIZE_INFO  *SizeTable;\r
 \r
   SizeTableIdx = GetSizeTableIndex (LocalTokenNumberTableIdx, Database);\r
 \r
@@ -1597,13 +1673,13 @@ GetPtrTypeSize (
   // PCD entry.\r
   //\r
   if ((LocalTokenNumber & PCD_TYPE_VPD) != 0) {\r
-      //\r
-      // We have only two entry for VPD enabled PCD entry:\r
-      // 1) MAX Size.\r
-      // 2) Current Size\r
-      // We consider current size is equal to MAX size.\r
-      //\r
-      return *MaxSize;\r
+    //\r
+    // We have only two entry for VPD enabled PCD entry:\r
+    // 1) MAX Size.\r
+    // 2) Current Size\r
+    // We consider current size is equal to MAX size.\r
+    //\r
+    return *MaxSize;\r
   } else {\r
     //\r
     // We have only two entry for Non-Sku enabled PCD entry:\r
@@ -1635,10 +1711,10 @@ SetPtrTypeSize (
   IN          PEI_PCD_DATABASE  *Database\r
   )\r
 {\r
-  INTN        SizeTableIdx;\r
-  UINTN       LocalTokenNumber;\r
-  SIZE_INFO   *SizeTable;\r
-  UINTN       MaxSize;\r
+  INTN       SizeTableIdx;\r
+  UINTN      LocalTokenNumber;\r
+  SIZE_INFO  *SizeTable;\r
+  UINTN      MaxSize;\r
 \r
   SizeTableIdx = GetSizeTableIndex (LocalTokenNumberTableIdx, Database);\r
 \r
@@ -1654,16 +1730,17 @@ SetPtrTypeSize (
   // PCD entry.\r
   //\r
   if ((LocalTokenNumber & PCD_TYPE_VPD) != 0) {\r
-      //\r
-      // We shouldn't come here as we don't support SET for VPD\r
-      //\r
-      ASSERT (FALSE);\r
-      return FALSE;\r
+    //\r
+    // We shouldn't come here as we don't support SET for VPD\r
+    //\r
+    ASSERT (FALSE);\r
+    return FALSE;\r
   } else {\r
     if ((*CurrentSize > MaxSize) ||\r
-      (*CurrentSize == MAX_ADDRESS)) {\r
-       *CurrentSize = MaxSize;\r
-       return FALSE;\r
+        (*CurrentSize == MAX_ADDRESS))\r
+    {\r
+      *CurrentSize = MaxSize;\r
+      return FALSE;\r
     }\r
 \r
     //\r
@@ -1671,8 +1748,7 @@ SetPtrTypeSize (
     // 1) MAX SIZE\r
     // 2) Current Size\r
     //\r
-    SizeTable[SizeTableIdx + 1] = (SIZE_INFO) *CurrentSize;\r
+    SizeTable[SizeTableIdx + 1] = (SIZE_INFO)*CurrentSize;\r
     return TRUE;\r
   }\r
-\r
 }\r