MdeModulePkg: Fix GraphicsConsole driver resolution out of sync issue

When the GOP doesn't support the resolution specified by
PcdVideoHorizontalResolution and PcdVideoVerticalResolution,
the code tries to set the resolution to 800x600 but uses the resolution
equals to the PCD when calculating the text print position.
The patch fixes the bug by updating the resolution to 800x600 for
this case.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Jeff Fan <jeff.fan@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@19630 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
index 74218c8b4e454eacc016d6d0e46e412fe3bc2bf5..7b1c37b357955582866537eaec996a4b16683e34 100644 (file)
--- a/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
+++ b/MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsole.c
@@ -1,7 +1,7 @@
 /** @file\r
   This is the main routine for initializing the Graphics Console support routines.\r
 \r
-Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2016, 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
@@ -480,10 +480,12 @@ GraphicsConsoleControllerDriverStart (
         //\r
         // if not supporting current mode, try 800x600 which is required by UEFI/EFI spec\r
         //\r
+        HorizontalResolution = 800;\r
+        VerticalResolution   = 600;\r
         Status = CheckModeSupported (\r
                      Private->GraphicsOutput,\r
-                     800,\r
-                     600,\r
+                     HorizontalResolution,\r
+                     VerticalResolution,\r
                      &ModeNumber\r
                      );\r
         Mode = Private->GraphicsOutput->Mode;\r