From: klu2 Date: Thu, 1 Feb 2007 08:42:59 +0000 (+0000) Subject: EFI_CONSOLE_CONTROL_PROTOCOL provide the SetMode interface to switch mode between... X-Git-Tag: edk2-stable201903~23532 X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=commitdiff_plain;h=db6efc11d350c27308e6e08d542f3f6ef17cd64f;hp=ca43e86ee7b30851028d2245bd91a52c3954ef1a EFI_CONSOLE_CONTROL_PROTOCOL provide the SetMode interface to switch mode between EfiConsoleControlScreenText and EfiConsoleControlScreenGraphics. And implementation of this interface is in EdkModlePkg\Universal\Console\ConSplitter\Dxe\ConSplitterGraphices.c ConSpliterConsoleControlSetMode(). In this function, it is better to judge current mode with wanted mode at first, if current mode is equal to wanted mode, then return and do nothing. Why need add judging: 1) Missing judge will lead an issue that screen splash when pressing key in front page. It is caused by ui.c will switch to textmode when first in, but current mode has been in text mode. 2) If switching to text mode, there are many works want to be done such as EnableCursor, ClearScreen. If original has been in text mode, judging will improve the performance of UI. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@2345 6f19259b-4bc3-4df7-8a09-765794883524 --- diff --git a/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitterGraphics.c b/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitterGraphics.c index cec8adb2fb..946835ed29 100644 --- a/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitterGraphics.c +++ b/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitterGraphics.c @@ -116,6 +116,13 @@ ConSpliterConsoleControlSetMode ( return EFI_INVALID_PARAMETER; } + // + // Judge current mode with wanted mode at first. + // + if (Private->ConsoleOutputMode == Mode) { + return EFI_SUCCESS; + } + Supported = FALSE; TextAndGop = &Private->TextOutList[0]; for (Index = 0; Index < Private->CurrentNumberOfConsoles; Index++, TextAndGop++) {