MdeModulePkg PCD: Add more debug message to show SkuId update

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>

diff --git a/MdeModulePkg/Universal/PCD/Dxe/Pcd.c b/MdeModulePkg/Universal/PCD/Dxe/Pcd.c
index ee53ae332b5702dcc12e5f7b5096e45356fede4e..c30e9d3b158d851313187912a260e1418e407b0e 100644 (file)
--- a/MdeModulePkg/Universal/PCD/Dxe/Pcd.c
+++ b/MdeModulePkg/Universal/PCD/Dxe/Pcd.c
@@ -273,10 +273,13 @@ DxePcdSetSku (
   UINTN      Index;\r
   EFI_STATUS Status;\r
 \r
+  DEBUG ((DEBUG_INFO, "PcdDxe - SkuId 0x%lx is to be set.\n", (SKU_ID) SkuId));\r
+\r
   if (SkuId == mPcdDatabase.DxeDb->SystemSkuId) {\r
     //\r
     // The input SKU Id is equal to current SKU Id, return directly.\r
     //\r
+    DEBUG ((DEBUG_INFO, "PcdDxe - SkuId is same to current system Sku.\n"));\r
     return;\r
   }\r
 \r
@@ -295,6 +298,7 @@ DxePcdSetSku (
   SkuIdTable = (SKU_ID *) ((UINT8 *) mPcdDatabase.DxeDb + mPcdDatabase.DxeDb->SkuIdTableOffset);\r
   for (Index = 0; Index < SkuIdTable[0]; Index++) {\r
     if (SkuId == SkuIdTable[Index + 1]) {\r
+      DEBUG ((DEBUG_INFO, "PcdDxe - SkuId is found in SkuId table.\n"));\r
       Status = UpdatePcdDatabase (SkuId, TRUE);\r
       if (!EFI_ERROR (Status)) {\r
         mPcdDatabase.DxeDb->SystemSkuId = (SKU_ID) SkuId;\r
@@ -307,7 +311,7 @@ DxePcdSetSku (
   //\r
   // Invalid input SkuId, the default SKU Id will be still used for the system.\r
   //\r
-  DEBUG ((DEBUG_INFO, "PcdDxe - Invalid input SkuId, the default SKU Id will be still used.\n"));\r
+  DEBUG ((DEBUG_ERROR, "PcdDxe - Invalid input SkuId, the default SKU Id will be still used.\n"));\r
   return;\r
 }\r
 \r

diff --git a/MdeModulePkg/Universal/PCD/Pei/Pcd.c b/MdeModulePkg/Universal/PCD/Pei/Pcd.c
index f8213953fa2b4fa7600a72335376a0324533d4cd..8d9328b54479fe8db53656741ed62d666b36f994 100644 (file)
--- a/MdeModulePkg/Universal/PCD/Pei/Pcd.c
+++ b/MdeModulePkg/Universal/PCD/Pei/Pcd.c
@@ -475,12 +475,15 @@ PeiPcdSetSku (
   PCD_DATABASE_SKU_DELTA *SkuDelta;\r
   PCD_DATA_DELTA         *SkuDeltaData;\r
 \r
+  DEBUG ((DEBUG_INFO, "PcdPei - SkuId 0x%lx is to be set.\n", (SKU_ID) SkuId));\r
+\r
   PeiPcdDb = GetPcdDatabase();\r
 \r
   if (SkuId == PeiPcdDb->SystemSkuId) {\r
     //\r
     // The input SKU Id is equal to current SKU Id, return directly.\r
     //\r
+    DEBUG ((DEBUG_INFO, "PcdPei - SkuId is same to current system Sku.\n"));\r
     return;\r
   }\r
 \r
@@ -499,6 +502,7 @@ PeiPcdSetSku (
   SkuIdTable = (SKU_ID *) ((UINT8 *) PeiPcdDb + PeiPcdDb->SkuIdTableOffset);\r
   for (Index = 0; Index < SkuIdTable[0]; Index++) {\r
     if (SkuId == SkuIdTable[Index + 1]) {\r
+      DEBUG ((DEBUG_INFO, "PcdPei - SkuId is found in SkuId table.\n"));\r
       break;\r
     }\r
   }\r
@@ -570,7 +574,7 @@ PeiPcdSetSku (
   //\r
   // Invalid input SkuId, the default SKU Id will be still used for the system.\r
   //\r
-  DEBUG ((EFI_D_INFO, "PcdPei - Invalid input SkuId, the default SKU Id will be still used.\n"));\r
+  DEBUG ((DEBUG_ERROR, "PcdPei - Invalid input SkuId, the default SKU Id will be still used.\n"));\r
 \r
   return;\r
 }\r