From db6efc11d350c27308e6e08d542f3f6ef17cd64f Mon Sep 17 00:00:00 2001 From: klu2 Date: Thu, 1 Feb 2007 08:42:59 +0000 Subject: [PATCH] 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 --- .../Console/ConSplitter/Dxe/ConSplitterGraphics.c | 7 +++++++ 1 file changed, 7 insertions(+) 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++) { -- 2.39.2