]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/DxePcdLib/DxePcdLib.c
MdePkg and MdeModulePkg Pcd: Implement PCD Driver for External PCD Database and SKU...
[mirror_edk2.git] / MdePkg / Library / DxePcdLib / DxePcdLib.c
index 276998fcd48107cf0e8cdf8e0dd9e7c30c59430f..cd8212824fc74d773c2107221fd7ac87f8168dd3 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
 Implementation of PcdLib class library for DXE phase.\r
 \r
-Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>\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
@@ -24,43 +24,53 @@ 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
+PCD_PROTOCOL     *mPcd   = NULL;\r
 EFI_PCD_PROTOCOL *mPiPcd = NULL;\r
 \r
 /**\r
-  The constructor function caches the PCD_PROTOCOL pointer.\r
+  Retrieves the PI PCD protocol from the handle database.\r
+**/\r
+EFI_PCD_PROTOCOL *\r
+EFIAPI\r
+GetPiPcdProtocol (\r
+  VOID\r
+  )\r
+{\r
+  EFI_STATUS  Status;\r
 \r
-  @param[in] ImageHandle The firmware allocated handle for the EFI image.  \r
-  @param[in] SystemTable A pointer to the EFI System Table.\r
-  \r
-  @retval EFI_SUCCESS The constructor always return EFI_SUCCESS.\r
+  if (mPiPcd == NULL) {\r
+    //\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
+    ASSERT_EFI_ERROR (Status);\r
+    ASSERT (mPiPcd != NULL);\r
+  }\r
+  return mPiPcd;\r
+}\r
 \r
+/**\r
+  Retrieves the PCD protocol from the handle database.\r
 **/\r
-EFI_STATUS\r
+PCD_PROTOCOL *\r
 EFIAPI\r
-PcdLibConstructor (\r
-  IN EFI_HANDLE        ImageHandle,\r
-  IN EFI_SYSTEM_TABLE  *SystemTable\r
+GetPcdProtocol (\r
+  VOID\r
   )\r
 {\r
   EFI_STATUS  Status;\r
 \r
-  //\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
-  gBS->LocateProtocol (&gPcdProtocolGuid, NULL, (VOID **)&mPcd);\r
-  \r
-  //\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
-  \r
-  ASSERT_EFI_ERROR (Status);\r
-  ASSERT (mPiPcd!= NULL);\r
-\r
-  return Status;\r
+  if (mPcd == NULL) {\r
+    //\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
+    Status = gBS->LocateProtocol (&gPcdProtocolGuid, NULL, (VOID **)&mPcd);\r
+    ASSERT_EFI_ERROR (Status);\r
+    ASSERT (mPcd != NULL);\r
+  }\r
+  return mPcd;\r
 }\r
 \r
 \r
@@ -82,10 +92,9 @@ LibPcdSetSku (
   IN UINTN   SkuId\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
   ASSERT (SkuId < PCD_MAX_SKU_ID);\r
 \r
-  mPcd->SetSku (SkuId);\r
+  GetPcdProtocol()->SetSku (SkuId);\r
 \r
   return SkuId;\r
 }\r
@@ -108,8 +117,7 @@ LibPcdGet8 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->Get8 (TokenNumber);\r
+  return GetPcdProtocol()->Get8 (TokenNumber);\r
 }\r
 \r
 \r
@@ -130,8 +138,7 @@ LibPcdGet16 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->Get16 (TokenNumber);\r
+  return GetPcdProtocol()->Get16 (TokenNumber);\r
 }\r
 \r
 \r
@@ -152,8 +159,7 @@ LibPcdGet32 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->Get32 (TokenNumber);\r
+  return GetPcdProtocol()->Get32 (TokenNumber);\r
 }\r
 \r
 \r
@@ -174,8 +180,7 @@ LibPcdGet64 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->Get64 (TokenNumber);\r
+  return GetPcdProtocol()->Get64 (TokenNumber);\r
 }\r
 \r
 \r
@@ -196,8 +201,7 @@ LibPcdGetPtr (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->GetPtr (TokenNumber);\r
+  return GetPcdProtocol()->GetPtr (TokenNumber);\r
 }\r
 \r
 \r
@@ -218,8 +222,7 @@ LibPcdGetBool (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->GetBool (TokenNumber);\r
+  return GetPcdProtocol()->GetBool (TokenNumber);\r
 }\r
 \r
 \r
@@ -238,8 +241,7 @@ LibPcdGetSize (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  ASSERT (mPcd != NULL);\r
-  return mPcd->GetSize (TokenNumber);\r
+  return GetPcdProtocol()->GetSize (TokenNumber);\r
 }\r
 \r
 \r
@@ -251,7 +253,7 @@ LibPcdGetSize (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid         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
@@ -267,7 +269,7 @@ LibPcdGetEx8 (
 {\r
   ASSERT (Guid != NULL);\r
   \r
-  return mPiPcd->Get8 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->Get8 (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -278,7 +280,7 @@ LibPcdGetEx8 (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid         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
@@ -294,7 +296,7 @@ LibPcdGetEx16 (
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return mPiPcd->Get16 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->Get16 (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -302,7 +304,7 @@ LibPcdGetEx16 (
   Returns the 32-bit value for the token specified by TokenNumber and Guid.\r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid         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
@@ -318,7 +320,7 @@ LibPcdGetEx32 (
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return mPiPcd->Get32 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->Get32 (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -330,7 +332,7 @@ LibPcdGetEx32 (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          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
@@ -346,7 +348,7 @@ LibPcdGetEx64 (
 {\r
   ASSERT (Guid != NULL);\r
   \r
-  return mPiPcd->Get64 (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->Get64 (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -358,7 +360,7 @@ LibPcdGetEx64 (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          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
@@ -374,7 +376,7 @@ LibPcdGetExPtr (
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return mPiPcd->GetPtr (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->GetPtr (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -386,7 +388,7 @@ LibPcdGetExPtr (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          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
@@ -402,7 +404,7 @@ LibPcdGetExBool (
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return mPiPcd->GetBool (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->GetBool (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -414,7 +416,7 @@ LibPcdGetExBool (
   \r
   If Guid is NULL, then ASSERT(). \r
 \r
-  @param[in]  Guid          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
@@ -430,7 +432,7 @@ LibPcdGetExSize (
 {\r
   ASSERT (Guid != NULL);\r
 \r
-  return mPiPcd->GetSize (Guid, TokenNumber);\r
+  return GetPiPcdProtocol()->GetSize (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -444,7 +446,7 @@ LibPcdGetExSize (
   @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 been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT8\r
@@ -456,9 +458,7 @@ LibPcdSet8 (
 {\r
   EFI_STATUS Status;\r
 \r
-  ASSERT (mPcd != NULL);\r
-  Status = mPcd->Set8 (TokenNumber, Value);\r
-\r
+  Status = GetPcdProtocol()->Set8 (TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
   \r
   return Value;\r
@@ -475,7 +475,7 @@ LibPcdSet8 (
   @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 been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT16\r
@@ -487,9 +487,7 @@ LibPcdSet16 (
 {\r
   EFI_STATUS Status;\r
 \r
-  ASSERT (mPcd != NULL);\r
-  Status = mPcd->Set16 (TokenNumber, Value);\r
-\r
+  Status = GetPcdProtocol()->Set16 (TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
   \r
   return Value;\r
@@ -506,7 +504,7 @@ LibPcdSet16 (
   @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 been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT32\r
@@ -518,9 +516,7 @@ LibPcdSet32 (
 {\r
   EFI_STATUS Status;\r
   \r
-  ASSERT (mPcd != NULL);\r
-  Status = mPcd->Set32 (TokenNumber, Value);\r
-\r
+  Status = GetPcdProtocol()->Set32 (TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -537,7 +533,7 @@ LibPcdSet32 (
   @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 been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT64\r
@@ -549,9 +545,7 @@ LibPcdSet64 (
 {\r
   EFI_STATUS Status;\r
 \r
-  ASSERT (mPcd != NULL);\r
-  Status = mPcd->Set64 (TokenNumber, Value);\r
-\r
+  Status = GetPcdProtocol()->Set64 (TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -591,20 +585,18 @@ LibPcdSetPtr (
 {\r
   EFI_STATUS Status;\r
 \r
-  ASSERT (mPcd != NULL);\r
   ASSERT (SizeOfBuffer != NULL);\r
 \r
   if (*SizeOfBuffer > 0) {\r
     ASSERT (Buffer != NULL);\r
   }\r
 \r
-  Status = mPcd->SetPtr (TokenNumber, SizeOfBuffer, (VOID *) Buffer);\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 (VOID *)Buffer;\r
 }\r
 \r
 \r
@@ -618,7 +610,7 @@ LibPcdSetPtr (
   @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 been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 BOOLEAN\r
@@ -630,9 +622,7 @@ LibPcdSetBool (
 {\r
   EFI_STATUS Status;\r
 \r
-  ASSERT (mPcd != NULL);\r
-  Status = mPcd->SetBool (TokenNumber, Value);\r
-\r
+  Status = GetPcdProtocol()->SetBool (TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -648,12 +638,12 @@ LibPcdSetBool (
   \r
   If Guid is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid          Pointer to a 128-bit unique value that \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
 \r
-  @return Return the value been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT8\r
@@ -668,8 +658,7 @@ LibPcdSetEx8 (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = mPiPcd->Set8 (Guid, TokenNumber, Value);\r
-\r
+  Status = GetPiPcdProtocol()->Set8 (Guid, TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -685,12 +674,12 @@ LibPcdSetEx8 (
   \r
   If Guid is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid          Pointer to a 128-bit unique value that \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
 \r
-  @return Return the value been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT16\r
@@ -705,8 +694,7 @@ LibPcdSetEx16 (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = mPiPcd->Set16 (Guid, TokenNumber, Value);\r
-\r
+  Status = GetPiPcdProtocol()->Set16 (Guid, TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -722,12 +710,12 @@ LibPcdSetEx16 (
   \r
   If Guid is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid          Pointer to a 128-bit unique value that \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
 \r
-  @return Return the value been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT32\r
@@ -742,8 +730,7 @@ LibPcdSetEx32 (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = mPiPcd->Set32 (Guid, TokenNumber, Value);\r
-\r
+  Status = GetPiPcdProtocol()->Set32 (Guid, TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -758,12 +745,12 @@ LibPcdSetEx32 (
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid          Pointer to a 128-bit unique value that \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
 \r
-  @return Return the value been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 UINT64\r
@@ -778,8 +765,7 @@ LibPcdSetEx64 (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = mPiPcd->Set64 (Guid, TokenNumber, Value);\r
-\r
+  Status = GetPiPcdProtocol()->Set64 (Guid, TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -800,13 +786,13 @@ LibPcdSetEx64 (
   If SizeOfBuffer is NULL, then ASSERT().\r
   If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid              Pointer to a 128-bit unique value that \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, out] SizeOfBuffer  The size, in bytes, of Buffer.\r
   @param[in]  Buffer            A pointer to the buffer to set.\r
 \r
-  @return Return the pinter to the buffer been set.\r
+  @return Return the pointer to the buffer been set.\r
 \r
 **/\r
 VOID *\r
@@ -828,8 +814,7 @@ LibPcdSetExPtr (
     ASSERT (Buffer != NULL);\r
   }\r
 \r
-  Status = mPiPcd->SetPtr (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
-\r
+  Status = GetPiPcdProtocol()->SetPtr (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
   if (EFI_ERROR (Status)) {\r
     return NULL;\r
   }\r
@@ -847,12 +832,12 @@ LibPcdSetExPtr (
   \r
   If Guid is NULL, then ASSERT().\r
   \r
-  @param[in]  Guid          Pointer to a 128-bit unique value that \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
 \r
-  @return Return the value been set.\r
+  @return Return the value that was set.\r
 \r
 **/\r
 BOOLEAN\r
@@ -867,8 +852,7 @@ LibPcdSetExBool (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = mPiPcd->SetBool (Guid, TokenNumber, Value);\r
-\r
+  Status = GetPiPcdProtocol()->SetBool (Guid, TokenNumber, Value);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return Value;\r
@@ -884,7 +868,7 @@ LibPcdSetExBool (
   If Guid is NULL, then the default token space is used.\r
   If NotificationFunction is NULL, then ASSERT().\r
 \r
-  @param[in]  Guid                  Pointer to a 128-bit unique value that designates which \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
@@ -904,8 +888,7 @@ LibPcdCallbackOnSet (
 \r
   ASSERT (NotificationFunction != NULL);\r
 \r
-  Status = mPiPcd->CallbackOnSet (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK) NotificationFunction);\r
-\r
+  Status = GetPiPcdProtocol()->CallbackOnSet (Guid, TokenNumber, (EFI_PCD_PROTOCOL_CALLBACK) NotificationFunction);\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   return;\r
@@ -938,8 +921,7 @@ LibPcdCancelCallback (
 \r
   ASSERT (NotificationFunction != NULL);\r
     \r
-  Status = mPiPcd->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
@@ -958,7 +940,7 @@ LibPcdCancelCallback (
   \r
   If TokenNumber is not 0 and is not in the token space specified by Guid, then ASSERT().\r
 \r
-  @param[in]  Guid        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
                           token number.\r
@@ -973,11 +955,7 @@ LibPcdGetNextToken (
   IN UINTN                    TokenNumber\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = mPiPcd->GetNextToken (Guid, &TokenNumber);\r
-\r
-  ASSERT_EFI_ERROR (Status);\r
+  GetPiPcdProtocol()->GetNextToken (Guid, &TokenNumber);\r
 \r
   return TokenNumber;\r
 }\r
@@ -992,7 +970,7 @@ LibPcdGetNextToken (
   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
-  @param  TokenSpaceGuid  Pointer to the a PCD token space GUID\r
+  @param  TokenSpaceGuid  The pointer to the a PCD token space GUID.\r
 \r
   @return The next valid token namespace.\r
 \r
@@ -1003,13 +981,9 @@ LibPcdGetNextTokenSpace (
   IN CONST GUID  *TokenSpaceGuid\r
   )\r
 {\r
-  EFI_STATUS Status;\r
-\r
-  Status = mPiPcd->GetNextTokenSpace (&TokenSpaceGuid);\r
-\r
-  ASSERT_EFI_ERROR (Status);\r
+  GetPiPcdProtocol()->GetNextTokenSpace (&TokenSpaceGuid);\r
 \r
-  return (GUID *) TokenSpaceGuid;\r
+  return (GUID *)TokenSpaceGuid;\r
 }\r
 \r
 \r