]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
Remove SafeFreePool from MemoryAllocationLib as this API's name is misleading. Its...
[mirror_edk2.git] / MdeModulePkg / Universal / BdsDxe / BootMaint / UpdatePage.c
index d1608bf528bef724b08bc6c652f376e54a79e03d..2ffb1ff037a7259751f37598e72dff5ade905175 100644 (file)
@@ -1,6 +1,7 @@
-/*++\r
+/** @file\r
+Dynamically update the pages.\r
 \r
-Copyright (c) 2004 - 2008, Intel Corporation\r
+Copyright (c) 2004 - 2008, Intel Corporation. <BR>\r
 All rights reserved. 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
@@ -9,42 +10,29 @@ http://opensource.org/licenses/bsd-license.php
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-Module Name:\r
-\r
-  UpdatePage.c\r
-\r
-Abstract:\r
-\r
-  Dynamically Update the pages\r
-\r
---*/\r
+**/\r
 \r
 #include "BootMaint.h"\r
 \r
-EFI_GUID gTerminalDriverGuid = {\r
-  0x10634d8e, 0x1c05, 0x46cb, {0xbb, 0xc, 0x5a, 0xfd, 0xc8, 0x29, 0xa8, 0xc8}\r
-};\r
+/**\r
+  Refresh the global UpdateData structure.\r
 \r
+**/\r
 VOID\r
 RefreshUpdateData (\r
   VOID\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Refresh the global UpdateData structure.\r
-\r
-Arguments:\r
-  None.\r
-\r
-Returns:\r
-  None.\r
-\r
---*/\r
 {\r
   gUpdateData.Offset = 0;\r
 }\r
 \r
+/**\r
+  Add a "Go back to main page" tag in front of the form when there are no\r
+  "Apply changes" and "Discard changes" tags in the end of the form.\r
\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdatePageStart (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -69,6 +57,13 @@ UpdatePageStart (
 \r
 }\r
 \r
+/**\r
+  Create the "Apply changes" and "Discard changes" tags. And\r
+  ensure user can return to the main page.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdatePageEnd (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -118,6 +113,15 @@ UpdatePageEnd (
     );\r
 }\r
 \r
+/**\r
+  Clean up the dynamic opcode at label and form specified by\r
+  both LabelId. \r
+\r
+  @param LabelId         It is both the Form ID and Label ID for\r
+                         opcode deletion.\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 CleanUpPage (\r
   IN UINT16                           LabelId,\r
@@ -139,6 +143,16 @@ CleanUpPage (
     );\r
 }\r
 \r
+/**\r
+  Boot a file selected by user at File Expoloer of BMM.\r
+\r
+  @param FileContext     The file context data, which contains the device path\r
+                         of the file to be boot from.\r
+\r
+  @retval EFI_SUCCESS    The function completed successfull.\r
+  @return                 Other value if the boot from the file fails.\r
+\r
+**/\r
 EFI_STATUS\r
 BootThisFile (\r
   IN BM_FILE_CONTEXT                   *FileContext\r
@@ -160,18 +174,20 @@ BootThisFile (
   //\r
   gST->ConOut->ClearScreen (gST->ConOut);\r
 \r
-  gBS->RaiseTPL (TPL_APPLICATION);\r
-\r
   ExitDataSize  = 0;\r
 \r
   Status        = BdsLibBootViaBootOption (Option, Option->DevicePath, &ExitDataSize, &ExitData);\r
 \r
-  gBS->RestoreTPL (TPL_APPLICATION);\r
-\r
   return Status;\r
 \r
 }\r
 \r
+/**\r
+  Create a list of Goto Opcode for all terminal devices logged\r
+  by TerminaMenu. This list will be inserted to form FORM_CON_COM_SETUP_ID.\r
+\r
+  @param CallbackData    The BMM context data.\r
+**/\r
 VOID\r
 UpdateConCOMPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -179,32 +195,35 @@ UpdateConCOMPage (
 {\r
   BM_MENU_ENTRY *NewMenuEntry;\r
   UINT16        Index;\r
-  EFI_STATUS    Status;\r
-  VOID          *Interface;\r
 \r
   CallbackData->BmmAskSaveOrNot = FALSE;\r
 \r
   UpdatePageStart (CallbackData);\r
 \r
-  Status = EfiLibLocateProtocol (&gTerminalDriverGuid, (VOID **) &Interface);\r
-  if (!EFI_ERROR (Status)) {\r
-    for (Index = 0; Index < TerminalMenu.MenuNumber; Index++) {\r
-      NewMenuEntry = BOpt_GetMenuEntry (&TerminalMenu, Index);\r
 \r
-      CreateGotoOpCode (\r
-        FORM_CON_COM_SETUP_ID,\r
-        NewMenuEntry->DisplayStringToken,\r
-        STRING_TOKEN (STR_NULL_STRING),\r
-        EFI_IFR_FLAG_CALLBACK,\r
-        (UINT16) (TERMINAL_OPTION_OFFSET + Index),\r
-        &gUpdateData\r
-        );\r
-    }\r
+  for (Index = 0; Index < TerminalMenu.MenuNumber; Index++) {\r
+    NewMenuEntry = BOpt_GetMenuEntry (&TerminalMenu, Index);\r
+\r
+    CreateGotoOpCode (\r
+      FORM_CON_COM_SETUP_ID,\r
+      NewMenuEntry->DisplayStringToken,\r
+      STRING_TOKEN (STR_NULL_STRING),\r
+      EFI_IFR_FLAG_CALLBACK,\r
+      (UINT16) (TERMINAL_OPTION_OFFSET + Index),\r
+      &gUpdateData\r
+      );\r
   }\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Create a lit of boot option from global BootOptionMenu. It\r
+  allow user to delete the boot option.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateBootDelPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -244,6 +263,12 @@ UpdateBootDelPage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Create a lit of driver option from global DriverMenu.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateDrvAddHandlePage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -272,6 +297,13 @@ UpdateDrvAddHandlePage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Create a lit of driver option from global DriverOptionMenu. It\r
+  allow user to delete the driver option.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateDrvDelPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -309,6 +341,13 @@ UpdateDrvDelPage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Prepare the page to allow user to add description for \r
+  a Driver Option.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateDriverAddHandleDescPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -371,6 +410,14 @@ UpdateDriverAddHandleDescPage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Update console page.\r
+\r
+  @param UpdatePageId    The form ID to be updated.\r
+  @param ConsoleMenu     The console menu list.\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateConsolePage (\r
   IN UINT16                           UpdatePageId,\r
@@ -384,9 +431,7 @@ UpdateConsolePage (
   UINT16              Index;\r
   UINT16              Index2;\r
   UINT8               CheckFlags;\r
-  EFI_STATUS          Status;\r
-  VOID                *Interface;\r
-\r
\r
   CallbackData->BmmAskSaveOrNot = TRUE;\r
 \r
   UpdatePageStart (CallbackData);\r
@@ -414,41 +459,47 @@ UpdateConsolePage (
       );\r
   }\r
 \r
-  Status = EfiLibLocateProtocol (&gTerminalDriverGuid, (VOID **) &Interface);\r
-  if (!EFI_ERROR (Status)) {\r
-    for (Index2 = 0; Index2 < TerminalMenu.MenuNumber; Index2++) {\r
-      CheckFlags          = 0;\r
-      NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, Index2);\r
-      NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
-\r
-      if ((NewTerminalContext->IsConIn && (UpdatePageId == FORM_CON_IN_ID)) ||\r
-          (NewTerminalContext->IsConOut && (UpdatePageId == FORM_CON_OUT_ID)) ||\r
-          (NewTerminalContext->IsStdErr && (UpdatePageId == FORM_CON_ERR_ID))\r
-          ) {\r
-        CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;\r
-        CallbackData->BmmFakeNvData.ConsoleCheck[Index] = TRUE;\r
-      } else {\r
-        CallbackData->BmmFakeNvData.ConsoleCheck[Index] = FALSE;\r
-      }\r
+  for (Index2 = 0; Index2 < TerminalMenu.MenuNumber; Index2++) {\r
+    CheckFlags          = 0;\r
+    NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, Index2);\r
+    NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
 \r
-      CreateCheckBoxOpCode (\r
-        (EFI_QUESTION_ID) (CON_DEVICE_QUESTION_ID + Index),\r
-        VARSTORE_ID_BOOT_MAINT,\r
-        (UINT16) (CON_DEVICE_VAR_OFFSET + Index),\r
-        NewMenuEntry->DisplayStringToken,\r
-        NewMenuEntry->HelpStringToken,\r
-        0,\r
-        CheckFlags,\r
-        &gUpdateData\r
-        );\r
-\r
-      Index++;\r
+    if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||\r
+        ((NewTerminalContext->IsConOut != 0) && (UpdatePageId == FORM_CON_OUT_ID)) ||\r
+        ((NewTerminalContext->IsStdErr != 0) && (UpdatePageId == FORM_CON_ERR_ID))\r
+        ) {\r
+      CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;\r
+      CallbackData->BmmFakeNvData.ConsoleCheck[Index] = TRUE;\r
+    } else {\r
+      CallbackData->BmmFakeNvData.ConsoleCheck[Index] = FALSE;\r
     }\r
+\r
+    CreateCheckBoxOpCode (\r
+      (EFI_QUESTION_ID) (CON_DEVICE_QUESTION_ID + Index),\r
+      VARSTORE_ID_BOOT_MAINT,\r
+      (UINT16) (CON_DEVICE_VAR_OFFSET + Index),\r
+      NewMenuEntry->DisplayStringToken,\r
+      NewMenuEntry->HelpStringToken,\r
+      0,\r
+      CheckFlags,\r
+      &gUpdateData\r
+      );\r
+\r
+    Index++;\r
   }\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Update the page's NV Map if user has changed the order\r
+  a list. This list can be Boot Order or Driver Order.\r
+\r
+  @param UpdatePageId    The form ID to be updated.\r
+  @param OptionMenu      The new list.\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateOrderPage (\r
   IN UINT16                           UpdatePageId,\r
@@ -468,7 +519,7 @@ UpdateOrderPage (
 \r
   ZeroMem (CallbackData->BmmFakeNvData.OptionOrder, 100);\r
 \r
-  IfrOptionList = EfiAllocateZeroPool (sizeof (IFR_OPTION) * OptionMenu->MenuNumber);\r
+  IfrOptionList = AllocateZeroPool (sizeof (IFR_OPTION) * OptionMenu->MenuNumber);\r
   if (NULL == IfrOptionList) {\r
     return ;\r
   }\r
@@ -498,7 +549,7 @@ UpdateOrderPage (
       );\r
   }\r
 \r
-  SafeFreePool (IfrOptionList);\r
+  FreePool (IfrOptionList);\r
 \r
   UpdatePageEnd (CallbackData);\r
 \r
@@ -509,6 +560,13 @@ UpdateOrderPage (
     );\r
 }\r
 \r
+/**\r
+  Create the dynamic page to allow user to set\r
+  the "BootNext" vaule.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateBootNextPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -528,7 +586,7 @@ UpdateBootNextPage (
   CreateMenuStringToken (CallbackData, CallbackData->BmmHiiHandle, &BootOptionMenu);\r
 \r
   if (NumberOfOptions > 0) {\r
-    IfrOptionList = EfiAllocateZeroPool ((NumberOfOptions + 1) * sizeof (IFR_OPTION));\r
+    IfrOptionList = AllocateZeroPool ((NumberOfOptions + 1) * sizeof (IFR_OPTION));\r
 \r
     ASSERT (IfrOptionList);\r
 \r
@@ -568,12 +626,19 @@ UpdateBootNextPage (
       &gUpdateData\r
       );\r
 \r
-    SafeFreePool (IfrOptionList);\r
+    FreePool (IfrOptionList);\r
   }\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Create the dynamic page to allow user to set\r
+  the "TimeOut" vaule.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateTimeOutPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -607,22 +672,16 @@ UpdateTimeOutPage (
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Refresh the text mode page\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateConModePage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
   )\r
-/*++\r
-\r
-Routine Description:\r
-  Refresh the text mode page\r
-\r
-Arguments:\r
-  CallbackData      - BMM_CALLBACK_DATA\r
-\r
-Returns:\r
-  None.\r
-\r
---*/\r
 {\r
   UINTN                         Mode;\r
   UINTN                         Index;\r
@@ -661,10 +720,10 @@ Returns:
     return;\r
   }\r
 \r
-  IfrOptionList       = EfiAllocateZeroPool (sizeof (IFR_OPTION) * ValidMode);\r
+  IfrOptionList       = AllocateZeroPool (sizeof (IFR_OPTION) * ValidMode);\r
   ASSERT(IfrOptionList != NULL);\r
 \r
-  ModeToken           = EfiAllocateZeroPool (sizeof (EFI_STRING_ID) * ValidMode);\r
+  ModeToken           = AllocateZeroPool (sizeof (EFI_STRING_ID) * ValidMode);\r
   ASSERT(ModeToken != NULL);\r
 \r
   //\r
@@ -688,7 +747,7 @@ Returns:
     UnicodeValueToString (RowString, 0, Row, 0);\r
     StrCat (ModeString, RowString);\r
 \r
-    IfrLibNewString (CallbackData->BmmHiiHandle, &ModeToken[Index], ModeString);\r
+    HiiLibNewString (CallbackData->BmmHiiHandle, &ModeToken[Index], ModeString);\r
 \r
     IfrOptionList[Index].StringToken  = ModeToken[Index];\r
     IfrOptionList[Index].Value.u16    = (UINT16) Mode;\r
@@ -712,12 +771,20 @@ Returns:
     ValidMode,\r
     &gUpdateData\r
     );\r
-  SafeFreePool (IfrOptionList);\r
-  SafeFreePool (ModeToken);\r
+  FreePool (IfrOptionList);\r
+  FreePool (ModeToken);\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Create the dynamic page which allows user to \r
+  set the property such as Baud Rate, Data Bits,\r
+  Parity, Stop Bits, Terminal Type.\r
+\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdateTerminalPage (\r
   IN BMM_CALLBACK_DATA                *CallbackData\r
@@ -744,12 +811,12 @@ UpdateTerminalPage (
 \r
   NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;\r
 \r
-  IfrOptionList       = EfiAllocateZeroPool (sizeof (IFR_OPTION) * 19);\r
+  IfrOptionList       = AllocateZeroPool (sizeof (IFR_OPTION) * 19);\r
   if (IfrOptionList == NULL) {\r
     return ;\r
   }\r
 \r
-  for (Index = 0; Index < 19; Index++) {\r
+  for (Index = 0; Index < sizeof (BaudRateList) / sizeof (BaudRateList [0]); Index++) {\r
     CheckFlags = 0;\r
     if (NewTerminalContext->BaudRate == (UINT64) (BaudRateList[Index].Value)) {\r
       CheckFlags |= EFI_IFR_OPTION_DEFAULT;\r
@@ -775,7 +842,7 @@ UpdateTerminalPage (
     &gUpdateData\r
     );\r
 \r
-  for (Index = 0; Index < 4; Index++) {\r
+  for (Index = 0; Index < sizeof (DataBitsList) / sizeof (DataBitsList[0]); Index++) {\r
     CheckFlags = 0;\r
 \r
     if (NewTerminalContext->DataBits == DataBitsList[Index].Value) {\r
@@ -802,7 +869,7 @@ UpdateTerminalPage (
     &gUpdateData\r
     );\r
 \r
-  for (Index = 0; Index < 5; Index++) {\r
+  for (Index = 0; Index < sizeof (ParityList) / sizeof (ParityList[0]); Index++) {\r
     CheckFlags = 0;\r
     if (NewTerminalContext->Parity == ParityList[Index].Value) {\r
       CheckFlags |= EFI_IFR_OPTION_DEFAULT;\r
@@ -828,7 +895,7 @@ UpdateTerminalPage (
     &gUpdateData\r
     );\r
 \r
-  for (Index = 0; Index < 3; Index++) {\r
+  for (Index = 0; Index < sizeof (StopBitsList) / sizeof (StopBitsList[0]); Index++) {\r
     CheckFlags = 0;\r
     if (NewTerminalContext->StopBits == StopBitsList[Index].Value) {\r
       CheckFlags |= EFI_IFR_OPTION_DEFAULT;\r
@@ -879,11 +946,19 @@ UpdateTerminalPage (
     &gUpdateData\r
     );\r
 \r
-  SafeFreePool (IfrOptionList);\r
+  FreePool (IfrOptionList);\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Dispatch the correct update page function to call based on\r
+  the UpdatePageId.\r
+\r
+  @param UpdatePageId    The form ID.\r
+  @param CallbackData    The BMM context data.\r
+\r
+**/\r
 VOID\r
 UpdatePageBody (\r
   IN UINT16                           UpdatePageId,\r
@@ -917,6 +992,16 @@ UpdatePageBody (
   }\r
 }\r
 \r
+/**\r
+  Get the index number (#### in Boot####) for the boot option pointed to a BBS legacy device type\r
+  specified by DeviceType.\r
+\r
+  @param DeviceType      The legacy device type. It can be floppy, network, harddisk, cdrom,\r
+                         etc.\r
+  @param OptionIndex     Returns the index number (#### in Boot####).\r
+  @param OptionSize      Return the size of the Boot### variable.\r
+\r
+**/\r
 VOID *\r
 GetLegacyBootOptionVar (\r
   IN  UINTN                            DeviceType,\r
@@ -978,17 +1063,26 @@ GetLegacyBootOptionVar (
         (BBS_BBS_DP == DevicePath->SubType)\r
         ) {\r
       *OptionIndex = OrderBuffer[Index];\r
-      SafeFreePool (OrderBuffer);\r
+      FreePool (OrderBuffer);\r
       return OptionBuffer;\r
     } else {\r
-      SafeFreePool (OptionBuffer);\r
+      FreePool (OptionBuffer);\r
     }\r
   }\r
 \r
-  SafeFreePool (OrderBuffer);\r
+  FreePool (OrderBuffer);\r
   return NULL;\r
 }\r
 \r
+/**\r
+  Create a dynamic page so that Legacy Device boot order\r
+  can be set for specified device type.\r
+\r
+  @param UpdatePageId    The form ID. It also spefies the legacy device type.\r
+  @param CallbackData    The BMM context data.\r
+\r
+\r
+**/\r
 VOID\r
 UpdateSetLegacyDeviceOrderPage (\r
   IN UINT16                           UpdatePageId,\r
@@ -1040,8 +1134,8 @@ UpdateSetLegacyDeviceOrderPage (
   case FORM_SET_FD_ORDER_ID:\r
     OptionMenu  = (BM_MENU_OPTION *) &LegacyFDMenu;\r
     Key         = (UINT16) LEGACY_FD_QUESTION_ID;\r
-    TypeStr     = StrFloppy;\r
-    TypeStrHelp = StrFloppyHelp;\r
+    TypeStr     = STR_FLOPPY;\r
+    TypeStrHelp = STR_FLOPPY_HELP;\r
     BbsType     = BBS_FLOPPY;\r
     LegacyOrder = CallbackData->BmmFakeNvData.LegacyFD;\r
     OldData     = CallbackData->BmmOldFakeNVData.LegacyFD;\r
@@ -1050,8 +1144,8 @@ UpdateSetLegacyDeviceOrderPage (
   case FORM_SET_HD_ORDER_ID:\r
     OptionMenu  = (BM_MENU_OPTION *) &LegacyHDMenu;\r
     Key         = (UINT16) LEGACY_HD_QUESTION_ID;\r
-    TypeStr     = StrHardDisk;\r
-    TypeStrHelp = StrHardDiskHelp;\r
+    TypeStr     = STR_HARDDISK;\r
+    TypeStrHelp = STR_HARDDISK_HELP;\r
     BbsType     = BBS_HARDDISK;\r
     LegacyOrder = CallbackData->BmmFakeNvData.LegacyHD;\r
     OldData     = CallbackData->BmmOldFakeNVData.LegacyHD;\r
@@ -1060,8 +1154,8 @@ UpdateSetLegacyDeviceOrderPage (
   case FORM_SET_CD_ORDER_ID:\r
     OptionMenu  = (BM_MENU_OPTION *) &LegacyCDMenu;\r
     Key         = (UINT16) LEGACY_CD_QUESTION_ID;\r
-    TypeStr     = StrCDROM;\r
-    TypeStrHelp = StrCDROMHelp;\r
+    TypeStr     = STR_CDROM;\r
+    TypeStrHelp = STR_CDROM_HELP;\r
     BbsType     = BBS_CDROM;\r
     LegacyOrder = CallbackData->BmmFakeNvData.LegacyCD;\r
     OldData     = CallbackData->BmmOldFakeNVData.LegacyCD;\r
@@ -1070,8 +1164,8 @@ UpdateSetLegacyDeviceOrderPage (
   case FORM_SET_NET_ORDER_ID:\r
     OptionMenu  = (BM_MENU_OPTION *) &LegacyNETMenu;\r
     Key         = (UINT16) LEGACY_NET_QUESTION_ID;\r
-    TypeStr     = StrNET;\r
-    TypeStrHelp = StrNETHelp;\r
+    TypeStr     = STR_NET;\r
+    TypeStrHelp = STR_NET_HELP;\r
     BbsType     = BBS_EMBED_NETWORK;\r
     LegacyOrder = CallbackData->BmmFakeNvData.LegacyNET;\r
     OldData     = CallbackData->BmmOldFakeNVData.LegacyNET;\r
@@ -1080,8 +1174,8 @@ UpdateSetLegacyDeviceOrderPage (
   case FORM_SET_BEV_ORDER_ID:\r
     OptionMenu  = (BM_MENU_OPTION *) &LegacyBEVMenu;\r
     Key         = (UINT16) LEGACY_BEV_QUESTION_ID;\r
-    TypeStr     = StrBEV;\r
-    TypeStrHelp = StrBEVHelp;\r
+    TypeStr     = STR_BEV;\r
+    TypeStrHelp = STR_BEV_HELP;\r
     BbsType     = BBS_BEV_DEVICE;\r
     LegacyOrder = CallbackData->BmmFakeNvData.LegacyBEV;\r
     OldData     = CallbackData->BmmOldFakeNVData.LegacyBEV;\r
@@ -1091,7 +1185,7 @@ UpdateSetLegacyDeviceOrderPage (
 \r
   CreateMenuStringToken (CallbackData, CallbackData->BmmHiiHandle, OptionMenu);\r
 \r
-  IfrOptionList = EfiAllocateZeroPool (sizeof (IFR_OPTION) * (OptionMenu->MenuNumber + 1));\r
+  IfrOptionList = AllocateZeroPool (sizeof (IFR_OPTION) * (OptionMenu->MenuNumber + 1));\r
   if (NULL == IfrOptionList) {\r
     return ;\r
   }\r
@@ -1117,7 +1211,7 @@ UpdateSetLegacyDeviceOrderPage (
   // Get Device Order from variable\r
   //\r
   VarData = BdsLibGetVariableAndSize (\r
-              VarLegacyDevOrder,\r
+              VAR_LEGACY_DEV_ORDER,\r
               &EfiLegacyDevOrderGuid,\r
               &VarSize\r
               );\r
@@ -1142,11 +1236,11 @@ UpdateSetLegacyDeviceOrderPage (
       //\r
       UnicodeSPrint (String, sizeof (String), TypeStr, Index);\r
       StrRef = 0;\r
-      IfrLibNewString (CallbackData->BmmHiiHandle, &StrRef, String);\r
+      HiiLibNewString (CallbackData->BmmHiiHandle, &StrRef, String);\r
 \r
       UnicodeSPrint (String, sizeof (String), TypeStrHelp, Index);\r
       StrRefHelp = 0;\r
-      IfrLibNewString (CallbackData->BmmHiiHandle, &StrRefHelp, String);\r
+      HiiLibNewString (CallbackData->BmmHiiHandle, &StrRefHelp, String);\r
 \r
       CreateOneOfOpCode (\r
         (EFI_QUESTION_ID) (Key + Index),\r
@@ -1177,13 +1271,20 @@ UpdateSetLegacyDeviceOrderPage (
   CopyMem (OldData, LegacyOrder, 100);\r
 \r
   if (IfrOptionList != NULL) {\r
-    SafeFreePool (IfrOptionList);\r
+    FreePool (IfrOptionList);\r
     IfrOptionList = NULL;\r
   }\r
 \r
   UpdatePageEnd (CallbackData);\r
 }\r
 \r
+/**\r
+  Dispatch the display to the next page based on NewPageId.\r
+\r
+  @param Private         The BMM context data.\r
+  @param NewPageId       The original page ID.\r
+\r
+**/\r
 VOID\r
 UpdatePageId (\r
   BMM_CALLBACK_DATA              *Private,\r