]> git.proxmox.com Git - mirror_edk2.git/blobdiff - IntelFrameworkModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
IntelFrameworkModulePkg BdsDxe: Remove redundant functions
[mirror_edk2.git] / IntelFrameworkModulePkg / Universal / BdsDxe / BootMaint / ConsoleOption.c
index 1854b98c360fdd45ba00cee2c1c3aa6e5e9e38da..33b53bf3b673c33dbc1dc0cd9cac8720021932d4 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
   handles console redirection from boot manager\r
 \r
-Copyright (c) 2004 - 2014, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2018, 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
@@ -32,7 +32,7 @@ UART_FLOW_CONTROL_DEVICE_PATH mFlowControlDevicePath =
   Check the device path node whether it's the Flow Control node or not.\r
 \r
   @param[in] FlowControl    The device path node to be checked.\r
-  \r
+\r
   @retval TRUE              It's the Flow Control node.\r
   @retval FALSE             It's not.\r
 \r
@@ -85,11 +85,11 @@ UpdateComAttributeFromVariable (
 \r
 /**\r
   Update the multi-instance device path of Terminal Device based on\r
-  the global TerminalMenu. If ChangeTernimal is TRUE, the terminal \r
+  the global TerminalMenu. If ChangeTernimal is TRUE, the terminal\r
   device path in the Terminal Device in TerminalMenu is also updated.\r
 \r
   @param DevicePath      The multi-instance device path.\r
-  @param ChangeTerminal  TRUE, then device path in the Terminal Device \r
+  @param ChangeTerminal  TRUE, then device path in the Terminal Device\r
                          in TerminalMenu is also updated; FALSE, no update.\r
 \r
   @return EFI_SUCCESS    The function completes successfully.\r
@@ -218,71 +218,7 @@ ChangeTerminalDevicePath (
 \r
 }\r
 \r
-/**\r
-  Update the device path that describing a terminal device\r
-  based on the new BaudRate, Data Bits, parity and Stop Bits\r
-  set.\r
-\r
-  @param DevicePath terminal device's path\r
-\r
-**/\r
-VOID\r
-ChangeVariableDevicePath (\r
-  IN OUT EFI_DEVICE_PATH_PROTOCOL  *DevicePath\r
-  )\r
-{\r
-  EFI_DEVICE_PATH_PROTOCOL  *Node;\r
-  ACPI_HID_DEVICE_PATH      *Acpi;\r
-  UART_DEVICE_PATH          *Uart;\r
-  UINTN                     Com;\r
-  BM_TERMINAL_CONTEXT       *NewTerminalContext;\r
-  BM_MENU_ENTRY             *NewMenuEntry;\r
-\r
-  Node  = DevicePath;\r
-  Node  = NextDevicePathNode (Node);\r
-  Com   = 0;\r
-  while (!IsDevicePathEnd (Node)) {\r
-    Acpi = (ACPI_HID_DEVICE_PATH *) Node;\r
-    if (IsIsaSerialNode (Acpi)) {\r
-      CopyMem (&Com, &Acpi->UID, sizeof (UINT32));\r
-    }\r
-\r
-    if ((DevicePathType (Node) == MESSAGING_DEVICE_PATH) && (DevicePathSubType (Node) == MSG_UART_DP)) {\r
-      NewMenuEntry = BOpt_GetMenuEntry (\r
-                      &TerminalMenu,\r
-                      Com\r
-                      );\r
-      ASSERT (NewMenuEntry != NULL);\r
-      NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
-      Uart                = (UART_DEVICE_PATH *) Node;\r
-      CopyMem (\r
-        &Uart->BaudRate,\r
-        &NewTerminalContext->BaudRate,\r
-        sizeof (UINT64)\r
-        );\r
-\r
-      CopyMem (\r
-        &Uart->DataBits,\r
-        &NewTerminalContext->DataBits,\r
-        sizeof (UINT8)\r
-        );\r
-\r
-      CopyMem (\r
-        &Uart->Parity,\r
-        &NewTerminalContext->Parity,\r
-        sizeof (UINT8)\r
-        );\r
-\r
-      CopyMem (\r
-        &Uart->StopBits,\r
-        &NewTerminalContext->StopBits,\r
-        sizeof (UINT8)\r
-        );\r
-    }\r
 \r
-    Node = NextDevicePathNode (Node);\r
-  }\r
-}\r
 \r
 /**\r
   Retrieve ACPI UID of UART from device path\r
@@ -760,7 +696,7 @@ UpdateComAttributeFromVariable (
                          and BM_CONSOLE_ERR_CONTEXT_SELECT.\r
 \r
   @retval EFI_UNSUPPORTED The type passed in is not in the 3 types defined.\r
-  @retval EFI_NOT_FOUND   If the EFI Variable defined in UEFI spec with name "ConOutDev", \r
+  @retval EFI_NOT_FOUND   If the EFI Variable defined in UEFI spec with name "ConOutDev",\r
                           "ConInDev" or "ConErrDev" doesn't exists.\r
   @retval EFI_OUT_OF_RESOURCES Not enough resource to complete the operations.\r
   @retval EFI_SUCCESS          Function completes successfully.\r
@@ -1057,24 +993,24 @@ GetConsoleOutMode (
 \r
   @param CallbackData    The BMM context data.\r
 \r
-**/  \r
-VOID  \r
+**/\r
+VOID\r
 GetConsoleInCheck (\r
   IN  BMM_CALLBACK_DATA    *CallbackData\r
   )\r
 {\r
   UINT16              Index;\r
-  BM_MENU_ENTRY       *NewMenuEntry; \r
+  BM_MENU_ENTRY       *NewMenuEntry;\r
   UINT8               *ConInCheck;\r
   BM_CONSOLE_CONTEXT  *NewConsoleContext;\r
-  \r
+\r
   ASSERT (CallbackData != NULL);\r
-    \r
+\r
   ConInCheck = &CallbackData->BmmFakeNvData.ConsoleInCheck[0];\r
   for (Index = 0; ((Index < ConsoleInpMenu.MenuNumber) && \\r
-       (Index < MAX_MENU_NUMBER)) ; Index++) {  \r
+       (Index < MAX_MENU_NUMBER)) ; Index++) {\r
     NewMenuEntry      = BOpt_GetMenuEntry (&ConsoleInpMenu, Index);\r
-    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;  \r
+    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;\r
     ConInCheck[Index] = NewConsoleContext->IsActive;\r
   }\r
 }\r
@@ -1086,27 +1022,27 @@ GetConsoleInCheck (
 \r
   @param CallbackData    The BMM context data.\r
 \r
-**/      \r
-VOID    \r
+**/\r
+VOID\r
 GetConsoleOutCheck (\r
   IN  BMM_CALLBACK_DATA    *CallbackData\r
   )\r
 {\r
   UINT16              Index;\r
-  BM_MENU_ENTRY       *NewMenuEntry; \r
+  BM_MENU_ENTRY       *NewMenuEntry;\r
   UINT8               *ConOutCheck;\r
   BM_CONSOLE_CONTEXT  *NewConsoleContext;\r
-  \r
+\r
   ASSERT (CallbackData != NULL);\r
   ConOutCheck = &CallbackData->BmmFakeNvData.ConsoleOutCheck[0];\r
   for (Index = 0; ((Index < ConsoleOutMenu.MenuNumber) && \\r
-       (Index < MAX_MENU_NUMBER)) ; Index++) {  \r
+       (Index < MAX_MENU_NUMBER)) ; Index++) {\r
     NewMenuEntry      = BOpt_GetMenuEntry (&ConsoleOutMenu, Index);\r
-    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;  \r
+    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;\r
     ConOutCheck[Index] = NewConsoleContext->IsActive;\r
   }\r
 }\r
-  \r
+\r
 /**\r
 \r
   Initialize standard error output device check box to ConsoleErrCheck[MAX_MENU_NUMBER]\r
@@ -1114,23 +1050,23 @@ GetConsoleOutCheck (
 \r
   @param CallbackData    The BMM context data.\r
 \r
-**/        \r
-VOID  \r
+**/\r
+VOID\r
 GetConsoleErrCheck (\r
   IN  BMM_CALLBACK_DATA    *CallbackData\r
   )\r
 {\r
   UINT16              Index;\r
-  BM_MENU_ENTRY       *NewMenuEntry; \r
+  BM_MENU_ENTRY       *NewMenuEntry;\r
   UINT8               *ConErrCheck;\r
   BM_CONSOLE_CONTEXT  *NewConsoleContext;\r
-  \r
+\r
   ASSERT (CallbackData != NULL);\r
   ConErrCheck = &CallbackData->BmmFakeNvData.ConsoleErrCheck[0];\r
   for (Index = 0; ((Index < ConsoleErrMenu.MenuNumber) && \\r
-       (Index < MAX_MENU_NUMBER)) ; Index++) {  \r
+       (Index < MAX_MENU_NUMBER)) ; Index++) {\r
     NewMenuEntry      = BOpt_GetMenuEntry (&ConsoleErrMenu, Index);\r
-    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;  \r
+    NewConsoleContext = (BM_CONSOLE_CONTEXT *) NewMenuEntry->VariableContext;\r
     ConErrCheck[Index] = NewConsoleContext->IsActive;\r
   }\r
 }\r
@@ -1142,23 +1078,23 @@ GetConsoleErrCheck (
 \r
   @param CallbackData    The BMM context data.\r
 \r
-**/        \r
-VOID  \r
+**/\r
+VOID\r
 GetTerminalAttribute (\r
   IN  BMM_CALLBACK_DATA    *CallbackData\r
   )\r
 {\r
   BMM_FAKE_NV_DATA     *CurrentFakeNVMap;\r
   BM_MENU_ENTRY        *NewMenuEntry;\r
-  BM_TERMINAL_CONTEXT  *NewTerminalContext;    \r
-  UINT16               TerminalIndex;  \r
+  BM_TERMINAL_CONTEXT  *NewTerminalContext;\r
+  UINT16               TerminalIndex;\r
   UINT8                AttributeIndex;\r
 \r
   ASSERT (CallbackData != NULL);\r
-  \r
-  CurrentFakeNVMap = &CallbackData->BmmFakeNvData;     \r
+\r
+  CurrentFakeNVMap = &CallbackData->BmmFakeNvData;\r
   for (TerminalIndex = 0; ((TerminalIndex < TerminalMenu.MenuNumber) && \\r
-       (TerminalIndex < MAX_MENU_NUMBER)); TerminalIndex++) {  \r
+       (TerminalIndex < MAX_MENU_NUMBER)); TerminalIndex++) {\r
     NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, TerminalIndex);\r
     NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
     for (AttributeIndex = 0; AttributeIndex < sizeof (BaudRateList) / sizeof (BaudRateList [0]); AttributeIndex++) {\r
@@ -1167,32 +1103,32 @@ GetTerminalAttribute (
         break;\r
       }\r
     }\r
-    for (AttributeIndex = 0; AttributeIndex < sizeof (DataBitsList) / sizeof (DataBitsList[0]); AttributeIndex++) {\r
+    for (AttributeIndex = 0; AttributeIndex < ARRAY_SIZE (DataBitsList); AttributeIndex++) {\r
       if (NewTerminalContext->DataBits == (UINT64) (DataBitsList[AttributeIndex].Value)) {\r
         NewTerminalContext->DataBitsIndex = AttributeIndex;\r
         break;\r
       }\r
-    }    \r
-    \r
-    for (AttributeIndex = 0; AttributeIndex < sizeof (ParityList) / sizeof (ParityList[0]); AttributeIndex++) {\r
+    }\r
+\r
+    for (AttributeIndex = 0; AttributeIndex < ARRAY_SIZE (ParityList); AttributeIndex++) {\r
       if (NewTerminalContext->Parity == (UINT64) (ParityList[AttributeIndex].Value)) {\r
         NewTerminalContext->ParityIndex = AttributeIndex;\r
         break;\r
       }\r
-    }            \r
-    \r
-    for (AttributeIndex = 0; AttributeIndex < sizeof (StopBitsList) / sizeof (StopBitsList[0]); AttributeIndex++) {\r
+    }\r
+\r
+    for (AttributeIndex = 0; AttributeIndex < ARRAY_SIZE (StopBitsList); AttributeIndex++) {\r
       if (NewTerminalContext->StopBits == (UINT64) (StopBitsList[AttributeIndex].Value)) {\r
         NewTerminalContext->StopBitsIndex = AttributeIndex;\r
         break;\r
       }\r
-    }        \r
+    }\r
     CurrentFakeNVMap->COMBaudRate[TerminalIndex]     = NewTerminalContext->BaudRateIndex;\r
     CurrentFakeNVMap->COMDataRate[TerminalIndex]     = NewTerminalContext->DataBitsIndex;\r
     CurrentFakeNVMap->COMStopBits[TerminalIndex]     = NewTerminalContext->StopBitsIndex;\r
-    CurrentFakeNVMap->COMParity[TerminalIndex]       = NewTerminalContext->ParityIndex; \r
+    CurrentFakeNVMap->COMParity[TerminalIndex]       = NewTerminalContext->ParityIndex;\r
     CurrentFakeNVMap->COMTerminalType[TerminalIndex] = NewTerminalContext->TerminalType;\r
     CurrentFakeNVMap->COMFlowControl[TerminalIndex]  = NewTerminalContext->FlowControl;\r
-  } \r
+  }\r
 }\r
 \r