summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--MdeModulePkg/Include/Protocol/ConsoleControl.h17
-rw-r--r--MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c5
-rw-r--r--MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterGraphics.c13
3 files changed, 17 insertions, 18 deletions
diff --git a/MdeModulePkg/Include/Protocol/ConsoleControl.h b/MdeModulePkg/Include/Protocol/ConsoleControl.h
index c7bbb32ff0..f41ffa7207 100644
--- a/MdeModulePkg/Include/Protocol/ConsoleControl.h
+++ b/MdeModulePkg/Include/Protocol/ConsoleControl.h
@@ -30,8 +30,8 @@ typedef enum {
/**
Return the current video mode information. Also returns info about existence
- of Graphics Output devices or UGA Draw devices in system, and whether the Std In device is locked.
- GopUgaExists and StdInLocked parameters are optional.
+ of Graphics Output devices or UGA Draw devices in system, and whether the Std
+ In device is locked. GopUgaExists and StdInLocked parameters are optional.
@param This Protocol instance pointer.
@param Mode Current video mode.
@@ -52,7 +52,8 @@ EFI_STATUS
);
/**
- Set the current video mode to either text or graphics.
+ Set the current video mode to either text or graphics. Graphics is
+ for Quiet Boot.
@param This Protocol instance pointer.
@param Mode Video mode is to be set.
@@ -70,17 +71,15 @@ EFI_STATUS
);
/**
- Store the password, enable state variable and arm the periodic timer.
- If Password is NULL unlock the password state variable and set the event
- timer. If the Password is too big return an error. If the Password is valid
- Copy the Password and enable state variable and then arm the periodic timer
+ If Password is NULL or the Password is too big, then return an error. If the
+ Password is valid, then store the password, lock StdIn and arm the periodic timer.
@param This Console Control protocol pointer.
@param Password The password input.
- @retval EFI_SUCCESS Lock the StdIn device successfully.
+ @retval EFI_SUCCESS Lock the StdIn device
@retval EFI_INVALID_PARAMETER Password is NULL
- @retval EFI_OUT_OF_RESOURCES Buffer allocation to store the big password fails
+ @retval EFI_OUT_OF_RESOURCES Buffer allocation to store the password fails
**/
typedef
diff --git a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
index 234f733bd6..b61f7bdc28 100644
--- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
+++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.c
@@ -3434,9 +3434,8 @@ ConSpliterConsoleControlLockStdInEvent (
/**
- If Password is NULL unlock the password state variable and set the event
- timer. If the Password is too big return an error. If the Password is valid
- Copy the Password and enable state variable and then arm the periodic timer
+ If Password is NULL or the Password is too big, then return an error. If the
+ Password is valid, then store the password, lock StdIn and arm the periodic timer.
@param This Console Control protocol pointer.
@param Password The password input.
diff --git a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterGraphics.c b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterGraphics.c
index d67fdb2f50..7d8e632f95 100644
--- a/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterGraphics.c
+++ b/MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterGraphics.c
@@ -25,16 +25,17 @@ CHAR16 mCrLfString[3] = { CHAR_CARRIAGE_RETURN, CHAR_LINEFEED, CHAR_NULL };
/**
Return the current video mode information. Also returns info about existence
- of Graphics Output devices or UGA Draw devices in system, and if the Std In device is locked. All the
- arguments are optional and only returned if a non NULL pointer is passed in.
+ of Graphics Output devices or UGA Draw devices in system, and whether the Std
+ In device is locked. GopUgaExists and StdInLocked parameters are optional, and
+ only returned if a non NULL pointer is passed in.
@param This Protocol instance pointer.
- @param Mode Are we in text of grahics mode.
- @param GopUgaExists TRUE if Console Spliter has found a GOP or UGA device
+ @param Mode Current video mode.
+ @param GopExists TRUE if GOP Spliter has found a GOP/UGA device
@param StdInLocked TRUE if StdIn device is keyboard locked
- @retval EFI_SUCCESS Mode information returned.
- @retval EFI_INVALID_PARAMETER Invalid parameters.
+ @retval EFI_SUCCESS Video mode information is returned.
+ @retval EFI_INVALID_PARAMETER Invalid parameters if Mode == NULL.
**/
EFI_STATUS