]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitter.h
Update consplitter driver to use dynamic PCD to set console output mode instead of...
[mirror_edk2.git] / MdeModulePkg / Universal / Console / ConSplitterDxe / ConSplitter.h
index 578b49a58e2553230f45224027bd98d49d83b889..b4b8f1d322a7e15b4ea69d304eecae35537aeeff 100644 (file)
@@ -16,21 +16,27 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #define _CON_SPLITTER_H_\r
 \r
 #include <Uefi.h>\r
-#include <Guid/PrimaryStandardErrorDevice.h>\r
-#include <Guid/PrimaryConsoleOutDevice.h>\r
-#include <Protocol/GraphicsOutput.h>\r
-#include <Guid/PrimaryConsoleInDevice.h>\r
-#include <Guid/GenericPlatformVariable.h>\r
+\r
+#include <Protocol/DevicePath.h>\r
+#include <Protocol/ComponentName.h>\r
+#include <Protocol/DriverBinding.h>\r
+#include <Protocol/ConsoleControl.h>\r
 #include <Protocol/SimplePointer.h>\r
 #include <Protocol/AbsolutePointer.h>\r
 #include <Protocol/SimpleTextOut.h>\r
-#include <Guid/ConsoleInDevice.h>\r
 #include <Protocol/SimpleTextIn.h>\r
 #include <Protocol/SimpleTextInEx.h>\r
-#include <Protocol/ConsoleControl.h>\r
+#include <Protocol/GraphicsOutput.h>\r
+#include <Protocol/UgaDraw.h>\r
+#include <Protocol/SimpleTextInExNotify.h>\r
+\r
+#include <Guid/PrimaryStandardErrorDevice.h>\r
+#include <Guid/PrimaryConsoleOutDevice.h>\r
+#include <Guid/PrimaryConsoleInDevice.h>\r
+#include <Guid/ConsoleInDevice.h>\r
 #include <Guid/StandardErrorDevice.h>\r
 #include <Guid/ConsoleOutDevice.h>\r
-#include <Protocol/UgaDraw.h>\r
+\r
 #include <Library/PcdLib.h>\r
 #include <Library/DebugLib.h>\r
 #include <Library/UefiDriverEntryPoint.h>\r
@@ -61,10 +67,11 @@ extern EFI_DRIVER_BINDING_PROTOCOL  gConSplitterStdErrDriverBinding;
 extern EFI_COMPONENT_NAME_PROTOCOL  gConSplitterStdErrComponentName;\r
 extern EFI_COMPONENT_NAME2_PROTOCOL gConSplitterStdErrComponentName2;\r
 \r
-extern EFI_GUID                     gSimpleTextInExNotifyGuid;\r
 \r
+//\r
 // These definitions were in the old Hii protocol, but are not in the new UEFI\r
 // version. So they are defined locally.\r
+//\r
 #define UNICODE_NARROW_CHAR   0xFFF0\r
 #define UNICODE_WIDE_CHAR     0xFFF1\r
 \r
@@ -76,7 +83,6 @@ extern EFI_GUID                     gSimpleTextInExNotifyGuid;
 #define CONSOLE_SPLITTER_MODES_ALLOC_UNIT     32\r
 #define MAX_STD_IN_PASSWORD                   80\r
 \r
-#define VARCONOUTMODE L"ConOutMode"\r
 \r
 typedef struct {\r
   UINTN   Column;\r
@@ -84,17 +90,16 @@ typedef struct {
 } CONSOLE_OUT_MODE;\r
 \r
 typedef struct {\r
-  UINTN Columns;\r
-  UINTN Rows;\r
+  UINTN   Columns;\r
+  UINTN   Rows;\r
 } TEXT_OUT_SPLITTER_QUERY_DATA;\r
 \r
-//\r
-// Private data for the EFI_SIMPLE_TEXT_INPUT_PROTOCOL splitter\r
-//\r
-#define TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE EFI_SIGNATURE_32 ('T', 'i', 'S', 'p')\r
 \r
-#define TEXT_IN_EX_SPLITTER_NOTIFY_SIGNATURE EFI_SIGNATURE_32 ('T', 'i', 'S', 'n')\r
+#define TEXT_IN_EX_SPLITTER_NOTIFY_SIGNATURE    SIGNATURE_32 ('T', 'i', 'S', 'n')\r
 \r
+//\r
+// Private data for Text In Ex Splitter Notify\r
+//\r
 typedef struct _TEXT_IN_EX_SPLITTER_NOTIFY {\r
   UINTN                                 Signature;\r
   EFI_HANDLE                            *NotifyHandleList;\r
@@ -104,6 +109,18 @@ typedef struct _TEXT_IN_EX_SPLITTER_NOTIFY {
   LIST_ENTRY                            NotifyEntry;\r
 } TEXT_IN_EX_SPLITTER_NOTIFY;\r
 \r
+#define TEXT_IN_EX_SPLITTER_NOTIFY_FROM_THIS(a)  \\r
+  CR ((a),                                       \\r
+      TEXT_IN_EX_SPLITTER_NOTIFY,                \\r
+      NotifyEntry,                               \\r
+      TEXT_IN_EX_SPLITTER_NOTIFY_SIGNATURE       \\r
+      )\r
+\r
+#define TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE SIGNATURE_32 ('T', 'i', 'S', 'p')\r
+\r
+//\r
+// Private data for the Console In splitter\r
+//\r
 typedef struct {\r
   UINT64                             Signature;\r
   EFI_HANDLE                         VirtualHandle;\r
@@ -144,36 +161,34 @@ typedef struct {
 } TEXT_IN_SPLITTER_PRIVATE_DATA;\r
 \r
 #define TEXT_IN_SPLITTER_PRIVATE_DATA_FROM_THIS(a)  \\r
-  CR ((a),                                            \\r
+  CR ((a),                                          \\r
       TEXT_IN_SPLITTER_PRIVATE_DATA,                \\r
       TextIn,                                       \\r
       TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE       \\r
       )\r
 \r
 #define TEXT_IN_SPLITTER_PRIVATE_DATA_FROM_SIMPLE_POINTER_THIS(a) \\r
-  CR ((a),                                                          \\r
+  CR ((a),                                                        \\r
       TEXT_IN_SPLITTER_PRIVATE_DATA,                              \\r
       SimplePointer,                                              \\r
       TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE                     \\r
       )\r
 #define TEXT_IN_EX_SPLITTER_PRIVATE_DATA_FROM_THIS(a) \\r
-  CR (a, \\r
-      TEXT_IN_SPLITTER_PRIVATE_DATA, \\r
-      TextInEx, \\r
-      TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE \\r
+  CR (a,                                              \\r
+      TEXT_IN_SPLITTER_PRIVATE_DATA,                  \\r
+      TextInEx,                                       \\r
+      TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE         \\r
       )\r
 \r
 #define TEXT_IN_SPLITTER_PRIVATE_DATA_FROM_ABSOLUTE_POINTER_THIS(a) \\r
-  CR (a, \\r
-      TEXT_IN_SPLITTER_PRIVATE_DATA, \\r
-      AbsolutePointer, \\r
-      TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE \\r
+  CR (a,                                                            \\r
+      TEXT_IN_SPLITTER_PRIVATE_DATA,                                \\r
+      AbsolutePointer,                                              \\r
+      TEXT_IN_SPLITTER_PRIVATE_DATA_SIGNATURE                       \\r
       )\r
 \r
-//\r
-// Private data for the EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL splitter\r
-//\r
-#define TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE  EFI_SIGNATURE_32 ('T', 'o', 'S', 'p')\r
+\r
+#define TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE  SIGNATURE_32 ('T', 'o', 'S', 'p')\r
 \r
 typedef struct {\r
   EFI_GRAPHICS_OUTPUT_PROTOCOL     *GraphicsOutput;\r
@@ -182,37 +197,40 @@ typedef struct {
   BOOLEAN                          TextOutEnabled;\r
 } TEXT_OUT_AND_GOP_DATA;\r
 \r
+//\r
+// Private data for the Console Out splitter\r
+//\r
 typedef struct {\r
   UINT64                                Signature;\r
   EFI_HANDLE                            VirtualHandle;\r
   EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL       TextOut;\r
   EFI_SIMPLE_TEXT_OUTPUT_MODE           TextOutMode;\r
-                                        \r
+\r
   EFI_UGA_DRAW_PROTOCOL                 UgaDraw;\r
   UINT32                                UgaHorizontalResolution;\r
   UINT32                                UgaVerticalResolution;\r
   UINT32                                UgaColorDepth;\r
   UINT32                                UgaRefreshRate;\r
   EFI_UGA_PIXEL                         *UgaBlt;\r
-                                        \r
+\r
   EFI_GRAPHICS_OUTPUT_PROTOCOL          GraphicsOutput;\r
   EFI_GRAPHICS_OUTPUT_BLT_PIXEL         *GraphicsOutputBlt;\r
   EFI_GRAPHICS_OUTPUT_MODE_INFORMATION  *GraphicsOutputModeBuffer;\r
   UINTN                                 CurrentNumberOfGraphicsOutput;\r
   UINTN                                 CurrentNumberOfUgaDraw;\r
   BOOLEAN                               HardwareNeedsStarting;\r
-                                        \r
+\r
   EFI_CONSOLE_CONTROL_PROTOCOL          ConsoleControl;\r
-                                        \r
+\r
   UINTN                                 CurrentNumberOfConsoles;\r
   TEXT_OUT_AND_GOP_DATA                 *TextOutList;\r
   UINTN                                 TextOutListCount;\r
   TEXT_OUT_SPLITTER_QUERY_DATA          *TextOutQueryData;\r
   UINTN                                 TextOutQueryDataCount;\r
   INT32                                 *TextOutModeMap;\r
-                                        \r
+\r
   EFI_CONSOLE_CONTROL_SCREEN_MODE       ConsoleOutputMode;\r
-                                        \r
+\r
   UINTN                                 DevNullColumns;\r
   UINTN                                 DevNullRows;\r
   CHAR16                                *DevNullScreen;\r
@@ -221,28 +239,28 @@ typedef struct {
 } TEXT_OUT_SPLITTER_PRIVATE_DATA;\r
 \r
 #define TEXT_OUT_SPLITTER_PRIVATE_DATA_FROM_THIS(a) \\r
-  CR ((a),                                            \\r
+  CR ((a),                                          \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA,               \\r
       TextOut,                                      \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE      \\r
       )\r
 \r
 #define GRAPHICS_OUTPUT_SPLITTER_PRIVATE_DATA_FROM_THIS(a)  \\r
-  CR ((a),                                                    \\r
+  CR ((a),                                                  \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA,                       \\r
       GraphicsOutput,                                       \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE              \\r
       )\r
 \r
 #define UGA_DRAW_SPLITTER_PRIVATE_DATA_FROM_THIS(a) \\r
-  CR ((a),                                            \\r
+  CR ((a),                                          \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA,               \\r
       UgaDraw,                                      \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE      \\r
       )\r
 \r
 #define CONSOLE_CONTROL_SPLITTER_PRIVATE_DATA_FROM_THIS(a)  \\r
-  CR ((a),                                                    \\r
+  CR ((a),                                                  \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA,                       \\r
       ConsoleControl,                                       \\r
       TEXT_OUT_SPLITTER_PRIVATE_DATA_SIGNATURE              \\r
@@ -257,7 +275,7 @@ typedef struct {
 \r
   Installs driver module protocols and. Creates virtual device handles for ConIn,\r
   ConOut, and StdErr. Installs Simple Text In protocol, Simple Text In Ex protocol,\r
-  Simple Pointer protocol, Absolute Pointer protocol on those virtual handlers. \r
+  Simple Pointer protocol, Absolute Pointer protocol on those virtual handlers.\r
   Installs Graphics Output protocol and/or UGA Draw protocol if needed.\r
 \r
   @param[in] ImageHandle    The firmware allocated handle for the EFI image.\r
@@ -272,8 +290,7 @@ EFIAPI
 ConSplitterDriverEntry (\r
   IN EFI_HANDLE                       ImageHandle,\r
   IN EFI_SYSTEM_TABLE                 *SystemTable\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Construct console input devices' private data.\r
@@ -282,20 +299,19 @@ ConSplitterDriverEntry (
                                    structure.\r
 \r
   @retval EFI_OUT_OF_RESOURCES     Out of resources.\r
-  @retval EFI_SUCCESS              Console Input Devcie's private data has been constructed.\r
+  @retval EFI_SUCCESS              Text Input Devcie's private data has been constructed.\r
   @retval other                    Failed to construct private data.\r
 \r
 **/\r
 EFI_STATUS\r
 ConSplitterTextInConstructor (\r
-  TEXT_IN_SPLITTER_PRIVATE_DATA       *Private\r
-  )\r
-;\r
+  TEXT_IN_SPLITTER_PRIVATE_DATA       *ConInPrivate\r
+  );\r
 \r
 /**\r
   Construct console output devices' private data.\r
 \r
-  @param  ConOutPrivate            A pointer to the TEXT_IN_SPLITTER_PRIVATE_DATA\r
+  @param  ConOutPrivate            A pointer to the TEXT_OUT_SPLITTER_PRIVATE_DATA\r
                                    structure.\r
 \r
   @retval EFI_OUT_OF_RESOURCES     Out of resources.\r
@@ -304,18 +320,14 @@ ConSplitterTextInConstructor (
 **/\r
 EFI_STATUS\r
 ConSplitterTextOutConstructor (\r
-  TEXT_OUT_SPLITTER_PRIVATE_DATA      *Private\r
-  )\r
-;\r
+  TEXT_OUT_SPLITTER_PRIVATE_DATA      *ConOutPrivate\r
+  );\r
 \r
-//\r
-// Driver Binding Functions\r
-//\r
 \r
 /**\r
-  Test to see if Console In Device could be supported on the ControllerHandle. \r
+  Test to see if Console In Device could be supported on the Controller.\r
 \r
-  @param  This                Protocol instance pointer.\r
+  @param  This                Driver Binding protocol instance pointer.\r
   @param  ControllerHandle    Handle of device to test.\r
   @param  RemainingDevicePath Optional parameter use to pick a specific child\r
                               device to start.\r
@@ -330,13 +342,12 @@ ConSplitterConInDriverBindingSupported (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Test to see if Simple Pointer protocol could be supported on the ControllerHandle. \r
+  Test to see if Simple Pointer protocol could be supported on the Controller.\r
 \r
-  @param  This                Protocol instance pointer.\r
+  @param  This                Driver Binding protocol instance pointer.\r
   @param  ControllerHandle    Handle of device to test.\r
   @param  RemainingDevicePath Optional parameter use to pick a specific child\r
                               device to start.\r
@@ -351,13 +362,12 @@ ConSplitterSimplePointerDriverBindingSupported (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Test to see if Console Out Device could be supported on the ControllerHandle. \r
+  Test to see if Console Out Device could be supported on the Controller.\r
 \r
-  @param  This                Protocol instance pointer.\r
+  @param  This                Driver Binding protocol instance pointer.\r
   @param  ControllerHandle    Handle of device to test.\r
   @param  RemainingDevicePath Optional parameter use to pick a specific child\r
                               device to start.\r
@@ -372,13 +382,12 @@ ConSplitterConOutDriverBindingSupported (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Test to see if Standard Error Device could be supported on the ControllerHandle. \r
+  Test to see if Standard Error Device could be supported on the Controller.\r
 \r
-  @param  This                Protocol instance pointer.\r
+  @param  This                Driver Binding protocol instance pointer.\r
   @param  ControllerHandle    Handle of device to test.\r
   @param  RemainingDevicePath Optional parameter use to pick a specific child\r
                               device to start.\r
@@ -393,13 +402,12 @@ ConSplitterStdErrDriverBindingSupported (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Start Console In Consplitter on device handle. \r
-  \r
-  @param  This                 Protocol instance pointer.\r
+  Start Console In Consplitter on device handle.\r
+\r
+  @param  This                 Driver Binding protocol instance pointer.\r
   @param  ControllerHandle     Handle of device to bind driver to.\r
   @param  RemainingDevicePath  Optional parameter use to pick a specific child\r
                                device to start.\r
@@ -414,13 +422,12 @@ ConSplitterConInDriverBindingStart (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Start Simple Pointer Consplitter on device handle. \r
-  \r
-  @param  This                 Protocol instance pointer.\r
+  Start Simple Pointer Consplitter on device handle.\r
+\r
+  @param  This                 Driver Binding protocol instance pointer.\r
   @param  ControllerHandle     Handle of device to bind driver to.\r
   @param  RemainingDevicePath  Optional parameter use to pick a specific child\r
                                device to start.\r
@@ -435,13 +442,12 @@ ConSplitterSimplePointerDriverBindingStart (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Start Console Out Consplitter on device handle. \r
-  \r
-  @param  This                 Protocol instance pointer.\r
+  Start Console Out Consplitter on device handle.\r
+\r
+  @param  This                 Driver Binding protocol instance pointer.\r
   @param  ControllerHandle     Handle of device to bind driver to.\r
   @param  RemainingDevicePath  Optional parameter use to pick a specific child\r
                                device to start.\r
@@ -456,13 +462,12 @@ ConSplitterConOutDriverBindingStart (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Start Standard Error Consplitter on device handle. \r
-  \r
-  @param  This                 Protocol instance pointer.\r
+  Start Standard Error Consplitter on device handle.\r
+\r
+  @param  This                 Driver Binding protocol instance pointer.\r
   @param  ControllerHandle     Handle of device to bind driver to.\r
   @param  RemainingDevicePath  Optional parameter use to pick a specific child\r
                                device to start.\r
@@ -477,13 +482,12 @@ ConSplitterStdErrDriverBindingStart (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Stop Console In ConSplitter on ControllerHandle by closing Console In Devcice GUID.\r
 \r
-  @param  This              Protocol instance pointer.\r
+  @param  This              Driver Binding protocol instance pointer.\r
   @param  ControllerHandle  Handle of device to stop driver on\r
   @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
                             children is zero stop the entire bus driver.\r
@@ -500,14 +504,13 @@ ConSplitterConInDriverBindingStop (
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  UINTN                           NumberOfChildren,\r
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Stop Simple Pointer protocol ConSplitter on ControllerHandle by closing\r
   Simple Pointer protocol.\r
 \r
-  @param  This              Protocol instance pointer.\r
+  @param  This              Driver Binding protocol instance pointer.\r
   @param  ControllerHandle  Handle of device to stop driver on\r
   @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
                             children is zero stop the entire bus driver.\r
@@ -524,13 +527,12 @@ ConSplitterSimplePointerDriverBindingStop (
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  UINTN                           NumberOfChildren,\r
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Stop Console Out ConSplitter on ControllerHandle by closing Console Out Devcice GUID.\r
+  Stop Console Out ConSplitter on device handle by closing Console Out Devcice GUID.\r
 \r
-  @param  This              Protocol instance pointer.\r
+  @param  This              Driver Binding protocol instance pointer.\r
   @param  ControllerHandle  Handle of device to stop driver on\r
   @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
                             children is zero stop the entire bus driver.\r
@@ -547,13 +549,12 @@ ConSplitterConOutDriverBindingStop (
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  UINTN                           NumberOfChildren,\r
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Stop Standard Error ConSplitter on ControllerHandle by closing Standard Error GUID.\r
 \r
-  @param  This              Protocol instance pointer.\r
+  @param  This              Driver Binding protocol instance pointer.\r
   @param  ControllerHandle  Handle of device to stop driver on\r
   @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
                             children is zero stop the entire bus driver.\r
@@ -570,14 +571,13 @@ ConSplitterStdErrDriverBindingStop (
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  UINTN                           NumberOfChildren,\r
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
-  Test to see if Absolute Pointer protocol could be supported on the ControllerHandle. \r
+  Test to see if Absolute Pointer protocol could be supported on the Controller.\r
 \r
-  @param  This                Protocol instance pointer.\r
+  @param  This                Driver Binding protocol instance pointer.\r
   @param  ControllerHandle    Handle of device to test.\r
   @param  RemainingDevicePath Optional parameter use to pick a specific child\r
                               device to start.\r
@@ -592,13 +592,12 @@ ConSplitterAbsolutePointerDriverBindingSupported (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Start Absolute Pointer Consplitter on device handle. \r
-  \r
-  @param  This                 Protocol instance pointer.\r
+  Start Absolute Pointer Consplitter on device handle.\r
+\r
+  @param  This                 Driver Binding protocol instance pointer.\r
   @param  ControllerHandle     Handle of device to bind driver to.\r
   @param  RemainingDevicePath  Optional parameter use to pick a specific child\r
                                device to start.\r
@@ -613,14 +612,13 @@ ConSplitterAbsolutePointerDriverBindingStart (
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  EFI_DEVICE_PATH_PROTOCOL        *RemainingDevicePath\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Stop Absolute Pointer protocol ConSplitter on ControllerHandle by closing\r
   Absolute Pointer protocol.\r
 \r
-  @param  This              Protocol instance pointer.\r
+  @param  This              Driver Binding protocol instance pointer.\r
   @param  ControllerHandle  Handle of device to stop driver on\r
   @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
                             children is zero stop the entire bus driver.\r
@@ -637,8 +635,7 @@ ConSplitterAbsolutePointerDriverBindingStop (
   IN  EFI_HANDLE                      ControllerHandle,\r
   IN  UINTN                           NumberOfChildren,\r
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Add Absolute Pointer Device in Consplitter Absolute Pointer list.\r
@@ -654,11 +651,10 @@ EFI_STATUS
 ConSplitterAbsolutePointerAddDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA     *Private,\r
   IN  EFI_ABSOLUTE_POINTER_PROTOCOL     *AbsolutePointer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Remove Absolute Pointer Device in Consplitter Absolute Pointer list.\r
+  Remove Absolute Pointer Device from Consplitter Absolute Pointer list.\r
 \r
   @param  Private                  Text In Splitter pointer.\r
   @param  AbsolutePointer          Absolute Pointer protocol pointer.\r
@@ -671,8 +667,7 @@ EFI_STATUS
 ConSplitterAbsolutePointerDeleteDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA     *Private,\r
   IN  EFI_ABSOLUTE_POINTER_PROTOCOL     *AbsolutePointer\r
-  )\r
-;\r
+  );\r
 \r
 //\r
 // Absolute Pointer protocol interfaces\r
@@ -695,8 +690,7 @@ EFIAPI
 ConSplitterAbsolutePointerReset (\r
   IN EFI_ABSOLUTE_POINTER_PROTOCOL   *This,\r
   IN BOOLEAN                         ExtendedVerification\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -719,8 +713,7 @@ EFIAPI
 ConSplitterAbsolutePointerGetState (\r
   IN EFI_ABSOLUTE_POINTER_PROTOCOL   *This,\r
   IN OUT EFI_ABSOLUTE_POINTER_STATE  *State\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   This event agregates all the events of the pointer devices in the splitter.\r
@@ -732,16 +725,13 @@ ConSplitterAbsolutePointerGetState (
   @param  Event                    The Event assoicated with callback.\r
   @param  Context                  Context registered when Event was created.\r
 \r
-  @return None\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
 ConSplitterAbsolutePointerWaitForInput (\r
   IN  EFI_EVENT                       Event,\r
   IN  VOID                            *Context\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Retrieves a Unicode string that is the user readable name of the driver.\r
@@ -1000,8 +990,7 @@ ConSplitterAbsolutePointerComponentNameGetControllerName (
   IN  EFI_HANDLE                                      ChildHandle        OPTIONAL,\r
   IN  CHAR8                                           *Language,\r
   OUT CHAR16                                          **ControllerName\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Retrieves a Unicode string that is the user readable name of the controller\r
@@ -1177,13 +1166,12 @@ ConSplitterStdErrComponentNameGetControllerName (
 **/\r
 EFI_STATUS\r
 ConSplitterTextInAddDevice (\r
-  IN  TEXT_IN_SPLITTER_PRIVATE_DATA      *Private,\r
-  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL     *TextIn\r
-  )\r
-;\r
+  IN  TEXT_IN_SPLITTER_PRIVATE_DATA   *Private,\r
+  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL  *TextIn\r
+  );\r
 \r
 /**\r
-  Remove Simple Text Device in Consplitter Absolute Pointer list.\r
+  Remove Text Input Device from Consplitter Text Input list.\r
 \r
   @param  Private                  Text In Splitter pointer.\r
   @param  TextIn                   Simple Text protocol pointer.\r
@@ -1194,10 +1182,9 @@ ConSplitterTextInAddDevice (
 **/\r
 EFI_STATUS\r
 ConSplitterTextInDeleteDevice (\r
-  IN  TEXT_IN_SPLITTER_PRIVATE_DATA      *Private,\r
-  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL     *TextIn\r
-  )\r
-;\r
+  IN  TEXT_IN_SPLITTER_PRIVATE_DATA   *Private,\r
+  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL  *TextIn\r
+  );\r
 \r
 //\r
 // SimplePointer Constuctor/Destructor functions\r
@@ -1217,11 +1204,10 @@ EFI_STATUS
 ConSplitterSimplePointerAddDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA   *Private,\r
   IN  EFI_SIMPLE_POINTER_PROTOCOL     *SimplePointer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Remove Simple Pointer Device in Consplitter Absolute Pointer list.\r
+  Remove Simple Pointer Device from Consplitter Simple Pointer list.\r
 \r
   @param  Private                  Text In Splitter pointer.\r
   @param  SimplePointer            Simple Pointer protocol pointer.\r
@@ -1234,8 +1220,7 @@ EFI_STATUS
 ConSplitterSimplePointerDeleteDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA   *Private,\r
   IN  EFI_SIMPLE_POINTER_PROTOCOL     *SimplePointer\r
-  )\r
-;\r
+  );\r
 \r
 //\r
 // TextOut Constuctor/Destructor functions\r
@@ -1259,8 +1244,7 @@ ConSplitterTextOutAddDevice (
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *TextOut,\r
   IN  EFI_GRAPHICS_OUTPUT_PROTOCOL       *GraphicsOutput,\r
   IN  EFI_UGA_DRAW_PROTOCOL              *UgaDraw\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Remove Text Out Device in Consplitter Text Out list.\r
@@ -1276,8 +1260,7 @@ EFI_STATUS
 ConSplitterTextOutDeleteDevice (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA     *Private,\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *TextOut\r
-  )\r
-;\r
+  );\r
 \r
 //\r
 // TextIn I/O Functions\r
@@ -1297,10 +1280,9 @@ ConSplitterTextOutDeleteDevice (
 EFI_STATUS\r
 EFIAPI\r
 ConSplitterTextInReset (\r
-  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL     *This,\r
-  IN  BOOLEAN                            ExtendedVerification\r
-  )\r
-;\r
+  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL  *This,\r
+  IN  BOOLEAN                         ExtendedVerification\r
+  );\r
 \r
 /**\r
   Reads the next keystroke from the input device. The WaitForKey Event can\r
@@ -1319,16 +1301,15 @@ ConSplitterTextInReset (
 EFI_STATUS\r
 EFIAPI\r
 ConSplitterTextInReadKeyStroke (\r
-  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL     *This,\r
-  OUT EFI_INPUT_KEY                      *Key\r
-  )\r
-;\r
+  IN  EFI_SIMPLE_TEXT_INPUT_PROTOCOL  *This,\r
+  OUT EFI_INPUT_KEY                   *Key\r
+  );\r
 \r
 /**\r
   Add Text Input Ex Device in Consplitter Text Input Ex list.\r
 \r
   @param  Private                  Text In Splitter pointer.\r
-  @param  TextInEx                 Simple Text Ex Input protocol pointer.\r
+  @param  TextInEx                 Simple Text Input Ex Input protocol pointer.\r
 \r
   @retval EFI_SUCCESS              Text Input Ex Device added successfully.\r
   @retval EFI_OUT_OF_RESOURCES     Could not grow the buffer size.\r
@@ -1338,31 +1319,28 @@ EFI_STATUS
 ConSplitterTextInExAddDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA         *Private,\r
   IN  EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL     *TextInEx\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Remove Simple Text Ex Device in Consplitter Absolute Pointer list.\r
+  Remove Text Ex Device from Consplitter Text Input Ex list.\r
 \r
   @param  Private                  Text In Splitter pointer.\r
   @param  TextInEx                 Simple Text Ex protocol pointer.\r
 \r
-  @retval EFI_SUCCESS              Simple Text Ex Device removed successfully.\r
-  @retval EFI_NOT_FOUND            No Simple Text Ex Device found.\r
+  @retval EFI_SUCCESS              Simple Text Input Ex Device removed successfully.\r
+  @retval EFI_NOT_FOUND            No Simple Text Input Ex Device found.\r
 \r
 **/\r
 EFI_STATUS\r
 ConSplitterTextInExDeleteDevice (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA         *Private,\r
   IN  EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL     *TextInEx\r
-  )\r
-;\r
+  );\r
 \r
 //\r
 // Simple Text Input Ex protocol function prototypes\r
 //\r
 \r
-\r
 /**\r
   Reset the input device and optionaly run diagnostics\r
 \r
@@ -1379,8 +1357,7 @@ EFIAPI
 ConSplitterTextInResetEx (\r
   IN EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL  *This,\r
   IN BOOLEAN                            ExtendedVerification\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -1404,8 +1381,7 @@ EFIAPI
 ConSplitterTextInReadKeyStrokeEx (\r
   IN  EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL *This,\r
   OUT EFI_KEY_DATA                      *KeyData\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -1428,8 +1404,7 @@ EFIAPI
 ConSplitterTextInSetState (\r
   IN EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL  *This,\r
   IN EFI_KEY_TOGGLE_STATE               *KeyToggleState\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -1448,7 +1423,7 @@ ConSplitterTextInSetState (
                                    successfully.\r
   @retval EFI_OUT_OF_RESOURCES     Unable to allocate resources for necesssary data\r
                                    structures.\r
-  @retval EFI_INVALID_PARAMETER    KeyData or NotifyHandle is NULL.\r
+  @retval EFI_INVALID_PARAMETER    KeyData or KeyNotificationFunction or NotifyHandle is NULL.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1458,8 +1433,7 @@ ConSplitterTextInRegisterKeyNotify (
   IN EFI_KEY_DATA                       *KeyData,\r
   IN EFI_KEY_NOTIFY_FUNCTION            KeyNotificationFunction,\r
   OUT EFI_HANDLE                        *NotifyHandle\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -1480,11 +1454,11 @@ EFIAPI
 ConSplitterTextInUnregisterKeyNotify (\r
   IN EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL  *This,\r
   IN EFI_HANDLE                         NotificationHandle\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  This event agregates all the events of the ConIn devices in the spliter.\r
+  This event aggregates all the events of the ConIn devices in the spliter.\r
+\r
   If the ConIn is password locked then return.\r
   If any events of physical ConIn devices are signaled, signal the ConIn\r
   spliter event. This will cause the calling code to call\r
@@ -1493,22 +1467,18 @@ ConSplitterTextInUnregisterKeyNotify (
   @param  Event                    The Event assoicated with callback.\r
   @param  Context                  Context registered when Event was created.\r
 \r
-  @return None\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
 ConSplitterTextInWaitForKey (\r
   IN  EFI_EVENT                       Event,\r
   IN  VOID                            *Context\r
-  )\r
-;\r
+  );\r
+\r
 /**\r
   Return TRUE if StdIn is locked. The ConIn device on the virtual handle is\r
   the only device locked.\r
 \r
-  NONE\r
-\r
   @retval TRUE                     StdIn locked\r
   @retval FALSE                    StdIn working normally\r
 \r
@@ -1516,28 +1486,26 @@ ConSplitterTextInWaitForKey (
 BOOLEAN\r
 ConSpliterConssoleControlStdInLocked (\r
   VOID\r
-  )\r
-;\r
+  );\r
+\r
 /**\r
-  This timer event will fire when StdIn is locked. It will check the key\r
-  sequence on StdIn to see if it matches the password. Any error in the\r
-  password will cause the check to reset. As long a mConIn.PasswordEnabled is\r
-  TRUE the StdIn splitter will not report any input.\r
+  Record and check key sequence on StdIn.\r
+\r
+  This timer event will fire when StdIn is locked. It will record the key sequence\r
+  on StdIn and also check to see if it matches the password. Any error in the\r
+  password will cause the check to reset. As long as a mConIn.PasswordEnabled is\r
+  TRUE, the StdIn splitter will not report any input.\r
 \r
   @param  Event                  The Event this notify function registered to.\r
   @param  Context                Pointer to the context data registerd to the\r
                                  Event.\r
-\r
-  @return None\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
 ConSpliterConsoleControlLockStdInEvent (\r
   IN  EFI_EVENT                       Event,\r
   IN  VOID                            *Context\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   If Password is NULL unlock the password state variable and set the event\r
@@ -1557,8 +1525,7 @@ EFIAPI
 ConSpliterConsoleControlLockStdIn (\r
   IN  EFI_CONSOLE_CONTROL_PROTOCOL    *This,\r
   IN  CHAR16                          *Password\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Reads the next keystroke from the input device. The WaitForKey Event can\r
@@ -1578,8 +1545,7 @@ EFIAPI
 ConSplitterTextInPrivateReadKeyStroke (\r
   IN  TEXT_IN_SPLITTER_PRIVATE_DATA   *Private,\r
   OUT EFI_INPUT_KEY                   *Key\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Reset the input device and optionaly run diagnostics\r
@@ -1597,8 +1563,7 @@ EFIAPI
 ConSplitterSimplePointerReset (\r
   IN  EFI_SIMPLE_POINTER_PROTOCOL     *This,\r
   IN  BOOLEAN                         ExtendedVerification\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Reads the next keystroke from the input device. The WaitForKey Event can\r
@@ -1619,8 +1584,7 @@ EFIAPI
 ConSplitterSimplePointerGetState (\r
   IN  EFI_SIMPLE_POINTER_PROTOCOL     *This,\r
   IN OUT EFI_SIMPLE_POINTER_STATE     *State\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   This event agregates all the events of the ConIn devices in the spliter.\r
@@ -1632,16 +1596,13 @@ ConSplitterSimplePointerGetState (
   @param  Event                    The Event assoicated with callback.\r
   @param  Context                  Context registered when Event was created.\r
 \r
-  @return None\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
 ConSplitterSimplePointerWaitForInput (\r
   IN  EFI_EVENT                       Event,\r
   IN  VOID                            *Context\r
-  )\r
-;\r
+  );\r
 \r
 //\r
 // TextOut I/O Functions\r
@@ -1664,8 +1625,7 @@ EFIAPI
 ConSplitterTextOutReset (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  BOOLEAN                            ExtendedVerification\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Write a Unicode string to the output device.\r
@@ -1691,8 +1651,7 @@ EFIAPI
 ConSplitterTextOutOutputString (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  CHAR16                             *WString\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Verifies that all characters in a Unicode string can be output to the\r
@@ -1714,8 +1673,7 @@ EFIAPI
 ConSplitterTextOutTestString (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  CHAR16                             *WString\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Returns information for an available text mode that the output device(s)\r
@@ -1741,8 +1699,7 @@ ConSplitterTextOutQueryMode (
   IN  UINTN                              ModeNumber,\r
   OUT UINTN                              *Columns,\r
   OUT UINTN                              *Rows\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Sets the output device(s) to a specified mode.\r
@@ -1761,8 +1718,7 @@ EFIAPI
 ConSplitterTextOutSetMode (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  UINTN                              ModeNumber\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Sets the background and foreground colors for the OutputString () and\r
@@ -1786,8 +1742,7 @@ EFIAPI
 ConSplitterTextOutSetAttribute (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  UINTN                              Attribute\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Clears the output device(s) display to the currently selected background\r
@@ -1805,8 +1760,7 @@ EFI_STATUS
 EFIAPI\r
 ConSplitterTextOutClearScreen (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Sets the current coordinates of the cursor position\r
@@ -1833,8 +1787,7 @@ ConSplitterTextOutSetCursorPosition (
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  UINTN                              Column,\r
   IN  UINTN                              Row\r
-  )\r
-;\r
+  );\r
 \r
 \r
 /**\r
@@ -1856,8 +1809,7 @@ EFIAPI
 ConSplitterTextOutEnableCursor (\r
   IN  EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL    *This,\r
   IN  BOOLEAN                            Visible\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Take the passed in Buffer of size SizeOfCount and grow the buffer\r
@@ -1865,12 +1817,12 @@ ConSplitterTextOutEnableCursor (
   bytes. Copy the current data in Buffer to the new version of Buffer\r
   and free the old version of buffer.\r
 \r
-  @param  SizeOfCount              Size of element in array\r
-  @param  Count                    Current number of elements in array\r
+  @param  SizeOfCount              Size of element in array.\r
+  @param  Count                    Current number of elements in array.\r
   @param  Buffer                   Bigger version of passed in Buffer with all the\r
-                                   data\r
+                                   data.\r
 \r
-  @retval EFI_SUCCESS              Buffer size has grown\r
+  @retval EFI_SUCCESS              Buffer size has grown.\r
   @retval EFI_OUT_OF_RESOURCES     Could not grow the buffer size.\r
 \r
 **/\r
@@ -1879,8 +1831,7 @@ ConSplitterGrowBuffer (
   IN  UINTN                           SizeOfCount,\r
   IN  UINTN                           *Count,\r
   IN OUT  VOID                        **Buffer\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Return the current video mode information. Also returns info about existence\r
@@ -1889,7 +1840,7 @@ ConSplitterGrowBuffer (
 \r
   @param  This                    Protocol instance pointer.\r
   @param  Mode                    Are we in text of grahics mode.\r
-  @param  GopExists               TRUE if GOP Spliter has found a GOP/UGA device\r
+  @param  GopUgaExists            TRUE if Console Spliter has found a GOP or UGA device\r
   @param  StdInLocked             TRUE if StdIn device is keyboard locked\r
 \r
   @retval EFI_SUCCESS             Mode information returned.\r
@@ -1901,17 +1852,16 @@ EFIAPI
 ConSpliterConsoleControlGetMode (\r
   IN  EFI_CONSOLE_CONTROL_PROTOCOL    *This,\r
   OUT EFI_CONSOLE_CONTROL_SCREEN_MODE *Mode,\r
-  OUT BOOLEAN                         *GopExists,\r
+  OUT BOOLEAN                         *GopUgaExists,\r
   OUT BOOLEAN                         *StdInLocked\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Set the current mode to either text or graphics. Graphics is\r
   for Quiet Boot.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  Mode                    Mode to set the\r
+  @param  This                    Console Control Protocol instance pointer.\r
+  @param  Mode                    Mode to set.\r
 \r
   @retval EFI_SUCCESS             Mode information returned.\r
   @retval EFI_INVALID_PARAMETER   Invalid parameter.\r
@@ -1923,47 +1873,45 @@ EFIAPI
 ConSpliterConsoleControlSetMode (\r
   IN  EFI_CONSOLE_CONTROL_PROTOCOL    *This,\r
   IN  EFI_CONSOLE_CONTROL_SCREEN_MODE Mode\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Return the current video mode information.\r
+  Returns information for an available graphics mode that the graphics device\r
+  and the set of active video output devices supports.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  ModeNumber              The mode number to return information on.\r
-  @param  SizeOfInfo              A pointer to the size, in bytes, of the Info\r
-                                  buffer.\r
-  @param  Info                    Caller allocated buffer that returns information\r
-                                  about ModeNumber.\r
+  @param  This                  The EFI_GRAPHICS_OUTPUT_PROTOCOL instance.\r
+  @param  ModeNumber            The mode number to return information on.\r
+  @param  SizeOfInfo            A pointer to the size, in bytes, of the Info buffer.\r
+  @param  Info                  A pointer to callee allocated buffer that returns information about ModeNumber.\r
 \r
-  @retval EFI_SUCCESS             Mode information returned.\r
-  @retval EFI_BUFFER_TOO_SMALL    The Info buffer was too small.\r
-  @retval EFI_DEVICE_ERROR        A hardware error occurred trying to retrieve the\r
-                                  video mode.\r
-  @retval EFI_NOT_STARTED         Video display is not initialized. Call SetMode ()\r
-  @retval EFI_INVALID_PARAMETER   One of the input args was NULL.\r
+  @retval EFI_SUCCESS           Mode information returned.\r
+  @retval EFI_BUFFER_TOO_SMALL  The Info buffer was too small.\r
+  @retval EFI_DEVICE_ERROR      A hardware error occurred trying to retrieve the video mode.\r
+  @retval EFI_NOT_STARTED       Video display is not initialized. Call SetMode ()\r
+  @retval EFI_INVALID_PARAMETER One of the input args was NULL.\r
+  @retval EFI_OUT_OF_RESOURCES  No resource available.\r
 \r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
 ConSpliterGraphicsOutputQueryMode (\r
-  IN  EFI_GRAPHICS_OUTPUT_PROTOCOL        *This,\r
-  IN  UINT32                            ModeNumber,\r
-  OUT UINTN                              *SizeOfInfo,\r
+  IN  EFI_GRAPHICS_OUTPUT_PROTOCOL          *This,\r
+  IN  UINT32                                ModeNumber,\r
+  OUT UINTN                                 *SizeOfInfo,\r
   OUT EFI_GRAPHICS_OUTPUT_MODE_INFORMATION  **Info\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Graphics output protocol interface to set video mode.\r
+  Set the video device into the specified mode and clears the visible portions of\r
+  the output display to black.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  ModeNumber              The mode number to be set.\r
+  @param  This                  The EFI_GRAPHICS_OUTPUT_PROTOCOL instance.\r
+  @param  ModeNumber            Abstraction that defines the current video mode.\r
 \r
-  @retval EFI_SUCCESS             Graphics mode was changed.\r
-  @retval EFI_DEVICE_ERROR        The device had an error and could not complete\r
-                                  the request.\r
-  @retval EFI_UNSUPPORTED         ModeNumber is not supported by this device.\r
+  @retval EFI_SUCCESS           The graphics mode specified by ModeNumber was selected.\r
+  @retval EFI_DEVICE_ERROR      The device had an error and could not complete the request.\r
+  @retval EFI_UNSUPPORTED       ModeNumber is not supported by this device.\r
+  @retval EFI_OUT_OF_RESOURCES  No resource available.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1971,8 +1919,7 @@ EFIAPI
 ConSpliterGraphicsOutputSetMode (\r
   IN  EFI_GRAPHICS_OUTPUT_PROTOCOL * This,\r
   IN  UINT32                       ModeNumber\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   The following table defines actions for BltOperations.\r
@@ -2011,7 +1958,7 @@ ConSpliterGraphicsOutputSetMode (
   @param  DestinationX            X coordinate of destination for the BltBuffer.\r
   @param  DestinationY            Y coordinate of destination for the BltBuffer.\r
   @param  Width                   Width of rectangle in BltBuffer in pixels.\r
-  @param  Height                  Hight of rectangle in BltBuffer in pixels. \r
+  @param  Height                  Hight of rectangle in BltBuffer in pixels.\r
   @param  Delta                   OPTIONAL.\r
 \r
   @retval EFI_SUCCESS             The Blt operation completed.\r
@@ -2033,11 +1980,10 @@ ConSpliterGraphicsOutputBlt (
   IN  UINTN                                         Width,\r
   IN  UINTN                                         Height,\r
   IN  UINTN                                         Delta         OPTIONAL\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Write data from the buffer to video display based on Graphics Output setting. \r
+  Write data from the buffer to video display based on Graphics Output setting.\r
 \r
   @param  Private                 Consplitter Text Out pointer.\r
   @param  GraphicsOutput          Graphics Output protocol pointer.\r
@@ -2047,7 +1993,7 @@ ConSpliterGraphicsOutputBlt (
   @retval EFI_SUCCESS             The Blt operation completed.\r
   @retval EFI_INVALID_PARAMETER   BltOperation is not valid.\r
   @retval EFI_DEVICE_ERROR        A hardware error occured writting to the video buffer.\r
-                 \r
+\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2055,21 +2001,20 @@ DevNullGopSync (
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  EFI_GRAPHICS_OUTPUT_PROTOCOL    *GraphicsOutput,\r
   IN  EFI_UGA_DRAW_PROTOCOL           *UgaDraw\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Return the current video mode information.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  HorizontalResolution    Current video horizontal resolution in pixels\r
-  @param  VerticalResolution      Current video vertical resolution in pixels\r
-  @param  ColorDepth              Current video color depth in bits per pixel\r
-  @param  RefreshRate             Current video refresh rate in Hz.\r
+  @param  This                  The EFI_UGA_DRAW_PROTOCOL instance.\r
+  @param  HorizontalResolution  The size of video screen in pixels in the X dimension.\r
+  @param  VerticalResolution    The size of video screen in pixels in the Y dimension.\r
+  @param  ColorDepth            Number of bits per pixel, currently defined to be 32.\r
+  @param  RefreshRate           The refresh rate of the monitor in Hertz.\r
 \r
-  @retval EFI_SUCCESS             Mode information returned.\r
-  @retval EFI_NOT_STARTED         Video display is not initialized. Call SetMode ()\r
-  @retval EFI_INVALID_PARAMETER   One of the input args was NULL.\r
+  @retval EFI_SUCCESS           Mode information returned.\r
+  @retval EFI_NOT_STARTED       Video display is not initialized. Call SetMode ()\r
+  @retval EFI_INVALID_PARAMETER One of the input args was NULL.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2080,21 +2025,20 @@ ConSpliterUgaDrawGetMode (
   OUT UINT32                          *VerticalResolution,\r
   OUT UINT32                          *ColorDepth,\r
   OUT UINT32                          *RefreshRate\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Return the current video mode information.\r
+  Set the current video mode information.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  HorizontalResolution    Current video horizontal resolution in pixels\r
-  @param  VerticalResolution      Current video vertical resolution in pixels\r
-  @param  ColorDepth              Current video color depth in bits per pixel\r
-  @param  RefreshRate             Current video refresh rate in Hz.\r
+  @param  This                 The EFI_UGA_DRAW_PROTOCOL instance.\r
+  @param  HorizontalResolution The size of video screen in pixels in the X dimension.\r
+  @param  VerticalResolution   The size of video screen in pixels in the Y dimension.\r
+  @param  ColorDepth           Number of bits per pixel, currently defined to be 32.\r
+  @param  RefreshRate          The refresh rate of the monitor in Hertz.\r
 \r
-  @retval EFI_SUCCESS             Mode information returned.\r
-  @retval EFI_NOT_STARTED         Video display is not initialized. Call SetMode ()\r
-  @retval EFI_OUT_OF_RESOURCES    Out of resources.\r
+  @retval EFI_SUCCESS          Mode information returned.\r
+  @retval EFI_NOT_STARTED      Video display is not initialized. Call SetMode ()\r
+  @retval EFI_OUT_OF_RESOURCES Out of resources.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2105,53 +2049,55 @@ ConSpliterUgaDrawSetMode (
   IN UINT32                           VerticalResolution,\r
   IN UINT32                           ColorDepth,\r
   IN UINT32                           RefreshRate\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  The following table defines actions for BltOperations.\r
-\r
-  EfiUgaVideoFill - Write data from the  BltBuffer pixel (SourceX, SourceY)\r
-  directly to every pixel of the video display rectangle\r
-  (DestinationX, DestinationY)\r
-  (DestinationX + Width, DestinationY + Height).\r
-  Only one pixel will be used from the BltBuffer. Delta is NOT used.\r
-  EfiUgaVideoToBltBuffer - Read data from the video display rectangle\r
-  (SourceX, SourceY) (SourceX + Width, SourceY + Height) and place it in\r
-  the BltBuffer rectangle (DestinationX, DestinationY )\r
-  (DestinationX + Width, DestinationY + Height). If DestinationX or\r
-  DestinationY is not zero then Delta must be set to the length in bytes\r
-  of a row in the BltBuffer.\r
-  EfiUgaBltBufferToVideo - Write data from the  BltBuffer rectangle\r
-  (SourceX, SourceY) (SourceX + Width, SourceY + Height) directly to the\r
-  video display rectangle (DestinationX, DestinationY)\r
-  (DestinationX + Width, DestinationY + Height). If SourceX or SourceY is\r
-  not zero then Delta must be set to the length in bytes of a row in the\r
-  BltBuffer.\r
-  EfiUgaVideoToVideo - Copy from the video display rectangle\r
-  (SourceX, SourceY) (SourceX + Width, SourceY + Height) .\r
-  to the video display rectangle (DestinationX, DestinationY)\r
-  (DestinationX + Width, DestinationY + Height).\r
-  The BltBuffer and Delta  are not used in this mode.\r
+  Blt a rectangle of pixels on the graphics screen.\r
 \r
-  @param  This                    Protocol instance pointer.\r
-  @param  BltBuffer               Buffer containing data to blit into video buffer.\r
-                                  This buffer has a size of\r
-                                  Width*Height*sizeof(EFI_UGA_PIXEL)\r
-  @param  BltOperation            Operation to perform on BlitBuffer and video\r
-                                  memory\r
-  @param  SourceX                 X coordinate of source for the BltBuffer.\r
-  @param  SourceY                 Y coordinate of source for the BltBuffer.\r
-  @param  DestinationX            X coordinate of destination for the BltBuffer.\r
-  @param  DestinationY            Y coordinate of destination for the BltBuffer.\r
-  @param  Width                   Width of rectangle in BltBuffer in pixels.\r
-  @param  Height                  Hight of rectangle in BltBuffer in pixels. \r
-  @param  Delta                   OPTIONAL.\r
+  The following table defines actions for BltOperations.\r
 \r
-  @retval EFI_SUCCESS             The Blt operation completed.\r
-  @retval EFI_INVALID_PARAMETER   BltOperation is not valid.\r
-  @retval EFI_DEVICE_ERROR        A hardware error occured writting to the video\r
-                                  buffer.\r
+  EfiUgaVideoFill:\r
+    Write data from the  BltBuffer pixel (SourceX, SourceY)\r
+    directly to every pixel of the video display rectangle\r
+    (DestinationX, DestinationY)\r
+    (DestinationX + Width, DestinationY + Height).\r
+    Only one pixel will be used from the BltBuffer. Delta is NOT used.\r
+  EfiUgaVideoToBltBuffer:\r
+    Read data from the video display rectangle\r
+    (SourceX, SourceY) (SourceX + Width, SourceY + Height) and place it in\r
+    the BltBuffer rectangle (DestinationX, DestinationY )\r
+    (DestinationX + Width, DestinationY + Height). If DestinationX or\r
+    DestinationY is not zero then Delta must be set to the length in bytes\r
+    of a row in the BltBuffer.\r
+  EfiUgaBltBufferToVideo:\r
+    Write data from the  BltBuffer rectangle\r
+    (SourceX, SourceY) (SourceX + Width, SourceY + Height) directly to the\r
+    video display rectangle (DestinationX, DestinationY)\r
+    (DestinationX + Width, DestinationY + Height). If SourceX or SourceY is\r
+    not zero then Delta must be set to the length in bytes of a row in the\r
+    BltBuffer.\r
+  EfiUgaVideoToVideo:\r
+    Copy from the video display rectangle\r
+    (SourceX, SourceY) (SourceX + Width, SourceY + Height) .\r
+    to the video display rectangle (DestinationX, DestinationY)\r
+    (DestinationX + Width, DestinationY + Height).\r
+    The BltBuffer and Delta  are not used in this mode.\r
+\r
+  @param  This           Protocol instance pointer.\r
+  @param  BltBuffer      Buffer containing data to blit into video buffer. This\r
+                         buffer has a size of Width*Height*sizeof(EFI_UGA_PIXEL)\r
+  @param  BltOperation   Operation to perform on BlitBuffer and video memory\r
+  @param  SourceX        X coordinate of source for the BltBuffer.\r
+  @param  SourceY        Y coordinate of source for the BltBuffer.\r
+  @param  DestinationX   X coordinate of destination for the BltBuffer.\r
+  @param  DestinationY   Y coordinate of destination for the BltBuffer.\r
+  @param  Width          Width of rectangle in BltBuffer in pixels.\r
+  @param  Height         Hight of rectangle in BltBuffer in pixels.\r
+  @param  Delta          OPTIONAL\r
+\r
+  @retval EFI_SUCCESS            The Blt operation completed.\r
+  @retval EFI_INVALID_PARAMETER  BltOperation is not valid.\r
+  @retval EFI_DEVICE_ERROR       A hardware error occured writting to the video buffer.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2167,11 +2113,10 @@ ConSpliterUgaDrawBlt (
   IN  UINTN                                         Width,\r
   IN  UINTN                                         Height,\r
   IN  UINTN                                         Delta         OPTIONAL\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Write data from the buffer to video display based on UGA Draw setting. \r
+  Write data from the buffer to video display based on UGA Draw setting.\r
 \r
   @param  Private                 Consplitter Text Out pointer.\r
   @param  GraphicsOutput          Graphics Output protocol pointer.\r
@@ -2181,15 +2126,14 @@ ConSpliterUgaDrawBlt (
   @retval EFI_SUCCESS             The Blt operation completed.\r
   @retval EFI_INVALID_PARAMETER   BltOperation is not valid.\r
   @retval EFI_DEVICE_ERROR        A hardware error occured writting to the video buffer.\r
-                  \r
+\r
 **/\r
 EFI_STATUS\r
 DevNullUgaSync (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  EFI_GRAPHICS_OUTPUT_PROTOCOL    *GraphicsOutput,\r
   IN  EFI_UGA_DRAW_PROTOCOL           *UgaDraw\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Write a Unicode string to the output device.\r
@@ -2215,13 +2159,12 @@ EFI_STATUS
 DevNullTextOutOutputString (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  CHAR16                          *WString\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Sets the output device(s) to a specified mode.\r
 \r
-  @param  Private                 Private data structure pointer.\r
+  @param  Private                 Text Out Splitter pointer.\r
   @param  ModeNumber              The mode number to set.\r
 \r
   @retval EFI_SUCCESS             The requested text mode was set.\r
@@ -2235,14 +2178,13 @@ EFI_STATUS
 DevNullTextOutSetMode (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  UINTN                           ModeNumber\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Clears the output device(s) display to the currently selected background\r
   color.\r
 \r
-  @param  Private                 Protocol instance pointer.\r
+  @param  Private                 Text Out Splitter pointer.\r
 \r
   @retval EFI_SUCCESS             The operation completed successfully.\r
   @retval EFI_DEVICE_ERROR        The device had an error and could not complete\r
@@ -2253,24 +2195,20 @@ DevNullTextOutSetMode (
 EFI_STATUS\r
 DevNullTextOutClearScreen (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Sets the current coordinates of the cursor position.\r
+  Sets the current coordinates of the cursor position on NULL device.\r
 \r
-  @param  Private                 Protocol instance pointer.\r
-  @param  Column                  \r
-  @param  Row                     the position to set the cursor to. Must be\r
-                                  greater than or equal to zero and less than the\r
-                                  number of columns and rows by QueryMode ().\r
+  @param  Private                  Text Out Splitter pointer.\r
+  @param  Column                   The column position to set the cursor to. Must be\r
+                                   greater than or equal to zero and less than the\r
+                                   number of columns by QueryMode ().\r
+  @param  Row                      The row position to set the cursor to. Must be\r
+                                   greater than or equal to zero and less than the\r
+                                   number of rows by QueryMode ().\r
 \r
-  @retval EFI_SUCCESS             The operation completed successfully.\r
-  @retval EFI_DEVICE_ERROR        The device had an error and could not complete\r
-                                  the request.\r
-  @retval EFI_UNSUPPORTED         The output device is not in a valid text mode, or\r
-                                  the cursor position is invalid for the current\r
-                                  mode.\r
+  @retval EFI_SUCCESS              Always returned.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -2278,32 +2216,29 @@ DevNullTextOutSetCursorPosition (
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  UINTN                           Column,\r
   IN  UINTN                           Row\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
-  Implements SIMPLE_TEXT_OUTPUT.EnableCursor().\r
-  In this driver, the cursor cannot be hidden.\r
+  Set cursor visibility property on NULL device.\r
 \r
-  @param  Private                 Indicates the calling context.\r
+  @param  Private                 Text Out Splitter pointer.\r
   @param  Visible                 If TRUE, the cursor is set to be visible, If\r
                                   FALSE, the cursor is set to be invisible.\r
 \r
-  @retval EFI_SUCCESS             The request is valid.\r
+  @retval EFI_SUCCESS             Always returned.\r
 \r
 **/\r
 EFI_STATUS\r
 DevNullTextOutEnableCursor (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
   IN  BOOLEAN                         Visible\r
-  )\r
-;\r
+  );\r
 \r
 /**\r
   Take the DevNull TextOut device and update the Simple Text Out on every\r
   UGA device.\r
 \r
-  @param  Private                 Indicates the calling context.\r
+  @param  Private                 Text Out Splitter pointer.\r
 \r
   @retval EFI_SUCCESS             The request is valid.\r
   @retval other                   Return status of TextOut->OutputString ()\r
@@ -2312,7 +2247,6 @@ DevNullTextOutEnableCursor (
 EFI_STATUS\r
 DevNullSyncStdOut (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private\r
-  )\r
-;\r
+  );\r
 \r
 #endif\r