]> git.proxmox.com Git - mirror_edk2.git/commitdiff
refine code.
authorklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Mon, 18 Aug 2008 05:43:42 +0000 (05:43 +0000)
committerklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Mon, 18 Aug 2008 05:43:42 +0000 (05:43 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5656 6f19259b-4bc3-4df7-8a09-765794883524

MdePkg/Library/PeiPcdLib/PeiPcdLib.c
MdePkg/Library/PeiPcdLib/PeiPcdLib.msa [deleted file]

index 1695ca561d0fd76d887e12d3e76771d9c73a3d70..03ef1d7f261c4cb1f4ee5981df955df820853ee0 100644 (file)
@@ -27,13 +27,16 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include <Library/BaseMemoryLib.h>\r
 \r
 /**\r
-  The constructor function retrieve the PCD_PPI pointer.\r
+  Retrieve the PCD_PPI pointer.\r
 \r
+  This function is to locate PCD_PPI PPI via PeiService. \r
+  If fail to locate PCD_PPI, then ASSERT_EFI_ERROR().\r
+  \r
   @retval PCD_PPI * The pointer to the PCD_PPI.\r
 \r
 **/\r
 PCD_PPI  *\r
-GetPcdPpiPtr (\r
+GetPcdPpiPointer (\r
   VOID\r
   ) \r
 {\r
@@ -48,9 +51,10 @@ GetPcdPpiPtr (
 \r
 /**\r
   Sets the current SKU in the PCD database to the value specified by SkuId.  SkuId is returned.\r
-\r
-  @param[in]  SkuId The SKU value that will be used when the PCD service will retrieve and \r
-              set values associated with a PCD token.\r
+  If SkuId not less than PCD_MAX_SKU_ID, then ASSERT().\r
+  \r
+  @param[in]  System sku id. The SKU value that will be used when the PCD service will retrieve and \r
+              set values.\r
 \r
   @retval SKU_ID Return the SKU ID that just be set.\r
 \r
@@ -62,9 +66,9 @@ LibPcdSetSku (
   )\r
 {\r
 \r
-  ASSERT (SkuId < 0x100);\r
+  ASSERT (SkuId < PCD_MAX_SKU_ID);\r
 \r
-  GetPcdPpiPtr()->SetSku (SkuId);;\r
+  GetPcdPpiPointer()->SetSku (SkuId);\r
 \r
   return SkuId;\r
 }\r
@@ -85,11 +89,7 @@ LibPcdGet8 (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get8 (TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get8 (TokenNumber);\r
 }\r
 \r
 \r
@@ -108,11 +108,7 @@ LibPcdGet16 (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI  *PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get16 (TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get16 (TokenNumber);\r
 }\r
 \r
 \r
@@ -131,11 +127,7 @@ LibPcdGet32 (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get32 (TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get32 (TokenNumber);\r
 }\r
 \r
 \r
@@ -154,11 +146,7 @@ LibPcdGet64 (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get64 (TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get64 (TokenNumber);\r
 }\r
 \r
 \r
@@ -177,11 +165,7 @@ LibPcdGetPtr (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetPtr (TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetPtr (TokenNumber);\r
 }\r
 \r
 \r
@@ -200,11 +184,7 @@ LibPcdGetBool (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetBool (TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetBool (TokenNumber);\r
 }\r
 \r
 \r
@@ -223,11 +203,7 @@ LibPcdGetSize (
   IN UINTN                        TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetSize (TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetSize (TokenNumber);\r
 }\r
 \r
 \r
@@ -250,13 +226,9 @@ LibPcdGetEx8 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get8Ex (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get8Ex (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -279,13 +251,10 @@ LibPcdGetEx16 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
 \r
   ASSERT (Guid != NULL);\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get16Ex (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get16Ex (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -308,13 +277,9 @@ LibPcdGetEx32 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get32Ex (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get32Ex (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -338,13 +303,8 @@ LibPcdGetEx64 (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->Get64Ex (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->Get64Ex (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -367,13 +327,9 @@ LibPcdGetExPtr (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetPtrEx (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetPtrEx (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -396,13 +352,8 @@ LibPcdGetExBool (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetBoolEx (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetBoolEx (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -425,13 +376,8 @@ LibPcdGetExSize (
   IN UINTN             TokenNumber\r
   )\r
 {\r
-  PCD_PPI * PcdPpi;\r
-\r
   ASSERT (Guid != NULL);\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  return PcdPpi->GetSizeEx (Guid, TokenNumber);\r
+  return (GetPcdPpiPointer ())->GetSizeEx (Guid, TokenNumber);\r
 }\r
 \r
 \r
@@ -439,6 +385,7 @@ LibPcdGetExSize (
 /**\r
   Sets the 8-bit value for the token specified by TokenNumber \r
   to the value specified by Value.  Value is returned.\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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
@@ -454,11 +401,8 @@ LibPcdSet8 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
-  Status = PcdPpi->Set8 (TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set8 (TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
   \r
@@ -470,6 +414,7 @@ LibPcdSet8 (
 /**\r
   Sets the 16-bit value for the token specified by TokenNumber \r
   to the value specified by Value.  Value is returned.\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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
@@ -485,11 +430,8 @@ LibPcdSet16 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  Status = PcdPpi->Set16 (TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set16 (TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
   \r
@@ -501,6 +443,7 @@ LibPcdSet16 (
 /**\r
   Sets the 32-bit value for the token specified by TokenNumber \r
   to the value specified by Value.  Value is returned.\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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
@@ -516,11 +459,8 @@ LibPcdSet32 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
-  Status = PcdPpi->Set32 (TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set32 (TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -532,6 +472,7 @@ LibPcdSet32 (
 /**\r
   Sets the 64-bit value for the token specified by TokenNumber \r
   to the value specified by Value.  Value is returned.\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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
@@ -547,11 +488,8 @@ LibPcdSet64 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  Status = PcdPpi->Set64 (TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set64 (TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -562,20 +500,21 @@ LibPcdSet64 (
 \r
 /**\r
   Sets a buffer for the token specified by TokenNumber to \r
-  the value specified by Buffer and SizeOfValue.  Buffer to\r
+  the value specified by Buffer and SizeOfBuffer.  Buffer to\r
   be set is returned. The content of the buffer could be \r
   overwritten if a Callback on SET is registered with this\r
   TokenNumber.\r
   \r
-  If SizeOfValue is greater than the maximum \r
-  size support by TokenNumber, then set SizeOfValue to the \r
+  If SizeOfBuffer is greater than the maximum \r
+  size support by TokenNumber, then set SizeOfBuffer to the \r
   maximum size supported by TokenNumber and return NULL to \r
   indicate that the set operation was not actually performed. \r
   \r
-  If SizeOfValue > 0 and Buffer is NULL, then ASSERT().\r
+  If SizeOfBuffer > 0 and Buffer is NULL, then ASSERT().\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
+                                    In out, returns actual size of buffer is set.\r
   @param[in]        Buffer          A pointer to the buffer to set.\r
 \r
   @retval VOID* Return the pointer for the buffer been set.\r
@@ -590,17 +529,14 @@ LibPcdSetPtr (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI    *PcdPpi;\r
 \r
   ASSERT (SizeOfBuffer != NULL);\r
 \r
   if (*SizeOfBuffer > 0) {\r
     ASSERT (Buffer != NULL);\r
   }\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
   \r
-  Status = PcdPpi->SetPtr (TokenNumber, SizeOfBuffer, Buffer);\r
+  Status = (GetPcdPpiPointer ())->SetPtr (TokenNumber, SizeOfBuffer, Buffer);\r
 \r
   if (EFI_ERROR (Status)) {\r
     return NULL;\r
@@ -614,6 +550,7 @@ LibPcdSetPtr (
 /**\r
   Sets the Boolean value for the token specified by TokenNumber \r
   to the value specified by Value.  Value is returned.\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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
@@ -629,11 +566,8 @@ LibPcdSetBool (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
 \r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
-  Status = PcdPpi->SetBool (TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->SetBool (TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -646,6 +580,7 @@ LibPcdSetBool (
   Sets the 8-bit value for the token specified by TokenNumber and \r
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\r
   \r
   @param[in]  Guid Pointer to a 128-bit unique value that \r
               designates which namespace to set a value from.\r
@@ -664,13 +599,10 @@ LibPcdSetEx8 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
   ASSERT (Guid != NULL);\r
 \r
-  Status = PcdPpi->Set8Ex (Guid, TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set8Ex (Guid, TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -683,13 +615,14 @@ LibPcdSetEx8 (
   Sets the 16-bit value for the token specified by TokenNumber and \r
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\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]  Value The 16-bit value to set.\r
 \r
-  @retval UINT8 Return the value been set.\r
+  @retval UINT16 Return the value been set.\r
 \r
 **/\r
 UINT16\r
@@ -701,12 +634,8 @@ LibPcdSetEx16 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
   ASSERT (Guid != NULL);\r
-  Status = PcdPpi->Set16Ex (Guid, TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set16Ex (Guid, TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -719,6 +648,7 @@ LibPcdSetEx16 (
   Sets the 32-bit value for the token specified by TokenNumber and \r
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\r
   \r
   @param[in]  Guid Pointer to a 128-bit unique value that \r
               designates which namespace to set a value from.\r
@@ -737,12 +667,10 @@ LibPcdSetEx32 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-  ASSERT (Guid != NULL);\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
+  ASSERT (Guid != NULL);\r
 \r
-  Status = PcdPpi->Set32Ex (Guid, TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set32Ex (Guid, TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -755,6 +683,7 @@ LibPcdSetEx32 (
   Sets the 64-bit value for the token specified by TokenNumber and \r
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\r
   \r
   @param[in]  Guid Pointer to a 128-bit unique value that \r
               designates which namespace to set a value from.\r
@@ -773,12 +702,9 @@ LibPcdSetEx64 (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
   ASSERT (Guid != NULL);\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
-\r
-  Status = PcdPpi->Set64Ex (Guid, TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->Set64Ex (Guid, TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -789,17 +715,18 @@ LibPcdSetEx64 (
 \r
 /**\r
   Sets a buffer for the token specified by TokenNumber to the value specified by \r
-  Buffer and SizeOfValue.  Buffer is returned.  If SizeOfValue is greater than \r
-  the maximum size support by TokenNumber, then set SizeOfValue to the maximum size \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
-  If SizeOfValue > 0 and Buffer 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
               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
+                  In out, returns actual size of buffer is set.\r
   @param[in]  Buffer A pointer to the buffer to set.\r
 \r
   @retval VOID * Return the pinter to the buffer been set.\r
@@ -815,15 +742,13 @@ LibPcdSetExPtr (
   )\r
 {\r
   EFI_STATUS      Status;\r
-  PCD_PPI         *PcdPpi;\r
- ASSERT (SizeOfBuffer != NULL);\r
+  ASSERT (SizeOfBuffer != NULL);\r
   if (*SizeOfBuffer > 0) {\r
     ASSERT (Buffer != NULL);\r
   }\r
   ASSERT (Guid != NULL);\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
-  Status = PcdPpi->SetPtrEx (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
+  Status = (GetPcdPpiPointer ())->SetPtrEx (Guid, TokenNumber, SizeOfBuffer, Buffer);\r
 \r
   if (EFI_ERROR (Status)) {\r
     return NULL;\r
@@ -838,6 +763,7 @@ LibPcdSetExPtr (
   Sets the Boolean value for the token specified by TokenNumber and \r
   Guid to the value specified by Value. Value is returned.\r
   If Guid is NULL, then ASSERT().\r
+  If fail to set pcd value, then ASSERT_EFI_ERROR().\r
   \r
   @param[in]  Guid Pointer to a 128-bit unique value that \r
               designates which namespace to set a value from.\r
@@ -856,12 +782,9 @@ LibPcdSetExBool (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
 \r
   ASSERT (Guid != NULL);\r
-  Status = PcdPpi->SetBoolEx (Guid, TokenNumber, Value);\r
+  Status = (GetPcdPpiPointer ())->SetBoolEx (Guid, TokenNumber, Value);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -875,7 +798,7 @@ LibPcdSetExBool (
   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
+  If fail to set callback, then ASSERT_EFI_ERROR().\r
   @param[in]  Guid 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
@@ -895,12 +818,8 @@ LibPcdCallbackOnSet (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
 \r
-  Status = PcdPpi->CallbackOnSet (Guid, TokenNumber, NotificationFunction);\r
+  Status = (GetPcdPpiPointer ())->CallbackOnSet (Guid, TokenNumber, NotificationFunction);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -912,7 +831,8 @@ LibPcdCallbackOnSet (
 /**\r
   Disable a notification function that was established with LibPcdCallbackonSet().\r
   If NotificationFunction is NULL, then ASSERT().\r
-\r
+  If fail to cancel callback, then ASSERT_EFI_ERROR().\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
@@ -929,12 +849,8 @@ LibPcdCancelCallback (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
 \r
-  Status = PcdPpi->CancelCallback (Guid, TokenNumber, NotificationFunction);\r
+  Status = (GetPcdPpiPointer ())->CancelCallback (Guid, TokenNumber, NotificationFunction);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -950,7 +866,8 @@ LibPcdCancelCallback (
   follows TokenNumber in the token space is returned.  If TokenNumber is the last \r
   token number in the token space, then 0 is returned.  If TokenNumber is not 0 and \r
   is not in the token space specified by Guid, then ASSERT().\r
-\r
+  If fail to get token in given token space, then ASSERT_EFI_ERROR().\r
+  \r
   @param[in]  Guid          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
@@ -967,12 +884,8 @@ LibPcdGetNextToken (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
 \r
-  Status = PcdPpi->GetNextToken (Guid, &TokenNumber);\r
+  Status = (GetPcdPpiPointer ())->GetNextToken (Guid, &TokenNumber);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -986,7 +899,8 @@ LibPcdGetNextToken (
   platform. If Guid is NULL, then the GUID of the first non-local token space of the \r
   current platform is returned. If Guid is the last non-local token space, \r
   then NULL is returned. \r
-\r
+  If fail to get next token space, then ASSERT_EFI_ERROR().\r
+  \r
   If Guid is not NULL and is not a valid token space in the current platform, then ASSERT().\r
 \r
 \r
@@ -1004,12 +918,8 @@ LibPcdGetNextTokenSpace (
   )\r
 {\r
   EFI_STATUS Status;\r
-  PCD_PPI * PcdPpi;\r
-\r
-  PcdPpi = GetPcdPpiPtr ();\r
-\r
 \r
-  Status = PcdPpi->GetNextTokenSpace (&Guid);\r
+  Status = (GetPcdPpiPointer ())->GetNextTokenSpace (&Guid);\r
 \r
   ASSERT_EFI_ERROR (Status);\r
 \r
@@ -1020,20 +930,21 @@ LibPcdGetNextTokenSpace (
 \r
 /**\r
   Sets the PCD entry specified by PatchVariable to the value specified by Buffer \r
-  and SizeOfValue.  Buffer is returned.  If SizeOfValue is greater than \r
-  MaximumDatumSize, then set SizeOfValue to MaximumDatumSize and return \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 SizeOfValue is set to MAX_ADDRESS, then SizeOfValue must be set to \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 SizeOfValue is NULL, then ASSERT().\r
-  If SizeOfValue > 0 and Buffer 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
                                 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
+                                In out, returns actual size of buffer is set.\r
   @param[in] Buffer             A pointer to the buffer to used to set the target variable.\r
 \r
 **/\r
diff --git a/MdePkg/Library/PeiPcdLib/PeiPcdLib.msa b/MdePkg/Library/PeiPcdLib/PeiPcdLib.msa
deleted file mode 100644 (file)
index 363cdb4..0000000
+++ /dev/null
@@ -1,55 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>\r
-<ModuleSurfaceArea xmlns="http://www.TianoCore.org/2006/Edk2.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">\r
-  <MsaHeader>\r
-    <ModuleName>PeiPcdLib</ModuleName>\r
-    <ModuleType>PEIM</ModuleType>\r
-    <GuidValue>9dbf6f25-0da2-4a1d-8e12-e78de6ab4d0e</GuidValue>\r
-    <Version>1.0</Version>\r
-    <Abstract>PCD Library instance implemented with PCD PPI.</Abstract>\r
-    <Description>This library instance implement the APIs listed 
-    in PCD library class defined in MDE library specification.
-    It is used by module in PEI phase.</Description>\r
-    <Copyright>Copyright (c) 2006, Intel Corporation</Copyright>\r
-    <License>All rights reserved. This program and the accompanying materials
-      are licensed and made available under the terms and conditions of the BSD License
-      which accompanies this distribution.  The full text of the license may be found at
-      http://opensource.org/licenses/bsd-license.php
-      THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
-      WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.</License>\r
-    <Specification>FRAMEWORK_BUILD_PACKAGING_SPECIFICATION   0x00000052</Specification>\r
-  </MsaHeader>\r
-  <ModuleDefinitions>\r
-    <SupportedArchitectures>IA32 X64 IPF EBC</SupportedArchitectures>\r
-    <BinaryModule>false</BinaryModule>\r
-    <OutputFileBasename>PeiPcdLib</OutputFileBasename>\r
-  </ModuleDefinitions>\r
-  <LibraryClassDefinitions>\r
-    <LibraryClass Usage="ALWAYS_PRODUCED" SupModuleList="PEIM PEI_CORE">\r
-      <Keyword>PcdLib</Keyword>\r
-    </LibraryClass>\r
-    <LibraryClass Usage="ALWAYS_CONSUMED">\r
-      <Keyword>DebugLib</Keyword>\r
-    </LibraryClass>\r
-    <LibraryClass Usage="ALWAYS_CONSUMED">\r
-      <Keyword>PeiServicesTablePointerLib</Keyword>\r
-    </LibraryClass>\r
-    <LibraryClass Usage="ALWAYS_CONSUMED">\r
-      <Keyword>BaseMemoryLib</Keyword>\r
-    </LibraryClass>\r
-  </LibraryClassDefinitions>\r
-  <SourceFiles>\r
-    <Filename>PeiPcdLib.c</Filename>\r
-  </SourceFiles>\r
-  <PackageDependencies>\r
-    <Package PackageGuid="5e0e9358-46b6-4ae2-8218-4ab8b9bbdcec"/>\r
-  </PackageDependencies>\r
-  <PPIs>\r
-    <Ppi Usage="ALWAYS_CONSUMED">\r
-      <PpiCName>gPcdPpiGuid</PpiCName>\r
-    </Ppi>\r
-  </PPIs>\r
-  <Externs>\r
-    <Specification>EFI_SPECIFICATION_VERSION 0x00020000</Specification>\r
-    <Specification>EDK_RELEASE_VERSION 0x00020000</Specification>\r
-  </Externs>\r
-</ModuleSurfaceArea>
\ No newline at end of file