Revision: 19166
          http://sourceforge.net/p/edk2/code/19166
Author:   shenshushi
Date:     2015-12-08 08:03:42 +0000 (Tue, 08 Dec 2015)
Log Message:
-----------
MdeModulePkg: Use PcdSet##S to replace PcdSet##

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Dandan Bi <[email protected]>
Reviewed-by: Qiu Shumin <[email protected]>

Modified Paths:
--------------
    trunk/edk2/MdeModulePkg/Library/BootMaintenanceManagerLib/BootMaintenance.c
    trunk/edk2/MdeModulePkg/Library/BootManagerLib/BootManager.c

Modified: 
trunk/edk2/MdeModulePkg/Library/BootMaintenanceManagerLib/BootMaintenance.c
===================================================================
--- trunk/edk2/MdeModulePkg/Library/BootMaintenanceManagerLib/BootMaintenance.c 
2015-12-08 07:51:29 UTC (rev 19165)
+++ trunk/edk2/MdeModulePkg/Library/BootMaintenanceManagerLib/BootMaintenance.c 
2015-12-08 08:03:42 UTC (rev 19166)
@@ -237,8 +237,10 @@
                   //
                   // Update text mode PCD.
                   //
-                  PcdSet32 (PcdConOutColumn, mBmmSetupTextModeColumn);
-                  PcdSet32 (PcdConOutRow, mBmmSetupTextModeRow);
+                  Status = PcdSet32S (PcdConOutColumn, 
mBmmSetupTextModeColumn);
+                  ASSERT_EFI_ERROR (Status);
+                  Status = PcdSet32S (PcdConOutRow, mBmmSetupTextModeRow);
+                  ASSERT_EFI_ERROR (Status);
                   FreePool (Info);
                   return EFI_SUCCESS;
                 }
@@ -279,10 +281,14 @@
   // Set PCD to Inform GraphicsConsole to change video resolution.
   // Set PCD to Inform Consplitter to change text mode.
   //
-  PcdSet32 (PcdVideoHorizontalResolution, NewHorizontalResolution);
-  PcdSet32 (PcdVideoVerticalResolution, NewVerticalResolution);
-  PcdSet32 (PcdConOutColumn, NewColumns);
-  PcdSet32 (PcdConOutRow, NewRows);
+  Status = PcdSet32S (PcdVideoHorizontalResolution, NewHorizontalResolution);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdVideoVerticalResolution, NewVerticalResolution);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdConOutColumn, NewColumns);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdConOutRow, NewRows);
+  ASSERT_EFI_ERROR (Status);
 
   //
   // Video mode is changed, so restart graphics console driver and higher 
level driver.

Modified: trunk/edk2/MdeModulePkg/Library/BootManagerLib/BootManager.c
===================================================================
--- trunk/edk2/MdeModulePkg/Library/BootManagerLib/BootManager.c        
2015-12-08 07:51:29 UTC (rev 19165)
+++ trunk/edk2/MdeModulePkg/Library/BootManagerLib/BootManager.c        
2015-12-08 08:03:42 UTC (rev 19166)
@@ -212,8 +212,10 @@
                   //
                   // Update text mode PCD.
                   //
-                  PcdSet32 (PcdConOutColumn, mBmSetupTextModeColumn);
-                  PcdSet32 (PcdConOutRow, mBmSetupTextModeRow);
+                  Status = PcdSet32S (PcdConOutColumn, mBmSetupTextModeColumn);
+                  ASSERT_EFI_ERROR (Status);
+                  Status = PcdSet32S (PcdConOutRow, mBmSetupTextModeRow);
+                  ASSERT_EFI_ERROR (Status);
                   FreePool (Info);
                   return EFI_SUCCESS;
                 }
@@ -254,10 +256,14 @@
   // Set PCD to Inform GraphicsConsole to change video resolution.
   // Set PCD to Inform Consplitter to change text mode.
   //
-  PcdSet32 (PcdVideoHorizontalResolution, NewHorizontalResolution);
-  PcdSet32 (PcdVideoVerticalResolution, NewVerticalResolution);
-  PcdSet32 (PcdConOutColumn, NewColumns);
-  PcdSet32 (PcdConOutRow, NewRows);
+  Status = PcdSet32S (PcdVideoHorizontalResolution, NewHorizontalResolution);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdVideoVerticalResolution, NewVerticalResolution);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdConOutColumn, NewColumns);
+  ASSERT_EFI_ERROR (Status);
+  Status = PcdSet32S (PcdConOutRow, NewRows);
+  ASSERT_EFI_ERROR (Status);
 
   //
   // Video mode is changed, so restart graphics console driver and higher 
level driver.


------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to