MdeModulePkg/TerminalDxe: Replace macro with enum for terminal types

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
index a209bf3cacf8e729432c9677af42f5030ee5de94..cf7f4e6ebc13a363dcfa2eea1875f138ffae05cc 100644 (file)
--- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.c
@@ -2,7 +2,7 @@
   Produces Simple Text Input Protocol, Simple Text Input Extended Protocol and\r
   Simple Text Output Protocol upon Serial IO Protocol.\r
 \r
-Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2017, 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
@@ -255,7 +255,7 @@ BuildTerminalDevpath  (
   )\r
 {\r
   EFI_DEVICE_PATH_PROTOCOL          *TerminalDevicePath;\r
-  UINT8                             TerminalType;\r
+  TERMINAL_TYPE                     TerminalType;\r
   VENDOR_DEVICE_PATH                *Node;\r
   EFI_STATUS                        Status;\r
 \r
@@ -270,23 +270,23 @@ BuildTerminalDevpath  (
 \r
   } else if (CompareGuid (&Node->Guid, &gEfiPcAnsiGuid)) {\r
 \r
-    TerminalType = PCANSITYPE;\r
+    TerminalType = TerminalTypePcAnsi;\r
 \r
   } else if (CompareGuid (&Node->Guid, &gEfiVT100Guid)) {\r
 \r
-    TerminalType = VT100TYPE;\r
+    TerminalType = TerminalTypeVt100;\r
 \r
   } else if (CompareGuid (&Node->Guid, &gEfiVT100PlusGuid)) {\r
 \r
-    TerminalType = VT100PLUSTYPE;\r
+    TerminalType = TerminalTypeVt100Plus;\r
 \r
   } else if (CompareGuid (&Node->Guid, &gEfiVTUTF8Guid)) {\r
 \r
-    TerminalType = VTUTF8TYPE;\r
+    TerminalType = TerminalTypeVtUtf8;\r
 \r
   } else if (CompareGuid (&Node->Guid, &gEfiTtyTermGuid)) {\r
 \r
-    TerminalType = TTYTERMTYPE;\r
+    TerminalType = TerminalTypeTtyTerm;\r
 \r
   } else {\r
     return NULL;\r
@@ -549,7 +549,7 @@ TerminalDriverBindingStart (
   EFI_SERIAL_IO_MODE                  *Mode;\r
   UINTN                               SerialInTimeOut;\r
   TERMINAL_DEV                        *TerminalDevice;\r
-  UINT8                               TerminalType;\r
+  TERMINAL_TYPE                       TerminalType;\r
   EFI_OPEN_PROTOCOL_INFORMATION_ENTRY *OpenInfoBuffer;\r
   UINTN                               EntryCount;\r
   UINTN                               Index;\r
@@ -713,9 +713,9 @@ TerminalDriverBindingStart (
     if (RemainingDevicePath == NULL) {\r
       TerminalType = PcdGet8 (PcdDefaultTerminalType);\r
       //\r
-      // Must be between PCANSITYPE (0) and TTYTERMTYPE (4)\r
+      // Must be between TerminalTypePcAnsi (0) and TerminalTypeTtyTerm (4)\r
       //\r
-      ASSERT (TerminalType <= TTYTERMTYPE);\r
+      ASSERT (TerminalType <= TerminalTypeTtyTerm);\r
     } else if (!IsDevicePathEnd (RemainingDevicePath)) {\r
       //\r
       // If RemainingDevicePath isn't the End of Device Path Node,\r
@@ -723,15 +723,15 @@ TerminalDriverBindingStart (
       //\r
       Node = (VENDOR_DEVICE_PATH *)RemainingDevicePath;\r
       if (CompareGuid (&Node->Guid, &gEfiPcAnsiGuid)) {\r
-        TerminalType = PCANSITYPE;\r
+        TerminalType = TerminalTypePcAnsi;\r
       } else if (CompareGuid (&Node->Guid, &gEfiVT100Guid)) {\r
-        TerminalType = VT100TYPE;\r
+        TerminalType = TerminalTypeVt100;\r
       } else if (CompareGuid (&Node->Guid, &gEfiVT100PlusGuid)) {\r
-        TerminalType = VT100PLUSTYPE;\r
+        TerminalType = TerminalTypeVt100Plus;\r
       } else if (CompareGuid (&Node->Guid, &gEfiVTUTF8Guid)) {\r
-        TerminalType = VTUTF8TYPE;\r
+        TerminalType = TerminalTypeVtUtf8;\r
       } else if (CompareGuid (&Node->Guid, &gEfiTtyTermGuid)) {\r
-        TerminalType = TTYTERMTYPE;\r
+        TerminalType = TerminalTypeTtyTerm;\r
       } else {\r
         goto Error;\r
       }\r
@@ -863,7 +863,7 @@ TerminalDriverBindingStart (
     //\r
     TerminalDevice->ControllerNameTable = NULL;\r
     switch (TerminalDevice->TerminalType) {\r
-    case PCANSITYPE:\r
+    case TerminalTypePcAnsi:\r
       AddUnicodeString2 (\r
         "eng",\r
         gTerminalComponentName.SupportedLanguages,\r
@@ -881,7 +881,7 @@ TerminalDriverBindingStart (
 \r
       break;\r
 \r
-    case VT100TYPE:\r
+    case TerminalTypeVt100:\r
       AddUnicodeString2 (\r
         "eng",\r
         gTerminalComponentName.SupportedLanguages,\r
@@ -899,7 +899,7 @@ TerminalDriverBindingStart (
 \r
       break;\r
 \r
-    case VT100PLUSTYPE:\r
+    case TerminalTypeVt100Plus:\r
       AddUnicodeString2 (\r
         "eng",\r
         gTerminalComponentName.SupportedLanguages,\r
@@ -917,7 +917,7 @@ TerminalDriverBindingStart (
 \r
       break;\r
 \r
-    case VTUTF8TYPE:\r
+    case TerminalTypeVtUtf8:\r
       AddUnicodeString2 (\r
         "eng",\r
         gTerminalComponentName.SupportedLanguages,\r
@@ -935,7 +935,7 @@ TerminalDriverBindingStart (
 \r
       break;\r
 \r
-    case TTYTERMTYPE:\r
+    case TerminalTypeTtyTerm:\r
       AddUnicodeString2 (\r
         "eng",\r
         gTerminalComponentName.SupportedLanguages,\r
@@ -1456,7 +1456,7 @@ TerminalUpdateConsoleDevVariable (
   EFI_STATUS                Status;\r
   UINTN                     NameSize;\r
   UINTN                     VariableSize;\r
-  UINT8                     TerminalType;\r
+  TERMINAL_TYPE             TerminalType;\r
   EFI_DEVICE_PATH_PROTOCOL  *Variable;\r
   EFI_DEVICE_PATH_PROTOCOL  *NewVariable;\r
   EFI_DEVICE_PATH_PROTOCOL  *TempDevicePath;\r
@@ -1473,7 +1473,7 @@ TerminalUpdateConsoleDevVariable (
   //\r
   // Append terminal device path onto the variable.\r
   //\r
-  for (TerminalType = PCANSITYPE; TerminalType <= TTYTERMTYPE; TerminalType++) {\r
+  for (TerminalType = TerminalTypePcAnsi; TerminalType <= TerminalTypeTtyTerm; TerminalType++) {\r
     SetTerminalDevicePath (TerminalType, ParentDevicePath, &TempDevicePath);\r
     NewVariable = AppendDevicePathInstance (Variable, TempDevicePath);\r
     ASSERT (NewVariable != NULL);\r
@@ -1548,7 +1548,7 @@ TerminalRemoveConsoleDevVariable (
   BOOLEAN                   Match;\r
   UINTN                     VariableSize;\r
   UINTN                     InstanceSize;\r
-  UINT8                     TerminalType;\r
+  TERMINAL_TYPE             TerminalType;\r
   EFI_DEVICE_PATH_PROTOCOL  *Instance;\r
   EFI_DEVICE_PATH_PROTOCOL  *Variable;\r
   EFI_DEVICE_PATH_PROTOCOL  *OriginalVariable;\r
@@ -1586,7 +1586,7 @@ TerminalRemoveConsoleDevVariable (
     // Loop through all the terminal types that this driver supports\r
     //\r
     Match = FALSE;\r
-    for (TerminalType = PCANSITYPE; TerminalType <= TTYTERMTYPE; TerminalType++) {\r
+    for (TerminalType = TerminalTypePcAnsi; TerminalType <= TerminalTypeTtyTerm; TerminalType++) {\r
 \r
       SetTerminalDevicePath (TerminalType, ParentDevicePath, &TempDevicePath);\r
 \r
@@ -1658,7 +1658,7 @@ TerminalRemoveConsoleDevVariable (
 **/\r
 EFI_STATUS\r
 SetTerminalDevicePath (\r
-  IN  UINT8                       TerminalType,\r
+  IN  TERMINAL_TYPE               TerminalType,\r
   IN  EFI_DEVICE_PATH_PROTOCOL    *ParentDevicePath,\r
   OUT EFI_DEVICE_PATH_PROTOCOL    **TerminalDevicePath\r
   )\r
@@ -1674,23 +1674,23 @@ SetTerminalDevicePath (
   //\r
   switch (TerminalType) {\r
 \r
-  case PCANSITYPE:\r
+  case TerminalTypePcAnsi:\r
     CopyGuid (&Node.Guid, &gEfiPcAnsiGuid);\r
     break;\r
 \r
-  case VT100TYPE:\r
+  case TerminalTypeVt100:\r
     CopyGuid (&Node.Guid, &gEfiVT100Guid);\r
     break;\r
 \r
-  case VT100PLUSTYPE:\r
+  case TerminalTypeVt100Plus:\r
     CopyGuid (&Node.Guid, &gEfiVT100PlusGuid);\r
     break;\r
 \r
-  case VTUTF8TYPE:\r
+  case TerminalTypeVtUtf8:\r
     CopyGuid (&Node.Guid, &gEfiVTUTF8Guid);\r
     break;\r
 \r
-  case TTYTERMTYPE:\r
+  case TerminalTypeTtyTerm:\r
     CopyGuid (&Node.Guid, &gEfiTtyTermGuid);\r
     break;\r
 \r

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.h b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.h
index e16b89c264ce2a203d519c5c70fa5667e26227df..c15d17cd0b65914e7e2c53e6bdfae7b3456de695 100644 (file)
--- a/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.h
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/Terminal.h
@@ -1,7 +1,7 @@
 /** @file\r
   Header file for Terminal driver.\r
 \r
-Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
 Copyright (C) 2016 Silicon Graphics, Inc. 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
@@ -81,10 +81,19 @@ typedef struct _TERMINAL_CONSOLE_IN_EX_NOTIFY {
   EFI_KEY_NOTIFY_FUNCTION               KeyNotificationFn;\r
   LIST_ENTRY                            NotifyEntry;\r
 } TERMINAL_CONSOLE_IN_EX_NOTIFY;\r
+\r
+typedef enum {\r
+  TerminalTypePcAnsi,\r
+  TerminalTypeVt100,\r
+  TerminalTypeVt100Plus,\r
+  TerminalTypeVtUtf8,\r
+  TerminalTypeTtyTerm\r
+} TERMINAL_TYPE;\r
+\r
 typedef struct {\r
   UINTN                               Signature;\r
   EFI_HANDLE                          Handle;\r
-  UINT8                               TerminalType;\r
+  TERMINAL_TYPE                       TerminalType;\r
   EFI_SERIAL_IO_PROTOCOL              *SerialIo;\r
   EFI_DEVICE_PATH_PROTOCOL            *DevicePath;\r
   EFI_SIMPLE_TEXT_INPUT_PROTOCOL      SimpleInput;\r
@@ -139,12 +148,6 @@ typedef union {
   UINT8 Utf8_3[3];\r
 } UTF8_CHAR;\r
 \r
-#define PCANSITYPE                0\r
-#define VT100TYPE                 1\r
-#define VT100PLUSTYPE             2\r
-#define VTUTF8TYPE                3\r
-#define TTYTERMTYPE               4\r
-\r
 #define LEFTOPENBRACKET           0x5b  // '['\r
 #define ACAP                      0x41\r
 #define BCAP                      0x42\r
@@ -859,7 +862,7 @@ TerminalRemoveConsoleDevVariable (
 **/\r
 EFI_STATUS\r
 SetTerminalDevicePath (\r
-  IN  UINT8                       TerminalType,\r
+  IN  TERMINAL_TYPE               TerminalType,\r
   IN  EFI_DEVICE_PATH_PROTOCOL    *ParentDevicePath,\r
   OUT EFI_DEVICE_PATH_PROTOCOL    **TerminalDevicePath\r
   );\r

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
index 016241017a42693ec05ab57fa4d84953256cdd84..1392f16670305c24d9858a2d4dedc6141cb9e2eb 100644 (file)
--- a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConIn.c
@@ -2,7 +2,7 @@
   Implementation for EFI_SIMPLE_TEXT_INPUT_PROTOCOL protocol.\r
 \r
 (C) Copyright 2014 Hewlett-Packard Development Company, L.P.<BR>\r
-Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
 Copyright (C) 2016 Silicon Graphics, Inc. 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
@@ -453,15 +453,15 @@ TranslateRawDataToEfiKey (
 {\r
   switch (TerminalDevice->TerminalType) {\r
 \r
-  case PCANSITYPE:\r
-  case VT100TYPE:\r
-  case VT100PLUSTYPE:\r
-  case TTYTERMTYPE:\r
+  case TerminalTypePcAnsi:\r
+  case TerminalTypeVt100:\r
+  case TerminalTypeVt100Plus:\r
+  case TerminalTypeTtyTerm:\r
     AnsiRawDataToUnicode (TerminalDevice);\r
     UnicodeToEfiKey (TerminalDevice);\r
     break;\r
 \r
-  case VTUTF8TYPE:\r
+  case TerminalTypeVtUtf8:\r
     //\r
     // Process all the raw data in the RawFIFO,\r
     // put the processed key into UnicodeFIFO.\r
@@ -1405,8 +1405,8 @@ UnicodeToEfiKey (
         continue;\r
       }\r
 \r
-      if (UnicodeChar == 'O' && (TerminalDevice->TerminalType == VT100TYPE ||\r
-                                 TerminalDevice->TerminalType == TTYTERMTYPE)) {\r
+      if (UnicodeChar == 'O' && (TerminalDevice->TerminalType == TerminalTypeVt100 ||\r
+                                 TerminalDevice->TerminalType == TerminalTypeTtyTerm)) {\r
         TerminalDevice->InputState |= INPUT_STATE_O;\r
         TerminalDevice->ResetState = RESET_STATE_DEFAULT;\r
         continue;\r
@@ -1414,8 +1414,8 @@ UnicodeToEfiKey (
 \r
       Key.ScanCode = SCAN_NULL;\r
 \r
-      if (TerminalDevice->TerminalType == VT100PLUSTYPE ||\r
-          TerminalDevice->TerminalType == VTUTF8TYPE) {\r
+      if (TerminalDevice->TerminalType == TerminalTypeVt100Plus ||\r
+          TerminalDevice->TerminalType == TerminalTypeVtUtf8) {\r
         switch (UnicodeChar) {\r
         case '1':\r
           Key.ScanCode = SCAN_F1;\r
@@ -1519,7 +1519,7 @@ UnicodeToEfiKey (
 \r
       Key.ScanCode = SCAN_NULL;\r
 \r
-      if (TerminalDevice->TerminalType == VT100TYPE) {\r
+      if (TerminalDevice->TerminalType == TerminalTypeVt100) {\r
         switch (UnicodeChar) {\r
         case 'P':\r
           Key.ScanCode = SCAN_F1;\r
@@ -1554,7 +1554,7 @@ UnicodeToEfiKey (
         default :\r
           break;\r
         }\r
-      } else if (TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+      } else if (TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
         /* Also accept VT100 escape codes for F1-F4, HOME and END for TTY term */\r
         switch (UnicodeChar) {\r
         case 'P':\r
@@ -1596,11 +1596,11 @@ UnicodeToEfiKey (
 \r
       Key.ScanCode = SCAN_NULL;\r
 \r
-      if (TerminalDevice->TerminalType == PCANSITYPE    ||\r
-          TerminalDevice->TerminalType == VT100TYPE     ||\r
-          TerminalDevice->TerminalType == VT100PLUSTYPE ||\r
-          TerminalDevice->TerminalType == VTUTF8TYPE    ||\r
-          TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+      if (TerminalDevice->TerminalType == TerminalTypePcAnsi    ||\r
+          TerminalDevice->TerminalType == TerminalTypeVt100     ||\r
+          TerminalDevice->TerminalType == TerminalTypeVt100Plus ||\r
+          TerminalDevice->TerminalType == TerminalTypeVtUtf8    ||\r
+          TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
         switch (UnicodeChar) {\r
         case 'A':\r
           Key.ScanCode = SCAN_UP;\r
@@ -1615,104 +1615,104 @@ UnicodeToEfiKey (
           Key.ScanCode = SCAN_LEFT;\r
           break;\r
         case 'H':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE ||\r
-              TerminalDevice->TerminalType == VT100TYPE  ||\r
-              TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi ||\r
+              TerminalDevice->TerminalType == TerminalTypeVt100  ||\r
+              TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
             Key.ScanCode = SCAN_HOME;\r
           }\r
           break;\r
         case 'F':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE ||\r
-              TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi ||\r
+              TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
             Key.ScanCode = SCAN_END;\r
           }\r
           break;\r
         case 'K':\r
-          if (TerminalDevice->TerminalType == VT100TYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypeVt100) {\r
             Key.ScanCode = SCAN_END;\r
           }\r
           break;\r
         case 'L':\r
         case '@':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE ||\r
-              TerminalDevice->TerminalType == VT100TYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi ||\r
+              TerminalDevice->TerminalType == TerminalTypeVt100) {\r
             Key.ScanCode = SCAN_INSERT;\r
           }\r
           break;\r
         case 'X':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_DELETE;\r
           }\r
           break;\r
         case 'P':\r
-          if (TerminalDevice->TerminalType == VT100TYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypeVt100) {\r
             Key.ScanCode = SCAN_DELETE;\r
-          } else if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          } else if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F4;\r
           }\r
           break;\r
         case 'I':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_PAGE_UP;\r
           }\r
           break;\r
         case 'V':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F10;\r
           }\r
           break;\r
         case '?':\r
-          if (TerminalDevice->TerminalType == VT100TYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypeVt100) {\r
             Key.ScanCode = SCAN_PAGE_UP;\r
           }\r
           break;\r
         case 'G':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_PAGE_DOWN;\r
           }\r
           break;\r
         case 'U':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F9;\r
           }\r
           break;\r
         case '/':\r
-          if (TerminalDevice->TerminalType == VT100TYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypeVt100) {\r
             Key.ScanCode = SCAN_PAGE_DOWN;\r
           }\r
           break;\r
         case 'M':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F1;\r
           }\r
           break;\r
         case 'N':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F2;\r
           }\r
           break;\r
         case 'O':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F3;\r
           }\r
           break;\r
         case 'Q':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F5;\r
           }\r
           break;\r
         case 'R':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F6;\r
           }\r
           break;\r
         case 'S':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F7;\r
           }\r
           break;\r
         case 'T':\r
-          if (TerminalDevice->TerminalType == PCANSITYPE) {\r
+          if (TerminalDevice->TerminalType == TerminalTypePcAnsi) {\r
             Key.ScanCode = SCAN_F8;\r
           }\r
           break;\r
@@ -1726,7 +1726,7 @@ UnicodeToEfiKey (
        * numeric codes, and there are no ambiguous prefixes shared with\r
        * other terminal types.\r
        */\r
-      if (TerminalDevice->TerminalType == TTYTERMTYPE &&\r
+      if (TerminalDevice->TerminalType == TerminalTypeTtyTerm &&\r
           Key.ScanCode == SCAN_NULL &&\r
           UnicodeChar >= '0' &&\r
           UnicodeChar <= '9') {\r
@@ -1755,7 +1755,7 @@ UnicodeToEfiKey (
        * state is only used by the TTY terminal type.\r
        */\r
       Key.ScanCode = SCAN_NULL;\r
-      if (TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+      if (TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
 \r
         if (UnicodeChar == '~' && TerminalDevice->TtyEscapeIndex <= 2) {\r
           UINT16 EscCode;\r
@@ -1851,7 +1851,7 @@ UnicodeToEfiKey (
     }\r
 \r
     if (UnicodeChar == DEL) {\r
-      if (TerminalDevice->TerminalType == TTYTERMTYPE) {\r
+      if (TerminalDevice->TerminalType == TerminalTypeTtyTerm) {\r
         Key.ScanCode    = SCAN_NULL;\r
         Key.UnicodeChar = CHAR_BACKSPACE;\r
       }\r

diff --git a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
index c9b4ffc59ffc0c2012bba8de31b7d8ac7fc811c6..e677a76e6b19b7402c5527e7251fea94bea80cc2 100644 (file)
--- a/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
+++ b/MdeModulePkg/Universal/Console/TerminalDxe/TerminalConOut.c
@@ -1,7 +1,7 @@
 /** @file\r
   Implementation for EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL protocol.\r
 \r
-Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
 Copyright (C) 2016 Silicon Graphics, Inc. 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
@@ -224,10 +224,10 @@ TerminalConOutOutputString (
 \r
     switch (TerminalDevice->TerminalType) {\r
 \r
-    case PCANSITYPE:\r
-    case VT100TYPE:\r
-    case VT100PLUSTYPE:\r
-    case TTYTERMTYPE:\r
+    case TerminalTypePcAnsi:\r
+    case TerminalTypeVt100:\r
+    case TerminalTypeVt100Plus:\r
+    case TerminalTypeTtyTerm:\r
 \r
       if (!TerminalIsValidTextGraphics (*WString, &GraphicChar, &AsciiChar)) {\r
         //\r
@@ -253,7 +253,7 @@ TerminalConOutOutputString (
 \r
       }\r
 \r
-      if (TerminalDevice->TerminalType != PCANSITYPE) {\r
+      if (TerminalDevice->TerminalType != TerminalTypePcAnsi) {\r
         GraphicChar = AsciiChar;\r
       }\r
 \r
@@ -271,7 +271,7 @@ TerminalConOutOutputString (
 \r
       break;\r
 \r
-    case VTUTF8TYPE:\r
+    case TerminalTypeVtUtf8:\r
       UnicodeToUtf8 (*WString, &Utf8Char, &ValidBytes);\r
       Length = ValidBytes;\r
       Status = TerminalDevice->SerialIo->Write (\r
@@ -317,7 +317,7 @@ TerminalConOutOutputString (
           Mode->CursorRow++;\r
         }\r
 \r
-        if (TerminalDevice->TerminalType == TTYTERMTYPE &&\r
+        if (TerminalDevice->TerminalType == TerminalTypeTtyTerm &&\r
             !TerminalDevice->OutputEscChar) {\r
           //\r
           // We've written the last character on the line.  The\r
@@ -398,14 +398,14 @@ TerminalConOutTestString (
 \r
   switch (TerminalDevice->TerminalType) {\r
 \r
-  case PCANSITYPE:\r
-  case VT100TYPE:\r
-  case VT100PLUSTYPE:\r
-  case TTYTERMTYPE:\r
+  case TerminalTypePcAnsi:\r
+  case TerminalTypeVt100:\r
+  case TerminalTypeVt100Plus:\r
+  case TerminalTypeTtyTerm:\r
     Status = AnsiTestString (TerminalDevice, WString);\r
     break;\r
 \r
-  case VTUTF8TYPE:\r
+  case TerminalTypeVtUtf8:\r
     Status = VTUTF8TestString (TerminalDevice, WString);\r
     break;\r
 \r
@@ -791,7 +791,7 @@ TerminalConOutSetCursorPosition (
   // within the current line if possible, and don't output anyting if\r
   // it isn't necessary.\r
   //\r
-  if (TerminalDevice->TerminalType == TTYTERMTYPE &&\r
+  if (TerminalDevice->TerminalType == TerminalTypeTtyTerm &&\r
       (UINTN)Mode->CursorRow == Row) {\r
     if ((UINTN)Mode->CursorColumn > Column) {\r
       mCursorBackwardString[FW_BACK_OFFSET + 0] = (CHAR16) ('0' + ((Mode->CursorColumn - Column) / 10));\r