]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/DxePcdLib/DxePcdLib.c
MdePkg: Apply uncrustify changes
[mirror_edk2.git] / MdePkg / Library / DxePcdLib / DxePcdLib.c
index f14a74652978bb38f6c1af867ab4294c7faa6e77..b2a8bf22a365bcb51785717b2a7ac5b8de1c7f96 100644 (file)
@@ -1,19 +1,12 @@
 /** @file\r
 Implementation of PcdLib class library for DXE phase.\r
 \r
-Copyright (c) 2006 - 2013, Intel Corporation. All rights reserved.<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
+Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>\r
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 \r
 **/\r
 \r
-\r
 #include <PiDxe.h>\r
 \r
 #include <Protocol/Pcd.h>\r
@@ -26,10 +19,10 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/UefiBootServicesTableLib.h>\r
 #include <Library/BaseMemoryLib.h>\r
 \r
-PCD_PROTOCOL                *mPcd       = NULL;\r
-EFI_PCD_PROTOCOL            *mPiPcd     = NULL;\r
-GET_PCD_INFO_PROTOCOL       *mPcdInfo   = NULL;\r
-EFI_GET_PCD_INFO_PROTOCOL   *mPiPcdInfo = NULL;\r
+PCD_PROTOCOL               *mPcd       = NULL;\r
+EFI_PCD_PROTOCOL           *mPiPcd     = NULL;\r
+GET_PCD_INFO_PROTOCOL      *mPcdInfo   = NULL;\r
+EFI_GET_PCD_INFO_PROTOCOL  *mPiPcdInfo = NULL;\r
 \r
 /**\r
   Retrieves the PI PCD protocol from the handle database.\r
@@ -46,13 +39,14 @@ GetPiPcdProtocol (
 \r
   if (mPiPcd == NULL) {\r
     //\r
-    // PI Pcd protocol defined in PI 1.2 vol3 should be installed before the module \r
+    // PI Pcd protocol defined in PI 1.2 vol3 should be installed before the module\r
     // access DynamicEx type PCD.\r
     //\r
-    Status = gBS->LocateProtocol (&gEfiPcdProtocolGuid, NULL, (VOID **) &mPiPcd);\r
+    Status = gBS->LocateProtocol (&gEfiPcdProtocolGuid, NULL, (VOID **)&mPiPcd);\r
     ASSERT_EFI_ERROR (Status);\r
     ASSERT (mPiPcd != NULL);\r
   }\r
+\r
   return mPiPcd;\r
 }\r
 \r
@@ -73,11 +67,12 @@ GetPcdProtocol (
     //\r
     // PCD protocol need to be installed before the module access Dynamic type PCD.\r
     // But dynamic type PCD is not required in PI 1.2 specification.\r
-    // \r
+    //\r
     Status = gBS->LocateProtocol (&gPcdProtocolGuid, NULL, (VOID **)&mPcd);\r
     ASSERT_EFI_ERROR (Status);\r
     ASSERT (mPcd != NULL);\r
   }\r
+\r
   return mPcd;\r
 }\r
 \r
@@ -98,6 +93,7 @@ GetPiPcdInfoProtocolPointer (
     ASSERT_EFI_ERROR (Status);\r
     ASSERT (mPiPcdInfo != NULL);\r
   }\r
+\r
   return mPiPcdInfo;\r
 }\r
 \r
@@ -109,7 +105,7 @@ GetPiPcdInfoProtocolPointer (
 GET_PCD_INFO_PROTOCOL *\r
 GetPcdInfoProtocolPointer (\r
   VOID\r
-  ) \r
+  )\r
 {\r
   EFI_STATUS  Status;\r
 \r
@@ -118,6 +114,7 @@ GetPcdInfoProtocolPointer (
     ASSERT_EFI_ERROR (Status);\r
     ASSERT (mPcdInfo != NULL);\r
   }\r
+\r
   return mPcdInfo;\r
 }\r
 \r
@@ -125,7 +122,6 @@ GetPcdInfoProtocolPointer (
   This function provides a means by which SKU support can be established in the PCD infrastructure.\r
 \r
   Sets the current SKU in the PCD database to the value specified by SkuId.  SkuId is returned.\r
-  If SkuId >= PCD_MAX_SKU_ID, then ASSERT(). \r
 \r
   @param  SkuId   The SKU value that will be used when the PCD service retrieves and sets values\r
                   associated with a PCD token.\r
@@ -136,64 +132,56 @@ GetPcdInfoProtocolPointer (
 UINTN\r
 EFIAPI\r
 LibPcdSetSku (\r
-  IN UINTN   SkuId\r
+  IN UINTN  SkuId\r
   )\r
 {\r
-  ASSERT (SkuId < PCD_MAX_SKU_ID);\r
-\r
-  GetPcdProtocol()->SetSku (SkuId);\r
+  GetPcdProtocol ()->SetSku (SkuId);\r
 \r
   return SkuId;\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
-  Returns the 8-bit value for the token specified by TokenNumber. \r
+\r
+  Returns the 8-bit value for the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
 \r
-  @return Returns the 8-bit value for the token specified by TokenNumber. \r
+  @return Returns the 8-bit value for the token specified by TokenNumber.\r
 \r
 **/\r
 UINT8\r
 EFIAPI\r
 LibPcdGet8 (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->Get8 (TokenNumber);\r
+  return GetPcdProtocol ()->Get8 (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
-  Returns the 16-bit value for the token specified by TokenNumber. \r
+\r
+  Returns the 16-bit value for the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
 \r
-  @return Returns the 16-bit value for the token specified by TokenNumber. \r
+  @return Returns the 16-bit value for the token specified by TokenNumber.\r
 \r
 **/\r
 UINT16\r
 EFIAPI\r
 LibPcdGet16 (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->Get16 (TokenNumber);\r
+  return GetPcdProtocol ()->Get16 (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
-  Returns the 32-bit value for the token specified by TokenNumber. \r
+\r
+  Returns the 32-bit value for the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
 \r
@@ -203,17 +191,15 @@ LibPcdGet16 (
 UINT32\r
 EFIAPI\r
 LibPcdGet32 (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->Get32 (TokenNumber);\r
+  return GetPcdProtocol ()->Get32 (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
+\r
   Returns the 64-bit value for the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
@@ -224,17 +210,15 @@ LibPcdGet32 (
 UINT64\r
 EFIAPI\r
 LibPcdGet64 (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->Get64 (TokenNumber);\r
+  return GetPcdProtocol ()->Get64 (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
+\r
   Returns the pointer to the buffer of the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
@@ -245,62 +229,56 @@ LibPcdGet64 (
 VOID *\r
 EFIAPI\r
 LibPcdGetPtr (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->GetPtr (TokenNumber);\r
+  return GetPcdProtocol ()->GetPtr (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
-  Returns the Boolean value of the token specified by TokenNumber. \r
+\r
+  Returns the Boolean value of the token specified by TokenNumber.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
 \r
-  @return Returns the Boolean value of the token specified by TokenNumber. \r
+  @return Returns the Boolean value of the token specified by TokenNumber.\r
 \r
 **/\r
-BOOLEAN \r
+BOOLEAN\r
 EFIAPI\r
 LibPcdGetBool (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->GetBool (TokenNumber);\r
+  return GetPcdProtocol ()->GetBool (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve the size of a given PCD token.\r
 \r
   @param[in]  TokenNumber The PCD token number to retrieve a current value for.\r
 \r
-  @return Returns the size of the token specified by TokenNumber. \r
+  @return Returns the size of the token specified by TokenNumber.\r
 \r
 **/\r
 UINTN\r
 EFIAPI\r
 LibPcdGetSize (\r
-  IN UINTN             TokenNumber\r
+  IN UINTN  TokenNumber\r
   )\r
 {\r
-  return GetPcdProtocol()->GetSize (TokenNumber);\r
+  return GetPcdProtocol ()->GetSize (TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
+\r
   Returns the 8-bit value for the token specified by TokenNumber and Guid.\r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid         The pointer to a 128-bit unique value that designates \r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid         The pointer to a 128-bit unique value that designates\r
                            which namespace to retrieve a value from.\r
   @param[in]  TokenNumber  The PCD token number to retrieve a current value for.\r
 \r
@@ -310,24 +288,23 @@ LibPcdGetSize (
 UINT8\r
 EFIAPI\r
 LibPcdGetEx8 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
-  \r
-  return GetPiPcdProtocol()->Get8 (Guid, TokenNumber);\r
-}\r
 \r
+  return GetPiPcdProtocol ()->Get8 (Guid, TokenNumber);\r
+}\r
 \r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
 \r
   Returns the 16-bit value for the token specified by TokenNumber and Guid.\r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid         The pointer to a 128-bit unique value that designates \r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid         The pointer to a 128-bit unique value that designates\r
                            which namespace to retrieve a value from.\r
   @param[in]  TokenNumber  The PCD token number to retrieve a current value for.\r
 \r
@@ -337,21 +314,20 @@ LibPcdGetEx8 (
 UINT16\r
 EFIAPI\r
 LibPcdGetEx16 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return GetPiPcdProtocol()->Get16 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol ()->Get16 (Guid, TokenNumber);\r
 }\r
 \r
-\r
 /**\r
   Returns the 32-bit value for the token specified by TokenNumber and Guid.\r
-  If Guid is NULL, then ASSERT(). \r
+  If Guid is NULL, then ASSERT().\r
 \r
-  @param[in]  Guid         The pointer to a 128-bit unique value that designates \r
+  @param[in]  Guid         The pointer to a 128-bit unique value that designates\r
                            which namespace to retrieve a value from.\r
   @param[in]  TokenNumber  The PCD token number to retrieve a current value for.\r
 \r
@@ -361,25 +337,23 @@ LibPcdGetEx16 (
 UINT32\r
 EFIAPI\r
 LibPcdGetEx32 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return GetPiPcdProtocol()->Get32 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol ()->Get32 (Guid, TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
+\r
   Returns the 64-bit value for the token specified by TokenNumber and Guid.\r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that designates \r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid          The pointer to a 128-bit unique value that designates\r
                             which namespace to retrieve a value from.\r
   @param[in]  TokenNumber   The PCD token number to retrieve a current value for.\r
 \r
@@ -389,25 +363,23 @@ LibPcdGetEx32 (
 UINT64\r
 EFIAPI\r
 LibPcdGetEx64 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
-  \r
-  return GetPiPcdProtocol()->Get64 (Guid, TokenNumber);\r
-}\r
-\r
 \r
+  return GetPiPcdProtocol ()->Get64 (Guid, TokenNumber);\r
+}\r
 \r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
+\r
   Returns the pointer to the buffer of token specified by TokenNumber and Guid.\r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that designates \r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid          The pointer to a 128-bit unique value that designates\r
                             which namespace to retrieve a value from.\r
   @param[in]  TokenNumber   The PCD token number to retrieve a current value for.\r
 \r
@@ -417,25 +389,23 @@ LibPcdGetEx64 (
 VOID *\r
 EFIAPI\r
 LibPcdGetExPtr (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return GetPiPcdProtocol()->GetPtr (Guid, TokenNumber);\r
+  return GetPiPcdProtocol ()->GetPtr (Guid, TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve a value for a given PCD token.\r
-  \r
-  Returns the Boolean value of the token specified by TokenNumber and Guid. \r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that designates \r
+  Returns the Boolean value of the token specified by TokenNumber and Guid.\r
+\r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid          The pointer to a 128-bit unique value that designates\r
                             which namespace to retrieve a value from.\r
   @param[in]  TokenNumber   The PCD token number to retrieve a current value for.\r
 \r
@@ -445,25 +415,23 @@ LibPcdGetExPtr (
 BOOLEAN\r
 EFIAPI\r
 LibPcdGetExBool (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return GetPiPcdProtocol()->GetBool (Guid, TokenNumber);\r
+  return GetPiPcdProtocol ()->GetBool (Guid, TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to retrieve the size of a given PCD token.\r
-  \r
-  Returns the size of the token specified by TokenNumber and Guid. \r
-  \r
-  If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that designates \r
+  Returns the size of the token specified by TokenNumber and Guid.\r
+\r
+  If Guid is NULL, then ASSERT().\r
+\r
+  @param[in]  Guid          The pointer to a 128-bit unique value that designates\r
                             which namespace to retrieve a value from.\r
   @param[in]  TokenNumber   The PCD token number to retrieve a current value for.\r
 \r
@@ -473,386 +441,311 @@ LibPcdGetExBool (
 UINTN\r
 EFIAPI\r
 LibPcdGetExSize (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return GetPiPcdProtocol()->GetSize (Guid, TokenNumber);\r
+  return GetPiPcdProtocol ()->GetSize (Guid, TokenNumber);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 8-bit value for the token specified by TokenNumber \r
-  to the value specified by Value.  Value is returned.\r
-  \r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 8-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  Sets the 8-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 8-bit value to set.\r
+\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT8\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSet8 (\r
-  IN UINTN             TokenNumber,\r
-  IN UINT8             Value\r
+LibPcdSet8S (\r
+  IN UINTN  TokenNumber,\r
+  IN UINT8  Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = GetPcdProtocol()->Set8 (TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-  \r
-  return Value;\r
+  return GetPcdProtocol ()->Set8 (TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 16-bit value for the token specified by TokenNumber \r
-  to the value specified by Value.  Value is returned.\r
-  \r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 16-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  Sets the 16-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 16-bit value to set.\r
+\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT16\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSet16 (\r
-  IN UINTN             TokenNumber,\r
-  IN UINT16            Value\r
+LibPcdSet16S (\r
+  IN UINTN   TokenNumber,\r
+  IN UINT16  Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = GetPcdProtocol()->Set16 (TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-  \r
-  return Value;\r
+  return GetPcdProtocol ()->Set16 (TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 32-bit value for the token specified by TokenNumber \r
-  to the value specified by Value.  Value is returned.\r
-  \r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 32-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  Sets the 32-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 32-bit value to set.\r
+\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT32\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSet32 (\r
-  IN UINTN             TokenNumber,\r
-  IN UINT32            Value\r
+LibPcdSet32S (\r
+  IN UINTN   TokenNumber,\r
+  IN UINT32  Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-  \r
-  Status = GetPcdProtocol()->Set32 (TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPcdProtocol ()->Set32 (TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 64-bit value for the token specified by TokenNumber \r
-  to the value specified by Value.  Value is returned.\r
-  \r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 64-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  Sets the 64-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 64-bit value to set.\r
+\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT64\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSet64 (\r
-  IN UINTN             TokenNumber,\r
-  IN UINT64            Value\r
+LibPcdSet64S (\r
+  IN UINTN   TokenNumber,\r
+  IN UINT64  Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = GetPcdProtocol()->Set64 (TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPcdProtocol ()->Set64 (TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets a buffer for the token specified by TokenNumber to the value \r
-  specified by Buffer and SizeOfBuffer.  Buffer is returned.  \r
-  If SizeOfBuffer is greater than the maximum size support by TokenNumber, \r
-  then set SizeOfBuffer to the maximum size supported by TokenNumber and \r
-  return NULL to indicate that the set operation was not actually performed.  \r
-\r
-  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to the \r
-  maximum size supported by TokenName and NULL must be returned.\r
-  \r
+\r
+  Sets a buffer for the token specified by TokenNumber to the value specified\r
+  by Buffer and SizeOfBuffer. If SizeOfBuffer is greater than the maximum size\r
+  support by TokenNumber, then set SizeOfBuffer to the maximum size supported by\r
+  TokenNumber and return EFI_INVALID_PARAMETER to indicate that the set operation\r
+  was not actually performed.\r
+\r
+  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to the\r
+  maximum size supported by TokenName and EFI_INVALID_PARAMETER must be returned.\r
+\r
   If SizeOfBuffer is NULL, then ASSERT().\r
   If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
-  \r
+\r
   @param[in]      TokenNumber   The PCD token number to set a current value for.\r
   @param[in, out] SizeOfBuffer  The size, in bytes, of Buffer.\r
   @param[in]      Buffer        A pointer to the buffer to set.\r
 \r
-  @return Return the pointer for the buffer been set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-VOID *\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetPtr (\r
-  IN        UINTN             TokenNumber,\r
-  IN OUT    UINTN             *SizeOfBuffer,\r
-  IN CONST  VOID              *Buffer\r
+LibPcdSetPtrS (\r
+  IN       UINTN  TokenNumber,\r
+  IN OUT   UINTN  *SizeOfBuffer,\r
+  IN CONST VOID   *Buffer\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (SizeOfBuffer != NULL);\r
 \r
   if (*SizeOfBuffer > 0) {\r
     ASSERT (Buffer != NULL);\r
   }\r
 \r
-  Status = GetPcdProtocol()->SetPtr (TokenNumber, SizeOfBuffer, (VOID *) Buffer);\r
-  if (EFI_ERROR (Status)) {\r
-    return NULL;\r
-  }\r
-\r
-  return (VOID *)Buffer;\r
+  return GetPcdProtocol ()->SetPtr (TokenNumber, SizeOfBuffer, (VOID *)Buffer);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the Boolean value for the token specified by TokenNumber \r
-  to the value specified by Value.  Value is returned.\r
-  \r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The boolean value to set.\r
 \r
-  @return Return the value that was set.\r
+  Sets the boolean value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The boolean value to set.\r
+\r
+  @return The status of the set operation.\r
 \r
 **/\r
-BOOLEAN\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetBool (\r
-  IN UINTN             TokenNumber,\r
-  IN BOOLEAN           Value\r
+LibPcdSetBoolS (\r
+  IN UINTN    TokenNumber,\r
+  IN BOOLEAN  Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = GetPcdProtocol()->SetBool (TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPcdProtocol ()->SetBool (TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 8-bit value for the token specified by TokenNumber and \r
-  Guid to the value specified by Value. Value is returned.\r
-  \r
+\r
+  Sets the 8-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
   If Guid is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that \r
+\r
+  @param[in] Guid           The pointer to a 128-bit unique value that\r
                             designates which namespace to set a value from.\r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 8-bit value to set.\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 8-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT8\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetEx8 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber,\r
-  IN UINT8             Value\r
+LibPcdSetEx8S (\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber,\r
+  IN UINT8       Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->Set8 (Guid, TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPiPcdProtocol ()->Set8 (Guid, TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 16-bit value for the token specified by TokenNumber and \r
-  Guid to the value specified by Value. Value is returned.\r
-  \r
+\r
+  Sets the 16-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
   If Guid is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that \r
+\r
+  @param[in] Guid           The pointer to a 128-bit unique value that\r
                             designates which namespace to set a value from.\r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 16-bit value to set.\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 16-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT16\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetEx16 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber,\r
-  IN UINT16            Value\r
+LibPcdSetEx16S (\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber,\r
+  IN UINT16      Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->Set16 (Guid, TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPiPcdProtocol ()->Set16 (Guid, TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 32-bit value for the token specified by TokenNumber and \r
-  Guid to the value specified by Value. Value is returned.\r
-  \r
+\r
+  Sets the 32-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
   If Guid is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that \r
+\r
+  @param[in] Guid           The pointer to a 128-bit unique value that\r
                             designates which namespace to set a value from.\r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 32-bit value to set.\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 32-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT32\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetEx32 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber,\r
-  IN UINT32            Value\r
+LibPcdSetEx32S (\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber,\r
+  IN UINT32      Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->Set32 (Guid, TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPiPcdProtocol ()->Set32 (Guid, TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the 64-bit value for the token specified by TokenNumber and \r
-  Guid to the value specified by Value. Value is returned.\r
+\r
+  Sets the 64-bit value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
   If Guid is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that \r
+\r
+  @param[in] Guid           The pointer to a 128-bit unique value that\r
                             designates which namespace to set a value from.\r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The 64-bit value to set.\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The 64-bit value to set.\r
 \r
-  @return Return the value that was set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-UINT64\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetEx64 (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber,\r
-  IN UINT64            Value\r
+LibPcdSetEx64S (\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber,\r
+  IN UINT64      Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->Set64 (Guid, TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPiPcdProtocol ()->Set64 (Guid, TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets a buffer for the token specified by TokenNumber to the value specified by \r
-  Buffer and SizeOfBuffer.  Buffer is returned.  If SizeOfBuffer is greater than \r
-  the maximum size support by TokenNumber, then set SizeOfBuffer to the maximum size \r
-  supported by TokenNumber and return NULL to indicate that the set operation \r
-  was not actually performed. \r
-  \r
+\r
+  Sets a buffer for the token specified by TokenNumber to the value specified by\r
+  Buffer and SizeOfBuffer. If SizeOfBuffer is greater than the maximum size\r
+  support by TokenNumber, then set SizeOfBuffer to the maximum size supported by\r
+  TokenNumber and return EFI_INVALID_PARAMETER to indicate that the set operation\r
+  was not actually performed.\r
+\r
   If Guid is NULL, then ASSERT().\r
   If SizeOfBuffer is NULL, then ASSERT().\r
   If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid              The pointer to a 128-bit unique value that \r
+\r
+  @param[in]      Guid          Pointer to a 128-bit unique value that\r
                                 designates which namespace to set a value from.\r
-  @param[in]  TokenNumber       The PCD token number to set a current value for.\r
+  @param[in]      TokenNumber   The PCD token number to set a current value for.\r
   @param[in, out] SizeOfBuffer  The size, in bytes, of Buffer.\r
-  @param[in]  Buffer            A pointer to the buffer to set.\r
+  @param[in]      Buffer        A pointer to the buffer to set.\r
 \r
-  @return Return the pointer to the buffer been set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-VOID *\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetExPtr (\r
-  IN      CONST GUID        *Guid,\r
-  IN      UINTN             TokenNumber,\r
-  IN OUT  UINTN             *SizeOfBuffer,\r
-  IN      VOID              *Buffer\r
+LibPcdSetExPtrS (\r
+  IN CONST GUID   *Guid,\r
+  IN       UINTN  TokenNumber,\r
+  IN OUT   UINTN  *SizeOfBuffer,\r
+  IN       VOID   *Buffer\r
   )\r
 {\r
-  EFI_STATUS  Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
   ASSERT (SizeOfBuffer != NULL);\r
@@ -861,96 +754,80 @@ LibPcdSetExPtr (
     ASSERT (Buffer != NULL);\r
   }\r
 \r
-  Status = GetPiPcdProtocol()->SetPtr (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
-  if (EFI_ERROR (Status)) {\r
-    return NULL;\r
-  }\r
-\r
-  return Buffer;\r
+  return GetPiPcdProtocol ()->SetPtr (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
 }\r
 \r
-\r
-\r
 /**\r
   This function provides a means by which to set a value for a given PCD token.\r
-  \r
-  Sets the Boolean value for the token specified by TokenNumber and \r
-  Guid to the value specified by Value. Value is returned.\r
-  \r
+\r
+  Sets the boolean value for the token specified by TokenNumber\r
+  to the value specified by Value.\r
+\r
   If Guid is NULL, then ASSERT().\r
-  \r
-  @param[in]  Guid          The pointer to a 128-bit unique value that \r
+\r
+  @param[in] Guid           The pointer to a 128-bit unique value that\r
                             designates which namespace to set a value from.\r
-  @param[in]  TokenNumber   The PCD token number to set a current value for.\r
-  @param[in]  Value         The Boolean value to set.\r
+  @param[in] TokenNumber    The PCD token number to set a current value for.\r
+  @param[in] Value          The boolean value to set.\r
 \r
-  @return Return the value that was set.\r
+  @return The status of the set operation.\r
 \r
 **/\r
-BOOLEAN\r
+RETURN_STATUS\r
 EFIAPI\r
-LibPcdSetExBool (\r
-  IN CONST GUID        *Guid,\r
-  IN UINTN             TokenNumber,\r
-  IN BOOLEAN           Value\r
+LibPcdSetExBoolS (\r
+  IN CONST GUID  *Guid,\r
+  IN UINTN       TokenNumber,\r
+  IN BOOLEAN     Value\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->SetBool (Guid, TokenNumber, Value);\r
-  ASSERT_EFI_ERROR (Status);\r
-\r
-  return Value;\r
+  return GetPiPcdProtocol ()->SetBool (Guid, TokenNumber, Value);\r
 }\r
 \r
-\r
-\r
 /**\r
   Set up a notification function that is called when a specified token is set.\r
-  \r
-  When the token specified by TokenNumber and Guid is set, \r
-  then notification function specified by NotificationFunction is called.  \r
+\r
+  When the token specified by TokenNumber and Guid is set,\r
+  then notification function specified by NotificationFunction is called.\r
   If Guid is NULL, then the default token space is used.\r
   If NotificationFunction is NULL, then ASSERT().\r
 \r
-  @param[in]  Guid                  The pointer to a 128-bit unique value that designates which \r
-                                    namespace to set a value from.  If NULL, then the default \r
+  @param[in]  Guid                  The pointer to a 128-bit unique value that designates which\r
+                                    namespace to set a value from.  If NULL, then the default\r
                                     token space is used.\r
   @param[in]  TokenNumber           The PCD token number to monitor.\r
-  @param[in]  NotificationFunction  The function to call when the token \r
+  @param[in]  NotificationFunction  The function to call when the token\r
                                     specified by Guid and TokenNumber is set.\r
 \r
 **/\r
 VOID\r
 EFIAPI\r
 LibPcdCallbackOnSet (\r
-  IN CONST GUID               *Guid,       OPTIONAL\r
-  IN UINTN                    TokenNumber,\r
-  IN PCD_CALLBACK             NotificationFunction\r
+  IN CONST GUID    *Guid        OPTIONAL,\r
+  IN UINTN         TokenNumber,\r
+  IN PCD_CALLBACK  NotificationFunction\r
   )\r
 {\r
-  EFI_STATUS Status;\r
+  EFI_STATUS  Status;\r
 \r
   ASSERT (NotificationFunction != NULL);\r
 \r
-  Status = GetPiPcdProtocol()->CallbackOnSet (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK) NotificationFunction);\r
+  Status = GetPiPcdProtocol ()->CallbackOnSet (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK)NotificationFunction);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return;\r
 }\r
 \r
-\r
-\r
 /**\r
   Disable a notification function that was established with LibPcdCallbackonSet().\r
-  \r
+\r
   Disable a notification function that was previously established with LibPcdCallbackOnSet().\r
   If NotificationFunction is NULL, then ASSERT().\r
-  If LibPcdCallbackOnSet() was not previously called with Guid, TokenNumber, \r
+  If LibPcdCallbackOnSet() was not previously called with Guid, TokenNumber,\r
   and NotificationFunction, then ASSERT().\r
-  \r
+\r
   @param[in]  Guid                 Specify the GUID token space.\r
   @param[in]  TokenNumber          Specify the token number.\r
   @param[in]  NotificationFunction The callback function to be unregistered.\r
@@ -959,67 +836,63 @@ LibPcdCallbackOnSet (
 VOID\r
 EFIAPI\r
 LibPcdCancelCallback (\r
-  IN CONST GUID               *Guid,       OPTIONAL\r
-  IN UINTN                    TokenNumber,\r
-  IN PCD_CALLBACK             NotificationFunction\r
+  IN CONST GUID    *Guid        OPTIONAL,\r
+  IN UINTN         TokenNumber,\r
+  IN PCD_CALLBACK  NotificationFunction\r
   )\r
 {\r
-  EFI_STATUS Status;\r
+  EFI_STATUS  Status;\r
 \r
   ASSERT (NotificationFunction != NULL);\r
-    \r
-  Status = GetPiPcdProtocol()->CancelCallback (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK) NotificationFunction);\r
+\r
+  Status = GetPiPcdProtocol ()->CancelCallback (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK)NotificationFunction);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return;\r
 }\r
 \r
-\r
-\r
 /**\r
   Retrieves the next token in a token space.\r
-  \r
-  Retrieves the next PCD token number from the token space specified by Guid.  \r
-  If Guid is NULL, then the default token space is used.  If TokenNumber is 0, \r
-  then the first token number is returned.  Otherwise, the token number that \r
-  follows TokenNumber in the token space is returned.  If TokenNumber is the last \r
-  token number in the token space, then 0 is returned.  \r
-  \r
+\r
+  Retrieves the next PCD token number from the token space specified by Guid.\r
+  If Guid is NULL, then the default token space is used.  If TokenNumber is 0,\r
+  then the first token number is returned.  Otherwise, the token number that\r
+  follows TokenNumber in the token space is returned.  If TokenNumber is the last\r
+  token number in the token space, then 0 is returned.\r
+\r
   If TokenNumber is not 0 and is not in the token space specified by Guid, then ASSERT().\r
 \r
-  @param[in]  Guid        The pointer to a 128-bit unique value that designates which namespace \r
+  @param[in]  Guid        The pointer to a 128-bit unique value that designates which namespace\r
                           to set a value from.  If NULL, then the default token space is used.\r
-  @param[in]  TokenNumber The previous PCD token number.  If 0, then retrieves the first PCD \r
+  @param[in]  TokenNumber The previous PCD token number.  If 0, then retrieves the first PCD\r
                           token number.\r
 \r
   @return The next valid token number.\r
 \r
 **/\r
-UINTN           \r
+UINTN\r
 EFIAPI\r
 LibPcdGetNextToken (\r
-  IN CONST GUID               *Guid,       OPTIONAL\r
-  IN UINTN                    TokenNumber\r
+  IN CONST GUID  *Guid        OPTIONAL,\r
+  IN UINTN       TokenNumber\r
   )\r
 {\r
-  EFI_STATUS    Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status = GetPiPcdProtocol()->GetNextToken (Guid, &TokenNumber);\r
+  Status = GetPiPcdProtocol ()->GetNextToken (Guid, &TokenNumber);\r
   ASSERT (!EFI_ERROR (Status) || TokenNumber == 0);\r
 \r
   return TokenNumber;\r
 }\r
 \r
-\r
-\r
 /**\r
   Used to retrieve the list of available PCD token space GUIDs.\r
-  \r
+\r
   Returns the PCD token space GUID that follows TokenSpaceGuid in the list of token spaces\r
   in the platform.\r
   If TokenSpaceGuid is NULL, then a pointer to the first PCD token spaces returned.\r
   If TokenSpaceGuid is the last PCD token space GUID in the list, then NULL is returned.\r
-  \r
+\r
   @param  TokenSpaceGuid  The pointer to the a PCD token space GUID.\r
 \r
   @return The next valid token namespace.\r
@@ -1031,60 +904,223 @@ LibPcdGetNextTokenSpace (
   IN CONST GUID  *TokenSpaceGuid\r
   )\r
 {\r
-  GetPiPcdProtocol()->GetNextTokenSpace (&TokenSpaceGuid);\r
+  GetPiPcdProtocol ()->GetNextTokenSpace (&TokenSpaceGuid);\r
 \r
   return (GUID *)TokenSpaceGuid;\r
 }\r
 \r
-\r
 /**\r
   Sets a value of a patchable PCD entry that is type pointer.\r
-  \r
-  Sets the PCD entry specified by PatchVariable to the value specified by Buffer \r
-  and SizeOfBuffer.  Buffer is returned.  If SizeOfBuffer is greater than \r
-  MaximumDatumSize, then set SizeOfBuffer to MaximumDatumSize and return \r
-  NULL to indicate that the set operation was not actually performed.  \r
-  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to \r
+\r
+  Sets the PCD entry specified by PatchVariable to the value specified by Buffer\r
+  and SizeOfBuffer.  Buffer is returned.  If SizeOfBuffer is greater than\r
+  MaximumDatumSize, then set SizeOfBuffer to MaximumDatumSize and return\r
+  NULL to indicate that the set operation was not actually performed.\r
+  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to\r
   MaximumDatumSize and NULL must be returned.\r
-  \r
+\r
   If PatchVariable is NULL, then ASSERT().\r
   If SizeOfBuffer is NULL, then ASSERT().\r
   If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
 \r
-  @param[in] PatchVariable      A pointer to the global variable in a module that is \r
+  @param[out] PatchVariable     A pointer to the global variable in a module that is\r
                                 the target of the set operation.\r
   @param[in] MaximumDatumSize   The maximum size allowed for the PCD entry specified by PatchVariable.\r
   @param[in, out] SizeOfBuffer  A pointer to the size, in bytes, of Buffer.\r
   @param[in] Buffer             A pointer to the buffer to used to set the target variable.\r
-  \r
+\r
   @return Return the pointer to the buffer been set.\r
 \r
 **/\r
 VOID *\r
 EFIAPI\r
 LibPatchPcdSetPtr (\r
-  IN        VOID        *PatchVariable,\r
-  IN        UINTN       MaximumDatumSize,\r
-  IN OUT    UINTN       *SizeOfBuffer,\r
-  IN CONST  VOID        *Buffer\r
+  OUT       VOID   *PatchVariable,\r
+  IN        UINTN  MaximumDatumSize,\r
+  IN OUT    UINTN  *SizeOfBuffer,\r
+  IN CONST  VOID   *Buffer\r
   )\r
 {\r
   ASSERT (PatchVariable != NULL);\r
   ASSERT (SizeOfBuffer  != NULL);\r
-  \r
+\r
   if (*SizeOfBuffer > 0) {\r
     ASSERT (Buffer != NULL);\r
   }\r
 \r
   if ((*SizeOfBuffer > MaximumDatumSize) ||\r
-      (*SizeOfBuffer == MAX_ADDRESS)) {\r
+      (*SizeOfBuffer == MAX_ADDRESS))\r
+  {\r
     *SizeOfBuffer = MaximumDatumSize;\r
     return NULL;\r
   }\r
-    \r
+\r
+  CopyMem (PatchVariable, Buffer, *SizeOfBuffer);\r
+\r
+  return (VOID *)Buffer;\r
+}\r
+\r
+/**\r
+  Sets a value of a patchable PCD entry that is type pointer.\r
+\r
+  Sets the PCD entry specified by PatchVariable to the value specified\r
+  by Buffer and SizeOfBuffer. If SizeOfBuffer is greater than MaximumDatumSize,\r
+  then set SizeOfBuffer to MaximumDatumSize and return RETURN_INVALID_PARAMETER\r
+  to indicate that the set operation was not actually performed.\r
+  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to\r
+  MaximumDatumSize and RETURN_INVALID_PARAMETER must be returned.\r
+\r
+  If PatchVariable is NULL, then ASSERT().\r
+  If SizeOfBuffer is NULL, then ASSERT().\r
+  If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
+\r
+  @param[out] PatchVariable     A pointer to the global variable in a module that is\r
+                                the target of the set operation.\r
+  @param[in] MaximumDatumSize   The maximum size allowed for the PCD entry specified by PatchVariable.\r
+  @param[in, out] SizeOfBuffer  A pointer to the size, in bytes, of Buffer.\r
+  @param[in] Buffer             A pointer to the buffer to used to set the target variable.\r
+\r
+  @return The status of the set operation.\r
+\r
+**/\r
+RETURN_STATUS\r
+EFIAPI\r
+LibPatchPcdSetPtrS (\r
+  OUT       VOID  *PatchVariable,\r
+  IN       UINTN  MaximumDatumSize,\r
+  IN OUT   UINTN  *SizeOfBuffer,\r
+  IN CONST VOID   *Buffer\r
+  )\r
+{\r
+  ASSERT (PatchVariable != NULL);\r
+  ASSERT (SizeOfBuffer  != NULL);\r
+\r
+  if (*SizeOfBuffer > 0) {\r
+    ASSERT (Buffer != NULL);\r
+  }\r
+\r
+  if ((*SizeOfBuffer > MaximumDatumSize) ||\r
+      (*SizeOfBuffer == MAX_ADDRESS))\r
+  {\r
+    *SizeOfBuffer = MaximumDatumSize;\r
+    return RETURN_INVALID_PARAMETER;\r
+  }\r
+\r
   CopyMem (PatchVariable, Buffer, *SizeOfBuffer);\r
-  \r
-  return (VOID *) Buffer;\r
+\r
+  return RETURN_SUCCESS;\r
+}\r
+\r
+/**\r
+  Sets a value and size of a patchable PCD entry that is type pointer.\r
+\r
+  Sets the PCD entry specified by PatchVariable to the value specified by Buffer\r
+  and SizeOfBuffer.  Buffer is returned.  If SizeOfBuffer is greater than\r
+  MaximumDatumSize, then set SizeOfBuffer to MaximumDatumSize and return\r
+  NULL to indicate that the set operation was not actually performed.\r
+  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to\r
+  MaximumDatumSize and NULL must be returned.\r
+\r
+  If PatchVariable is NULL, then ASSERT().\r
+  If SizeOfPatchVariable is NULL, then ASSERT().\r
+  If SizeOfBuffer is NULL, then ASSERT().\r
+  If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
+\r
+  @param[out] PatchVariable     A pointer to the global variable in a module that is\r
+                                the target of the set operation.\r
+  @param[out] SizeOfPatchVariable A pointer to the size, in bytes, of PatchVariable.\r
+  @param[in] MaximumDatumSize   The maximum size allowed for the PCD entry specified by PatchVariable.\r
+  @param[in, out] SizeOfBuffer  A pointer to the size, in bytes, of Buffer.\r
+  @param[in] Buffer             A pointer to the buffer to used to set the target variable.\r
+\r
+  @return Return the pointer to the buffer been set.\r
+\r
+**/\r
+VOID *\r
+EFIAPI\r
+LibPatchPcdSetPtrAndSize (\r
+  OUT       VOID   *PatchVariable,\r
+  OUT       UINTN  *SizeOfPatchVariable,\r
+  IN        UINTN  MaximumDatumSize,\r
+  IN OUT    UINTN  *SizeOfBuffer,\r
+  IN CONST  VOID   *Buffer\r
+  )\r
+{\r
+  ASSERT (PatchVariable != NULL);\r
+  ASSERT (SizeOfPatchVariable != NULL);\r
+  ASSERT (SizeOfBuffer  != NULL);\r
+\r
+  if (*SizeOfBuffer > 0) {\r
+    ASSERT (Buffer != NULL);\r
+  }\r
+\r
+  if ((*SizeOfBuffer > MaximumDatumSize) ||\r
+      (*SizeOfBuffer == MAX_ADDRESS))\r
+  {\r
+    *SizeOfBuffer = MaximumDatumSize;\r
+    return NULL;\r
+  }\r
+\r
+  CopyMem (PatchVariable, Buffer, *SizeOfBuffer);\r
+  *SizeOfPatchVariable = *SizeOfBuffer;\r
+\r
+  return (VOID *)Buffer;\r
+}\r
+\r
+/**\r
+  Sets a value and size of a patchable PCD entry that is type pointer.\r
+\r
+  Sets the PCD entry specified by PatchVariable to the value specified\r
+  by Buffer and SizeOfBuffer. If SizeOfBuffer is greater than MaximumDatumSize,\r
+  then set SizeOfBuffer to MaximumDatumSize and return RETURN_INVALID_PARAMETER\r
+  to indicate that the set operation was not actually performed.\r
+  If SizeOfBuffer is set to MAX_ADDRESS, then SizeOfBuffer must be set to\r
+  MaximumDatumSize and RETURN_INVALID_PARAMETER must be returned.\r
+\r
+  If PatchVariable is NULL, then ASSERT().\r
+  If SizeOfPatchVariable is NULL, then ASSERT().\r
+  If SizeOfBuffer is NULL, then ASSERT().\r
+  If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
+\r
+  @param[out] PatchVariable     A pointer to the global variable in a module that is\r
+                                the target of the set operation.\r
+  @param[out] SizeOfPatchVariable A pointer to the size, in bytes, of PatchVariable.\r
+  @param[in] MaximumDatumSize   The maximum size allowed for the PCD entry specified by PatchVariable.\r
+  @param[in, out] SizeOfBuffer  A pointer to the size, in bytes, of Buffer.\r
+  @param[in] Buffer             A pointer to the buffer to used to set the target variable.\r
+\r
+  @return The status of the set operation.\r
+\r
+**/\r
+RETURN_STATUS\r
+EFIAPI\r
+LibPatchPcdSetPtrAndSizeS (\r
+  OUT      VOID   *PatchVariable,\r
+  OUT      UINTN  *SizeOfPatchVariable,\r
+  IN       UINTN  MaximumDatumSize,\r
+  IN OUT   UINTN  *SizeOfBuffer,\r
+  IN CONST VOID   *Buffer\r
+  )\r
+{\r
+  ASSERT (PatchVariable != NULL);\r
+  ASSERT (SizeOfPatchVariable != NULL);\r
+  ASSERT (SizeOfBuffer  != NULL);\r
+\r
+  if (*SizeOfBuffer > 0) {\r
+    ASSERT (Buffer != NULL);\r
+  }\r
+\r
+  if ((*SizeOfBuffer > MaximumDatumSize) ||\r
+      (*SizeOfBuffer == MAX_ADDRESS))\r
+  {\r
+    *SizeOfBuffer = MaximumDatumSize;\r
+    return RETURN_INVALID_PARAMETER;\r
+  }\r
+\r
+  CopyMem (PatchVariable, Buffer, *SizeOfBuffer);\r
+  *SizeOfPatchVariable = *SizeOfBuffer;\r
+\r
+  return RETURN_SUCCESS;\r
 }\r
 \r
 /**\r
@@ -1102,13 +1138,13 @@ LibPatchPcdSetPtr (
 VOID\r
 EFIAPI\r
 LibPcdGetInfo (\r
-  IN        UINTN           TokenNumber,\r
-  OUT       PCD_INFO        *PcdInfo\r
+  IN        UINTN     TokenNumber,\r
+  OUT       PCD_INFO  *PcdInfo\r
   )\r
 {\r
-  EFI_STATUS Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status = GetPcdInfoProtocolPointer()->GetInfo (TokenNumber, (EFI_PCD_INFO *) PcdInfo);\r
+  Status = GetPcdInfoProtocolPointer ()->GetInfo (TokenNumber, (EFI_PCD_INFO *)PcdInfo);\r
   ASSERT_EFI_ERROR (Status);\r
 }\r
 \r
@@ -1128,22 +1164,20 @@ LibPcdGetInfo (
 VOID\r
 EFIAPI\r
 LibPcdGetInfoEx (\r
-  IN CONST  GUID            *Guid,\r
-  IN        UINTN           TokenNumber,\r
-  OUT       PCD_INFO        *PcdInfo\r
+  IN CONST  GUID      *Guid,\r
+  IN        UINTN     TokenNumber,\r
+  OUT       PCD_INFO  *PcdInfo\r
   )\r
 {\r
-  EFI_STATUS Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status = GetPiPcdInfoProtocolPointer()->GetInfo (Guid, TokenNumber, (EFI_PCD_INFO *) PcdInfo);\r
+  Status = GetPiPcdInfoProtocolPointer ()->GetInfo (Guid, TokenNumber, (EFI_PCD_INFO *)PcdInfo);\r
   ASSERT_EFI_ERROR (Status);\r
 }\r
 \r
 /**\r
   Retrieve the currently set SKU Id.\r
 \r
-  If the sku id got >= PCD_MAX_SKU_ID, then ASSERT().\r
-\r
   @return   The currently set SKU Id. If the platform has not set at a SKU Id, then the\r
             default SKU Id value of 0 is returned. If the platform has set a SKU Id, then the currently set SKU\r
             Id is returned.\r
@@ -1154,11 +1188,5 @@ LibPcdGetSku (
   VOID\r
   )\r
 {\r
-  UINTN SkuId;\r
-\r
-  SkuId = GetPiPcdInfoProtocolPointer()->GetSku ();\r
-  ASSERT (SkuId < PCD_MAX_SKU_ID);\r
-\r
-  return SkuId;\r
+  return GetPiPcdInfoProtocolPointer ()->GetSku ();\r
 }\r
-\r