Add in support for MaxSize and CurrentSize for PCD entry.

Change the behavior for PcdSET for Variable_Enabled Pcd entry. If a Variable does not exist, we will only save the value to a volatile space. We save it to variable ONLY when the variable exist. 

Fix a few bugs in Pcd Build tool when generated Pcd database for Pcd entry with Unicode String type.

Support PcdSet to return MaxSize if the input SizeOfBuffer is greater than the Max Size declared in FPD file.

Add SetValueWorker and GetValueWoker for each PcdSET and PcdGET function to reduce code size.

Sync function prototype definition for PCD_PPI_SET_POINTER, PCD_PPI_SET_EX_POINTER, PCD_PROTOCOL_SET_POINTER and PCD_PROTOCOL_SET_EX_POINTER with PCD arch spec 0.52.

Add ASSERTION in LibPcdSetPtr.





git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@789 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 7 insertions, 17 deletions

diff --git a/MdePkg/Library/DxePcdLib/DxePcdLib.c b/MdePkg/Library/DxePcdLib/DxePcdLib.c
index f0b0cbd8f6..2747df35c0 100644
--- a/MdePkg/Library/DxePcdLib/DxePcdLib.c
+++ b/MdePkg/Library/DxePcdLib/DxePcdLib.c
@@ -514,23 +514,19 @@ LibPcdSetPtr (
   )

 {

   EFI_STATUS Status;

-  UINTN      Size;

+

+  ASSERT (SizeOfBuffer != NULL);

 

   if (*SizeOfBuffer > 0) {

     ASSERT (Buffer != NULL);

   }

 

-  Size = LibPcdGetSize (TokenNumber);

-  

-  if (*SizeOfBuffer > Size) {

-    *SizeOfBuffer = Size;

+  Status = mPcd->SetPtr (TokenNumber, SizeOfBuffer, Buffer);

+

+  if (EFI_ERROR (Status)) {

     return NULL;

   }

 

-  Status = mPcd->SetPtr (TokenNumber, *SizeOfBuffer, Buffer);

-

-  ASSERT_EFI_ERROR (Status);

-

   return Buffer;

 }

 

@@ -728,7 +724,6 @@ LibPcdSetExPtr (
   )

 {

   EFI_STATUS  Status;

-  UINTN       Size;

 

   ASSERT (Guid != NULL);

 

@@ -738,17 +733,12 @@ LibPcdSetExPtr (
     ASSERT (Buffer != NULL);

   }

 

+  Status = mPcd->SetPtrEx (Guid, TokenNumber, SizeOfBuffer, Buffer);

 

-  Size = LibPcdGetExSize (Guid, TokenNumber);

-  if (*SizeOfBuffer > Size) {

-    *SizeOfBuffer = Size;

+  if (EFI_ERROR (Status)) {

     return NULL;

   }

 

-  Status = mPcd->SetPtrEx (Guid, TokenNumber, *SizeOfBuffer, Buffer);

-

-  ASSERT_EFI_ERROR (Status);

-

   return Buffer;

 }