]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/SetupBrowserDxe/Setup.c
UefiCpuPkg: Move AsmRelocateApLoopStart from Mpfuncs.nasm to AmdSev.nasm
[mirror_edk2.git] / MdeModulePkg / Universal / SetupBrowserDxe / Setup.c
index f936a4b8e8b667e2a3ce576eefac432516642b1a..5158baf5bd55e9cd9c87c7c7744aed4aee012e3f 100644 (file)
@@ -2,7 +2,7 @@
 Entry and initialization module for the browser.\r
 \r
 Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>\r
-(C) Copyright 2020 Hewlett Packard Enterprise Development LP<BR>\r
+(C) Copyright 2020 - 2022 Hewlett Packard Enterprise Development LP<BR>\r
 SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
@@ -29,46 +29,45 @@ SETUP_DRIVER_PRIVATE_DATA  mPrivateData = {
     RegiserExitHandler,\r
     IsBrowserDataModified,\r
     ExecuteAction,\r
-    {NULL,NULL},\r
-    {NULL,NULL},\r
+    { NULL,                        NULL  },\r
+    { NULL,                        NULL  },\r
     IsResetRequired\r
   }\r
 };\r
 \r
-EFI_HII_DATABASE_PROTOCOL         *mHiiDatabase;\r
-EFI_HII_CONFIG_ROUTING_PROTOCOL   *mHiiConfigRouting;\r
-EFI_DEVICE_PATH_FROM_TEXT_PROTOCOL *mPathFromText;\r
-EDKII_FORM_DISPLAY_ENGINE_PROTOCOL *mFormDisplay;\r
-\r
-UINTN           gBrowserContextCount = 0;\r
-LIST_ENTRY      gBrowserContextList = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserContextList);\r
-LIST_ENTRY      gBrowserFormSetList = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserFormSetList);\r
-LIST_ENTRY      gBrowserHotKeyList  = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserHotKeyList);\r
-LIST_ENTRY      gBrowserStorageList = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserStorageList);\r
-LIST_ENTRY      gBrowserSaveFailFormSetList = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserSaveFailFormSetList);\r
-\r
-BOOLEAN               mSystemSubmit = FALSE;\r
-BOOLEAN               gResetRequiredFormLevel;\r
-BOOLEAN               gResetRequiredSystemLevel = FALSE;\r
-BOOLEAN               gExitRequired;\r
-BOOLEAN               gFlagReconnect;\r
-BOOLEAN               gCallbackReconnect;\r
-BROWSER_SETTING_SCOPE gBrowserSettingScope = FormSetLevel;\r
-BOOLEAN               mBrowserScopeFirstSet = TRUE;\r
-EXIT_HANDLER          ExitHandlerFunction = NULL;\r
-FORM_BROWSER_FORMSET  *mSystemLevelFormSet;\r
+EFI_HII_DATABASE_PROTOCOL           *mHiiDatabase;\r
+EFI_HII_CONFIG_ROUTING_PROTOCOL     *mHiiConfigRouting;\r
+EFI_DEVICE_PATH_FROM_TEXT_PROTOCOL  *mPathFromText;\r
+EDKII_FORM_DISPLAY_ENGINE_PROTOCOL  *mFormDisplay;\r
+\r
+UINTN       gBrowserContextCount        = 0;\r
+LIST_ENTRY  gBrowserContextList         = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserContextList);\r
+LIST_ENTRY  gBrowserFormSetList         = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserFormSetList);\r
+LIST_ENTRY  gBrowserHotKeyList          = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserHotKeyList);\r
+LIST_ENTRY  gBrowserStorageList         = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserStorageList);\r
+LIST_ENTRY  gBrowserSaveFailFormSetList = INITIALIZE_LIST_HEAD_VARIABLE (gBrowserSaveFailFormSetList);\r
+\r
+BOOLEAN                mSystemSubmit = FALSE;\r
+BOOLEAN                gResetRequiredFormLevel;\r
+BOOLEAN                gResetRequiredSystemLevel = FALSE;\r
+BOOLEAN                gExitRequired;\r
+BOOLEAN                gFlagReconnect;\r
+BOOLEAN                gCallbackReconnect;\r
+BROWSER_SETTING_SCOPE  gBrowserSettingScope  = FormSetLevel;\r
+BOOLEAN                mBrowserScopeFirstSet = TRUE;\r
+EXIT_HANDLER           ExitHandlerFunction   = NULL;\r
+FORM_BROWSER_FORMSET   *mSystemLevelFormSet;\r
 \r
 //\r
 // Browser Global Strings\r
 //\r
-CHAR16            *gEmptyString;\r
-CHAR16            *mUnknownString = L"!";\r
+CHAR16  *gEmptyString;\r
+CHAR16  *mUnknownString = L"!";\r
 \r
-extern EFI_GUID        mCurrentFormSetGuid;\r
-extern EFI_HII_HANDLE  mCurrentHiiHandle;\r
-extern UINT16          mCurrentFormId;\r
-extern FORM_DISPLAY_ENGINE_FORM gDisplayFormData;\r
-extern BOOLEAN         mDynamicFormUpdated;\r
+extern EFI_GUID                  mCurrentFormSetGuid;\r
+extern EFI_HII_HANDLE            mCurrentHiiHandle;\r
+extern UINT16                    mCurrentFormId;\r
+extern FORM_DISPLAY_ENGINE_FORM  gDisplayFormData;\r
 \r
 /**\r
   Create a menu with specified formset GUID and form ID, and add it as a child\r
@@ -84,10 +83,10 @@ extern BOOLEAN         mDynamicFormUpdated;
 **/\r
 FORM_ENTRY_INFO *\r
 UiAddMenuList (\r
-  IN EFI_HII_HANDLE       HiiHandle,\r
-  IN EFI_GUID             *FormSetGuid,\r
-  IN UINT16               FormId,\r
-  IN UINT16               QuestionId\r
+  IN EFI_HII_HANDLE  HiiHandle,\r
+  IN EFI_GUID        *FormSetGuid,\r
+  IN UINT16          FormId,\r
+  IN UINT16          QuestionId\r
   )\r
 {\r
   FORM_ENTRY_INFO  *MenuList;\r
@@ -99,7 +98,7 @@ UiAddMenuList (
 \r
   MenuList->Signature = FORM_ENTRY_INFO_SIGNATURE;\r
 \r
-  MenuList->HiiHandle  = HiiHandle;\r
+  MenuList->HiiHandle = HiiHandle;\r
   CopyMem (&MenuList->FormSetGuid, FormSetGuid, sizeof (EFI_GUID));\r
   MenuList->FormId     = FormId;\r
   MenuList->QuestionId = QuestionId;\r
@@ -123,8 +122,8 @@ UiAddMenuList (
 **/\r
 EFI_FORM_ID\r
 GetFirstFormId (\r
-  IN EFI_HII_HANDLE       HiiHandle,\r
-  IN EFI_GUID             *FormSetGuid\r
+  IN EFI_HII_HANDLE  HiiHandle,\r
+  IN EFI_GUID        *FormSetGuid\r
   )\r
 {\r
   LIST_ENTRY         *Link;\r
@@ -148,22 +147,22 @@ GetFirstFormId (
 **/\r
 FORM_ENTRY_INFO *\r
 UiFindMenuList (\r
-  IN EFI_HII_HANDLE       HiiHandle,\r
-  IN EFI_GUID             *FormSetGuid,\r
-  IN UINT16               FormId\r
+  IN EFI_HII_HANDLE  HiiHandle,\r
+  IN EFI_GUID        *FormSetGuid,\r
+  IN UINT16          FormId\r
   )\r
 {\r
-  LIST_ENTRY         *Link;\r
-  FORM_ENTRY_INFO    *MenuList;\r
-  FORM_ENTRY_INFO    *RetMenu;\r
-  EFI_FORM_ID        FirstFormId;\r
+  LIST_ENTRY       *Link;\r
+  FORM_ENTRY_INFO  *MenuList;\r
+  FORM_ENTRY_INFO  *RetMenu;\r
+  EFI_FORM_ID      FirstFormId;\r
 \r
   RetMenu = NULL;\r
 \r
   Link = GetFirstNode (&mPrivateData.FormBrowserEx2.FormViewHistoryHead);\r
   while (!IsNull (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, Link)) {\r
     MenuList = FORM_ENTRY_INFO_FROM_LINK (Link);\r
-    Link = GetNextNode (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, Link);\r
+    Link     = GetNextNode (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, Link);\r
 \r
     //\r
     // If already find the menu, free the menus behind it.\r
@@ -186,9 +185,9 @@ UiFindMenuList (
       } else if (CompareGuid (&MenuList->FormSetGuid, FormSetGuid)) {\r
         if (MenuList->FormId == FormId) {\r
           RetMenu = MenuList;\r
-        } else if (FormId == 0 || MenuList->FormId == 0 ) {\r
+        } else if ((FormId == 0) || (MenuList->FormId == 0)) {\r
           FirstFormId = GetFirstFormId (HiiHandle, FormSetGuid);\r
-          if ((FormId == 0 && FirstFormId == MenuList->FormId) || (MenuList->FormId ==0 && FirstFormId == FormId)) {\r
+          if (((FormId == 0) && (FirstFormId == MenuList->FormId)) || ((MenuList->FormId == 0) && (FirstFormId == FormId))) {\r
             RetMenu = MenuList;\r
           }\r
         }\r
@@ -212,12 +211,12 @@ UiFindMenuList (
 **/\r
 FORM_ENTRY_INFO *\r
 UiFindParentMenu (\r
-  IN FORM_ENTRY_INFO          *CurrentMenu,\r
-  IN BROWSER_SETTING_SCOPE    SettingLevel\r
+  IN FORM_ENTRY_INFO        *CurrentMenu,\r
+  IN BROWSER_SETTING_SCOPE  SettingLevel\r
   )\r
 {\r
-  FORM_ENTRY_INFO    *ParentMenu;\r
-  LIST_ENTRY         *Link;\r
+  FORM_ENTRY_INFO  *ParentMenu;\r
+  LIST_ENTRY       *Link;\r
 \r
   ASSERT (SettingLevel == FormLevel || SettingLevel == FormSetLevel);\r
 \r
@@ -266,10 +265,10 @@ UiFindParentMenu (
 **/\r
 VOID\r
 UiFreeMenuList (\r
-  LIST_ENTRY   *MenuListHead\r
+  LIST_ENTRY  *MenuListHead\r
   )\r
 {\r
-  FORM_ENTRY_INFO    *MenuList;\r
+  FORM_ENTRY_INFO  *MenuList;\r
 \r
   while (!IsListEmpty (MenuListHead)) {\r
     MenuList = FORM_ENTRY_INFO_FROM_LINK (MenuListHead->ForwardLink);\r
@@ -288,13 +287,13 @@ UiFreeMenuList (
 **/\r
 VOID\r
 UiCopyMenuList (\r
-  OUT LIST_ENTRY   *NewMenuListHead,\r
-  IN  LIST_ENTRY   *CurrentMenuListHead\r
+  OUT LIST_ENTRY  *NewMenuListHead,\r
+  IN  LIST_ENTRY  *CurrentMenuListHead\r
   )\r
 {\r
-  LIST_ENTRY         *Link;\r
-  FORM_ENTRY_INFO    *MenuList;\r
-  FORM_ENTRY_INFO    *NewMenuEntry;\r
+  LIST_ENTRY       *Link;\r
+  FORM_ENTRY_INFO  *MenuList;\r
+  FORM_ENTRY_INFO  *NewMenuEntry;\r
 \r
   //\r
   // If new menu list not empty, free it first.\r
@@ -304,12 +303,12 @@ UiCopyMenuList (
   Link = GetFirstNode (CurrentMenuListHead);\r
   while (!IsNull (CurrentMenuListHead, Link)) {\r
     MenuList = FORM_ENTRY_INFO_FROM_LINK (Link);\r
-    Link = GetNextNode (CurrentMenuListHead, Link);\r
+    Link     = GetNextNode (CurrentMenuListHead, Link);\r
 \r
     NewMenuEntry = AllocateZeroPool (sizeof (FORM_ENTRY_INFO));\r
     ASSERT (NewMenuEntry != NULL);\r
-    NewMenuEntry->Signature  = FORM_ENTRY_INFO_SIGNATURE;\r
-    NewMenuEntry->HiiHandle  = MenuList->HiiHandle;\r
+    NewMenuEntry->Signature = FORM_ENTRY_INFO_SIGNATURE;\r
+    NewMenuEntry->HiiHandle = MenuList->HiiHandle;\r
     CopyMem (&NewMenuEntry->FormSetGuid, &MenuList->FormSetGuid, sizeof (EFI_GUID));\r
     NewMenuEntry->FormId     = MenuList->FormId;\r
     NewMenuEntry->QuestionId = MenuList->QuestionId;\r
@@ -327,12 +326,12 @@ LoadAllHiiFormset (
   VOID\r
   )\r
 {\r
-  FORM_BROWSER_FORMSET    *LocalFormSet;\r
-  EFI_HII_HANDLE          *HiiHandles;\r
-  UINTN                   Index;\r
-  EFI_GUID                ZeroGuid;\r
-  EFI_STATUS              Status;\r
-  FORM_BROWSER_FORMSET    *OldFormset;\r
+  FORM_BROWSER_FORMSET  *LocalFormSet;\r
+  EFI_HII_HANDLE        *HiiHandles;\r
+  UINTN                 Index;\r
+  EFI_GUID              ZeroGuid;\r
+  EFI_STATUS            Status;\r
+  FORM_BROWSER_FORMSET  *OldFormset;\r
 \r
   OldFormset = mSystemLevelFormSet;\r
 \r
@@ -366,6 +365,7 @@ LoadAllHiiFormset (
       DestroyFormSet (LocalFormSet);\r
       continue;\r
     }\r
+\r
     InitializeCurrentSetting (LocalFormSet);\r
 \r
     //\r
@@ -397,21 +397,21 @@ LoadAllHiiFormset (
 **/\r
 UINT32\r
 PopupErrorMessage (\r
-  IN UINT32                BrowserStatus,\r
-  IN EFI_HII_HANDLE        HiiHandle,\r
-  IN EFI_IFR_OP_HEADER     *OpCode, OPTIONAL\r
-  IN CHAR16                *ErrorString\r
+  IN UINT32             BrowserStatus,\r
+  IN EFI_HII_HANDLE     HiiHandle,\r
+  IN EFI_IFR_OP_HEADER  *OpCode  OPTIONAL,\r
+  IN CHAR16             *ErrorString\r
   )\r
 {\r
-  FORM_DISPLAY_ENGINE_STATEMENT *Statement;\r
-  USER_INPUT                    UserInputData;\r
+  FORM_DISPLAY_ENGINE_STATEMENT  *Statement;\r
+  USER_INPUT                     UserInputData;\r
 \r
   Statement = NULL;\r
 \r
   if (OpCode != NULL) {\r
-    Statement = AllocateZeroPool (sizeof(FORM_DISPLAY_ENGINE_STATEMENT));\r
+    Statement = AllocateZeroPool (sizeof (FORM_DISPLAY_ENGINE_STATEMENT));\r
     ASSERT (Statement != NULL);\r
-    Statement->OpCode = OpCode;\r
+    Statement->OpCode                     = OpCode;\r
     gDisplayFormData.HighLightedStatement = Statement;\r
   }\r
 \r
@@ -423,7 +423,7 @@ PopupErrorMessage (
   gDisplayFormData.BrowserStatus = BrowserStatus;\r
 \r
   if (HiiHandle != NULL) {\r
-    gDisplayFormData.HiiHandle     = HiiHandle;\r
+    gDisplayFormData.HiiHandle = HiiHandle;\r
   }\r
 \r
   mFormDisplay->FormDisplay (&gDisplayFormData, &UserInputData);\r
@@ -466,21 +466,21 @@ PopupErrorMessage (
 EFI_STATUS\r
 EFIAPI\r
 SendForm (\r
-  IN  CONST EFI_FORM_BROWSER2_PROTOCOL *This,\r
-  IN  EFI_HII_HANDLE                   *Handles,\r
-  IN  UINTN                            HandleCount,\r
-  IN  EFI_GUID                         *FormSetGuid, OPTIONAL\r
-  IN  UINT16                           FormId, OPTIONAL\r
-  IN  CONST EFI_SCREEN_DESCRIPTOR      *ScreenDimensions, OPTIONAL\r
-  OUT EFI_BROWSER_ACTION_REQUEST       *ActionRequest  OPTIONAL\r
+  IN  CONST EFI_FORM_BROWSER2_PROTOCOL  *This,\r
+  IN  EFI_HII_HANDLE                    *Handles,\r
+  IN  UINTN                             HandleCount,\r
+  IN  EFI_GUID                          *FormSetGuid  OPTIONAL,\r
+  IN  UINT16                            FormId  OPTIONAL,\r
+  IN  CONST EFI_SCREEN_DESCRIPTOR       *ScreenDimensions  OPTIONAL,\r
+  OUT EFI_BROWSER_ACTION_REQUEST        *ActionRequest  OPTIONAL\r
   )\r
 {\r
-  EFI_STATUS                    Status;\r
-  UI_MENU_SELECTION             *Selection;\r
-  UINTN                         Index;\r
-  FORM_BROWSER_FORMSET          *FormSet;\r
-  FORM_ENTRY_INFO               *MenuList;\r
-  BOOLEAN                       RetVal;\r
+  EFI_STATUS            Status;\r
+  UI_MENU_SELECTION     *Selection;\r
+  UINTN                 Index;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
+  FORM_ENTRY_INFO       *MenuList;\r
+  BOOLEAN               RetVal;\r
 \r
   //\r
   // If EDKII_FORM_DISPLAY_ENGINE_PROTOCOL not found, return EFI_UNSUPPORTED.\r
@@ -495,13 +495,13 @@ SendForm (
   //\r
   SaveBrowserContext ();\r
 \r
-  gFlagReconnect = FALSE;\r
-  gResetRequiredFormLevel = FALSE;\r
-  gExitRequired  = FALSE;\r
-  gCallbackReconnect = FALSE;\r
-  Status         = EFI_SUCCESS;\r
-  gEmptyString   = L"";\r
-  gDisplayFormData.ScreenDimensions = (EFI_SCREEN_DESCRIPTOR *) ScreenDimensions;\r
+  gFlagReconnect                    = FALSE;\r
+  gResetRequiredFormLevel           = FALSE;\r
+  gExitRequired                     = FALSE;\r
+  gCallbackReconnect                = FALSE;\r
+  Status                            = EFI_SUCCESS;\r
+  gEmptyString                      = L"";\r
+  gDisplayFormData.ScreenDimensions = (EFI_SCREEN_DESCRIPTOR *)ScreenDimensions;\r
 \r
   for (Index = 0; Index < HandleCount; Index++) {\r
     Selection = AllocateZeroPool (sizeof (UI_MENU_SELECTION));\r
@@ -523,7 +523,7 @@ SendForm (
       // Validate the HiiHandle\r
       // if validate failed, find the first validate parent HiiHandle.\r
       //\r
-      if (!ValidateHiiHandle(Selection->Handle)) {\r
+      if (!ValidateHiiHandle (Selection->Handle)) {\r
         FindNextMenu (Selection, FormSetLevel);\r
       }\r
 \r
@@ -535,9 +535,9 @@ SendForm (
         DestroyFormSet (FormSet);\r
         break;\r
       }\r
-      Selection->FormSet = FormSet;\r
+\r
+      Selection->FormSet  = FormSet;\r
       mSystemLevelFormSet = FormSet;\r
-      mDynamicFormUpdated = FALSE;\r
 \r
       //\r
       // Display this formset\r
@@ -546,19 +546,19 @@ SendForm (
 \r
       Status = SetupBrowser (Selection);\r
 \r
-      gCurrentSelection = NULL;\r
+      gCurrentSelection   = NULL;\r
       mSystemLevelFormSet = NULL;\r
 \r
       //\r
-      // If callback update form dynamically, it's not exiting of the formset for user so system do not reconnect driver hanlde\r
-      // this time.\r
+      // Check incoming formset whether is same with previous. If yes, that means action is not exiting of formset so do not reconnect controller.\r
       //\r
-      if (!mDynamicFormUpdated && (gFlagReconnect || gCallbackReconnect)) {\r
+      if ((gFlagReconnect || gCallbackReconnect) && !CompareGuid (&FormSet->Guid, &Selection->FormSetGuid)) {\r
         RetVal = ReconnectController (FormSet->DriverHandle);\r
         if (!RetVal) {\r
-          PopupErrorMessage(BROWSER_RECONNECT_FAIL, NULL, NULL, NULL);\r
+          PopupErrorMessage (BROWSER_RECONNECT_FAIL, NULL, NULL, NULL);\r
         }\r
-        gFlagReconnect = FALSE;\r
+\r
+        gFlagReconnect     = FALSE;\r
         gCallbackReconnect = FALSE;\r
       }\r
 \r
@@ -566,7 +566,7 @@ SendForm (
       // If no data is changed, don't need to save current FormSet into the maintain list.\r
       //\r
       if (!IsNvUpdateRequiredForFormSet (FormSet)) {\r
-        CleanBrowserStorage(FormSet);\r
+        CleanBrowserStorage (FormSet);\r
         RemoveEntryList (&FormSet->Link);\r
         DestroyFormSet (FormSet);\r
       }\r
@@ -586,7 +586,7 @@ SendForm (
     }\r
   }\r
 \r
-  mFormDisplay->ExitDisplay();\r
+  mFormDisplay->ExitDisplay ();\r
 \r
   //\r
   // Clear the menu history data.\r
@@ -625,19 +625,19 @@ SendForm (
 **/\r
 EFI_STATUS\r
 ProcessStorage (\r
-  IN OUT UINTN                         *ResultsDataSize,\r
-  IN OUT EFI_STRING                    *ResultsData,\r
-  IN BOOLEAN                           RetrieveData,\r
-  IN BROWSER_STORAGE                   *Storage\r
+  IN OUT UINTN        *ResultsDataSize,\r
+  IN OUT EFI_STRING   *ResultsData,\r
+  IN BOOLEAN          RetrieveData,\r
+  IN BROWSER_STORAGE  *Storage\r
   )\r
 {\r
-  CHAR16                *ConfigResp;\r
-  EFI_STATUS            Status;\r
-  CHAR16                *StrPtr;\r
-  UINTN                 BufferSize;\r
-  UINTN                 TmpSize;\r
-  UINTN                 MaxLen;\r
-  FORMSET_STORAGE       *BrowserStorage;\r
+  CHAR16           *ConfigResp;\r
+  EFI_STATUS       Status;\r
+  CHAR16           *StrPtr;\r
+  UINTN            BufferSize;\r
+  UINTN            TmpSize;\r
+  UINTN            MaxLen;\r
+  FORMSET_STORAGE  *BrowserStorage;\r
 \r
   if (RetrieveData) {\r
     //\r
@@ -654,8 +654,8 @@ ProcessStorage (
     //\r
     StrPtr = StrStr (ConfigResp, L"PATH");\r
     ASSERT (StrPtr != NULL);\r
-    StrPtr = StrStr (StrPtr, L"&");\r
-    StrPtr += 1;\r
+    StrPtr     = StrStr (StrPtr, L"&");\r
+    StrPtr    += 1;\r
     BufferSize = StrSize (StrPtr);\r
 \r
     //\r
@@ -673,9 +673,9 @@ ProcessStorage (
     //\r
     BrowserStorage = GetFstStgFromBrsStg (Storage);\r
     ASSERT (BrowserStorage != NULL);\r
-    TmpSize = StrLen (*ResultsData);\r
+    TmpSize    = StrLen (*ResultsData);\r
     BufferSize = (TmpSize + StrLen (BrowserStorage->ConfigHdr) + 2) * sizeof (CHAR16);\r
-    MaxLen = BufferSize / sizeof (CHAR16);\r
+    MaxLen     = BufferSize / sizeof (CHAR16);\r
     ConfigResp = AllocateZeroPool (BufferSize);\r
     ASSERT (ConfigResp != NULL);\r
 \r
@@ -730,18 +730,18 @@ BrowserCallback (
   IN OUT UINTN                         *ResultsDataSize,\r
   IN OUT EFI_STRING                    ResultsData,\r
   IN BOOLEAN                           RetrieveData,\r
-  IN CONST EFI_GUID                    *VariableGuid, OPTIONAL\r
+  IN CONST EFI_GUID                    *VariableGuid  OPTIONAL,\r
   IN CONST CHAR16                      *VariableName  OPTIONAL\r
   )\r
 {\r
-  EFI_STATUS            Status;\r
-  LIST_ENTRY            *Link;\r
-  BROWSER_STORAGE       *Storage;\r
-  FORMSET_STORAGE       *FormsetStorage;\r
-  UINTN                 TotalSize;\r
-  BOOLEAN               Found;\r
+  EFI_STATUS       Status;\r
+  LIST_ENTRY       *Link;\r
+  BROWSER_STORAGE  *Storage;\r
+  FORMSET_STORAGE  *FormsetStorage;\r
+  UINTN            TotalSize;\r
+  BOOLEAN          Found;\r
 \r
-  if (ResultsDataSize == NULL || ResultsData == NULL) {\r
+  if ((ResultsDataSize == NULL) || (ResultsData == NULL)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -757,16 +757,17 @@ BrowserCallback (
     Link = GetFirstNode (&gBrowserStorageList);\r
     while (!IsNull (&gBrowserStorageList, Link)) {\r
       Storage = BROWSER_STORAGE_FROM_LINK (Link);\r
-      Link = GetNextNode (&gBrowserStorageList, Link);\r
+      Link    = GetNextNode (&gBrowserStorageList, Link);\r
       //\r
       // Check the current storage.\r
       //\r
-      if (!CompareGuid (&Storage->Guid, (EFI_GUID *) VariableGuid)) {\r
+      if (!CompareGuid (&Storage->Guid, (EFI_GUID *)VariableGuid)) {\r
         continue;\r
       }\r
 \r
-      if (Storage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-          Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+      if ((Storage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+          (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER))\r
+      {\r
         //\r
         // Buffer storage require both GUID and Name\r
         //\r
@@ -774,14 +775,15 @@ BrowserCallback (
           return EFI_NOT_FOUND;\r
         }\r
 \r
-        if (StrCmp (Storage->Name, (CHAR16 *) VariableName) != 0) {\r
+        if (StrCmp (Storage->Name, (CHAR16 *)VariableName) != 0) {\r
           continue;\r
         }\r
       }\r
 \r
-      if (Storage->Type == EFI_HII_VARSTORE_NAME_VALUE ||\r
-          Storage->Type == EFI_HII_VARSTORE_BUFFER) {\r
-        if (mSystemLevelFormSet == NULL || mSystemLevelFormSet->HiiHandle == NULL) {\r
+      if ((Storage->Type == EFI_HII_VARSTORE_NAME_VALUE) ||\r
+          (Storage->Type == EFI_HII_VARSTORE_BUFFER))\r
+      {\r
+        if ((mSystemLevelFormSet == NULL) || (mSystemLevelFormSet->HiiHandle == NULL)) {\r
           return EFI_NOT_FOUND;\r
         }\r
 \r
@@ -835,15 +837,13 @@ BrowserCallback (
   }\r
 \r
   if (RetrieveData) {\r
-    Status = TotalSize <= *ResultsDataSize ? EFI_SUCCESS : EFI_BUFFER_TOO_SMALL;\r
+    Status           = TotalSize <= *ResultsDataSize ? EFI_SUCCESS : EFI_BUFFER_TOO_SMALL;\r
     *ResultsDataSize = TotalSize;\r
   }\r
 \r
   return Status;\r
-\r
 }\r
 \r
-\r
 /**\r
   Callback function for SimpleTextInEx protocol install events\r
 \r
@@ -854,8 +854,8 @@ BrowserCallback (
 VOID\r
 EFIAPI\r
 FormDisplayCallback (\r
-  IN EFI_EVENT    Event,\r
-  IN VOID         *Context\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
   )\r
 {\r
   if (mFormDisplay != NULL) {\r
@@ -863,10 +863,10 @@ FormDisplayCallback (
   }\r
 \r
   gBS->LocateProtocol (\r
-                  &gEdkiiFormDisplayEngineProtocolGuid,\r
-                  NULL,\r
-                  (VOID **) &mFormDisplay\r
-                  );\r
+         &gEdkiiFormDisplayEngineProtocolGuid,\r
+         NULL,\r
+         (VOID **)&mFormDisplay\r
+         );\r
 }\r
 \r
 /**\r
@@ -882,12 +882,12 @@ FormDisplayCallback (
 EFI_STATUS\r
 EFIAPI\r
 InitializeSetup (\r
-  IN EFI_HANDLE           ImageHandle,\r
-  IN EFI_SYSTEM_TABLE     *SystemTable\r
+  IN EFI_HANDLE        ImageHandle,\r
+  IN EFI_SYSTEM_TABLE  *SystemTable\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  VOID                        *Registration;\r
+  EFI_STATUS  Status;\r
+  VOID        *Registration;\r
 \r
   //\r
   // Locate required Hii relative protocols\r
@@ -895,33 +895,33 @@ InitializeSetup (
   Status = gBS->LocateProtocol (\r
                   &gEfiHiiDatabaseProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &mHiiDatabase\r
+                  (VOID **)&mHiiDatabase\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   Status = gBS->LocateProtocol (\r
                   &gEfiHiiConfigRoutingProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &mHiiConfigRouting\r
+                  (VOID **)&mHiiConfigRouting\r
                   );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   Status = gBS->LocateProtocol (\r
                   &gEfiDevicePathFromTextProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &mPathFromText\r
+                  (VOID **)&mPathFromText\r
                   );\r
 \r
   //\r
   // Install FormBrowser2 protocol\r
   //\r
   mPrivateData.Handle = NULL;\r
-  Status = gBS->InstallProtocolInterface (\r
-                  &mPrivateData.Handle,\r
-                  &gEfiFormBrowser2ProtocolGuid,\r
-                  EFI_NATIVE_INTERFACE,\r
-                  &mPrivateData.FormBrowser2\r
-                  );\r
+  Status              = gBS->InstallProtocolInterface (\r
+                               &mPrivateData.Handle,\r
+                               &gEfiFormBrowser2ProtocolGuid,\r
+                               EFI_NATIVE_INTERFACE,\r
+                               &mPrivateData.FormBrowser2\r
+                               );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   //\r
@@ -930,12 +930,12 @@ InitializeSetup (
   InitializeListHead (&mPrivateData.FormBrowserEx2.FormViewHistoryHead);\r
   InitializeListHead (&mPrivateData.FormBrowserEx2.OverrideQestListHead);\r
   mPrivateData.Handle = NULL;\r
-  Status = gBS->InstallProtocolInterface (\r
-                  &mPrivateData.Handle,\r
-                  &gEdkiiFormBrowserEx2ProtocolGuid,\r
-                  EFI_NATIVE_INTERFACE,\r
-                  &mPrivateData.FormBrowserEx2\r
-                  );\r
+  Status              = gBS->InstallProtocolInterface (\r
+                               &mPrivateData.Handle,\r
+                               &gEdkiiFormBrowserEx2ProtocolGuid,\r
+                               EFI_NATIVE_INTERFACE,\r
+                               &mPrivateData.FormBrowserEx2\r
+                               );\r
   ASSERT_EFI_ERROR (Status);\r
 \r
   Status = gBS->InstallProtocolInterface (\r
@@ -951,7 +951,7 @@ InitializeSetup (
   Status = gBS->LocateProtocol (\r
                   &gEdkiiFormDisplayEngineProtocolGuid,\r
                   NULL,\r
-                  (VOID **) &mFormDisplay\r
+                  (VOID **)&mFormDisplay\r
                   );\r
 \r
   if (EFI_ERROR (Status)) {\r
@@ -967,7 +967,6 @@ InitializeSetup (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Create a new string in HII Package List.\r
 \r
@@ -980,8 +979,8 @@ InitializeSetup (
 **/\r
 EFI_STRING_ID\r
 NewString (\r
-  IN  CHAR16                   *String,\r
-  IN  EFI_HII_HANDLE           HiiHandle\r
+  IN  CHAR16          *String,\r
+  IN  EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
   EFI_STRING_ID  StringId;\r
@@ -992,7 +991,6 @@ NewString (
   return StringId;\r
 }\r
 \r
-\r
 /**\r
   Delete a string from HII Package List.\r
 \r
@@ -1004,8 +1002,8 @@ NewString (
 **/\r
 EFI_STATUS\r
 DeleteString (\r
-  IN  EFI_STRING_ID            StringId,\r
-  IN  EFI_HII_HANDLE           HiiHandle\r
+  IN  EFI_STRING_ID   StringId,\r
+  IN  EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
   CHAR16  NullChar;\r
@@ -1015,7 +1013,6 @@ DeleteString (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Get the string based on the StringId and HII Package List Handle.\r
 \r
@@ -1028,8 +1025,8 @@ DeleteString (
 **/\r
 CHAR16 *\r
 GetToken (\r
-  IN  EFI_STRING_ID                Token,\r
-  IN  EFI_HII_HANDLE               HiiHandle\r
+  IN  EFI_STRING_ID   Token,\r
+  IN  EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
   EFI_STRING  String;\r
@@ -1043,9 +1040,9 @@ GetToken (
     String = AllocateCopyPool (StrSize (mUnknownString), mUnknownString);\r
     ASSERT (String != NULL);\r
   }\r
-  return (CHAR16 *) String;\r
-}\r
 \r
+  return (CHAR16 *)String;\r
+}\r
 \r
 /**\r
   Allocate new memory and then copy the Unicode string Source to Destination.\r
@@ -1056,18 +1053,18 @@ GetToken (
 **/\r
 VOID\r
 NewStringCpy (\r
-  IN OUT CHAR16       **Dest,\r
-  IN CHAR16           *Src\r
+  IN OUT CHAR16  **Dest,\r
+  IN CHAR16      *Src\r
   )\r
 {\r
   if (*Dest != NULL) {\r
     FreePool (*Dest);\r
   }\r
+\r
   *Dest = AllocateCopyPool (StrSize (Src), Src);\r
   ASSERT (*Dest != NULL);\r
 }\r
 \r
-\r
 /**\r
   Allocate new memory and concatinate Source on the end of Destination.\r
 \r
@@ -1077,8 +1074,8 @@ NewStringCpy (
 **/\r
 VOID\r
 NewStringCat (\r
-  IN OUT CHAR16       **Dest,\r
-  IN CHAR16           *Src\r
+  IN OUT CHAR16  **Dest,\r
+  IN CHAR16      *Src\r
   )\r
 {\r
   CHAR16  *NewString;\r
@@ -1089,7 +1086,7 @@ NewStringCat (
     return;\r
   }\r
 \r
-  MaxLen = ( StrSize (*Dest) + StrSize (Src) - 1) / sizeof (CHAR16);\r
+  MaxLen    = (StrSize (*Dest) + StrSize (Src) - 1) / sizeof (CHAR16);\r
   NewString = AllocateZeroPool (MaxLen * sizeof (CHAR16));\r
   ASSERT (NewString != NULL);\r
 \r
@@ -1114,16 +1111,16 @@ NewStringCat (
 **/\r
 EFI_STATUS\r
 GetValueByName (\r
-  IN BROWSER_STORAGE             *Storage,\r
-  IN CHAR16                      *Name,\r
-  IN OUT CHAR16                  **Value,\r
-  IN GET_SET_QUESTION_VALUE_WITH GetValueFrom\r
+  IN BROWSER_STORAGE              *Storage,\r
+  IN CHAR16                       *Name,\r
+  IN OUT CHAR16                   **Value,\r
+  IN GET_SET_QUESTION_VALUE_WITH  GetValueFrom\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  NAME_VALUE_NODE         *Node;\r
+  LIST_ENTRY       *Link;\r
+  NAME_VALUE_NODE  *Node;\r
 \r
-  if (GetValueFrom != GetSetValueWithEditBuffer && GetValueFrom != GetSetValueWithBuffer) {\r
+  if ((GetValueFrom != GetSetValueWithEditBuffer) && (GetValueFrom != GetSetValueWithBuffer)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1139,6 +1136,7 @@ GetValueByName (
       } else {\r
         NewStringCpy (Value, Node->Value);\r
       }\r
+\r
       return EFI_SUCCESS;\r
     }\r
 \r
@@ -1148,7 +1146,6 @@ GetValueByName (
   return EFI_NOT_FOUND;\r
 }\r
 \r
-\r
 /**\r
   Set Value of given Name in a NameValue Storage.\r
 \r
@@ -1164,18 +1161,18 @@ GetValueByName (
 **/\r
 EFI_STATUS\r
 SetValueByName (\r
-  IN  BROWSER_STORAGE             *Storage,\r
-  IN  CHAR16                      *Name,\r
-  IN  CHAR16                      *Value,\r
-  IN  GET_SET_QUESTION_VALUE_WITH SetValueTo,\r
-  OUT NAME_VALUE_NODE             **ReturnNode\r
+  IN  BROWSER_STORAGE              *Storage,\r
+  IN  CHAR16                       *Name,\r
+  IN  CHAR16                       *Value,\r
+  IN  GET_SET_QUESTION_VALUE_WITH  SetValueTo,\r
+  OUT NAME_VALUE_NODE              **ReturnNode\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  NAME_VALUE_NODE         *Node;\r
-  CHAR16                  *Buffer;\r
+  LIST_ENTRY       *Link;\r
+  NAME_VALUE_NODE  *Node;\r
+  CHAR16           *Buffer;\r
 \r
-  if (SetValueTo != GetSetValueWithEditBuffer && SetValueTo != GetSetValueWithBuffer) {\r
+  if ((SetValueTo != GetSetValueWithEditBuffer) && (SetValueTo != GetSetValueWithBuffer)) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -1189,9 +1186,11 @@ SetValueByName (
       } else {\r
         Buffer = Node->Value;\r
       }\r
+\r
       if (Buffer != NULL) {\r
         FreePool (Buffer);\r
       }\r
+\r
       Buffer = AllocateCopyPool (StrSize (Value), Value);\r
       ASSERT (Buffer != NULL);\r
       if (SetValueTo == GetSetValueWithEditBuffer) {\r
@@ -1213,7 +1212,6 @@ SetValueByName (
   return EFI_NOT_FOUND;\r
 }\r
 \r
-\r
 /**\r
   Convert setting of Buffer Storage or NameValue Storage to <ConfigResp>.\r
 \r
@@ -1228,69 +1226,70 @@ SetValueByName (
 **/\r
 EFI_STATUS\r
 StorageToConfigResp (\r
-  IN BROWSER_STORAGE         *Storage,\r
-  IN CHAR16                  **ConfigResp,\r
-  IN CHAR16                  *ConfigRequest,\r
-  IN BOOLEAN                 GetEditBuf\r
+  IN BROWSER_STORAGE  *Storage,\r
+  IN CHAR16           **ConfigResp,\r
+  IN CHAR16           *ConfigRequest,\r
+  IN BOOLEAN          GetEditBuf\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_STRING              Progress;\r
-  LIST_ENTRY              *Link;\r
-  NAME_VALUE_NODE         *Node;\r
-  UINT8                   *SourceBuf;\r
-  FORMSET_STORAGE         *FormsetStorage;\r
+  EFI_STATUS       Status;\r
+  EFI_STRING       Progress;\r
+  LIST_ENTRY       *Link;\r
+  NAME_VALUE_NODE  *Node;\r
+  UINT8            *SourceBuf;\r
+  FORMSET_STORAGE  *FormsetStorage;\r
 \r
   Status = EFI_SUCCESS;\r
 \r
   switch (Storage->Type) {\r
-  case EFI_HII_VARSTORE_BUFFER:\r
-  case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
-    SourceBuf = GetEditBuf ? Storage->EditBuffer : Storage->Buffer;\r
-    Status = mHiiConfigRouting->BlockToConfig (\r
-                                  mHiiConfigRouting,\r
-                                  ConfigRequest,\r
-                                  SourceBuf,\r
-                                  Storage->Size,\r
-                                  ConfigResp,\r
-                                  &Progress\r
-                                  );\r
-    break;\r
-\r
-  case EFI_HII_VARSTORE_NAME_VALUE:\r
-    *ConfigResp = NULL;\r
-    FormsetStorage = GetFstStgFromBrsStg(Storage);\r
-    ASSERT (FormsetStorage != NULL);\r
-    NewStringCat (ConfigResp, FormsetStorage->ConfigHdr);\r
-\r
-    Link = GetFirstNode (&Storage->NameValueListHead);\r
-    while (!IsNull (&Storage->NameValueListHead, Link)) {\r
-      Node = NAME_VALUE_NODE_FROM_LINK (Link);\r
+    case EFI_HII_VARSTORE_BUFFER:\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
+      SourceBuf = GetEditBuf ? Storage->EditBuffer : Storage->Buffer;\r
+      Status    = mHiiConfigRouting->BlockToConfig (\r
+                                       mHiiConfigRouting,\r
+                                       ConfigRequest,\r
+                                       SourceBuf,\r
+                                       Storage->Size,\r
+                                       ConfigResp,\r
+                                       &Progress\r
+                                       );\r
+      break;\r
 \r
-      if (StrStr (ConfigRequest, Node->Name) != NULL) {\r
-        NewStringCat (ConfigResp, L"&");\r
-        NewStringCat (ConfigResp, Node->Name);\r
-        NewStringCat (ConfigResp, L"=");\r
-        if (GetEditBuf) {\r
-          NewStringCat (ConfigResp, Node->EditValue);\r
-        } else {\r
-          NewStringCat (ConfigResp, Node->Value);\r
+    case EFI_HII_VARSTORE_NAME_VALUE:\r
+      *ConfigResp    = NULL;\r
+      FormsetStorage = GetFstStgFromBrsStg (Storage);\r
+      ASSERT (FormsetStorage != NULL);\r
+      NewStringCat (ConfigResp, FormsetStorage->ConfigHdr);\r
+\r
+      Link = GetFirstNode (&Storage->NameValueListHead);\r
+      while (!IsNull (&Storage->NameValueListHead, Link)) {\r
+        Node = NAME_VALUE_NODE_FROM_LINK (Link);\r
+\r
+        if (StrStr (ConfigRequest, Node->Name) != NULL) {\r
+          NewStringCat (ConfigResp, L"&");\r
+          NewStringCat (ConfigResp, Node->Name);\r
+          NewStringCat (ConfigResp, L"=");\r
+          if (GetEditBuf) {\r
+            NewStringCat (ConfigResp, Node->EditValue);\r
+          } else {\r
+            NewStringCat (ConfigResp, Node->Value);\r
+          }\r
         }\r
+\r
+        Link = GetNextNode (&Storage->NameValueListHead, Link);\r
       }\r
-      Link = GetNextNode (&Storage->NameValueListHead, Link);\r
-    }\r
-    break;\r
 \r
-  case EFI_HII_VARSTORE_EFI_VARIABLE:\r
-  default:\r
-    Status = EFI_INVALID_PARAMETER;\r
-    break;\r
+      break;\r
+\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE:\r
+    default:\r
+      Status = EFI_INVALID_PARAMETER;\r
+      break;\r
   }\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Convert <ConfigResp> to settings in Buffer Storage or NameValue Storage.\r
 \r
@@ -1303,8 +1302,8 @@ StorageToConfigResp (
 **/\r
 EFI_STATUS\r
 ConfigRespToStorage (\r
-  IN BROWSER_STORAGE         *Storage,\r
-  IN CHAR16                  *ConfigResp\r
+  IN BROWSER_STORAGE  *Storage,\r
+  IN CHAR16           *ConfigResp\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -1317,53 +1316,57 @@ ConfigRespToStorage (
   Status = EFI_SUCCESS;\r
 \r
   switch (Storage->Type) {\r
-  case EFI_HII_VARSTORE_BUFFER:\r
-  case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
-    BufferSize = Storage->Size;\r
-    Status = mHiiConfigRouting->ConfigToBlock (\r
-                                  mHiiConfigRouting,\r
-                                  ConfigResp,\r
-                                  Storage->EditBuffer,\r
-                                  &BufferSize,\r
-                                  &Progress\r
-                                  );\r
-    break;\r
-\r
-  case EFI_HII_VARSTORE_NAME_VALUE:\r
-    StrPtr = StrStr (ConfigResp, L"PATH");\r
-    if (StrPtr == NULL) {\r
+    case EFI_HII_VARSTORE_BUFFER:\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
+      BufferSize = Storage->Size;\r
+      Status     = mHiiConfigRouting->ConfigToBlock (\r
+                                        mHiiConfigRouting,\r
+                                        ConfigResp,\r
+                                        Storage->EditBuffer,\r
+                                        &BufferSize,\r
+                                        &Progress\r
+                                        );\r
       break;\r
-    }\r
-    StrPtr = StrStr (ConfigResp, L"&");\r
-    while (StrPtr != NULL) {\r
-      //\r
-      // Skip '&'\r
-      //\r
-      StrPtr = StrPtr + 1;\r
-      Name = StrPtr;\r
-      StrPtr = StrStr (StrPtr, L"=");\r
+\r
+    case EFI_HII_VARSTORE_NAME_VALUE:\r
+      StrPtr = StrStr (ConfigResp, L"PATH");\r
       if (StrPtr == NULL) {\r
         break;\r
       }\r
-      *StrPtr = 0;\r
 \r
-      //\r
-      // Skip '='\r
-      //\r
-      StrPtr = StrPtr + 1;\r
-      Value = StrPtr;\r
-      StrPtr = StrStr (StrPtr, L"&");\r
-      if (StrPtr != NULL) {\r
+      StrPtr = StrStr (ConfigResp, L"&");\r
+      while (StrPtr != NULL) {\r
+        //\r
+        // Skip '&'\r
+        //\r
+        StrPtr = StrPtr + 1;\r
+        Name   = StrPtr;\r
+        StrPtr = StrStr (StrPtr, L"=");\r
+        if (StrPtr == NULL) {\r
+          break;\r
+        }\r
+\r
         *StrPtr = 0;\r
+\r
+        //\r
+        // Skip '='\r
+        //\r
+        StrPtr = StrPtr + 1;\r
+        Value  = StrPtr;\r
+        StrPtr = StrStr (StrPtr, L"&");\r
+        if (StrPtr != NULL) {\r
+          *StrPtr = 0;\r
+        }\r
+\r
+        SetValueByName (Storage, Name, Value, GetSetValueWithEditBuffer, NULL);\r
       }\r
-      SetValueByName (Storage, Name, Value, GetSetValueWithEditBuffer, NULL);\r
-    }\r
-    break;\r
 \r
-  case EFI_HII_VARSTORE_EFI_VARIABLE:\r
-  default:\r
-    Status = EFI_INVALID_PARAMETER;\r
-    break;\r
+      break;\r
+\r
+    case EFI_HII_VARSTORE_EFI_VARIABLE:\r
+    default:\r
+      Status = EFI_INVALID_PARAMETER;\r
+      break;\r
   }\r
 \r
   return Status;\r
@@ -1379,19 +1382,19 @@ ConfigRespToStorage (
 **/\r
 VOID\r
 GetBitsQuestionValue (\r
-  IN  FORM_BROWSER_STATEMENT *Question,\r
-  IN  UINT8                  *Buffer\r
+  IN  FORM_BROWSER_STATEMENT  *Question,\r
+  IN  UINT8                   *Buffer\r
   )\r
 {\r
-  UINTN    StartBit;\r
-  UINTN    EndBit;\r
-  UINT32   RetVal;\r
-  UINT32   BufferValue;\r
+  UINTN   StartBit;\r
+  UINTN   EndBit;\r
+  UINT32  RetVal;\r
+  UINT32  BufferValue;\r
 \r
   StartBit = Question->BitVarOffset % 8;\r
-  EndBit = StartBit + Question->BitStorageWidth - 1;\r
+  EndBit   = StartBit + Question->BitStorageWidth - 1;\r
 \r
-  CopyMem ((UINT8 *) &BufferValue, Buffer, Question->StorageWidth);\r
+  CopyMem ((UINT8 *)&BufferValue, Buffer, Question->StorageWidth);\r
 \r
   RetVal = BitFieldRead32 (BufferValue, StartBit, EndBit);\r
 \r
@@ -1400,7 +1403,7 @@ GetBitsQuestionValue (
   // Note: Since Question with BufferValue (orderedlist, password, string)are not supported to refer bit field.\r
   // Only oneof/checkbox/oneof can support bit field.So we can copy the value to the Hiivalue of Question directly.\r
   //\r
-  CopyMem ((UINT8 *) &Question->HiiValue.Value, (UINT8 *) &RetVal, Question->StorageWidth);\r
+  CopyMem ((UINT8 *)&Question->HiiValue.Value, (UINT8 *)&RetVal, Question->StorageWidth);\r
 }\r
 \r
 /**\r
@@ -1414,24 +1417,24 @@ GetBitsQuestionValue (
 **/\r
 VOID\r
 SetBitsQuestionValue (\r
-  IN     FORM_BROWSER_STATEMENT *Question,\r
-  IN OUT UINT8                  *Buffer,\r
-  IN     UINT32                 Value\r
+  IN     FORM_BROWSER_STATEMENT  *Question,\r
+  IN OUT UINT8                   *Buffer,\r
+  IN     UINT32                  Value\r
   )\r
 {\r
-  UINT32   Operand;\r
-  UINTN    StartBit;\r
-  UINTN    EndBit;\r
-  UINT32   RetVal;\r
+  UINT32  Operand;\r
+  UINTN   StartBit;\r
+  UINTN   EndBit;\r
+  UINT32  RetVal;\r
 \r
   StartBit = Question->BitVarOffset % 8;\r
-  EndBit = StartBit + Question->BitStorageWidth - 1;\r
+  EndBit   = StartBit + Question->BitStorageWidth - 1;\r
 \r
-  CopyMem ((UINT8*) &Operand, Buffer, Question->StorageWidth);\r
+  CopyMem ((UINT8 *)&Operand, Buffer, Question->StorageWidth);\r
 \r
   RetVal = BitFieldWrite32 (Operand, StartBit, EndBit, Value);\r
 \r
-  CopyMem (Buffer, (UINT8*) &RetVal, Question->StorageWidth);\r
+  CopyMem (Buffer, (UINT8 *)&RetVal, Question->StorageWidth);\r
 }\r
 \r
 /**\r
@@ -1445,29 +1448,30 @@ SetBitsQuestionValue (
 **/\r
 EFI_STATUS\r
 BufferToValue (\r
-  IN OUT FORM_BROWSER_STATEMENT           *Question,\r
-  IN     CHAR16                           *Value\r
+  IN OUT FORM_BROWSER_STATEMENT  *Question,\r
+  IN     CHAR16                  *Value\r
   )\r
 {\r
-  CHAR16                       *StringPtr;\r
-  BOOLEAN                      IsBufferStorage;\r
-  CHAR16                       *DstBuf;\r
-  CHAR16                       TempChar;\r
-  UINTN                        LengthStr;\r
-  UINT8                        *Dst;\r
-  CHAR16                       TemStr[5];\r
-  UINTN                        Index;\r
-  UINT8                        DigitUint8;\r
-  BOOLEAN                      IsString;\r
-  UINTN                        Length;\r
-  EFI_STATUS                   Status;\r
-  UINT8                        *Buffer;\r
+  CHAR16      *StringPtr;\r
+  BOOLEAN     IsBufferStorage;\r
+  CHAR16      *DstBuf;\r
+  CHAR16      TempChar;\r
+  UINTN       LengthStr;\r
+  UINT8       *Dst;\r
+  CHAR16      TemStr[5];\r
+  UINTN       Index;\r
+  UINT8       DigitUint8;\r
+  BOOLEAN     IsString;\r
+  UINTN       Length;\r
+  EFI_STATUS  Status;\r
+  UINT8       *Buffer;\r
 \r
   Buffer = NULL;\r
 \r
-  IsString = (BOOLEAN) ((Question->HiiValue.Type == EFI_IFR_TYPE_STRING) ?  TRUE : FALSE);\r
-  if (Question->Storage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-      Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+  IsString = (BOOLEAN)((Question->HiiValue.Type == EFI_IFR_TYPE_STRING) ?  TRUE : FALSE);\r
+  if ((Question->Storage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+      (Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER))\r
+  {\r
     IsBufferStorage = TRUE;\r
   } else {\r
     IsBufferStorage = FALSE;\r
@@ -1490,9 +1494,10 @@ BufferToValue (
       if (Buffer == NULL) {\r
         return EFI_OUT_OF_RESOURCES;\r
       }\r
+\r
       Dst = Buffer;\r
     } else {\r
-      Dst = (UINT8 *) &Question->HiiValue.Value;\r
+      Dst = (UINT8 *)&Question->HiiValue.Value;\r
     }\r
   }\r
 \r
@@ -1503,7 +1508,8 @@ BufferToValue (
   while (*StringPtr != L'\0' && *StringPtr != L'&') {\r
     StringPtr++;\r
   }\r
-  TempChar = *StringPtr;\r
+\r
+  TempChar   = *StringPtr;\r
   *StringPtr = L'\0';\r
 \r
   LengthStr = StrLen (Value);\r
@@ -1514,44 +1520,45 @@ BufferToValue (
   // So the maximum value string length of a question is : Question->StorageWidth * 2.\r
   // If the value string length > Question->StorageWidth * 2, only set the string length as Question->StorageWidth * 2, then convert.\r
   //\r
-  if (LengthStr > (UINTN) Question->StorageWidth * 2) {\r
-    Length = (UINTN) Question->StorageWidth * 2;\r
+  if (LengthStr > (UINTN)Question->StorageWidth * 2) {\r
+    Length = (UINTN)Question->StorageWidth * 2;\r
   } else {\r
     Length = LengthStr;\r
   }\r
 \r
-  Status    = EFI_SUCCESS;\r
+  Status = EFI_SUCCESS;\r
   if (!IsBufferStorage && IsString) {\r
     //\r
     // Convert Config String to Unicode String, e.g "0041004200430044" => "ABCD"\r
     // Add string tail char L'\0' into Length\r
     //\r
-    DstBuf = (CHAR16 *) Dst;\r
+    DstBuf = (CHAR16 *)Dst;\r
     ZeroMem (TemStr, sizeof (TemStr));\r
     for (Index = 0; Index < Length; Index += 4) {\r
       StrnCpyS (TemStr, sizeof (TemStr) / sizeof (CHAR16), Value + Index, 4);\r
-      DstBuf[Index/4] = (CHAR16) StrHexToUint64 (TemStr);\r
+      DstBuf[Index/4] = (CHAR16)StrHexToUint64 (TemStr);\r
     }\r
+\r
     //\r
     // Add tailing L'\0' character\r
     //\r
     DstBuf[Index/4] = L'\0';\r
   } else {\r
     ZeroMem (TemStr, sizeof (TemStr));\r
-    for (Index = 0; Index < Length; Index ++) {\r
-      TemStr[0] = Value[LengthStr - Index - 1];\r
-      DigitUint8 = (UINT8) StrHexToUint64 (TemStr);\r
+    for (Index = 0; Index < Length; Index++) {\r
+      TemStr[0]  = Value[LengthStr - Index - 1];\r
+      DigitUint8 = (UINT8)StrHexToUint64 (TemStr);\r
       if ((Index & 1) == 0) {\r
-        Dst [Index/2] = DigitUint8;\r
+        Dst[Index/2] = DigitUint8;\r
       } else {\r
-        Dst [Index/2] = (UINT8) ((DigitUint8 << 4) + Dst [Index/2]);\r
+        Dst[Index/2] = (UINT8)((DigitUint8 << 4) + Dst[Index/2]);\r
       }\r
     }\r
   }\r
 \r
   *StringPtr = TempChar;\r
 \r
-  if (Buffer != NULL && Question->QuestionReferToBitField) {\r
+  if ((Buffer != NULL) && Question->QuestionReferToBitField) {\r
     GetBitsQuestionValue (Question, Buffer);\r
     FreePool (Buffer);\r
   }\r
@@ -1572,10 +1579,10 @@ BufferToValue (
 **/\r
 EFI_STATUS\r
 GetQuestionValue (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN OUT FORM_BROWSER_STATEMENT       *Question,\r
-  IN GET_SET_QUESTION_VALUE_WITH      GetValueFrom\r
+  IN FORM_BROWSER_FORMSET         *FormSet,\r
+  IN FORM_BROWSER_FORM            *Form,\r
+  IN OUT FORM_BROWSER_STATEMENT   *Question,\r
+  IN GET_SET_QUESTION_VALUE_WITH  GetValueFrom\r
   )\r
 {\r
   EFI_STATUS          Status;\r
@@ -1618,21 +1625,25 @@ GetQuestionValue (
           CopyMem (Question->HiiValue.Buffer, Question->ValueExpression->Result.Buffer, Question->StorageWidth);\r
           Question->HiiValue.BufferLen = Question->StorageWidth;\r
         }\r
+\r
         FreePool (Question->ValueExpression->Result.Buffer);\r
       }\r
+\r
       Question->HiiValue.Type = Question->ValueExpression->Result.Type;\r
       CopyMem (&Question->HiiValue.Value, &Question->ValueExpression->Result.Value, sizeof (EFI_IFR_TYPE_VALUE));\r
     }\r
+\r
     return Status;\r
   }\r
 \r
   //\r
   // Get question value by read expression.\r
   //\r
-  if (Question->ReadExpression != NULL && Form->FormType == STANDARD_MAP_FORM_TYPE) {\r
+  if ((Question->ReadExpression != NULL) && (Form->FormType == STANDARD_MAP_FORM_TYPE)) {\r
     Status = EvaluateExpression (FormSet, Form, Question->ReadExpression);\r
     if (!EFI_ERROR (Status) &&\r
-      ((Question->ReadExpression->Result.Type < EFI_IFR_TYPE_OTHER) || (Question->ReadExpression->Result.Type == EFI_IFR_TYPE_BUFFER))) {\r
+        ((Question->ReadExpression->Result.Type < EFI_IFR_TYPE_OTHER) || (Question->ReadExpression->Result.Type == EFI_IFR_TYPE_BUFFER)))\r
+    {\r
       //\r
       // Only update question value to the valid result.\r
       //\r
@@ -1645,8 +1656,10 @@ GetQuestionValue (
           CopyMem (Question->HiiValue.Buffer, Question->ReadExpression->Result.Buffer, Question->StorageWidth);\r
           Question->HiiValue.BufferLen = Question->StorageWidth;\r
         }\r
+\r
         FreePool (Question->ReadExpression->Result.Buffer);\r
       }\r
+\r
       Question->HiiValue.Type = Question->ReadExpression->Result.Type;\r
       CopyMem (&Question->HiiValue.Value, &Question->ReadExpression->Result.Value, sizeof (EFI_IFR_TYPE_VALUE));\r
       return EFI_SUCCESS;\r
@@ -1656,35 +1669,35 @@ GetQuestionValue (
   //\r
   // Question value is provided by RTC\r
   //\r
-  Storage = Question->Storage;\r
+  Storage       = Question->Storage;\r
   QuestionValue = &Question->HiiValue.Value;\r
   if (Storage == NULL) {\r
     //\r
     // It's a Question without storage, or RTC date/time\r
     //\r
-    if (Question->Operand == EFI_IFR_DATE_OP || Question->Operand == EFI_IFR_TIME_OP) {\r
+    if ((Question->Operand == EFI_IFR_DATE_OP) || (Question->Operand == EFI_IFR_TIME_OP)) {\r
       //\r
       // Date and time define the same Flags bit\r
       //\r
       switch (Question->Flags & EFI_QF_DATE_STORAGE) {\r
-      case QF_DATE_STORAGE_TIME:\r
-        Status = gRT->GetTime (&EfiTime, NULL);\r
-        break;\r
+        case QF_DATE_STORAGE_TIME:\r
+          Status = gRT->GetTime (&EfiTime, NULL);\r
+          break;\r
 \r
-      case QF_DATE_STORAGE_WAKEUP:\r
-        Status = gRT->GetWakeupTime (&Enabled, &Pending, &EfiTime);\r
-        break;\r
+        case QF_DATE_STORAGE_WAKEUP:\r
+          Status = gRT->GetWakeupTime (&Enabled, &Pending, &EfiTime);\r
+          break;\r
 \r
-      case QF_DATE_STORAGE_NORMAL:\r
-      default:\r
-        //\r
-        // For date/time without storage\r
-        //\r
-        return EFI_SUCCESS;\r
+        case QF_DATE_STORAGE_NORMAL:\r
+        default:\r
+          //\r
+          // For date/time without storage\r
+          //\r
+          return EFI_SUCCESS;\r
       }\r
 \r
       if (EFI_ERROR (Status)) {\r
-        if (Question->Operand == EFI_IFR_DATE_OP){\r
+        if (Question->Operand == EFI_IFR_DATE_OP) {\r
           QuestionValue->date.Year  = 0xff;\r
           QuestionValue->date.Month = 0xff;\r
           QuestionValue->date.Day   = 0xff;\r
@@ -1693,6 +1706,7 @@ GetQuestionValue (
           QuestionValue->time.Minute = 0xff;\r
           QuestionValue->time.Second = 0xff;\r
         }\r
+\r
         return EFI_SUCCESS;\r
       }\r
 \r
@@ -1718,16 +1732,16 @@ GetQuestionValue (
     if (Question->BufferValue != NULL) {\r
       Dst = Question->BufferValue;\r
     } else {\r
-      Dst = (UINT8 *) QuestionValue;\r
+      Dst = (UINT8 *)QuestionValue;\r
     }\r
 \r
     Status = gRT->GetVariable (\r
-                     Question->VariableName,\r
-                     &Storage->Guid,\r
-                     NULL,\r
-                     &StorageWidth,\r
-                     Dst\r
-                     );\r
+                    Question->VariableName,\r
+                    &Storage->Guid,\r
+                    NULL,\r
+                    &StorageWidth,\r
+                    Dst\r
+                    );\r
     //\r
     // Always return success, even this EFI variable doesn't exist\r
     //\r
@@ -1746,16 +1760,18 @@ GetQuestionValue (
     //\r
     // Other type of Questions\r
     //\r
-    Dst = (UINT8 *) &Question->HiiValue.Value;\r
+    Dst = (UINT8 *)&Question->HiiValue.Value;\r
   }\r
 \r
-  if (Storage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-      Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+  if ((Storage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+      (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER))\r
+  {\r
     IsBufferStorage = TRUE;\r
   } else {\r
     IsBufferStorage = FALSE;\r
   }\r
-  if (GetValueFrom == GetSetValueWithEditBuffer || GetValueFrom == GetSetValueWithBuffer ) {\r
+\r
+  if ((GetValueFrom == GetSetValueWithEditBuffer) || (GetValueFrom == GetSetValueWithBuffer)) {\r
     if (IsBufferStorage) {\r
       if (GetValueFrom == GetSetValueWithEditBuffer) {\r
         //\r
@@ -1779,7 +1795,7 @@ GetQuestionValue (
         }\r
       }\r
     } else {\r
-      Value = NULL;\r
+      Value  = NULL;\r
       Status = GetValueByName (Storage, Question->VariableName, &Value, GetValueFrom);\r
       if (EFI_ERROR (Status)) {\r
         return Status;\r
@@ -1790,21 +1806,22 @@ GetQuestionValue (
       FreePool (Value);\r
     }\r
   } else {\r
-    FormsetStorage = GetFstStgFromVarId(FormSet, Question->VarStoreId);\r
+    FormsetStorage = GetFstStgFromVarId (FormSet, Question->VarStoreId);\r
     ASSERT (FormsetStorage != NULL);\r
     //\r
     // <ConfigRequest> ::= <ConfigHdr> + <BlockName> ||\r
     //                   <ConfigHdr> + "&" + <VariableName>\r
     //\r
     if (IsBufferStorage) {\r
-      Length = StrLen (FormsetStorage->ConfigHdr);\r
+      Length  = StrLen (FormsetStorage->ConfigHdr);\r
       Length += StrLen (Question->BlockName);\r
     } else {\r
-      Length = StrLen (FormsetStorage->ConfigHdr);\r
+      Length  = StrLen (FormsetStorage->ConfigHdr);\r
       Length += StrLen (Question->VariableName) + 1;\r
     }\r
+\r
     // Allocate buffer include '\0'\r
-    MaxLen = Length + 1;\r
+    MaxLen        = Length + 1;\r
     ConfigRequest = AllocateZeroPool (MaxLen * sizeof (CHAR16));\r
     ASSERT (ConfigRequest != NULL);\r
 \r
@@ -1820,11 +1837,11 @@ GetQuestionValue (
     // Request current settings from Configuration Driver\r
     //\r
     Status = mHiiConfigRouting->ExtractConfig (\r
-                                      mHiiConfigRouting,\r
-                                      ConfigRequest,\r
-                                      &Progress,\r
-                                      &Result\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  ConfigRequest,\r
+                                  &Progress,\r
+                                  &Result\r
+                                  );\r
     FreePool (ConfigRequest);\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
@@ -1839,6 +1856,7 @@ GetQuestionValue (
         FreePool (Result);\r
         return EFI_NOT_FOUND;\r
       }\r
+\r
       //\r
       // Skip "&VALUE"\r
       //\r
@@ -1846,10 +1864,12 @@ GetQuestionValue (
     } else {\r
       Value = Result + Length;\r
     }\r
+\r
     if (*Value != '=') {\r
       FreePool (Result);\r
       return EFI_NOT_FOUND;\r
     }\r
+\r
     //\r
     // Skip '=', point to value\r
     //\r
@@ -1878,7 +1898,6 @@ GetQuestionValue (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Save Question Value to edit copy(cached) or Storage(uncached).\r
 \r
@@ -1892,10 +1911,10 @@ GetQuestionValue (
 **/\r
 EFI_STATUS\r
 SetQuestionValue (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN OUT FORM_BROWSER_STATEMENT       *Question,\r
-  IN GET_SET_QUESTION_VALUE_WITH      SetValueTo\r
+  IN FORM_BROWSER_FORMSET         *FormSet,\r
+  IN FORM_BROWSER_FORM            *Form,\r
+  IN OUT FORM_BROWSER_STATEMENT   *Question,\r
+  IN GET_SET_QUESTION_VALUE_WITH  SetValueTo\r
   )\r
 {\r
   EFI_STATUS          Status;\r
@@ -1938,7 +1957,7 @@ SetQuestionValue (
   //\r
   // Before set question value, evaluate its write expression.\r
   //\r
-  if (Question->WriteExpression != NULL && Form->FormType == STANDARD_MAP_FORM_TYPE) {\r
+  if ((Question->WriteExpression != NULL) && (Form->FormType == STANDARD_MAP_FORM_TYPE)) {\r
     Status = EvaluateExpression (FormSet, Form, Question->WriteExpression);\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
@@ -1948,31 +1967,31 @@ SetQuestionValue (
   //\r
   // Question value is provided by RTC\r
   //\r
-  Storage = Question->Storage;\r
+  Storage       = Question->Storage;\r
   QuestionValue = &Question->HiiValue.Value;\r
   if (Storage == NULL) {\r
     //\r
     // It's a Question without storage, or RTC date/time\r
     //\r
-    if (Question->Operand == EFI_IFR_DATE_OP || Question->Operand == EFI_IFR_TIME_OP) {\r
+    if ((Question->Operand == EFI_IFR_DATE_OP) || (Question->Operand == EFI_IFR_TIME_OP)) {\r
       //\r
       // Date and time define the same Flags bit\r
       //\r
       switch (Question->Flags & EFI_QF_DATE_STORAGE) {\r
-      case QF_DATE_STORAGE_TIME:\r
-        Status = gRT->GetTime (&EfiTime, NULL);\r
-        break;\r
+        case QF_DATE_STORAGE_TIME:\r
+          Status = gRT->GetTime (&EfiTime, NULL);\r
+          break;\r
 \r
-      case QF_DATE_STORAGE_WAKEUP:\r
-        Status = gRT->GetWakeupTime (&Enabled, &Pending, &EfiTime);\r
-        break;\r
+        case QF_DATE_STORAGE_WAKEUP:\r
+          Status = gRT->GetWakeupTime (&Enabled, &Pending, &EfiTime);\r
+          break;\r
 \r
-      case QF_DATE_STORAGE_NORMAL:\r
-      default:\r
-        //\r
-        // For date/time without storage\r
-        //\r
-        return EFI_SUCCESS;\r
+        case QF_DATE_STORAGE_NORMAL:\r
+        default:\r
+          //\r
+          // For date/time without storage\r
+          //\r
+          return EFI_SUCCESS;\r
       }\r
 \r
       if (EFI_ERROR (Status)) {\r
@@ -2007,16 +2026,16 @@ SetQuestionValue (
     if (Question->BufferValue != NULL) {\r
       Src = Question->BufferValue;\r
     } else {\r
-      Src = (UINT8 *) QuestionValue;\r
+      Src = (UINT8 *)QuestionValue;\r
     }\r
 \r
     Status = gRT->SetVariable (\r
-                     Question->VariableName,\r
-                     &Storage->Guid,\r
-                     Storage->Attributes,\r
-                     StorageWidth,\r
-                     Src\r
-                     );\r
+                    Question->VariableName,\r
+                    &Storage->Guid,\r
+                    Storage->Attributes,\r
+                    StorageWidth,\r
+                    Src\r
+                    );\r
     return Status;\r
   }\r
 \r
@@ -2026,18 +2045,20 @@ SetQuestionValue (
   if (Question->BufferValue != NULL) {\r
     Src = Question->BufferValue;\r
   } else {\r
-    Src = (UINT8 *) &Question->HiiValue.Value;\r
+    Src = (UINT8 *)&Question->HiiValue.Value;\r
   }\r
 \r
-  if (Storage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-      Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
+  if ((Storage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+      (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER))\r
+  {\r
     IsBufferStorage = TRUE;\r
   } else {\r
     IsBufferStorage = FALSE;\r
   }\r
-  IsString = (BOOLEAN) ((Question->HiiValue.Type == EFI_IFR_TYPE_STRING) ?  TRUE : FALSE);\r
 \r
-  if (SetValueTo == GetSetValueWithEditBuffer || SetValueTo == GetSetValueWithBuffer) {\r
+  IsString = (BOOLEAN)((Question->HiiValue.Type == EFI_IFR_TYPE_STRING) ?  TRUE : FALSE);\r
+\r
+  if ((SetValueTo == GetSetValueWithEditBuffer) || (SetValueTo == GetSetValueWithBuffer)) {\r
     if (IsBufferStorage) {\r
       if (SetValueTo == GetSetValueWithEditBuffer) {\r
         //\r
@@ -2065,16 +2086,16 @@ SetQuestionValue (
         //\r
         // Allocate enough string buffer.\r
         //\r
-        Value = NULL;\r
-        BufferLen = ((StrLen ((CHAR16 *) Src) * 4) + 1) * sizeof (CHAR16);\r
-        Value = AllocateZeroPool (BufferLen);\r
+        Value     = NULL;\r
+        BufferLen = ((StrLen ((CHAR16 *)Src) * 4) + 1) * sizeof (CHAR16);\r
+        Value     = AllocateZeroPool (BufferLen);\r
         ASSERT (Value != NULL);\r
         //\r
         // Convert Unicode String to Config String, e.g. "ABCD" => "0041004200430044"\r
         //\r
-        TemName = (CHAR16 *) Src;\r
+        TemName   = (CHAR16 *)Src;\r
         TemString = Value;\r
-        for (; *TemName != L'\0'; TemName++) {\r
+        for ( ; *TemName != L'\0'; TemName++) {\r
           UnicodeValueToStringS (\r
             TemString,\r
             BufferLen - ((UINTN)TemString - (UINTN)Value),\r
@@ -2086,14 +2107,14 @@ SetQuestionValue (
         }\r
       } else {\r
         BufferLen = StorageWidth * 2 + 1;\r
-        Value = AllocateZeroPool (BufferLen * sizeof (CHAR16));\r
+        Value     = AllocateZeroPool (BufferLen * sizeof (CHAR16));\r
         ASSERT (Value != NULL);\r
         //\r
         // Convert Buffer to Hex String\r
         //\r
         TemBuffer = Src + StorageWidth - 1;\r
         TemString = Value;\r
-        for (Index = 0; Index < StorageWidth; Index ++, TemBuffer --) {\r
+        for (Index = 0; Index < StorageWidth; Index++, TemBuffer--) {\r
           UnicodeValueToStringS (\r
             TemString,\r
             BufferLen * sizeof (CHAR16) - ((UINTN)TemString - (UINTN)Value),\r
@@ -2121,14 +2142,16 @@ SetQuestionValue (
     } else {\r
       Length = StrLen (Question->VariableName) + 2;\r
     }\r
+\r
     if (!IsBufferStorage && IsString) {\r
-      Length += (StrLen ((CHAR16 *) Src) * 4);\r
+      Length += (StrLen ((CHAR16 *)Src) * 4);\r
     } else {\r
       Length += (StorageWidth * 2);\r
     }\r
-    FormsetStorage = GetFstStgFromVarId(FormSet, Question->VarStoreId);\r
+\r
+    FormsetStorage = GetFstStgFromVarId (FormSet, Question->VarStoreId);\r
     ASSERT (FormsetStorage != NULL);\r
-    MaxLen = StrLen (FormsetStorage->ConfigHdr) + Length + 1;\r
+    MaxLen     = StrLen (FormsetStorage->ConfigHdr) + Length + 1;\r
     ConfigResp = AllocateZeroPool (MaxLen * sizeof (CHAR16));\r
     ASSERT (ConfigResp != NULL);\r
 \r
@@ -2148,9 +2171,9 @@ SetQuestionValue (
       //\r
       // Convert Unicode String to Config String, e.g. "ABCD" => "0041004200430044"\r
       //\r
-      TemName = (CHAR16 *) Src;\r
+      TemName   = (CHAR16 *)Src;\r
       TemString = Value;\r
-      for (; *TemName != L'\0'; TemName++) {\r
+      for ( ; *TemName != L'\0'; TemName++) {\r
         UnicodeValueToStringS (\r
           TemString,\r
           MaxLen * sizeof (CHAR16) - ((UINTN)TemString - (UINTN)ConfigResp),\r
@@ -2166,7 +2189,7 @@ SetQuestionValue (
       //\r
       TemBuffer = Src + StorageWidth - 1;\r
       TemString = Value;\r
-      for (Index = 0; Index < StorageWidth; Index ++, TemBuffer --) {\r
+      for (Index = 0; Index < StorageWidth; Index++, TemBuffer--) {\r
         UnicodeValueToStringS (\r
           TemString,\r
           MaxLen * sizeof (CHAR16) - ((UINTN)TemString - (UINTN)ConfigResp),\r
@@ -2182,8 +2205,8 @@ SetQuestionValue (
     // Convert to lower char.\r
     //\r
     for (TemString = Value; *Value != L'\0'; Value++) {\r
-      if (*Value >= L'A' && *Value <= L'Z') {\r
-        *Value = (CHAR16) (*Value - L'A' + L'a');\r
+      if ((*Value >= L'A') && (*Value <= L'Z')) {\r
+        *Value = (CHAR16)(*Value - L'A' + L'a');\r
       }\r
     }\r
 \r
@@ -2191,14 +2214,15 @@ SetQuestionValue (
     // Submit Question Value to Configuration Driver\r
     //\r
     Status = mHiiConfigRouting->RouteConfig (\r
-                                      mHiiConfigRouting,\r
-                                      ConfigResp,\r
-                                      &Progress\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  ConfigResp,\r
+                                  &Progress\r
+                                  );\r
     if (EFI_ERROR (Status)) {\r
       FreePool (ConfigResp);\r
       return Status;\r
     }\r
+\r
     FreePool (ConfigResp);\r
 \r
     //\r
@@ -2210,7 +2234,6 @@ SetQuestionValue (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Perform nosubmitif check for a Form.\r
 \r
@@ -2225,38 +2248,38 @@ SetQuestionValue (
 **/\r
 EFI_STATUS\r
 ValidateQuestion (\r
-  IN  FORM_BROWSER_FORMSET            *FormSet,\r
-  IN  FORM_BROWSER_FORM               *Form,\r
-  IN  FORM_BROWSER_STATEMENT          *Question,\r
-  IN  UINTN                           Type\r
+  IN  FORM_BROWSER_FORMSET    *FormSet,\r
+  IN  FORM_BROWSER_FORM       *Form,\r
+  IN  FORM_BROWSER_STATEMENT  *Question,\r
+  IN  UINTN                   Type\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  LIST_ENTRY              *Link;\r
-  LIST_ENTRY              *ListHead;\r
-  FORM_EXPRESSION         *Expression;\r
-  UINT32                  BrowserStatus;\r
-  CHAR16                  *ErrorStr;\r
+  EFI_STATUS       Status;\r
+  LIST_ENTRY       *Link;\r
+  LIST_ENTRY       *ListHead;\r
+  FORM_EXPRESSION  *Expression;\r
+  UINT32           BrowserStatus;\r
+  CHAR16           *ErrorStr;\r
 \r
   BrowserStatus = BROWSER_SUCCESS;\r
   ErrorStr      = NULL;\r
 \r
   switch (Type) {\r
-  case EFI_HII_EXPRESSION_INCONSISTENT_IF:\r
-    ListHead = &Question->InconsistentListHead;\r
-    break;\r
+    case EFI_HII_EXPRESSION_INCONSISTENT_IF:\r
+      ListHead = &Question->InconsistentListHead;\r
+      break;\r
 \r
-  case EFI_HII_EXPRESSION_WARNING_IF:\r
-    ListHead = &Question->WarningListHead;\r
-    break;\r
+    case EFI_HII_EXPRESSION_WARNING_IF:\r
+      ListHead = &Question->WarningListHead;\r
+      break;\r
 \r
-  case EFI_HII_EXPRESSION_NO_SUBMIT_IF:\r
-    ListHead = &Question->NoSubmitListHead;\r
-    break;\r
+    case EFI_HII_EXPRESSION_NO_SUBMIT_IF:\r
+      ListHead = &Question->NoSubmitListHead;\r
+      break;\r
 \r
-  default:\r
-    ASSERT (FALSE);\r
-    return EFI_UNSUPPORTED;\r
+    default:\r
+      ASSERT (FALSE);\r
+      return EFI_UNSUPPORTED;\r
   }\r
 \r
   Link = GetFirstNode (ListHead);\r
@@ -2273,28 +2296,29 @@ ValidateQuestion (
 \r
     if (IsTrue (&Expression->Result)) {\r
       switch (Type) {\r
-      case EFI_HII_EXPRESSION_INCONSISTENT_IF:\r
-        BrowserStatus = BROWSER_INCONSISTENT_IF;\r
-        break;\r
+        case EFI_HII_EXPRESSION_INCONSISTENT_IF:\r
+          BrowserStatus = BROWSER_INCONSISTENT_IF;\r
+          break;\r
 \r
-      case EFI_HII_EXPRESSION_WARNING_IF:\r
-        BrowserStatus = BROWSER_WARNING_IF;\r
-        break;\r
+        case EFI_HII_EXPRESSION_WARNING_IF:\r
+          BrowserStatus = BROWSER_WARNING_IF;\r
+          break;\r
 \r
-      case EFI_HII_EXPRESSION_NO_SUBMIT_IF:\r
-        BrowserStatus = BROWSER_NO_SUBMIT_IF;\r
-        //\r
-        // This code only used to compatible with old display engine,\r
-        // New display engine will not use this field.\r
-        //\r
-        if (Expression->Error != 0) {\r
-          ErrorStr = GetToken (Expression->Error, FormSet->HiiHandle);\r
-        }\r
-        break;\r
+        case EFI_HII_EXPRESSION_NO_SUBMIT_IF:\r
+          BrowserStatus = BROWSER_NO_SUBMIT_IF;\r
+          //\r
+          // This code only used to compatible with old display engine,\r
+          // New display engine will not use this field.\r
+          //\r
+          if (Expression->Error != 0) {\r
+            ErrorStr = GetToken (Expression->Error, FormSet->HiiHandle);\r
+          }\r
 \r
-      default:\r
-        ASSERT (FALSE);\r
-        break;\r
+          break;\r
+\r
+        default:\r
+          ASSERT (FALSE);\r
+          break;\r
       }\r
 \r
       if (!((Type == EFI_HII_EXPRESSION_NO_SUBMIT_IF) && mSystemSubmit)) {\r
@@ -2302,7 +2326,7 @@ ValidateQuestion (
         // If in system submit process and for no_submit_if check, not popup this error message.\r
         // Will process this fail again later in not system submit process.\r
         //\r
-        PopupErrorMessage(BrowserStatus, FormSet->HiiHandle, Expression->OpCode, ErrorStr);\r
+        PopupErrorMessage (BrowserStatus, FormSet->HiiHandle, Expression->OpCode, ErrorStr);\r
       }\r
 \r
       if (ErrorStr != NULL) {\r
@@ -2338,12 +2362,12 @@ ValidateQuestion (
 **/\r
 EFI_STATUS\r
 ValueChangedValidation (\r
-  IN  FORM_BROWSER_FORMSET            *FormSet,\r
-  IN  FORM_BROWSER_FORM               *Form,\r
-  IN  FORM_BROWSER_STATEMENT          *Question\r
+  IN  FORM_BROWSER_FORMSET    *FormSet,\r
+  IN  FORM_BROWSER_FORM       *Form,\r
+  IN  FORM_BROWSER_STATEMENT  *Question\r
   )\r
 {\r
-  EFI_STATUS   Status;\r
+  EFI_STATUS  Status;\r
 \r
   Status = EFI_SUCCESS;\r
 \r
@@ -2380,9 +2404,9 @@ ValueChangedValidation (
 **/\r
 EFI_STATUS\r
 NoSubmitCheck (\r
-  IN      FORM_BROWSER_FORMSET            *FormSet,\r
-  IN OUT  FORM_BROWSER_FORM               **CurrentForm,\r
-  OUT     FORM_BROWSER_STATEMENT          **Statement\r
+  IN      FORM_BROWSER_FORMSET    *FormSet,\r
+  IN OUT  FORM_BROWSER_FORM       **CurrentForm,\r
+  OUT     FORM_BROWSER_STATEMENT  **Statement\r
   )\r
 {\r
   EFI_STATUS              Status;\r
@@ -2393,24 +2417,26 @@ NoSubmitCheck (
 \r
   LinkForm = GetFirstNode (&FormSet->FormListHead);\r
   while (!IsNull (&FormSet->FormListHead, LinkForm)) {\r
-    Form = FORM_BROWSER_FORM_FROM_LINK (LinkForm);\r
+    Form     = FORM_BROWSER_FORM_FROM_LINK (LinkForm);\r
     LinkForm = GetNextNode (&FormSet->FormListHead, LinkForm);\r
 \r
-    if (*CurrentForm != NULL && *CurrentForm != Form) {\r
+    if ((*CurrentForm != NULL) && (*CurrentForm != Form)) {\r
       continue;\r
     }\r
 \r
     Link = GetFirstNode (&Form->StatementListHead);\r
     while (!IsNull (&Form->StatementListHead, Link)) {\r
       Question = FORM_BROWSER_STATEMENT_FROM_LINK (Link);\r
-      Status = ValidateQuestion (FormSet, Form, Question, EFI_HII_EXPRESSION_NO_SUBMIT_IF);\r
+      Status   = ValidateQuestion (FormSet, Form, Question, EFI_HII_EXPRESSION_NO_SUBMIT_IF);\r
       if (EFI_ERROR (Status)) {\r
         if (*CurrentForm == NULL) {\r
           *CurrentForm = Form;\r
         }\r
+\r
         if (Statement != NULL) {\r
           *Statement = Question;\r
         }\r
+\r
         return Status;\r
       }\r
 \r
@@ -2436,25 +2462,26 @@ NoSubmitCheck (
 **/\r
 EFI_STATUS\r
 SynchronizeStorage (\r
-  OUT BROWSER_STORAGE             *Storage,\r
-  IN  CHAR16                      *ConfigRequest,\r
-  IN  BOOLEAN                     SyncOrRestore\r
+  OUT BROWSER_STORAGE  *Storage,\r
+  IN  CHAR16           *ConfigRequest,\r
+  IN  BOOLEAN          SyncOrRestore\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  EFI_STRING              Progress;\r
-  EFI_STRING              Result;\r
-  UINTN                   BufferSize;\r
-  LIST_ENTRY              *Link;\r
-  NAME_VALUE_NODE         *Node;\r
-  UINT8                   *Src;\r
-  UINT8                   *Dst;\r
+  EFI_STATUS       Status;\r
+  EFI_STRING       Progress;\r
+  EFI_STRING       Result;\r
+  UINTN            BufferSize;\r
+  LIST_ENTRY       *Link;\r
+  NAME_VALUE_NODE  *Node;\r
+  UINT8            *Src;\r
+  UINT8            *Dst;\r
 \r
   Status = EFI_SUCCESS;\r
   Result = NULL;\r
 \r
-  if (Storage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-      (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER)) {\r
+  if ((Storage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+      (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER))\r
+  {\r
     BufferSize = Storage->Size;\r
 \r
     if (SyncOrRestore) {\r
@@ -2466,7 +2493,7 @@ SynchronizeStorage (
     }\r
 \r
     if (ConfigRequest != NULL) {\r
-      Status = mHiiConfigRouting->BlockToConfig(\r
+      Status = mHiiConfigRouting->BlockToConfig (\r
                                     mHiiConfigRouting,\r
                                     ConfigRequest,\r
                                     Src,\r
@@ -2496,8 +2523,9 @@ SynchronizeStorage (
     while (!IsNull (&Storage->NameValueListHead, Link)) {\r
       Node = NAME_VALUE_NODE_FROM_LINK (Link);\r
 \r
-      if ((ConfigRequest != NULL && StrStr (ConfigRequest, Node->Name) != NULL) ||\r
-          (ConfigRequest == NULL)) {\r
+      if (((ConfigRequest != NULL) && (StrStr (ConfigRequest, Node->Name) != NULL)) ||\r
+          (ConfigRequest == NULL))\r
+      {\r
         if (SyncOrRestore) {\r
           NewStringCpy (&Node->Value, Node->EditValue);\r
         } else {\r
@@ -2522,8 +2550,8 @@ SynchronizeStorage (
 **/\r
 VOID\r
 SendDiscardInfoToDriver (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
   LIST_ENTRY                  *Link;\r
@@ -2538,9 +2566,9 @@ SendDiscardInfoToDriver (
   Link = GetFirstNode (&Form->StatementListHead);\r
   while (!IsNull (&Form->StatementListHead, Link)) {\r
     Question = FORM_BROWSER_STATEMENT_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->StatementListHead, Link);\r
+    Link     = GetNextNode (&Form->StatementListHead, Link);\r
 \r
-    if (Question->Storage == NULL || Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
+    if ((Question->Storage == NULL) || (Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE)) {\r
       continue;\r
     }\r
 \r
@@ -2561,12 +2589,12 @@ SendDiscardInfoToDriver (
     //\r
     GetQuestionValue (FormSet, Form, Question, GetSetValueWithEditBuffer);\r
 \r
-    if (Question->Operand == EFI_IFR_STRING_OP){\r
-      HiiSetString (FormSet->HiiHandle, Question->HiiValue.Value.string, (CHAR16*)Question->BufferValue, NULL);\r
+    if (Question->Operand == EFI_IFR_STRING_OP) {\r
+      HiiSetString (FormSet->HiiHandle, Question->HiiValue.Value.string, (CHAR16 *)Question->BufferValue, NULL);\r
     }\r
 \r
     if (Question->HiiValue.Type == EFI_IFR_TYPE_BUFFER) {\r
-      TypeValue = (EFI_IFR_TYPE_VALUE *) Question->BufferValue;\r
+      TypeValue = (EFI_IFR_TYPE_VALUE *)Question->BufferValue;\r
     } else {\r
       TypeValue = &Question->HiiValue.Value;\r
     }\r
@@ -2593,8 +2621,8 @@ SendDiscardInfoToDriver (
 **/\r
 VOID\r
 SubmitCallbackForForm (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
   LIST_ENTRY                  *Link;\r
@@ -2609,22 +2637,22 @@ SubmitCallbackForForm (
   Link = GetFirstNode (&Form->StatementListHead);\r
   while (!IsNull (&Form->StatementListHead, Link)) {\r
     Question = FORM_BROWSER_STATEMENT_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->StatementListHead, Link);\r
+    Link     = GetNextNode (&Form->StatementListHead, Link);\r
 \r
-    if (Question->Storage == NULL || Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
+    if ((Question->Storage == NULL) || (Question->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE)) {\r
       continue;\r
     }\r
 \r
     if ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) != EFI_IFR_FLAG_CALLBACK) {\r
-       continue;\r
+      continue;\r
     }\r
 \r
     if (Question->Operand == EFI_IFR_PASSWORD_OP) {\r
-       continue;\r
+      continue;\r
     }\r
 \r
     if (Question->HiiValue.Type == EFI_IFR_TYPE_BUFFER) {\r
-      TypeValue = (EFI_IFR_TYPE_VALUE *) Question->BufferValue;\r
+      TypeValue = (EFI_IFR_TYPE_VALUE *)Question->BufferValue;\r
     } else {\r
       TypeValue = &Question->HiiValue.Value;\r
     }\r
@@ -2650,24 +2678,24 @@ SubmitCallbackForForm (
 **/\r
 VOID\r
 SubmitCallback (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
-  FORM_BROWSER_FORM       *CurrentForm;\r
-  LIST_ENTRY              *Link;\r
+  FORM_BROWSER_FORM  *CurrentForm;\r
+  LIST_ENTRY         *Link;\r
 \r
   if (Form != NULL) {\r
-    SubmitCallbackForForm(FormSet, Form);\r
+    SubmitCallbackForForm (FormSet, Form);\r
     return;\r
   }\r
 \r
   Link = GetFirstNode (&FormSet->FormListHead);\r
   while (!IsNull (&FormSet->FormListHead, Link)) {\r
     CurrentForm = FORM_BROWSER_FORM_FROM_LINK (Link);\r
-    Link = GetNextNode (&FormSet->FormListHead, Link);\r
+    Link        = GetNextNode (&FormSet->FormListHead, Link);\r
 \r
-    SubmitCallbackForForm(FormSet, CurrentForm);\r
+    SubmitCallbackForForm (FormSet, CurrentForm);\r
   }\r
 }\r
 \r
@@ -2682,12 +2710,12 @@ SubmitCallback (
 **/\r
 BOOLEAN\r
 ValidateHiiHandle (\r
-  EFI_HII_HANDLE          HiiHandle\r
+  EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
-  EFI_HII_HANDLE          *HiiHandles;\r
-  UINTN                   Index;\r
-  BOOLEAN                 Find;\r
+  EFI_HII_HANDLE  *HiiHandles;\r
+  UINTN           Index;\r
+  BOOLEAN         Find;\r
 \r
   if (HiiHandle == NULL) {\r
     return FALSE;\r
@@ -2721,25 +2749,26 @@ ValidateHiiHandle (
 **/\r
 BOOLEAN\r
 ValidateFormSet (\r
-  FORM_BROWSER_FORMSET    *FormSet\r
+  FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
   BOOLEAN  Find;\r
 \r
   ASSERT (FormSet != NULL);\r
 \r
-  Find = ValidateHiiHandle(FormSet->HiiHandle);\r
+  Find = ValidateHiiHandle (FormSet->HiiHandle);\r
   //\r
   // Should not remove the formset which is being used.\r
   //\r
   if (!Find && (FormSet != gCurrentSelection->FormSet)) {\r
-    CleanBrowserStorage(FormSet);\r
+    CleanBrowserStorage (FormSet);\r
     RemoveEntryList (&FormSet->Link);\r
     DestroyFormSet (FormSet);\r
   }\r
 \r
   return Find;\r
 }\r
+\r
 /**\r
   Check whether need to enable the reset flag in form level.\r
   Also clean all ValueChanged flag in question.\r
@@ -2751,9 +2780,9 @@ ValidateFormSet (
 **/\r
 VOID\r
 UpdateFlagForForm (\r
-  IN BOOLEAN                          SetFlag,\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN BOOLEAN               SetFlag,\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
   LIST_ENTRY              *Link;\r
@@ -2763,7 +2792,7 @@ UpdateFlagForForm (
   Link = GetFirstNode (&Form->StatementListHead);\r
   while (!IsNull (&Form->StatementListHead, Link)) {\r
     Question = FORM_BROWSER_STATEMENT_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->StatementListHead, Link);\r
+    Link     = GetNextNode (&Form->StatementListHead, Link);\r
 \r
     if (!Question->ValueChanged) {\r
       continue;\r
@@ -2774,14 +2803,14 @@ UpdateFlagForForm (
     //\r
     // Compare the buffer and editbuffer data to see whether the data has been saved.\r
     //\r
-    Question->ValueChanged = IsQuestionValueChanged(FormSet, Form, Question, GetSetValueWithBothBuffer);\r
+    Question->ValueChanged = IsQuestionValueChanged (FormSet, Form, Question, GetSetValueWithBothBuffer);\r
 \r
     //\r
     // Only the changed data has been saved, then need to set the reset flag.\r
     //\r
     if (SetFlag && OldValue && !Question->ValueChanged) {\r
       if ((Question->QuestionFlags & EFI_IFR_FLAG_RESET_REQUIRED) != 0) {\r
-        gResetRequiredFormLevel = TRUE;\r
+        gResetRequiredFormLevel   = TRUE;\r
         gResetRequiredSystemLevel = TRUE;\r
       }\r
 \r
@@ -2805,25 +2834,25 @@ UpdateFlagForForm (
 **/\r
 VOID\r
 ValueChangeResetFlagUpdate (\r
-  IN BOOLEAN                          SetFlag,\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN BOOLEAN               SetFlag,\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
-  FORM_BROWSER_FORM       *CurrentForm;\r
-  LIST_ENTRY              *Link;\r
+  FORM_BROWSER_FORM  *CurrentForm;\r
+  LIST_ENTRY         *Link;\r
 \r
   if (Form != NULL) {\r
-    UpdateFlagForForm(SetFlag, FormSet, Form);\r
+    UpdateFlagForForm (SetFlag, FormSet, Form);\r
     return;\r
   }\r
 \r
   Link = GetFirstNode (&FormSet->FormListHead);\r
   while (!IsNull (&FormSet->FormListHead, Link)) {\r
     CurrentForm = FORM_BROWSER_FORM_FROM_LINK (Link);\r
-    Link = GetNextNode (&FormSet->FormListHead, Link);\r
+    Link        = GetNextNode (&FormSet->FormListHead, Link);\r
 \r
-    UpdateFlagForForm(SetFlag, FormSet, CurrentForm);\r
+    UpdateFlagForForm (SetFlag, FormSet, CurrentForm);\r
   }\r
 }\r
 \r
@@ -2845,11 +2874,11 @@ ValueChangeResetFlagUpdate (
 **/\r
 BOOLEAN\r
 FindQuestionFromProgress (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN BROWSER_STORAGE                  *Storage,\r
-  IN EFI_STRING                       Progress,\r
-  OUT FORM_BROWSER_FORM               **RetForm,\r
-  OUT FORM_BROWSER_STATEMENT          **RetQuestion\r
+  IN FORM_BROWSER_FORMSET     *FormSet,\r
+  IN BROWSER_STORAGE          *Storage,\r
+  IN EFI_STRING               Progress,\r
+  OUT FORM_BROWSER_FORM       **RetForm,\r
+  OUT FORM_BROWSER_STATEMENT  **RetQuestion\r
   )\r
 {\r
   LIST_ENTRY                   *Link;\r
@@ -2932,7 +2961,7 @@ FindQuestionFromProgress (
     //\r
     LinkStorage = GetFirstNode (&Form->ConfigRequestHead);\r
     while (!IsNull (&Form->ConfigRequestHead, LinkStorage)) {\r
-      ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (LinkStorage);\r
+      ConfigInfo  = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (LinkStorage);\r
       LinkStorage = GetNextNode (&Form->ConfigRequestHead, LinkStorage);\r
 \r
       if (Storage != ConfigInfo->Storage) {\r
@@ -2951,15 +2980,15 @@ FindQuestionFromProgress (
     if (*RetForm != NULL) {\r
       LinkStatement = GetFirstNode (&Form->StatementListHead);\r
       while (!IsNull (&Form->StatementListHead, LinkStatement)) {\r
-        Statement = FORM_BROWSER_STATEMENT_FROM_LINK (LinkStatement);\r
+        Statement     = FORM_BROWSER_STATEMENT_FROM_LINK (LinkStatement);\r
         LinkStatement = GetNextNode (&Form->StatementListHead, LinkStatement);\r
 \r
-        if (Statement->BlockName != NULL && StrStr (Statement->BlockName, Progress) != NULL) {\r
+        if ((Statement->BlockName != NULL) && (StrStr (Statement->BlockName, Progress) != NULL)) {\r
           *RetQuestion = Statement;\r
           break;\r
         }\r
 \r
-        if (Statement->VariableName != NULL && StrStr (Statement->VariableName, Progress) != NULL) {\r
+        if ((Statement->VariableName != NULL) && (StrStr (Statement->VariableName, Progress) != NULL)) {\r
           *RetQuestion = Statement;\r
           break;\r
         }\r
@@ -2980,7 +3009,7 @@ FindQuestionFromProgress (
     *EndStr = '&';\r
   }\r
 \r
-  return (BOOLEAN) (*RetForm != NULL);\r
+  return (BOOLEAN)(*RetForm != NULL);\r
 }\r
 \r
 /**\r
@@ -2995,20 +3024,20 @@ FindQuestionFromProgress (
 \r
 **/\r
 VOID\r
-GetSyncRestoreConfigRequest(\r
-  IN  BROWSER_STORAGE   *Storage,\r
-  IN  EFI_STRING        ConfigRequest,\r
-  IN  EFI_STRING        Progress,\r
-  OUT EFI_STRING        *RestoreConfigRequest,\r
-  OUT EFI_STRING        *SyncConfigRequest\r
+GetSyncRestoreConfigRequest (\r
+  IN  BROWSER_STORAGE  *Storage,\r
+  IN  EFI_STRING       ConfigRequest,\r
+  IN  EFI_STRING       Progress,\r
+  OUT EFI_STRING       *RestoreConfigRequest,\r
+  OUT EFI_STRING       *SyncConfigRequest\r
   )\r
 {\r
-  EFI_STRING    EndStr;\r
-  EFI_STRING    ConfigHdrEndStr;\r
-  EFI_STRING    ElementStr;\r
-  UINTN         TotalSize;\r
-  UINTN         RestoreEleSize;\r
-  UINTN         SyncSize;\r
+  EFI_STRING  EndStr;\r
+  EFI_STRING  ConfigHdrEndStr;\r
+  EFI_STRING  ElementStr;\r
+  UINTN       TotalSize;\r
+  UINTN       RestoreEleSize;\r
+  UINTN       SyncSize;\r
 \r
   ASSERT ((*Progress == L'&') || (*Progress == L'G'));\r
   //\r
@@ -3053,6 +3082,7 @@ GetSyncRestoreConfigRequest(
     //\r
     ConfigHdrEndStr = StrStr (ConfigRequest, L"&OFFSET=");\r
   }\r
+\r
   //\r
   // Find the first fail pair in the ConfigRequest.\r
   //\r
@@ -3061,8 +3091,8 @@ GetSyncRestoreConfigRequest(
   //\r
   // To get the RestoreConfigRequest.\r
   //\r
-  RestoreEleSize = StrSize (ElementStr);\r
-  TotalSize = (ConfigHdrEndStr - ConfigRequest) * sizeof (CHAR16) + RestoreEleSize + sizeof (CHAR16);\r
+  RestoreEleSize        = StrSize (ElementStr);\r
+  TotalSize             = (ConfigHdrEndStr - ConfigRequest) * sizeof (CHAR16) + RestoreEleSize + sizeof (CHAR16);\r
   *RestoreConfigRequest = AllocateZeroPool (TotalSize);\r
   ASSERT (*RestoreConfigRequest != NULL);\r
   StrnCpyS (*RestoreConfigRequest, TotalSize / sizeof (CHAR16), ConfigRequest, ConfigHdrEndStr - ConfigRequest);\r
@@ -3070,7 +3100,7 @@ GetSyncRestoreConfigRequest(
   //\r
   // To get the SyncConfigRequest.\r
   //\r
-  SyncSize = StrSize (ConfigRequest) - RestoreEleSize + sizeof (CHAR16);\r
+  SyncSize           = StrSize (ConfigRequest) - RestoreEleSize + sizeof (CHAR16);\r
   *SyncConfigRequest = AllocateZeroPool (SyncSize);\r
   ASSERT (*SyncConfigRequest != NULL);\r
   StrnCpyS (*SyncConfigRequest, SyncSize / sizeof (CHAR16), ConfigRequest, SyncSize / sizeof (CHAR16) - 1);\r
@@ -3096,13 +3126,13 @@ GetSyncRestoreConfigRequest(
 **/\r
 UINT32\r
 ConfirmSaveFail (\r
-  IN EFI_STRING_ID    TitleId,\r
-  IN EFI_HII_HANDLE   HiiHandle\r
+  IN EFI_STRING_ID   TitleId,\r
+  IN EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
-  CHAR16                  *FormTitle;\r
-  CHAR16                  *StringBuffer;\r
-  UINT32                  RetVal;\r
+  CHAR16  *FormTitle;\r
+  CHAR16  *StringBuffer;\r
+  UINT32  RetVal;\r
 \r
   FormTitle = GetToken (TitleId, HiiHandle);\r
 \r
@@ -3116,7 +3146,7 @@ ConfirmSaveFail (
     FormTitle\r
     );\r
 \r
-  RetVal = PopupErrorMessage(BROWSER_SUBMIT_FAIL, NULL, NULL, StringBuffer);\r
+  RetVal = PopupErrorMessage (BROWSER_SUBMIT_FAIL, NULL, NULL, StringBuffer);\r
 \r
   FreePool (StringBuffer);\r
   FreePool (FormTitle);\r
@@ -3135,13 +3165,13 @@ ConfirmSaveFail (
 **/\r
 UINT32\r
 ConfirmNoSubmitFail (\r
-  IN EFI_STRING_ID    TitleId,\r
-  IN EFI_HII_HANDLE   HiiHandle\r
+  IN EFI_STRING_ID   TitleId,\r
+  IN EFI_HII_HANDLE  HiiHandle\r
   )\r
 {\r
-  CHAR16                  *FormTitle;\r
-  CHAR16                  *StringBuffer;\r
-  UINT32                  RetVal;\r
+  CHAR16  *FormTitle;\r
+  CHAR16  *StringBuffer;\r
+  UINT32  RetVal;\r
 \r
   FormTitle = GetToken (TitleId, HiiHandle);\r
 \r
@@ -3155,7 +3185,7 @@ ConfirmNoSubmitFail (
     FormTitle\r
     );\r
 \r
-  RetVal = PopupErrorMessage(BROWSER_SUBMIT_FAIL_NO_SUBMIT_IF, NULL, NULL, StringBuffer);\r
+  RetVal = PopupErrorMessage (BROWSER_SUBMIT_FAIL_NO_SUBMIT_IF, NULL, NULL, StringBuffer);\r
 \r
   FreePool (StringBuffer);\r
   FreePool (FormTitle);\r
@@ -3176,9 +3206,9 @@ ConfirmNoSubmitFail (
 **/\r
 EFI_STATUS\r
 DiscardForm (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN BROWSER_SETTING_SCOPE            SettingScope\r
+  IN FORM_BROWSER_FORMSET   *FormSet,\r
+  IN FORM_BROWSER_FORM      *Form,\r
+  IN BROWSER_SETTING_SCOPE  SettingScope\r
   )\r
 {\r
   LIST_ENTRY                   *Link;\r
@@ -3194,12 +3224,12 @@ DiscardForm (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  if (SettingScope == FormLevel && IsNvUpdateRequiredForForm (Form)) {\r
+  if ((SettingScope == FormLevel) && IsNvUpdateRequiredForForm (Form)) {\r
     ConfigInfo = NULL;\r
-    Link = GetFirstNode (&Form->ConfigRequestHead);\r
+    Link       = GetFirstNode (&Form->ConfigRequestHead);\r
     while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
       ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
-      Link = GetNextNode (&Form->ConfigRequestHead, Link);\r
+      Link       = GetNextNode (&Form->ConfigRequestHead, Link);\r
 \r
       if (ConfigInfo->Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
         continue;\r
@@ -3215,7 +3245,7 @@ DiscardForm (
       //\r
       // Prepare <ConfigResp>\r
       //\r
-      SynchronizeStorage(ConfigInfo->Storage, ConfigInfo->ConfigRequest, FALSE);\r
+      SynchronizeStorage (ConfigInfo->Storage, ConfigInfo->ConfigRequest, FALSE);\r
 \r
       //\r
       // Call callback with Changed type to inform the driver.\r
@@ -3224,15 +3254,14 @@ DiscardForm (
     }\r
 \r
     ValueChangeResetFlagUpdate (FALSE, FormSet, Form);\r
-  } else if (SettingScope == FormSetLevel && IsNvUpdateRequiredForFormSet (FormSet)) {\r
-\r
+  } else if ((SettingScope == FormSetLevel) && IsNvUpdateRequiredForFormSet (FormSet)) {\r
     //\r
     // Discard Buffer storage or Name/Value storage\r
     //\r
     Link = GetFirstNode (&FormSet->StorageListHead);\r
     while (!IsNull (&FormSet->StorageListHead, Link)) {\r
       Storage = FORMSET_STORAGE_FROM_LINK (Link);\r
-      Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+      Link    = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
       if (Storage->BrowserStorage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
         continue;\r
@@ -3245,7 +3274,7 @@ DiscardForm (
         continue;\r
       }\r
 \r
-      SynchronizeStorage(Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
+      SynchronizeStorage (Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
     }\r
 \r
     Link = GetFirstNode (&FormSet->FormListHead);\r
@@ -3259,7 +3288,7 @@ DiscardForm (
       SendDiscardInfoToDriver (FormSet, Form);\r
     }\r
 \r
-    ValueChangeResetFlagUpdate(FALSE, FormSet, NULL);\r
+    ValueChangeResetFlagUpdate (FALSE, FormSet, NULL);\r
   } else if (SettingScope == SystemLevel) {\r
     //\r
     // System Level Discard.\r
@@ -3272,8 +3301,8 @@ DiscardForm (
     Link = GetFirstNode (&gBrowserFormSetList);\r
     while (!IsNull (&gBrowserFormSetList, Link)) {\r
       LocalFormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-      Link = GetNextNode (&gBrowserFormSetList, Link);\r
-      if (!ValidateFormSet(LocalFormSet)) {\r
+      Link         = GetNextNode (&gBrowserFormSetList, Link);\r
+      if (!ValidateFormSet (LocalFormSet)) {\r
         continue;\r
       }\r
 \r
@@ -3284,7 +3313,7 @@ DiscardForm (
         //\r
         // Remove maintain backup list after discard except for the current using FormSet.\r
         //\r
-        CleanBrowserStorage(LocalFormSet);\r
+        CleanBrowserStorage (LocalFormSet);\r
         RemoveEntryList (&LocalFormSet->Link);\r
         DestroyFormSet (LocalFormSet);\r
       }\r
@@ -3308,17 +3337,17 @@ DiscardForm (
 **/\r
 EFI_STATUS\r
 SubmitForForm (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  LIST_ENTRY              *Link;\r
-  EFI_STRING              ConfigResp;\r
-  EFI_STRING              Progress;\r
-  BROWSER_STORAGE         *Storage;\r
+  EFI_STATUS                   Status;\r
+  LIST_ENTRY                   *Link;\r
+  EFI_STRING                   ConfigResp;\r
+  EFI_STRING                   Progress;\r
+  BROWSER_STORAGE              *Storage;\r
   FORM_BROWSER_CONFIG_REQUEST  *ConfigInfo;\r
-  BOOLEAN                 SubmitFormFail;\r
+  BOOLEAN                      SubmitFormFail;\r
 \r
   SubmitFormFail = FALSE;\r
 \r
@@ -3334,7 +3363,7 @@ SubmitForForm (
   Link = GetFirstNode (&Form->ConfigRequestHead);\r
   while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
     ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->ConfigRequestHead, Link);\r
+    Link       = GetNextNode (&Form->ConfigRequestHead, Link);\r
 \r
     Storage = ConfigInfo->Storage;\r
     if (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
@@ -3360,10 +3389,10 @@ SubmitForForm (
     // 2. Set value to hii config routine protocol.\r
     //\r
     Status = mHiiConfigRouting->RouteConfig (\r
-                                      mHiiConfigRouting,\r
-                                      ConfigResp,\r
-                                      &Progress\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  ConfigResp,\r
+                                  &Progress\r
+                                  );\r
 \r
     if (EFI_ERROR (Status)) {\r
       //\r
@@ -3391,7 +3420,7 @@ SubmitForForm (
       Link = GetFirstNode (&gBrowserSaveFailFormSetList);\r
       while (!IsNull (&gBrowserSaveFailFormSetList, Link)) {\r
         ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_SAVE_FAIL_LINK (Link);\r
-        Link = GetNextNode (&gBrowserSaveFailFormSetList, Link);\r
+        Link       = GetNextNode (&gBrowserSaveFailFormSetList, Link);\r
         //\r
         // Process the submit fail question, base on the RestoreConfigRequest to restore the EditBuffer\r
         // base on the SyncConfigRequest to Sync the buffer.\r
@@ -3400,14 +3429,15 @@ SubmitForForm (
         FreePool (ConfigInfo->RestoreConfigRequest);\r
         ConfigInfo->RestoreConfigRequest = NULL;\r
         if (ConfigInfo->SyncConfigRequest != NULL) {\r
-          SynchronizeStorage(ConfigInfo->Storage, ConfigInfo->SyncConfigRequest, TRUE);\r
+          SynchronizeStorage (ConfigInfo->Storage, ConfigInfo->SyncConfigRequest, TRUE);\r
           FreePool (ConfigInfo->SyncConfigRequest);\r
           ConfigInfo->SyncConfigRequest = NULL;\r
         }\r
 \r
         Status = EFI_SUCCESS;\r
       }\r
-      SendDiscardInfoToDriver (FormSet,Form);\r
+\r
+      SendDiscardInfoToDriver (FormSet, Form);\r
     } else {\r
       Status = EFI_UNSUPPORTED;\r
     }\r
@@ -3416,7 +3446,7 @@ SubmitForForm (
     // Free Form save fail list.\r
     //\r
     while (!IsListEmpty (&gBrowserSaveFailFormSetList)) {\r
-      Link = GetFirstNode (&gBrowserSaveFailFormSetList);\r
+      Link       = GetFirstNode (&gBrowserSaveFailFormSetList);\r
       ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_SAVE_FAIL_LINK (Link);\r
       RemoveEntryList (&ConfigInfo->SaveFailLink);\r
     }\r
@@ -3425,7 +3455,7 @@ SubmitForForm (
   //\r
   // 5. Update the NV flag.\r
   //\r
-  ValueChangeResetFlagUpdate(TRUE, FormSet, Form);\r
+  ValueChangeResetFlagUpdate (TRUE, FormSet, Form);\r
 \r
   //\r
   // 6 Call callback with Submitted type to inform the driver.\r
@@ -3455,8 +3485,8 @@ SubmitForForm (
 **/\r
 EFI_STATUS\r
 SubmitForFormSet (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN BOOLEAN                          SkipProcessFail\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN BOOLEAN               SkipProcessFail\r
   )\r
 {\r
   EFI_STATUS              Status;\r
@@ -3471,7 +3501,7 @@ SubmitForFormSet (
   BOOLEAN                 SubmitFormSetFail;\r
   BOOLEAN                 DiscardChange;\r
 \r
-  HasInserted = FALSE;\r
+  HasInserted       = FALSE;\r
   SubmitFormSetFail = FALSE;\r
   DiscardChange     = FALSE;\r
 \r
@@ -3479,14 +3509,14 @@ SubmitForFormSet (
     return EFI_SUCCESS;\r
   }\r
 \r
-  Form = NULL;\r
+  Form   = NULL;\r
   Status = NoSubmitCheck (FormSet, &Form, &Question);\r
   if (EFI_ERROR (Status)) {\r
     if (SkipProcessFail) {\r
       //\r
       // Process NO_SUBMIT check first, so insert it at head.\r
       //\r
-      FormSet->SaveFailForm = Form;\r
+      FormSet->SaveFailForm      = Form;\r
       FormSet->SaveFailStatement = Question;\r
       InsertHeadList (&gBrowserSaveFailFormSetList, &FormSet->SaveFailLink);\r
     }\r
@@ -3494,7 +3524,7 @@ SubmitForFormSet (
     return Status;\r
   }\r
 \r
-  Form = NULL;\r
+  Form     = NULL;\r
   Question = NULL;\r
   //\r
   // Submit Buffer storage or Name/Value storage\r
@@ -3503,7 +3533,7 @@ SubmitForFormSet (
   while (!IsNull (&FormSet->StorageListHead, Link)) {\r
     FormSetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
     Storage        = FormSetStorage->BrowserStorage;\r
-    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+    Link           = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
     if (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
       continue;\r
@@ -3528,10 +3558,10 @@ SubmitForFormSet (
     // 2. Send <ConfigResp> to Routine config Protocol.\r
     //\r
     Status = mHiiConfigRouting->RouteConfig (\r
-                                      mHiiConfigRouting,\r
-                                      ConfigResp,\r
-                                      &Progress\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  ConfigResp,\r
+                                  &Progress\r
+                                  );\r
     if (EFI_ERROR (Status)) {\r
       //\r
       // Submit fail, to get the RestoreConfigRequest and SyncConfigRequest.\r
@@ -3544,13 +3574,14 @@ SubmitForFormSet (
         // Call submit formset for system level, save the formset info\r
         // and process later.\r
         //\r
-        FindQuestionFromProgress(FormSet, Storage, Progress, &Form, &Question);\r
+        FindQuestionFromProgress (FormSet, Storage, Progress, &Form, &Question);\r
         ASSERT (Form != NULL && Question != NULL);\r
-        FormSet->SaveFailForm = Form;\r
+        FormSet->SaveFailForm      = Form;\r
         FormSet->SaveFailStatement = Question;\r
         if (SkipProcessFail) {\r
           InsertTailList (&gBrowserSaveFailFormSetList, &FormSet->SaveFailLink);\r
         }\r
+\r
         HasInserted = TRUE;\r
       }\r
 \r
@@ -3575,11 +3606,11 @@ SubmitForFormSet (
       //\r
       if (ConfirmSaveFail (Form->FormTitle, FormSet->HiiHandle) == BROWSER_ACTION_DISCARD) {\r
         DiscardChange = TRUE;\r
-        Link = GetFirstNode (&FormSet->SaveFailStorageListHead);\r
+        Link          = GetFirstNode (&FormSet->SaveFailStorageListHead);\r
         while (!IsNull (&FormSet->SaveFailStorageListHead, Link)) {\r
           FormSetStorage = FORMSET_STORAGE_FROM_SAVE_FAIL_LINK (Link);\r
           Storage        = FormSetStorage->BrowserStorage;\r
-          Link = GetNextNode (&FormSet->SaveFailStorageListHead, Link);\r
+          Link           = GetNextNode (&FormSet->SaveFailStorageListHead, Link);\r
           //\r
           // Process the submit fail question, base on the RestoreConfigRequest to restore the EditBuffer\r
           // base on the SyncConfigRequest to Sync the buffer.\r
@@ -3588,7 +3619,7 @@ SubmitForFormSet (
           FreePool (FormSetStorage->RestoreConfigRequest);\r
           FormSetStorage->RestoreConfigRequest = NULL;\r
           if (FormSetStorage->SyncConfigRequest != NULL) {\r
-            SynchronizeStorage(FormSetStorage->BrowserStorage, FormSetStorage->SyncConfigRequest, TRUE);\r
+            SynchronizeStorage (FormSetStorage->BrowserStorage, FormSetStorage->SyncConfigRequest, TRUE);\r
             FreePool (FormSetStorage->SyncConfigRequest);\r
             FormSetStorage->SyncConfigRequest = NULL;\r
           }\r
@@ -3596,12 +3627,12 @@ SubmitForFormSet (
           Status = EFI_SUCCESS;\r
         }\r
       } else {\r
-        UiCopyMenuList(&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &Form->FormViewListHead);\r
+        UiCopyMenuList (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &Form->FormViewListHead);\r
 \r
         gCurrentSelection->Action = UI_ACTION_REFRESH_FORMSET;\r
         gCurrentSelection->Handle = FormSet->HiiHandle;\r
         CopyGuid (&gCurrentSelection->FormSetGuid, &FormSet->Guid);\r
-        gCurrentSelection->FormId = Form->FormId;\r
+        gCurrentSelection->FormId     = Form->FormId;\r
         gCurrentSelection->QuestionId = Question->QuestionId;\r
 \r
         Status = EFI_UNSUPPORTED;\r
@@ -3611,7 +3642,7 @@ SubmitForFormSet (
       // Free FormSet save fail list.\r
       //\r
       while (!IsListEmpty (&FormSet->SaveFailStorageListHead)) {\r
-        Link = GetFirstNode (&FormSet->SaveFailStorageListHead);\r
+        Link           = GetFirstNode (&FormSet->SaveFailStorageListHead);\r
         FormSetStorage = FORMSET_STORAGE_FROM_SAVE_FAIL_LINK (Link);\r
         RemoveEntryList (&FormSetStorage->SaveFailLink);\r
       }\r
@@ -3641,7 +3672,7 @@ SubmitForFormSet (
   //\r
   // 5. Update the NV flag.\r
   //\r
-  ValueChangeResetFlagUpdate(TRUE, FormSet, NULL);\r
+  ValueChangeResetFlagUpdate (TRUE, FormSet, NULL);\r
 \r
   //\r
   // 6. Call callback with Submitted type to inform the driver.\r
@@ -3676,11 +3707,11 @@ SubmitForSystem (
   FORM_BROWSER_STATEMENT  *Question;\r
 \r
   mSystemSubmit = TRUE;\r
-  Link = GetFirstNode (&gBrowserFormSetList);\r
+  Link          = GetFirstNode (&gBrowserFormSetList);\r
   while (!IsNull (&gBrowserFormSetList, Link)) {\r
     LocalFormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-    Link = GetNextNode (&gBrowserFormSetList, Link);\r
-    if (!ValidateFormSet(LocalFormSet)) {\r
+    Link         = GetNextNode (&gBrowserFormSetList, Link);\r
+    if (!ValidateFormSet (LocalFormSet)) {\r
       continue;\r
     }\r
 \r
@@ -3693,11 +3724,12 @@ SubmitForSystem (
     // Remove maintain backup list after save except for the current using FormSet.\r
     //\r
     if (!IsHiiHandleInBrowserContext (LocalFormSet->HiiHandle)) {\r
-      CleanBrowserStorage(LocalFormSet);\r
+      CleanBrowserStorage (LocalFormSet);\r
       RemoveEntryList (&LocalFormSet->Link);\r
       DestroyFormSet (LocalFormSet);\r
     }\r
   }\r
+\r
   mSystemSubmit = FALSE;\r
 \r
   Status = EFI_SUCCESS;\r
@@ -3708,14 +3740,14 @@ SubmitForSystem (
   Link = GetFirstNode (&gBrowserSaveFailFormSetList);\r
   while (!IsNull (&gBrowserSaveFailFormSetList, Link)) {\r
     LocalFormSet = FORM_BROWSER_FORMSET_FROM_SAVE_FAIL_LINK (Link);\r
-    Link = GetNextNode (&gBrowserSaveFailFormSetList, Link);\r
+    Link         = GetNextNode (&gBrowserSaveFailFormSetList, Link);\r
 \r
-    if (!ValidateFormSet(LocalFormSet)) {\r
+    if (!ValidateFormSet (LocalFormSet)) {\r
       continue;\r
     }\r
 \r
-    Form = LocalFormSet->SaveFailForm;\r
-    Question= LocalFormSet->SaveFailStatement;\r
+    Form     = LocalFormSet->SaveFailForm;\r
+    Question = LocalFormSet->SaveFailStatement;\r
 \r
     //\r
     // Confirm with user, get user input.\r
@@ -3734,15 +3766,15 @@ SubmitForSystem (
         StorageLink = GetFirstNode (&LocalFormSet->StorageListHead);\r
         while (!IsNull (&LocalFormSet->StorageListHead, StorageLink)) {\r
           FormSetStorage = FORMSET_STORAGE_FROM_LINK (StorageLink);\r
-          StorageLink = GetNextNode (&LocalFormSet->StorageListHead, StorageLink);\r
+          StorageLink    = GetNextNode (&LocalFormSet->StorageListHead, StorageLink);\r
 \r
-          SynchronizeStorage(FormSetStorage->BrowserStorage, FormSetStorage->ConfigRequest, FALSE);\r
+          SynchronizeStorage (FormSetStorage->BrowserStorage, FormSetStorage->ConfigRequest, FALSE);\r
         }\r
       } else {\r
         StorageLink = GetFirstNode (&LocalFormSet->SaveFailStorageListHead);\r
         while (!IsNull (&LocalFormSet->SaveFailStorageListHead, StorageLink)) {\r
           FormSetStorage = FORMSET_STORAGE_FROM_SAVE_FAIL_LINK (StorageLink);\r
-          StorageLink = GetNextNode (&LocalFormSet->SaveFailStorageListHead, StorageLink);\r
+          StorageLink    = GetNextNode (&LocalFormSet->SaveFailStorageListHead, StorageLink);\r
           //\r
           // Process the submit fail question, base on the RestoreConfigRequest to restore the EditBuffer\r
           // base on the SyncConfigRequest to Sync the buffer.\r
@@ -3760,7 +3792,7 @@ SubmitForSystem (
 \r
       FormLink = GetFirstNode (&LocalFormSet->FormListHead);\r
       while (!IsNull (&LocalFormSet->FormListHead, FormLink)) {\r
-        Form = FORM_BROWSER_FORM_FROM_LINK (FormLink);\r
+        Form     = FORM_BROWSER_FORM_FROM_LINK (FormLink);\r
         FormLink = GetNextNode (&LocalFormSet->FormListHead, FormLink);\r
         //\r
         // Call callback with Changed type to inform the driver.\r
@@ -3769,24 +3801,24 @@ SubmitForSystem (
       }\r
 \r
       if (!IsHiiHandleInBrowserContext (LocalFormSet->HiiHandle)) {\r
-        CleanBrowserStorage(LocalFormSet);\r
+        CleanBrowserStorage (LocalFormSet);\r
         RemoveEntryList (&LocalFormSet->Link);\r
         RemoveEntryList (&LocalFormSet->SaveFailLink);\r
         DestroyFormSet (LocalFormSet);\r
       } else {\r
-        ValueChangeResetFlagUpdate(FALSE, LocalFormSet, NULL);\r
+        ValueChangeResetFlagUpdate (FALSE, LocalFormSet, NULL);\r
       }\r
     } else {\r
       if (IsListEmpty (&LocalFormSet->SaveFailStorageListHead)) {\r
         NoSubmitCheck (LocalFormSet, &Form, &Question);\r
       }\r
 \r
-      UiCopyMenuList(&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &Form->FormViewListHead);\r
+      UiCopyMenuList (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &Form->FormViewListHead);\r
 \r
       gCurrentSelection->Action = UI_ACTION_REFRESH_FORMSET;\r
       gCurrentSelection->Handle = LocalFormSet->HiiHandle;\r
       CopyGuid (&gCurrentSelection->FormSetGuid, &LocalFormSet->Guid);\r
-      gCurrentSelection->FormId = Form->FormId;\r
+      gCurrentSelection->FormId     = Form->FormId;\r
       gCurrentSelection->QuestionId = Question->QuestionId;\r
 \r
       Status = EFI_UNSUPPORTED;\r
@@ -3798,12 +3830,12 @@ SubmitForSystem (
   // Clean the list which will not process.\r
   //\r
   while (!IsListEmpty (&gBrowserSaveFailFormSetList)) {\r
-    Link = GetFirstNode (&gBrowserSaveFailFormSetList);\r
+    Link         = GetFirstNode (&gBrowserSaveFailFormSetList);\r
     LocalFormSet = FORM_BROWSER_FORMSET_FROM_SAVE_FAIL_LINK (Link);\r
     RemoveEntryList (&LocalFormSet->SaveFailLink);\r
 \r
     while (!IsListEmpty (&LocalFormSet->SaveFailStorageListHead)) {\r
-      StorageLink = GetFirstNode (&LocalFormSet->SaveFailStorageListHead);\r
+      StorageLink    = GetFirstNode (&LocalFormSet->SaveFailStorageListHead);\r
       FormSetStorage = FORMSET_STORAGE_FROM_SAVE_FAIL_LINK (StorageLink);\r
       RemoveEntryList (&FormSetStorage->SaveFailLink);\r
     }\r
@@ -3825,29 +3857,29 @@ SubmitForSystem (
 **/\r
 EFI_STATUS\r
 SubmitForm (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN BROWSER_SETTING_SCOPE            SettingScope\r
+  IN FORM_BROWSER_FORMSET   *FormSet,\r
+  IN FORM_BROWSER_FORM      *Form,\r
+  IN BROWSER_SETTING_SCOPE  SettingScope\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   switch (SettingScope) {\r
-  case FormLevel:\r
-    Status = SubmitForForm(FormSet, Form);\r
-    break;\r
+    case FormLevel:\r
+      Status = SubmitForForm (FormSet, Form);\r
+      break;\r
 \r
-  case FormSetLevel:\r
-    Status = SubmitForFormSet (FormSet, FALSE);\r
-    break;\r
+    case FormSetLevel:\r
+      Status = SubmitForFormSet (FormSet, FALSE);\r
+      break;\r
 \r
-  case SystemLevel:\r
-    Status = SubmitForSystem ();\r
-    break;\r
+    case SystemLevel:\r
+      Status = SubmitForSystem ();\r
+      break;\r
 \r
-  default:\r
-    Status = EFI_UNSUPPORTED;\r
-    break;\r
+    default:\r
+      Status = EFI_UNSUPPORTED;\r
+      break;\r
   }\r
 \r
   return Status;\r
@@ -3879,8 +3911,8 @@ HiiToLower (
       Lower = TRUE;\r
     } else if (*String == L'&') {\r
       Lower = FALSE;\r
-    } else if (Lower && *String >= L'A' && *String <= L'F') {\r
-      *String = (CHAR16) (*String - L'A' + L'a');\r
+    } else if (Lower && (*String >= L'A') && (*String <= L'F')) {\r
+      *String = (CHAR16)(*String - L'A' + L'a');\r
     }\r
   }\r
 }\r
@@ -3896,12 +3928,12 @@ HiiToLower (
 **/\r
 CHAR16 *\r
 GetOffsetFromConfigResp (\r
-  IN FORM_BROWSER_STATEMENT           *Question,\r
-  IN CHAR16                           *ConfigResp\r
+  IN FORM_BROWSER_STATEMENT  *Question,\r
+  IN CHAR16                  *ConfigResp\r
   )\r
 {\r
-  CHAR16                       *RequestElement;\r
-  CHAR16                       *BlockData;\r
+  CHAR16  *RequestElement;\r
+  CHAR16  *BlockData;\r
 \r
   //\r
   // Type is EFI_HII_VARSTORE_NAME_VALUE.\r
@@ -3942,7 +3974,7 @@ GetOffsetFromConfigResp (
   //\r
   // 2. Change all hex digits in Question->BlockName to lower and compare again.\r
   //\r
-  BlockData = AllocateCopyPool (StrSize(Question->BlockName), Question->BlockName);\r
+  BlockData = AllocateCopyPool (StrSize (Question->BlockName), Question->BlockName);\r
   ASSERT (BlockData != NULL);\r
   HiiToLower (BlockData);\r
   RequestElement = StrStr (ConfigResp, BlockData);\r
@@ -3970,9 +4002,9 @@ GetOffsetFromConfigResp (
 **/\r
 EFI_STATUS\r
 GetDefaultValueFromAltCfg (\r
-  IN     FORM_BROWSER_FORMSET             *FormSet,\r
-  IN     FORM_BROWSER_FORM                *Form,\r
-  IN OUT FORM_BROWSER_STATEMENT           *Question\r
+  IN     FORM_BROWSER_FORMSET    *FormSet,\r
+  IN     FORM_BROWSER_FORM       *Form,\r
+  IN OUT FORM_BROWSER_STATEMENT  *Question\r
   )\r
 {\r
   BROWSER_STORAGE              *Storage;\r
@@ -3991,11 +4023,11 @@ GetDefaultValueFromAltCfg (
   // Try to get AltCfg string from form. If not found it, then\r
   // try to get it from formset.\r
   //\r
-  ConfigResp    = NULL;\r
-  Link = GetFirstNode (&Form->ConfigRequestHead);\r
+  ConfigResp = NULL;\r
+  Link       = GetFirstNode (&Form->ConfigRequestHead);\r
   while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
     ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->ConfigRequestHead, Link);\r
+    Link       = GetNextNode (&Form->ConfigRequestHead, Link);\r
 \r
     if (Storage == ConfigInfo->Storage) {\r
       ConfigResp = ConfigInfo->ConfigAltResp;\r
@@ -4007,7 +4039,7 @@ GetDefaultValueFromAltCfg (
     Link = GetFirstNode (&FormSet->StorageListHead);\r
     while (!IsNull (&FormSet->StorageListHead, Link)) {\r
       FormSetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
-      Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+      Link           = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
       if (Storage == FormSetStorage->BrowserStorage) {\r
         ConfigResp = FormSetStorage->ConfigAltResp;\r
@@ -4038,7 +4070,7 @@ GetDefaultValueFromAltCfg (
 **/\r
 INTN\r
 GetDefaultIdForCallBack (\r
-  UINTN DefaultId\r
+  UINTN  DefaultId\r
   )\r
 {\r
   if (DefaultId == EFI_HII_DEFAULT_CLASS_STANDARD) {\r
@@ -4047,19 +4079,17 @@ GetDefaultIdForCallBack (
     return EFI_BROWSER_ACTION_DEFAULT_MANUFACTURING;\r
   } else if (DefaultId == EFI_HII_DEFAULT_CLASS_SAFE) {\r
     return EFI_BROWSER_ACTION_DEFAULT_SAFE;\r
-  } else if (DefaultId >= EFI_HII_DEFAULT_CLASS_PLATFORM_BEGIN && DefaultId < EFI_HII_DEFAULT_CLASS_PLATFORM_BEGIN + 0x1000) {\r
+  } else if ((DefaultId >= EFI_HII_DEFAULT_CLASS_PLATFORM_BEGIN) && (DefaultId < EFI_HII_DEFAULT_CLASS_PLATFORM_BEGIN + 0x1000)) {\r
     return EFI_BROWSER_ACTION_DEFAULT_PLATFORM + DefaultId - EFI_HII_DEFAULT_CLASS_PLATFORM_BEGIN;\r
-  } else if (DefaultId >= EFI_HII_DEFAULT_CLASS_HARDWARE_BEGIN && DefaultId < EFI_HII_DEFAULT_CLASS_HARDWARE_BEGIN + 0x1000) {\r
+  } else if ((DefaultId >= EFI_HII_DEFAULT_CLASS_HARDWARE_BEGIN) && (DefaultId < EFI_HII_DEFAULT_CLASS_HARDWARE_BEGIN + 0x1000)) {\r
     return EFI_BROWSER_ACTION_DEFAULT_HARDWARE + DefaultId - EFI_HII_DEFAULT_CLASS_HARDWARE_BEGIN;\r
-  } else if (DefaultId >= EFI_HII_DEFAULT_CLASS_FIRMWARE_BEGIN && DefaultId < EFI_HII_DEFAULT_CLASS_FIRMWARE_BEGIN + 0x1000) {\r
+  } else if ((DefaultId >= EFI_HII_DEFAULT_CLASS_FIRMWARE_BEGIN) && (DefaultId < EFI_HII_DEFAULT_CLASS_FIRMWARE_BEGIN + 0x1000)) {\r
     return EFI_BROWSER_ACTION_DEFAULT_FIRMWARE + DefaultId - EFI_HII_DEFAULT_CLASS_FIRMWARE_BEGIN;\r
   } else {\r
     return -1;\r
   }\r
 }\r
 \r
-\r
-\r
 /**\r
   Return data element in an Array by its Index.\r
 \r
@@ -4072,41 +4102,40 @@ GetDefaultIdForCallBack (
 **/\r
 UINT64\r
 GetArrayData (\r
-  IN VOID                     *Array,\r
-  IN UINT8                    Type,\r
-  IN UINTN                    Index\r
+  IN VOID   *Array,\r
+  IN UINT8  Type,\r
+  IN UINTN  Index\r
   )\r
 {\r
-  UINT64 Data;\r
+  UINT64  Data;\r
 \r
   ASSERT (Array != NULL);\r
 \r
   Data = 0;\r
   switch (Type) {\r
-  case EFI_IFR_TYPE_NUM_SIZE_8:\r
-    Data = (UINT64) *(((UINT8 *) Array) + Index);\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_8:\r
+      Data = (UINT64)*(((UINT8 *)Array) + Index);\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_16:\r
-    Data = (UINT64) *(((UINT16 *) Array) + Index);\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_16:\r
+      Data = (UINT64)*(((UINT16 *)Array) + Index);\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_32:\r
-    Data = (UINT64) *(((UINT32 *) Array) + Index);\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_32:\r
+      Data = (UINT64)*(((UINT32 *)Array) + Index);\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_64:\r
-    Data = (UINT64) *(((UINT64 *) Array) + Index);\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_64:\r
+      Data = (UINT64)*(((UINT64 *)Array) + Index);\r
+      break;\r
 \r
-  default:\r
-    break;\r
+    default:\r
+      break;\r
   }\r
 \r
   return Data;\r
 }\r
 \r
-\r
 /**\r
   Set value of a data element in an Array by its Index.\r
 \r
@@ -4118,34 +4147,33 @@ GetArrayData (
 **/\r
 VOID\r
 SetArrayData (\r
-  IN VOID                     *Array,\r
-  IN UINT8                    Type,\r
-  IN UINTN                    Index,\r
-  IN UINT64                   Value\r
+  IN VOID    *Array,\r
+  IN UINT8   Type,\r
+  IN UINTN   Index,\r
+  IN UINT64  Value\r
   )\r
 {\r
-\r
   ASSERT (Array != NULL);\r
 \r
   switch (Type) {\r
-  case EFI_IFR_TYPE_NUM_SIZE_8:\r
-    *(((UINT8 *) Array) + Index) = (UINT8) Value;\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_8:\r
+      *(((UINT8 *)Array) + Index) = (UINT8)Value;\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_16:\r
-    *(((UINT16 *) Array) + Index) = (UINT16) Value;\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_16:\r
+      *(((UINT16 *)Array) + Index) = (UINT16)Value;\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_32:\r
-    *(((UINT32 *) Array) + Index) = (UINT32) Value;\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_32:\r
+      *(((UINT32 *)Array) + Index) = (UINT32)Value;\r
+      break;\r
 \r
-  case EFI_IFR_TYPE_NUM_SIZE_64:\r
-    *(((UINT64 *) Array) + Index) = (UINT64) Value;\r
-    break;\r
+    case EFI_IFR_TYPE_NUM_SIZE_64:\r
+      *(((UINT64 *)Array) + Index) = (UINT64)Value;\r
+      break;\r
 \r
-  default:\r
-    break;\r
+    default:\r
+      break;\r
   }\r
 }\r
 \r
@@ -4161,8 +4189,8 @@ SetArrayData (
 **/\r
 QUESTION_OPTION *\r
 ValueToOption (\r
-  IN FORM_BROWSER_STATEMENT   *Question,\r
-  IN EFI_HII_VALUE            *OptionValue\r
+  IN FORM_BROWSER_STATEMENT  *Question,\r
+  IN EFI_HII_VALUE           *OptionValue\r
   )\r
 {\r
   LIST_ENTRY       *Link;\r
@@ -4178,7 +4206,8 @@ ValueToOption (
       // Check the suppressif condition, only a valid option can be return.\r
       //\r
       if ((Option->SuppressExpression == NULL) ||\r
-          ((EvaluateExpressionList(Option->SuppressExpression, FALSE, NULL, NULL) == ExpressFalse))) {\r
+          ((EvaluateExpressionList (Option->SuppressExpression, FALSE, NULL, NULL) == ExpressFalse)))\r
+      {\r
         return Option;\r
       }\r
     }\r
@@ -4189,7 +4218,6 @@ ValueToOption (
   return NULL;\r
 }\r
 \r
-\r
 /**\r
   Reset Question to its default value.\r
 \r
@@ -4203,19 +4231,19 @@ ValueToOption (
 **/\r
 EFI_STATUS\r
 GetQuestionDefault (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN FORM_BROWSER_STATEMENT           *Question,\r
-  IN UINT16                           DefaultId\r
+  IN FORM_BROWSER_FORMSET    *FormSet,\r
+  IN FORM_BROWSER_FORM       *Form,\r
+  IN FORM_BROWSER_STATEMENT  *Question,\r
+  IN UINT16                  DefaultId\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  LIST_ENTRY              *Link;\r
-  QUESTION_DEFAULT        *Default;\r
-  QUESTION_OPTION         *Option;\r
-  EFI_HII_VALUE           *HiiValue;\r
-  UINT8                   Index;\r
-  EFI_STRING              StrValue;\r
+  EFI_STATUS                      Status;\r
+  LIST_ENTRY                      *Link;\r
+  QUESTION_DEFAULT                *Default;\r
+  QUESTION_OPTION                 *Option;\r
+  EFI_HII_VALUE                   *HiiValue;\r
+  UINT8                           Index;\r
+  EFI_STRING                      StrValue;\r
   EFI_HII_CONFIG_ACCESS_PROTOCOL  *ConfigAccess;\r
   EFI_BROWSER_ACTION_REQUEST      ActionRequest;\r
   INTN                            Action;\r
@@ -4225,10 +4253,10 @@ GetQuestionDefault (
   FORMSET_DEFAULTSTORE            *DefaultStore;\r
   LIST_ENTRY                      *DefaultLink;\r
 \r
-  Status   = EFI_NOT_FOUND;\r
-  StrValue = NULL;\r
-  OriginalDefaultId  = DefaultId;\r
-  DefaultLink        = GetFirstNode (&FormSet->DefaultStoreListHead);\r
+  Status            = EFI_NOT_FOUND;\r
+  StrValue          = NULL;\r
+  OriginalDefaultId = DefaultId;\r
+  DefaultLink       = GetFirstNode (&FormSet->DefaultStoreListHead);\r
 \r
   //\r
   // Statement don't have storage, skip them\r
@@ -4246,30 +4274,30 @@ GetQuestionDefault (
   //  5, set flags of EFI_IFR_CHECKBOX (provide Standard and Manufacturing default) (lowest priority)\r
   //\r
 ReGetDefault:\r
-  HiiValue = &Question->HiiValue;\r
+  HiiValue  = &Question->HiiValue;\r
   TypeValue = &HiiValue->Value;\r
   if (HiiValue->Type == EFI_IFR_TYPE_BUFFER) {\r
     //\r
     // For orderedlist, need to pass the BufferValue to Callback function.\r
     //\r
-    TypeValue = (EFI_IFR_TYPE_VALUE *) Question->BufferValue;\r
+    TypeValue = (EFI_IFR_TYPE_VALUE *)Question->BufferValue;\r
   }\r
 \r
   //\r
   // Get Question defaut value from call back function.\r
   //\r
   ConfigAccess = FormSet->ConfigAccess;\r
-  Action = GetDefaultIdForCallBack (DefaultId);\r
+  Action       = GetDefaultIdForCallBack (DefaultId);\r
   if ((Action > 0) && ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) != 0) && (ConfigAccess != NULL)) {\r
     ActionRequest = EFI_BROWSER_ACTION_REQUEST_NONE;\r
-    Status = ConfigAccess->Callback (\r
-                             ConfigAccess,\r
-                             Action,\r
-                             Question->QuestionId,\r
-                             HiiValue->Type,\r
-                             TypeValue,\r
-                             &ActionRequest\r
-                             );\r
+    Status        = ConfigAccess->Callback (\r
+                                    ConfigAccess,\r
+                                    Action,\r
+                                    Question->QuestionId,\r
+                                    HiiValue->Type,\r
+                                    TypeValue,\r
+                                    &ActionRequest\r
+                                    );\r
     if (!EFI_ERROR (Status)) {\r
       if (HiiValue->Type == EFI_IFR_TYPE_STRING) {\r
         NewString = GetToken (Question->HiiValue.Value.string, FormSet->HiiHandle);\r
@@ -4285,6 +4313,7 @@ ReGetDefault:
 \r
         FreePool (NewString);\r
       }\r
+\r
       return Status;\r
     }\r
   }\r
@@ -4293,9 +4322,9 @@ ReGetDefault:
   // Get default value from altcfg string.\r
   //\r
   if (ConfigAccess != NULL) {\r
-    Status = GetDefaultValueFromAltCfg(FormSet, Form, Question);\r
+    Status = GetDefaultValueFromAltCfg (FormSet, Form, Question);\r
     if (!EFI_ERROR (Status)) {\r
-        return Status;\r
+      return Status;\r
     }\r
   }\r
 \r
@@ -4326,8 +4355,10 @@ ReGetDefault:
               CopyMem (Question->HiiValue.Buffer, Default->ValueExpression->Result.Buffer, Question->StorageWidth);\r
               Question->HiiValue.BufferLen = Question->StorageWidth;\r
             }\r
+\r
             FreePool (Default->ValueExpression->Result.Buffer);\r
           }\r
+\r
           HiiValue->Type = Default->ValueExpression->Result.Type;\r
           CopyMem (&HiiValue->Value, &Default->ValueExpression->Result.Value, sizeof (EFI_IFR_TYPE_VALUE));\r
         } else {\r
@@ -4347,6 +4378,7 @@ ReGetDefault:
           if (StrValue == NULL) {\r
             return EFI_NOT_FOUND;\r
           }\r
+\r
           if (Question->StorageWidth > StrSize (StrValue)) {\r
             ZeroMem (Question->BufferValue, Question->StorageWidth);\r
             CopyMem (Question->BufferValue, StrValue, StrSize (StrValue));\r
@@ -4366,23 +4398,25 @@ ReGetDefault:
   // EFI_ONE_OF_OPTION\r
   //\r
   if ((Question->Operand == EFI_IFR_ONE_OF_OP) && !IsListEmpty (&Question->OptionListHead)) {\r
-    if (DefaultId <= EFI_HII_DEFAULT_CLASS_MANUFACTURING)  {\r
+    if (DefaultId <= EFI_HII_DEFAULT_CLASS_MANUFACTURING) {\r
       //\r
       // OneOfOption could only provide Standard and Manufacturing default\r
       //\r
       Link = GetFirstNode (&Question->OptionListHead);\r
       while (!IsNull (&Question->OptionListHead, Link)) {\r
         Option = QUESTION_OPTION_FROM_LINK (Link);\r
-        Link = GetNextNode (&Question->OptionListHead, Link);\r
+        Link   = GetNextNode (&Question->OptionListHead, Link);\r
 \r
         if ((Option->SuppressExpression != NULL) &&\r
-            EvaluateExpressionList(Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse) {\r
+            (EvaluateExpressionList (Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse))\r
+        {\r
           continue;\r
         }\r
 \r
         if (((DefaultId == EFI_HII_DEFAULT_CLASS_STANDARD) && ((Option->Flags & EFI_IFR_OPTION_DEFAULT) != 0)) ||\r
             ((DefaultId == EFI_HII_DEFAULT_CLASS_MANUFACTURING) && ((Option->Flags & EFI_IFR_OPTION_DEFAULT_MFG) != 0))\r
-           ) {\r
+            )\r
+        {\r
           CopyMem (HiiValue, &Option->Value, sizeof (EFI_HII_VALUE));\r
 \r
           return EFI_SUCCESS;\r
@@ -4395,13 +4429,14 @@ ReGetDefault:
   // EFI_IFR_CHECKBOX - lowest priority\r
   //\r
   if (Question->Operand == EFI_IFR_CHECKBOX_OP) {\r
-    if (DefaultId <= EFI_HII_DEFAULT_CLASS_MANUFACTURING)  {\r
+    if (DefaultId <= EFI_HII_DEFAULT_CLASS_MANUFACTURING) {\r
       //\r
       // Checkbox could only provide Standard and Manufacturing default\r
       //\r
       if (((DefaultId == EFI_HII_DEFAULT_CLASS_STANDARD) && ((Question->Flags & EFI_IFR_CHECKBOX_DEFAULT) != 0)) ||\r
           ((DefaultId == EFI_HII_DEFAULT_CLASS_MANUFACTURING) && ((Question->Flags & EFI_IFR_CHECKBOX_DEFAULT_MFG) != 0))\r
-         ) {\r
+          )\r
+      {\r
         HiiValue->Value.b = TRUE;\r
       }\r
 \r
@@ -4414,13 +4449,14 @@ ReGetDefault:
   // If get, will exit the function, if not, will choose next default id in the DefaultStoreList.\r
   // The default id in DefaultStoreList are in ascending order to make sure choose the smallest default id every time.\r
   //\r
-  while (!IsNull(&FormSet->DefaultStoreListHead, DefaultLink)) {\r
-    DefaultStore = FORMSET_DEFAULTSTORE_FROM_LINK(DefaultLink);\r
-    DefaultLink = GetNextNode (&FormSet->DefaultStoreListHead,DefaultLink);\r
-    DefaultId = DefaultStore->DefaultId;\r
+  while (!IsNull (&FormSet->DefaultStoreListHead, DefaultLink)) {\r
+    DefaultStore = FORMSET_DEFAULTSTORE_FROM_LINK (DefaultLink);\r
+    DefaultLink  = GetNextNode (&FormSet->DefaultStoreListHead, DefaultLink);\r
+    DefaultId    = DefaultStore->DefaultId;\r
     if (DefaultId == OriginalDefaultId) {\r
       continue;\r
     }\r
+\r
     goto ReGetDefault;\r
   }\r
 \r
@@ -4429,104 +4465,113 @@ ReGetDefault:
   //\r
   Status = EFI_NOT_FOUND;\r
   switch (Question->Operand) {\r
-  case EFI_IFR_CHECKBOX_OP:\r
-    HiiValue->Value.b = FALSE;\r
-    Status = EFI_SUCCESS;\r
-    break;\r
+    case EFI_IFR_CHECKBOX_OP:\r
+      HiiValue->Value.b = FALSE;\r
+      Status            = EFI_SUCCESS;\r
+      break;\r
 \r
-  case EFI_IFR_NUMERIC_OP:\r
-    //\r
-    // Take minimum value as numeric default value\r
-    //\r
-    if ((Question->Flags & EFI_IFR_DISPLAY) == 0) {\r
+    case EFI_IFR_NUMERIC_OP:\r
       //\r
-      // In EFI_IFR_DISPLAY_INT_DEC type, should check value with int* type.\r
+      // Take minimum value as numeric default value\r
       //\r
-      switch (Question->Flags & EFI_IFR_NUMERIC_SIZE) {\r
-      case EFI_IFR_NUMERIC_SIZE_1:\r
-        if (((INT8) HiiValue->Value.u8 < (INT8) Question->Minimum) || ((INT8) HiiValue->Value.u8 > (INT8) Question->Maximum)) {\r
-          HiiValue->Value.u8 = (UINT8) Question->Minimum;\r
-          Status = EFI_SUCCESS;\r
-        }\r
-        break;\r
-      case EFI_IFR_NUMERIC_SIZE_2:\r
-        if (((INT16) HiiValue->Value.u16 < (INT16) Question->Minimum) || ((INT16) HiiValue->Value.u16 > (INT16) Question->Maximum)) {\r
-          HiiValue->Value.u16 = (UINT16) Question->Minimum;\r
-          Status = EFI_SUCCESS;\r
-        }\r
-        break;\r
-      case EFI_IFR_NUMERIC_SIZE_4:\r
-        if (((INT32) HiiValue->Value.u32 < (INT32) Question->Minimum) || ((INT32) HiiValue->Value.u32 > (INT32) Question->Maximum)) {\r
-          HiiValue->Value.u32 = (UINT32) Question->Minimum;\r
-          Status = EFI_SUCCESS;\r
+      if ((Question->Flags & EFI_IFR_DISPLAY) == 0) {\r
+        //\r
+        // In EFI_IFR_DISPLAY_INT_DEC type, should check value with int* type.\r
+        //\r
+        switch (Question->Flags & EFI_IFR_NUMERIC_SIZE) {\r
+          case EFI_IFR_NUMERIC_SIZE_1:\r
+            if (((INT8)HiiValue->Value.u8 < (INT8)Question->Minimum) || ((INT8)HiiValue->Value.u8 > (INT8)Question->Maximum)) {\r
+              HiiValue->Value.u8 = (UINT8)Question->Minimum;\r
+              Status             = EFI_SUCCESS;\r
+            }\r
+\r
+            break;\r
+          case EFI_IFR_NUMERIC_SIZE_2:\r
+            if (((INT16)HiiValue->Value.u16 < (INT16)Question->Minimum) || ((INT16)HiiValue->Value.u16 > (INT16)Question->Maximum)) {\r
+              HiiValue->Value.u16 = (UINT16)Question->Minimum;\r
+              Status              = EFI_SUCCESS;\r
+            }\r
+\r
+            break;\r
+          case EFI_IFR_NUMERIC_SIZE_4:\r
+            if (((INT32)HiiValue->Value.u32 < (INT32)Question->Minimum) || ((INT32)HiiValue->Value.u32 > (INT32)Question->Maximum)) {\r
+              HiiValue->Value.u32 = (UINT32)Question->Minimum;\r
+              Status              = EFI_SUCCESS;\r
+            }\r
+\r
+            break;\r
+          case EFI_IFR_NUMERIC_SIZE_8:\r
+            if (((INT64)HiiValue->Value.u64 < (INT64)Question->Minimum) || ((INT64)HiiValue->Value.u64 > (INT64)Question->Maximum)) {\r
+              HiiValue->Value.u64 = Question->Minimum;\r
+              Status              = EFI_SUCCESS;\r
+            }\r
+\r
+            break;\r
+          default:\r
+            break;\r
         }\r
-        break;\r
-      case EFI_IFR_NUMERIC_SIZE_8:\r
-        if (((INT64) HiiValue->Value.u64 < (INT64) Question->Minimum) || ((INT64) HiiValue->Value.u64 > (INT64) Question->Maximum)) {\r
+      } else {\r
+        if ((HiiValue->Value.u64 < Question->Minimum) || (HiiValue->Value.u64 > Question->Maximum)) {\r
           HiiValue->Value.u64 = Question->Minimum;\r
-          Status = EFI_SUCCESS;\r
+          Status              = EFI_SUCCESS;\r
         }\r
-        break;\r
-      default:\r
-        break;\r
       }\r
-    } else {\r
-      if ((HiiValue->Value.u64 < Question->Minimum) || (HiiValue->Value.u64 > Question->Maximum)) {\r
-        HiiValue->Value.u64 = Question->Minimum;\r
-        Status = EFI_SUCCESS;\r
+\r
+      break;\r
+\r
+    case EFI_IFR_ONE_OF_OP:\r
+      //\r
+      // Take first oneof option as oneof's default value\r
+      //\r
+      if (ValueToOption (Question, HiiValue) == NULL) {\r
+        Link = GetFirstNode (&Question->OptionListHead);\r
+        while (!IsNull (&Question->OptionListHead, Link)) {\r
+          Option = QUESTION_OPTION_FROM_LINK (Link);\r
+          Link   = GetNextNode (&Question->OptionListHead, Link);\r
+\r
+          if ((Option->SuppressExpression != NULL) &&\r
+              (EvaluateExpressionList (Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse))\r
+          {\r
+            continue;\r
+          }\r
+\r
+          CopyMem (HiiValue, &Option->Value, sizeof (EFI_HII_VALUE));\r
+          Status = EFI_SUCCESS;\r
+          break;\r
+        }\r
       }\r
-    }\r
-    break;\r
 \r
-  case EFI_IFR_ONE_OF_OP:\r
-    //\r
-    // Take first oneof option as oneof's default value\r
-    //\r
-    if (ValueToOption (Question, HiiValue) == NULL) {\r
-      Link = GetFirstNode (&Question->OptionListHead);\r
+      break;\r
+\r
+    case EFI_IFR_ORDERED_LIST_OP:\r
+      //\r
+      // Take option sequence in IFR as ordered list's default value\r
+      //\r
+      Index = 0;\r
+      Link  = GetFirstNode (&Question->OptionListHead);\r
       while (!IsNull (&Question->OptionListHead, Link)) {\r
+        Status = EFI_SUCCESS;\r
         Option = QUESTION_OPTION_FROM_LINK (Link);\r
-        Link = GetNextNode (&Question->OptionListHead, Link);\r
+        Link   = GetNextNode (&Question->OptionListHead, Link);\r
 \r
         if ((Option->SuppressExpression != NULL) &&\r
-            EvaluateExpressionList(Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse) {\r
+            (EvaluateExpressionList (Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse))\r
+        {\r
           continue;\r
         }\r
 \r
-        CopyMem (HiiValue, &Option->Value, sizeof (EFI_HII_VALUE));\r
-        Status = EFI_SUCCESS;\r
-        break;\r
-      }\r
-    }\r
-    break;\r
+        SetArrayData (Question->BufferValue, Question->ValueType, Index, Option->Value.Value.u64);\r
 \r
-  case EFI_IFR_ORDERED_LIST_OP:\r
-    //\r
-    // Take option sequence in IFR as ordered list's default value\r
-    //\r
-    Index = 0;\r
-    Link = GetFirstNode (&Question->OptionListHead);\r
-    while (!IsNull (&Question->OptionListHead, Link)) {\r
-      Status = EFI_SUCCESS;\r
-      Option = QUESTION_OPTION_FROM_LINK (Link);\r
-      Link = GetNextNode (&Question->OptionListHead, Link);\r
-\r
-      if ((Option->SuppressExpression != NULL) &&\r
-          EvaluateExpressionList(Option->SuppressExpression, FALSE, NULL, NULL) != ExpressFalse) {\r
-        continue;\r
+        Index++;\r
+        if (Index >= Question->MaxContainers) {\r
+          break;\r
+        }\r
       }\r
 \r
-      SetArrayData (Question->BufferValue, Question->ValueType, Index, Option->Value.Value.u64);\r
-\r
-      Index++;\r
-      if (Index >= Question->MaxContainers) {\r
-        break;\r
-      }\r
-    }\r
-    break;\r
+      break;\r
 \r
-  default:\r
-    break;\r
+    default:\r
+      break;\r
   }\r
 \r
   return Status;\r
@@ -4543,10 +4588,10 @@ ReGetDefault:
 **/\r
 VOID\r
 ExtractAltCfgForForm (\r
-  IN FORM_BROWSER_FORMSET   *FormSet,\r
-  IN FORM_BROWSER_FORM      *Form,\r
-  IN UINT16                 DefaultId,\r
-  IN BROWSER_STORAGE        *BrowserStorage\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORM_BROWSER_FORM     *Form,\r
+  IN UINT16                DefaultId,\r
+  IN BROWSER_STORAGE       *BrowserStorage\r
   )\r
 {\r
   EFI_STATUS                   Status;\r
@@ -4566,14 +4611,15 @@ ExtractAltCfgForForm (
   while (!IsNull (&FormSet->StorageListHead, Link)) {\r
     FormSetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
     Storage        = FormSetStorage->BrowserStorage;\r
-    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
-    if (BrowserStorage != NULL && BrowserStorage != Storage) {\r
+    Link           = GetNextNode (&FormSet->StorageListHead, Link);\r
+    if ((BrowserStorage != NULL) && (BrowserStorage != Storage)) {\r
       continue;\r
     }\r
 \r
-    if (Storage->Type != EFI_HII_VARSTORE_EFI_VARIABLE &&\r
-        FormSetStorage->ElementCount != 0 &&\r
-        FormSetStorage->HasCallAltCfg) {\r
+    if ((Storage->Type != EFI_HII_VARSTORE_EFI_VARIABLE) &&\r
+        (FormSetStorage->ElementCount != 0) &&\r
+        FormSetStorage->HasCallAltCfg)\r
+    {\r
       return;\r
     }\r
   }\r
@@ -4584,10 +4630,10 @@ ExtractAltCfgForForm (
   Link = GetFirstNode (&Form->ConfigRequestHead);\r
   while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
     ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->ConfigRequestHead, Link);\r
+    Link       = GetNextNode (&Form->ConfigRequestHead, Link);\r
 \r
     Storage = ConfigInfo->Storage;\r
-    if (BrowserStorage != NULL && BrowserStorage != Storage) {\r
+    if ((BrowserStorage != NULL) && (BrowserStorage != Storage)) {\r
       continue;\r
     }\r
 \r
@@ -4606,11 +4652,11 @@ ExtractAltCfgForForm (
     // 2. Get value through hii config routine protocol.\r
     //\r
     Status = mHiiConfigRouting->ExtractConfig (\r
-                                      mHiiConfigRouting,\r
-                                      ConfigInfo->ConfigRequest,\r
-                                      &Progress,\r
-                                      &Result\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  ConfigInfo->ConfigRequest,\r
+                                  &Progress,\r
+                                  &Result\r
+                                  );\r
     if (EFI_ERROR (Status)) {\r
       continue;\r
     }\r
@@ -4627,7 +4673,7 @@ ExtractAltCfgForForm (
                                   NULL,\r
                                   &DefaultId,  // it can be NULL to get the current setting.\r
                                   &ConfigResp\r
-                                );\r
+                                  );\r
     FreePool (Result);\r
     if (EFI_ERROR (Status)) {\r
       continue;\r
@@ -4645,16 +4691,16 @@ ExtractAltCfgForForm (
 **/\r
 VOID\r
 CleanAltCfgForForm (\r
-  IN FORM_BROWSER_FORM   *Form\r
+  IN FORM_BROWSER_FORM  *Form\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
+  LIST_ENTRY                   *Link;\r
   FORM_BROWSER_CONFIG_REQUEST  *ConfigInfo;\r
 \r
   Link = GetFirstNode (&Form->ConfigRequestHead);\r
   while (!IsNull (&Form->ConfigRequestHead, Link)) {\r
     ConfigInfo = FORM_BROWSER_CONFIG_REQUEST_FROM_LINK (Link);\r
-    Link = GetNextNode (&Form->ConfigRequestHead, Link);\r
+    Link       = GetNextNode (&Form->ConfigRequestHead, Link);\r
 \r
     if (ConfigInfo->ConfigAltResp != NULL) {\r
       FreePool (ConfigInfo->ConfigAltResp);\r
@@ -4673,26 +4719,26 @@ CleanAltCfgForForm (
 **/\r
 VOID\r
 ExtractAltCfgForFormSet (\r
-  IN FORM_BROWSER_FORMSET   *FormSet,\r
-  IN UINT16                 DefaultId,\r
-  IN BROWSER_STORAGE        *BrowserStorage\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN UINT16                DefaultId,\r
+  IN BROWSER_STORAGE       *BrowserStorage\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  LIST_ENTRY              *Link;\r
-  CHAR16                  *ConfigResp;\r
-  CHAR16                  *Progress;\r
-  CHAR16                  *Result;\r
-  BROWSER_STORAGE         *Storage;\r
-  FORMSET_STORAGE         *FormSetStorage;\r
+  EFI_STATUS       Status;\r
+  LIST_ENTRY       *Link;\r
+  CHAR16           *ConfigResp;\r
+  CHAR16           *Progress;\r
+  CHAR16           *Result;\r
+  BROWSER_STORAGE  *Storage;\r
+  FORMSET_STORAGE  *FormSetStorage;\r
 \r
   Link = GetFirstNode (&FormSet->StorageListHead);\r
   while (!IsNull (&FormSet->StorageListHead, Link)) {\r
     FormSetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
     Storage        = FormSetStorage->BrowserStorage;\r
-    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+    Link           = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
-    if (BrowserStorage != NULL && BrowserStorage != Storage) {\r
+    if ((BrowserStorage != NULL) && (BrowserStorage != Storage)) {\r
       continue;\r
     }\r
 \r
@@ -4713,11 +4759,11 @@ ExtractAltCfgForFormSet (
     // 2. Get value through hii config routine protocol.\r
     //\r
     Status = mHiiConfigRouting->ExtractConfig (\r
-                                      mHiiConfigRouting,\r
-                                      FormSetStorage->ConfigRequest,\r
-                                      &Progress,\r
-                                      &Result\r
-                                      );\r
+                                  mHiiConfigRouting,\r
+                                  FormSetStorage->ConfigRequest,\r
+                                  &Progress,\r
+                                  &Result\r
+                                  );\r
     if (EFI_ERROR (Status)) {\r
       continue;\r
     }\r
@@ -4734,7 +4780,7 @@ ExtractAltCfgForFormSet (
                                   NULL,\r
                                   &DefaultId,  // it can be NULL to get the current setting.\r
                                   &ConfigResp\r
-                                );\r
+                                  );\r
 \r
     FreePool (Result);\r
     if (EFI_ERROR (Status)) {\r
@@ -4743,7 +4789,6 @@ ExtractAltCfgForFormSet (
 \r
     FormSetStorage->ConfigAltResp = ConfigResp;\r
   }\r
-\r
 }\r
 \r
 /**\r
@@ -4754,16 +4799,16 @@ ExtractAltCfgForFormSet (
 **/\r
 VOID\r
 CleanAltCfgForFormSet (\r
-  IN FORM_BROWSER_FORMSET   *FormSet\r
+  IN FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  FORMSET_STORAGE         *FormSetStorage;\r
+  LIST_ENTRY       *Link;\r
+  FORMSET_STORAGE  *FormSetStorage;\r
 \r
   Link = GetFirstNode (&FormSet->StorageListHead);\r
   while (!IsNull (&FormSet->StorageListHead, Link)) {\r
     FormSetStorage = FORMSET_STORAGE_FROM_LINK (Link);\r
-    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+    Link           = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
     if (FormSetStorage->ConfigAltResp != NULL) {\r
       FreePool (FormSetStorage->ConfigAltResp);\r
@@ -4797,14 +4842,14 @@ CleanAltCfgForFormSet (
 **/\r
 EFI_STATUS\r
 ExtractDefault (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN UINT16                           DefaultId,\r
-  IN BROWSER_SETTING_SCOPE            SettingScope,\r
-  IN BROWSER_GET_DEFAULT_VALUE        GetDefaultValueScope,\r
-  IN BROWSER_STORAGE                  *Storage OPTIONAL,\r
-  IN BOOLEAN                          RetrieveValueFirst,\r
-  IN BOOLEAN                          SkipGetAltCfg\r
+  IN FORM_BROWSER_FORMSET       *FormSet,\r
+  IN FORM_BROWSER_FORM          *Form,\r
+  IN UINT16                     DefaultId,\r
+  IN BROWSER_SETTING_SCOPE      SettingScope,\r
+  IN BROWSER_GET_DEFAULT_VALUE  GetDefaultValueScope,\r
+  IN BROWSER_STORAGE            *Storage OPTIONAL,\r
+  IN BOOLEAN                    RetrieveValueFirst,\r
+  IN BOOLEAN                    SkipGetAltCfg\r
   )\r
 {\r
   EFI_STATUS              Status;\r
@@ -4819,11 +4864,11 @@ ExtractDefault (
   //\r
   // Check the supported setting level.\r
   //\r
-  if (SettingScope >= MaxLevel || GetDefaultValueScope >= GetDefaultForMax) {\r
+  if ((SettingScope >= MaxLevel) || (GetDefaultValueScope >= GetDefaultForMax)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  if (GetDefaultValueScope == GetDefaultForStorage && Storage == NULL) {\r
+  if ((GetDefaultValueScope == GetDefaultForStorage) && (Storage == NULL)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
@@ -4841,7 +4886,7 @@ ExtractDefault (
     Link = GetFirstNode (&Form->StatementListHead);\r
     while (!IsNull (&Form->StatementListHead, Link)) {\r
       Question = FORM_BROWSER_STATEMENT_FROM_LINK (Link);\r
-      Link = GetNextNode (&Form->StatementListHead, Link);\r
+      Link     = GetNextNode (&Form->StatementListHead, Link);\r
 \r
       //\r
       // If get default value only for this storage, check the storage first.\r
@@ -4861,7 +4906,7 @@ ExtractDefault (
       // If Question is disabled, don't reset it to default\r
       //\r
       if (Question->Expression != NULL) {\r
-        if (EvaluateExpressionList(Question->Expression, TRUE, FormSet, Form) == ExpressDisable) {\r
+        if (EvaluateExpressionList (Question->Expression, TRUE, FormSet, Form) == ExpressDisable) {\r
           continue;\r
         }\r
       }\r
@@ -4870,7 +4915,7 @@ ExtractDefault (
         //\r
         // Call the Retrieve call back to get the initial question value.\r
         //\r
-        Status = ProcessRetrieveForQuestion(FormSet->ConfigAccess, Question, FormSet);\r
+        Status = ProcessRetrieveForQuestion (FormSet->ConfigAccess, Question, FormSet);\r
       }\r
 \r
       //\r
@@ -4887,7 +4932,8 @@ ExtractDefault (
       // Synchronize Buffer storage's Edit buffer\r
       //\r
       if ((Question->Storage != NULL) &&\r
-          (Question->Storage->Type != EFI_HII_VARSTORE_EFI_VARIABLE)) {\r
+          (Question->Storage->Type != EFI_HII_VARSTORE_EFI_VARIABLE))\r
+      {\r
         SetQuestionValue (FormSet, Form, Question, GetSetValueWithEditBuffer);\r
       }\r
     }\r
@@ -4896,7 +4942,7 @@ ExtractDefault (
     // Clean the AltCfg String.\r
     //\r
     if (!SkipGetAltCfg && (GetDefaultValueScope != GetDefaultForNoStorage)) {\r
-      CleanAltCfgForForm(Form);\r
+      CleanAltCfgForForm (Form);\r
     }\r
   } else if (SettingScope == FormSetLevel) {\r
     //\r
@@ -4923,7 +4969,7 @@ ExtractDefault (
     //\r
     // Preload all Hii formset.\r
     //\r
-    LoadAllHiiFormset();\r
+    LoadAllHiiFormset ();\r
 \r
     OldFormSet = mSystemLevelFormSet;\r
 \r
@@ -4933,8 +4979,8 @@ ExtractDefault (
     Link = GetFirstNode (&gBrowserFormSetList);\r
     while (!IsNull (&gBrowserFormSetList, Link)) {\r
       LocalFormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-      Link = GetNextNode (&gBrowserFormSetList, Link);\r
-      if (!ValidateFormSet(LocalFormSet)) {\r
+      Link         = GetNextNode (&gBrowserFormSetList, Link);\r
+      if (!ValidateFormSet (LocalFormSet)) {\r
         continue;\r
       }\r
 \r
@@ -4949,7 +4995,6 @@ ExtractDefault (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Validate whether this question's value has changed.\r
 \r
@@ -4964,30 +5009,30 @@ ExtractDefault (
 **/\r
 BOOLEAN\r
 IsQuestionValueChanged (\r
-  IN FORM_BROWSER_FORMSET             *FormSet,\r
-  IN FORM_BROWSER_FORM                *Form,\r
-  IN OUT FORM_BROWSER_STATEMENT       *Question,\r
-  IN GET_SET_QUESTION_VALUE_WITH      GetValueFrom\r
+  IN FORM_BROWSER_FORMSET         *FormSet,\r
+  IN FORM_BROWSER_FORM            *Form,\r
+  IN OUT FORM_BROWSER_STATEMENT   *Question,\r
+  IN GET_SET_QUESTION_VALUE_WITH  GetValueFrom\r
   )\r
 {\r
-  EFI_HII_VALUE    BackUpValue;\r
-  CHAR8            *BackUpBuffer;\r
-  EFI_HII_VALUE    BackUpValue2;\r
-  CHAR8            *BackUpBuffer2;\r
-  EFI_STATUS       Status;\r
-  BOOLEAN          ValueChanged;\r
-  UINTN            BufferWidth;\r
+  EFI_HII_VALUE  BackUpValue;\r
+  CHAR8          *BackUpBuffer;\r
+  EFI_HII_VALUE  BackUpValue2;\r
+  CHAR8          *BackUpBuffer2;\r
+  EFI_STATUS     Status;\r
+  BOOLEAN        ValueChanged;\r
+  UINTN          BufferWidth;\r
 \r
   //\r
   // For quetion without storage, always mark it as data not changed.\r
   //\r
-  if (Question->Storage == NULL && Question->Operand != EFI_IFR_TIME_OP && Question->Operand != EFI_IFR_DATE_OP) {\r
+  if ((Question->Storage == NULL) && (Question->Operand != EFI_IFR_TIME_OP) && (Question->Operand != EFI_IFR_DATE_OP)) {\r
     return FALSE;\r
   }\r
 \r
-  BackUpBuffer = NULL;\r
+  BackUpBuffer  = NULL;\r
   BackUpBuffer2 = NULL;\r
-  ValueChanged = FALSE;\r
+  ValueChanged  = FALSE;\r
 \r
   switch (Question->Operand) {\r
     case EFI_IFR_ORDERED_LIST_OP:\r
@@ -4998,7 +5043,7 @@ IsQuestionValueChanged (
 \r
     case EFI_IFR_STRING_OP:\r
     case EFI_IFR_PASSWORD_OP:\r
-      BufferWidth  = (UINTN) Question->Maximum * sizeof (CHAR16);\r
+      BufferWidth  = (UINTN)Question->Maximum * sizeof (CHAR16);\r
       BackUpBuffer = AllocateCopyPool (BufferWidth, Question->BufferValue);\r
       ASSERT (BackUpBuffer != NULL);\r
       break;\r
@@ -5007,22 +5052,23 @@ IsQuestionValueChanged (
       BufferWidth = 0;\r
       break;\r
   }\r
+\r
   CopyMem (&BackUpValue, &Question->HiiValue, sizeof (EFI_HII_VALUE));\r
 \r
   if (GetValueFrom == GetSetValueWithBothBuffer) {\r
     Status = GetQuestionValue (FormSet, Form, Question, GetSetValueWithEditBuffer);\r
-    ASSERT_EFI_ERROR(Status);\r
+    ASSERT_EFI_ERROR (Status);\r
 \r
     switch (Question->Operand) {\r
       case EFI_IFR_ORDERED_LIST_OP:\r
-        BufferWidth  = Question->StorageWidth;\r
+        BufferWidth   = Question->StorageWidth;\r
         BackUpBuffer2 = AllocateCopyPool (BufferWidth, Question->BufferValue);\r
         ASSERT (BackUpBuffer2 != NULL);\r
         break;\r
 \r
       case EFI_IFR_STRING_OP:\r
       case EFI_IFR_PASSWORD_OP:\r
-        BufferWidth  = (UINTN) Question->Maximum * sizeof (CHAR16);\r
+        BufferWidth   = (UINTN)Question->Maximum * sizeof (CHAR16);\r
         BackUpBuffer2 = AllocateCopyPool (BufferWidth, Question->BufferValue);\r
         ASSERT (BackUpBuffer2 != NULL);\r
         break;\r
@@ -5031,21 +5077,24 @@ IsQuestionValueChanged (
         BufferWidth = 0;\r
         break;\r
     }\r
+\r
     CopyMem (&BackUpValue2, &Question->HiiValue, sizeof (EFI_HII_VALUE));\r
 \r
     Status = GetQuestionValue (FormSet, Form, Question, GetSetValueWithBuffer);\r
-    ASSERT_EFI_ERROR(Status);\r
+    ASSERT_EFI_ERROR (Status);\r
 \r
-    if (CompareMem (&BackUpValue2, &Question->HiiValue, sizeof (EFI_HII_VALUE)) != 0 ||\r
-        CompareMem (BackUpBuffer2, Question->BufferValue, BufferWidth) != 0) {\r
+    if ((CompareMem (&BackUpValue2, &Question->HiiValue, sizeof (EFI_HII_VALUE)) != 0) ||\r
+        (CompareMem (BackUpBuffer2, Question->BufferValue, BufferWidth) != 0))\r
+    {\r
       ValueChanged = TRUE;\r
     }\r
   } else {\r
     Status = GetQuestionValue (FormSet, Form, Question, GetValueFrom);\r
-    ASSERT_EFI_ERROR(Status);\r
+    ASSERT_EFI_ERROR (Status);\r
 \r
-    if (CompareMem (&BackUpValue, &Question->HiiValue, sizeof (EFI_HII_VALUE)) != 0 ||\r
-        CompareMem (BackUpBuffer, Question->BufferValue, BufferWidth) != 0) {\r
+    if ((CompareMem (&BackUpValue, &Question->HiiValue, sizeof (EFI_HII_VALUE)) != 0) ||\r
+        (CompareMem (BackUpBuffer, Question->BufferValue, BufferWidth) != 0))\r
+    {\r
       ValueChanged = TRUE;\r
     }\r
   }\r
@@ -5080,14 +5129,14 @@ IsQuestionValueChanged (
 **/\r
 EFI_STATUS\r
 LoadFormConfig (\r
-  IN OUT UI_MENU_SELECTION    *Selection,\r
-  IN FORM_BROWSER_FORMSET     *FormSet,\r
-  IN FORM_BROWSER_FORM        *Form\r
+  IN OUT UI_MENU_SELECTION  *Selection,\r
+  IN FORM_BROWSER_FORMSET   *FormSet,\r
+  IN FORM_BROWSER_FORM      *Form\r
   )\r
 {\r
-  EFI_STATUS                  Status;\r
-  LIST_ENTRY                  *Link;\r
-  FORM_BROWSER_STATEMENT      *Question;\r
+  EFI_STATUS              Status;\r
+  LIST_ENTRY              *Link;\r
+  FORM_BROWSER_STATEMENT  *Question;\r
 \r
   Link = GetFirstNode (&Form->StatementListHead);\r
   while (!IsNull (&Form->StatementListHead, Link)) {\r
@@ -5096,17 +5145,18 @@ LoadFormConfig (
     //\r
     // Initialize local copy of Value for each Question\r
     //\r
-    if (Question->Operand == EFI_IFR_PASSWORD_OP && (Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK)== 0) {\r
+    if ((Question->Operand == EFI_IFR_PASSWORD_OP) && ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) == 0)) {\r
       Status = GetQuestionValue (FormSet, Form, Question, GetSetValueWithHiiDriver);\r
     } else {\r
       Status = GetQuestionValue (FormSet, Form, Question, GetSetValueWithEditBuffer);\r
     }\r
+\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
 \r
     if ((Question->Operand == EFI_IFR_STRING_OP) || (Question->Operand == EFI_IFR_PASSWORD_OP)) {\r
-      HiiSetString (FormSet->HiiHandle, Question->HiiValue.Value.string, (CHAR16*)Question->BufferValue, NULL);\r
+      HiiSetString (FormSet->HiiHandle, Question->HiiValue.Value.string, (CHAR16 *)Question->BufferValue, NULL);\r
     }\r
 \r
     Link = GetNextNode (&Form->StatementListHead, Link);\r
@@ -5129,13 +5179,13 @@ LoadFormConfig (
 **/\r
 EFI_STATUS\r
 LoadFormSetConfig (\r
-  IN OUT UI_MENU_SELECTION    *Selection,\r
-  IN     FORM_BROWSER_FORMSET *FormSet\r
+  IN OUT UI_MENU_SELECTION     *Selection,\r
+  IN     FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
-  EFI_STATUS            Status;\r
-  LIST_ENTRY            *Link;\r
-  FORM_BROWSER_FORM     *Form;\r
+  EFI_STATUS         Status;\r
+  LIST_ENTRY         *Link;\r
+  FORM_BROWSER_FORM  *Form;\r
 \r
   Link = GetFirstNode (&FormSet->FormListHead);\r
   while (!IsNull (&FormSet->FormListHead, Link)) {\r
@@ -5169,12 +5219,12 @@ LoadFormSetConfig (
 **/\r
 VOID\r
 RemoveElement (\r
-  IN OUT BROWSER_STORAGE      *Storage,\r
-  IN     CHAR16               *RequestElement\r
+  IN OUT BROWSER_STORAGE  *Storage,\r
+  IN     CHAR16           *RequestElement\r
   )\r
 {\r
-  CHAR16   *NewStr;\r
-  CHAR16   *DestStr;\r
+  CHAR16  *NewStr;\r
+  CHAR16  *DestStr;\r
 \r
   ASSERT (Storage->ConfigRequest != NULL && RequestElement != NULL);\r
 \r
@@ -5203,13 +5253,13 @@ RemoveElement (
 **/\r
 VOID\r
 RemoveConfigRequest (\r
-  FORMSET_STORAGE   *Storage,\r
-  CHAR16            *ConfigRequest\r
+  FORMSET_STORAGE  *Storage,\r
+  CHAR16           *ConfigRequest\r
   )\r
 {\r
-  CHAR16       *RequestElement;\r
-  CHAR16       *NextRequestElement;\r
-  CHAR16       *SearchKey;\r
+  CHAR16  *RequestElement;\r
+  CHAR16  *NextRequestElement;\r
+  CHAR16  *SearchKey;\r
 \r
   //\r
   // No request element in it, just return.\r
@@ -5290,26 +5340,28 @@ CleanBrowserStorage (
   IN OUT FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
-  LIST_ENTRY            *Link;\r
-  FORMSET_STORAGE       *Storage;\r
+  LIST_ENTRY       *Link;\r
+  FORMSET_STORAGE  *Storage;\r
 \r
   Link = GetFirstNode (&FormSet->StorageListHead);\r
   while (!IsNull (&FormSet->StorageListHead, Link)) {\r
     Storage = FORMSET_STORAGE_FROM_LINK (Link);\r
-    Link = GetNextNode (&FormSet->StorageListHead, Link);\r
+    Link    = GetNextNode (&FormSet->StorageListHead, Link);\r
 \r
     if (Storage->BrowserStorage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER) {\r
-      if (Storage->ConfigRequest == NULL || Storage->BrowserStorage->ConfigRequest == NULL) {\r
+      if ((Storage->ConfigRequest == NULL) || (Storage->BrowserStorage->ConfigRequest == NULL)) {\r
         continue;\r
       }\r
 \r
       RemoveConfigRequest (Storage, Storage->ConfigRequest);\r
-    } else if (Storage->BrowserStorage->Type == EFI_HII_VARSTORE_BUFFER ||\r
-               Storage->BrowserStorage->Type == EFI_HII_VARSTORE_NAME_VALUE) {\r
+    } else if ((Storage->BrowserStorage->Type == EFI_HII_VARSTORE_BUFFER) ||\r
+               (Storage->BrowserStorage->Type == EFI_HII_VARSTORE_NAME_VALUE))\r
+    {\r
       if (Storage->BrowserStorage->ConfigRequest != NULL) {\r
         FreePool (Storage->BrowserStorage->ConfigRequest);\r
         Storage->BrowserStorage->ConfigRequest = NULL;\r
       }\r
+\r
       Storage->BrowserStorage->Initialized = FALSE;\r
     }\r
   }\r
@@ -5327,8 +5379,8 @@ CleanBrowserStorage (
 **/\r
 BOOLEAN\r
 ElementValidation (\r
-  BROWSER_STORAGE   *BrowserStorage,\r
-  CHAR16            *RequestElement\r
+  BROWSER_STORAGE  *BrowserStorage,\r
+  CHAR16           *RequestElement\r
   )\r
 {\r
   return StrStr (BrowserStorage->ConfigRequest, RequestElement) != NULL ? TRUE : FALSE;\r
@@ -5344,19 +5396,19 @@ ElementValidation (
 **/\r
 VOID\r
 AppendConfigRequest (\r
-  IN OUT CHAR16               **ConfigRequest,\r
-  IN OUT UINTN                *SpareStrLen,\r
-  IN     CHAR16               *RequestElement\r
+  IN OUT CHAR16  **ConfigRequest,\r
+  IN OUT UINTN   *SpareStrLen,\r
+  IN     CHAR16  *RequestElement\r
   )\r
 {\r
-  CHAR16   *NewStr;\r
-  UINTN    StringSize;\r
-  UINTN    StrLength;\r
-  UINTN    MaxLen;\r
+  CHAR16  *NewStr;\r
+  UINTN   StringSize;\r
+  UINTN   StrLength;\r
+  UINTN   MaxLen;\r
 \r
-  StrLength = StrLen (RequestElement);\r
+  StrLength  = StrLen (RequestElement);\r
   StringSize = (*ConfigRequest != NULL) ? StrSize (*ConfigRequest) : sizeof (CHAR16);\r
-  MaxLen = StringSize / sizeof (CHAR16) + *SpareStrLen;\r
+  MaxLen     = StringSize / sizeof (CHAR16) + *SpareStrLen;\r
 \r
   //\r
   // Append <RequestElement> to <ConfigRequest>\r
@@ -5373,6 +5425,7 @@ AppendConfigRequest (
       CopyMem (NewStr, *ConfigRequest, StringSize);\r
       FreePool (*ConfigRequest);\r
     }\r
+\r
     *ConfigRequest = NewStr;\r
     *SpareStrLen   = CONFIG_REQUEST_STRING_INCREMENTAL;\r
   }\r
@@ -5394,20 +5447,20 @@ AppendConfigRequest (
 **/\r
 BOOLEAN\r
 ConfigRequestAdjust (\r
-  IN  BROWSER_STORAGE         *Storage,\r
-  IN  CHAR16                  *Request,\r
-  IN  BOOLEAN                 RespString\r
+  IN  BROWSER_STORAGE  *Storage,\r
+  IN  CHAR16           *Request,\r
+  IN  BOOLEAN          RespString\r
   )\r
 {\r
-  CHAR16       *RequestElement;\r
-  CHAR16       *NextRequestElement;\r
-  CHAR16       *NextElementBakup;\r
-  CHAR16       *SearchKey;\r
-  CHAR16       *ValueKey;\r
-  BOOLEAN      RetVal;\r
-  CHAR16       *ConfigRequest;\r
-\r
-  RetVal         = FALSE;\r
+  CHAR16   *RequestElement;\r
+  CHAR16   *NextRequestElement;\r
+  CHAR16   *NextElementBakup;\r
+  CHAR16   *SearchKey;\r
+  CHAR16   *ValueKey;\r
+  BOOLEAN  RetVal;\r
+  CHAR16   *ConfigRequest;\r
+\r
+  RetVal           = FALSE;\r
   NextElementBakup = NULL;\r
   ValueKey         = NULL;\r
 \r
@@ -5447,7 +5500,6 @@ ConfigRequestAdjust (
   }\r
 \r
   while (RequestElement != NULL) {\r
-\r
     //\r
     // +1 to avoid find header itself.\r
     //\r
@@ -5458,17 +5510,18 @@ ConfigRequestAdjust (
     //\r
     if (NextRequestElement != NULL) {\r
       if (RespString && (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER)) {\r
-        NextElementBakup = NextRequestElement;\r
+        NextElementBakup   = NextRequestElement;\r
         NextRequestElement = StrStr (RequestElement, ValueKey);\r
         ASSERT (NextRequestElement != NULL);\r
       }\r
+\r
       //\r
       // Replace "&" with '\0'.\r
       //\r
       *NextRequestElement = L'\0';\r
     } else {\r
       if (RespString && (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER)) {\r
-        NextElementBakup = NextRequestElement;\r
+        NextElementBakup   = NextRequestElement;\r
         NextRequestElement = StrStr (RequestElement, ValueKey);\r
         ASSERT (NextRequestElement != NULL);\r
         //\r
@@ -5482,7 +5535,7 @@ ConfigRequestAdjust (
       //\r
       // Add this element to the Storage->BrowserStorage->AllRequestElement.\r
       //\r
-      AppendConfigRequest(&Storage->ConfigRequest, &Storage->SpareStrLen, RequestElement);\r
+      AppendConfigRequest (&Storage->ConfigRequest, &Storage->SpareStrLen, RequestElement);\r
       RetVal = TRUE;\r
     }\r
 \r
@@ -5512,8 +5565,8 @@ ConfigRequestAdjust (
 **/\r
 VOID\r
 LoadStorage (\r
-  IN FORM_BROWSER_FORMSET    *FormSet,\r
-  IN FORMSET_STORAGE         *Storage\r
+  IN FORM_BROWSER_FORMSET  *FormSet,\r
+  IN FORMSET_STORAGE       *Storage\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -5531,9 +5584,10 @@ LoadStorage (
 \r
     case EFI_HII_VARSTORE_EFI_VARIABLE_BUFFER:\r
       if (Storage->BrowserStorage->ConfigRequest != NULL) {\r
-        ConfigRequestAdjust(Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
+        ConfigRequestAdjust (Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
         return;\r
       }\r
+\r
       break;\r
 \r
     case EFI_HII_VARSTORE_BUFFER:\r
@@ -5549,7 +5603,7 @@ LoadStorage (
       // Just update the ConfigRequest, if storage already initialized.\r
       //\r
       if (Storage->BrowserStorage->Initialized) {\r
-        ConfigRequestAdjust(Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
+        ConfigRequestAdjust (Storage->BrowserStorage, Storage->ConfigRequest, FALSE);\r
         return;\r
       }\r
 \r
@@ -5566,15 +5620,16 @@ LoadStorage (
     // Allocate and fill a buffer large enough to hold the <ConfigHdr> template\r
     // followed by "&OFFSET=0&WIDTH=WWWW"followed by a Null-terminator\r
     //\r
-    StrLen = StrSize (Storage->ConfigHdr) + 20 * sizeof (CHAR16);\r
+    StrLen        = StrSize (Storage->ConfigHdr) + 20 * sizeof (CHAR16);\r
     ConfigRequest = AllocateZeroPool (StrLen);\r
     ASSERT (ConfigRequest != NULL);\r
     UnicodeSPrint (\r
-               ConfigRequest,\r
-               StrLen,\r
-               L"%s&OFFSET=0&WIDTH=%04x",\r
-               Storage->ConfigHdr,\r
-               Storage->BrowserStorage->Size);\r
+      ConfigRequest,\r
+      StrLen,\r
+      L"%s&OFFSET=0&WIDTH=%04x",\r
+      Storage->ConfigHdr,\r
+      Storage->BrowserStorage->Size\r
+      );\r
   } else {\r
     ConfigRequest = Storage->ConfigRequest;\r
   }\r
@@ -5583,11 +5638,11 @@ LoadStorage (
   // Request current settings from Configuration Driver\r
   //\r
   Status = mHiiConfigRouting->ExtractConfig (\r
-                                    mHiiConfigRouting,\r
-                                    ConfigRequest,\r
-                                    &Progress,\r
-                                    &Result\r
-                                    );\r
+                                mHiiConfigRouting,\r
+                                ConfigRequest,\r
+                                &Progress,\r
+                                &Result\r
+                                );\r
 \r
   //\r
   // If get value fail, extract default from IFR binary\r
@@ -5612,7 +5667,7 @@ LoadStorage (
   //\r
   // Input NULL for ConfigRequest field means sync all fields from editbuffer to buffer.\r
   //\r
-  SynchronizeStorage(Storage->BrowserStorage, NULL, TRUE);\r
+  SynchronizeStorage (Storage->BrowserStorage, NULL, TRUE);\r
 \r
   if (Storage->BrowserStorage->Type != EFI_HII_VARSTORE_NAME_VALUE) {\r
     if (ConfigRequest != NULL) {\r
@@ -5630,14 +5685,14 @@ LoadStorage (
 **/\r
 VOID\r
 SyncStatusForQuestion (\r
-  IN OUT FORM_BROWSER_FORMSET             *NewFormSet,\r
-  IN     FORM_BROWSER_STATEMENT           *OldQuestion\r
+  IN OUT FORM_BROWSER_FORMSET    *NewFormSet,\r
+  IN     FORM_BROWSER_STATEMENT  *OldQuestion\r
   )\r
 {\r
-  LIST_ENTRY                  *Link;\r
-  LIST_ENTRY                  *QuestionLink;\r
-  FORM_BROWSER_FORM           *Form;\r
-  FORM_BROWSER_STATEMENT      *Question;\r
+  LIST_ENTRY              *Link;\r
+  LIST_ENTRY              *QuestionLink;\r
+  FORM_BROWSER_FORM       *Form;\r
+  FORM_BROWSER_STATEMENT  *Question;\r
 \r
   //\r
   // For each form in one formset.\r
@@ -5652,7 +5707,7 @@ SyncStatusForQuestion (
     //\r
     QuestionLink = GetFirstNode (&Form->StatementListHead);\r
     while (!IsNull (&Form->StatementListHead, QuestionLink)) {\r
-      Question = FORM_BROWSER_STATEMENT_FROM_LINK (QuestionLink);\r
+      Question     = FORM_BROWSER_STATEMENT_FROM_LINK (QuestionLink);\r
       QuestionLink = GetNextNode (&Form->StatementListHead, QuestionLink);\r
 \r
       if (Question->QuestionId == OldQuestion->QuestionId) {\r
@@ -5672,14 +5727,14 @@ SyncStatusForQuestion (
 **/\r
 VOID\r
 SyncStatusForFormSet (\r
-  IN OUT FORM_BROWSER_FORMSET             *NewFormSet,\r
-  IN     FORM_BROWSER_FORMSET             *OldFormSet\r
+  IN OUT FORM_BROWSER_FORMSET  *NewFormSet,\r
+  IN     FORM_BROWSER_FORMSET  *OldFormSet\r
   )\r
 {\r
-  LIST_ENTRY                  *Link;\r
-  LIST_ENTRY                  *QuestionLink;\r
-  FORM_BROWSER_FORM           *Form;\r
-  FORM_BROWSER_STATEMENT      *Question;\r
+  LIST_ENTRY              *Link;\r
+  LIST_ENTRY              *QuestionLink;\r
+  FORM_BROWSER_FORM       *Form;\r
+  FORM_BROWSER_STATEMENT  *Question;\r
 \r
   //\r
   // For each form in one formset.\r
@@ -5694,7 +5749,7 @@ SyncStatusForFormSet (
     //\r
     QuestionLink = GetFirstNode (&Form->StatementListHead);\r
     while (!IsNull (&Form->StatementListHead, QuestionLink)) {\r
-      Question = FORM_BROWSER_STATEMENT_FROM_LINK (QuestionLink);\r
+      Question     = FORM_BROWSER_STATEMENT_FROM_LINK (QuestionLink);\r
       QuestionLink = GetNextNode (&Form->StatementListHead, QuestionLink);\r
 \r
       if (!Question->ValueChanged) {\r
@@ -5717,12 +5772,12 @@ SyncStatusForFormSet (
 **/\r
 VOID\r
 InitializeCurrentSetting (\r
-  IN OUT FORM_BROWSER_FORMSET             *FormSet\r
+  IN OUT FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  FORMSET_STORAGE         *Storage;\r
-  FORM_BROWSER_FORMSET    *OldFormSet;\r
+  LIST_ENTRY            *Link;\r
+  FORMSET_STORAGE       *Storage;\r
+  FORM_BROWSER_FORMSET  *OldFormSet;\r
 \r
   //\r
   // Try to find pre FormSet in the maintain backup list.\r
@@ -5734,6 +5789,7 @@ InitializeCurrentSetting (
     RemoveEntryList (&OldFormSet->Link);\r
     DestroyFormSet (OldFormSet);\r
   }\r
+\r
   InsertTailList (&gBrowserFormSetList, &FormSet->Link);\r
 \r
   //\r
@@ -5754,7 +5810,6 @@ InitializeCurrentSetting (
   }\r
 }\r
 \r
-\r
 /**\r
   Fetch the Ifr binary data of a FormSet.\r
 \r
@@ -5776,10 +5831,10 @@ InitializeCurrentSetting (
 **/\r
 EFI_STATUS\r
 GetIfrBinaryData (\r
-  IN  EFI_HII_HANDLE   Handle,\r
-  IN OUT EFI_GUID      *FormSetGuid,\r
-  OUT UINTN            *BinaryLength,\r
-  OUT UINT8            **BinaryData\r
+  IN  EFI_HII_HANDLE  Handle,\r
+  IN OUT EFI_GUID     *FormSetGuid,\r
+  OUT UINTN           *BinaryLength,\r
+  OUT UINT8           **BinaryData\r
   )\r
 {\r
   EFI_STATUS                   Status;\r
@@ -5798,7 +5853,7 @@ GetIfrBinaryData (
   EFI_GUID                     *ComparingGuid;\r
 \r
   OpCodeData = NULL;\r
-  Package = NULL;\r
+  Package    = NULL;\r
   ZeroMem (&PackageHeader, sizeof (EFI_HII_PACKAGE_HEADER));\r
 \r
   //\r
@@ -5813,30 +5868,32 @@ GetIfrBinaryData (
   //\r
   // Get HII PackageList\r
   //\r
-  BufferSize = 0;\r
+  BufferSize     = 0;\r
   HiiPackageList = NULL;\r
-  Status = mHiiDatabase->ExportPackageLists (mHiiDatabase, Handle, &BufferSize, HiiPackageList);\r
+  Status         = mHiiDatabase->ExportPackageLists (mHiiDatabase, Handle, &BufferSize, HiiPackageList);\r
   if (Status == EFI_BUFFER_TOO_SMALL) {\r
     HiiPackageList = AllocatePool (BufferSize);\r
     ASSERT (HiiPackageList != NULL);\r
 \r
     Status = mHiiDatabase->ExportPackageLists (mHiiDatabase, Handle, &BufferSize, HiiPackageList);\r
   }\r
+\r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
+\r
   ASSERT (HiiPackageList != NULL);\r
 \r
   //\r
   // Get Form package from this HII package List\r
   //\r
-  Offset = sizeof (EFI_HII_PACKAGE_LIST_HEADER);\r
+  Offset  = sizeof (EFI_HII_PACKAGE_LIST_HEADER);\r
   Offset2 = 0;\r
   CopyMem (&PackageListLength, &HiiPackageList->PackageLength, sizeof (UINT32));\r
 \r
   ClassGuidMatch = FALSE;\r
   while (Offset < PackageListLength) {\r
-    Package = ((UINT8 *) HiiPackageList) + Offset;\r
+    Package = ((UINT8 *)HiiPackageList) + Offset;\r
     CopyMem (&PackageHeader, Package, sizeof (EFI_HII_PACKAGE_HEADER));\r
 \r
     if (PackageHeader.Type == EFI_HII_PACKAGE_FORMS) {\r
@@ -5847,27 +5904,29 @@ GetIfrBinaryData (
       while (Offset2 < PackageHeader.Length) {\r
         OpCodeData = Package + Offset2;\r
 \r
-        if (((EFI_IFR_OP_HEADER *) OpCodeData)->OpCode == EFI_IFR_FORM_SET_OP) {\r
+        if (((EFI_IFR_OP_HEADER *)OpCodeData)->OpCode == EFI_IFR_FORM_SET_OP) {\r
           //\r
           // Try to compare against formset GUID\r
           //\r
           if (IsZeroGuid (ComparingGuid) ||\r
-              CompareGuid (ComparingGuid, (EFI_GUID *)(OpCodeData + sizeof (EFI_IFR_OP_HEADER)))) {\r
+              CompareGuid (ComparingGuid, (EFI_GUID *)(OpCodeData + sizeof (EFI_IFR_OP_HEADER))))\r
+          {\r
             break;\r
           }\r
 \r
-          if (((EFI_IFR_OP_HEADER *) OpCodeData)->Length > OFFSET_OF (EFI_IFR_FORM_SET, Flags)) {\r
+          if (((EFI_IFR_OP_HEADER *)OpCodeData)->Length > OFFSET_OF (EFI_IFR_FORM_SET, Flags)) {\r
             //\r
             // Try to compare against formset class GUID\r
             //\r
-            NumberOfClassGuid = (UINT8) (((EFI_IFR_FORM_SET *) OpCodeData)->Flags & 0x3);\r
-            ClassGuid         = (EFI_GUID *) (OpCodeData + sizeof (EFI_IFR_FORM_SET));\r
+            NumberOfClassGuid = (UINT8)(((EFI_IFR_FORM_SET *)OpCodeData)->Flags & 0x3);\r
+            ClassGuid         = (EFI_GUID *)(OpCodeData + sizeof (EFI_IFR_FORM_SET));\r
             for (Index = 0; Index < NumberOfClassGuid; Index++) {\r
               if (CompareGuid (ComparingGuid, ClassGuid + Index)) {\r
                 ClassGuidMatch = TRUE;\r
                 break;\r
               }\r
             }\r
+\r
             if (ClassGuidMatch) {\r
               break;\r
             }\r
@@ -5877,7 +5936,7 @@ GetIfrBinaryData (
           }\r
         }\r
 \r
-        Offset2 += ((EFI_IFR_OP_HEADER *) OpCodeData)->Length;\r
+        Offset2 += ((EFI_IFR_OP_HEADER *)OpCodeData)->Length;\r
       }\r
 \r
       if (Offset2 < PackageHeader.Length) {\r
@@ -5903,7 +5962,7 @@ GetIfrBinaryData (
     //\r
     // Return the FormSet GUID\r
     //\r
-    CopyMem (FormSetGuid, &((EFI_IFR_FORM_SET *) OpCodeData)->Guid, sizeof (EFI_GUID));\r
+    CopyMem (FormSetGuid, &((EFI_IFR_FORM_SET *)OpCodeData)->Guid, sizeof (EFI_GUID));\r
   }\r
 \r
   //\r
@@ -5912,7 +5971,7 @@ GetIfrBinaryData (
   // of the Form Package.\r
   //\r
   *BinaryLength = PackageHeader.Length - Offset2;\r
-  *BinaryData = AllocateCopyPool (*BinaryLength, OpCodeData);\r
+  *BinaryData   = AllocateCopyPool (*BinaryLength, OpCodeData);\r
 \r
   FreePool (HiiPackageList);\r
 \r
@@ -5923,7 +5982,6 @@ GetIfrBinaryData (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Initialize the internal data structure of a FormSet.\r
 \r
@@ -5941,13 +5999,13 @@ GetIfrBinaryData (
 **/\r
 EFI_STATUS\r
 InitializeFormSet (\r
-  IN  EFI_HII_HANDLE                   Handle,\r
-  IN OUT EFI_GUID                      *FormSetGuid,\r
-  OUT FORM_BROWSER_FORMSET             *FormSet\r
+  IN  EFI_HII_HANDLE        Handle,\r
+  IN OUT EFI_GUID           *FormSetGuid,\r
+  OUT FORM_BROWSER_FORMSET  *FormSet\r
   )\r
 {\r
-  EFI_STATUS                Status;\r
-  EFI_HANDLE                DriverHandle;\r
+  EFI_STATUS  Status;\r
+  EFI_HANDLE  DriverHandle;\r
 \r
   Status = GetIfrBinaryData (Handle, FormSetGuid, &FormSet->IfrBinaryLength, &FormSet->IfrBinaryData);\r
   if (EFI_ERROR (Status)) {\r
@@ -5966,12 +6024,13 @@ InitializeFormSet (
   if (EFI_ERROR (Status)) {\r
     return Status;\r
   }\r
+\r
   FormSet->DriverHandle = DriverHandle;\r
-  Status = gBS->HandleProtocol (\r
-                  DriverHandle,\r
-                  &gEfiHiiConfigAccessProtocolGuid,\r
-                  (VOID **) &FormSet->ConfigAccess\r
-                  );\r
+  Status                = gBS->HandleProtocol (\r
+                                 DriverHandle,\r
+                                 &gEfiHiiConfigAccessProtocolGuid,\r
+                                 (VOID **)&FormSet->ConfigAccess\r
+                                 );\r
   if (EFI_ERROR (Status)) {\r
     //\r
     // Configuration Driver don't attach ConfigAccess protocol to its HII package\r
@@ -5988,7 +6047,6 @@ InitializeFormSet (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Save globals used by previous call to SendForm(). SendForm() may be called from\r
   HiiConfigAccess.Callback(), this will cause SendForm() be reentried.\r
@@ -6000,9 +6058,9 @@ SaveBrowserContext (
   VOID\r
   )\r
 {\r
-  BROWSER_CONTEXT      *Context;\r
-  FORM_ENTRY_INFO      *MenuList;\r
-  FORM_BROWSER_FORMSET *FormSet;\r
+  BROWSER_CONTEXT       *Context;\r
+  FORM_ENTRY_INFO       *MenuList;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
 \r
   gBrowserContextCount++;\r
   if (gBrowserContextCount == 1) {\r
@@ -6020,39 +6078,39 @@ SaveBrowserContext (
   //\r
   // Save FormBrowser context\r
   //\r
-  Context->Selection            = gCurrentSelection;\r
-  Context->ResetRequired        = gResetRequiredFormLevel;\r
-  Context->FlagReconnect        = gFlagReconnect;\r
-  Context->CallbackReconnect    = gCallbackReconnect;\r
-  Context->ExitRequired         = gExitRequired;\r
-  Context->HiiHandle            = mCurrentHiiHandle;\r
-  Context->FormId               = mCurrentFormId;\r
+  Context->Selection         = gCurrentSelection;\r
+  Context->ResetRequired     = gResetRequiredFormLevel;\r
+  Context->FlagReconnect     = gFlagReconnect;\r
+  Context->CallbackReconnect = gCallbackReconnect;\r
+  Context->ExitRequired      = gExitRequired;\r
+  Context->HiiHandle         = mCurrentHiiHandle;\r
+  Context->FormId            = mCurrentFormId;\r
   CopyGuid (&Context->FormSetGuid, &mCurrentFormSetGuid);\r
-  Context->SystemLevelFormSet   = mSystemLevelFormSet;\r
-  Context->CurFakeQestId        = mCurFakeQestId;\r
+  Context->SystemLevelFormSet    = mSystemLevelFormSet;\r
+  Context->CurFakeQestId         = mCurFakeQestId;\r
   Context->HiiPackageListUpdated = mHiiPackageListUpdated;\r
-  Context->FinishRetrieveCall   = mFinishRetrieveCall;\r
+  Context->FinishRetrieveCall    = mFinishRetrieveCall;\r
 \r
   //\r
   // Save the menu history data.\r
   //\r
-  InitializeListHead(&Context->FormHistoryList);\r
+  InitializeListHead (&Context->FormHistoryList);\r
   while (!IsListEmpty (&mPrivateData.FormBrowserEx2.FormViewHistoryHead)) {\r
     MenuList = FORM_ENTRY_INFO_FROM_LINK (mPrivateData.FormBrowserEx2.FormViewHistoryHead.ForwardLink);\r
     RemoveEntryList (&MenuList->Link);\r
 \r
-    InsertTailList(&Context->FormHistoryList, &MenuList->Link);\r
+    InsertTailList (&Context->FormHistoryList, &MenuList->Link);\r
   }\r
 \r
   //\r
   // Save formset list.\r
   //\r
-  InitializeListHead(&Context->FormSetList);\r
+  InitializeListHead (&Context->FormSetList);\r
   while (!IsListEmpty (&gBrowserFormSetList)) {\r
     FormSet = FORM_BROWSER_FORMSET_FROM_LINK (gBrowserFormSetList.ForwardLink);\r
     RemoveEntryList (&FormSet->Link);\r
 \r
-    InsertTailList(&Context->FormSetList, &FormSet->Link);\r
+    InsertTailList (&Context->FormSetList, &FormSet->Link);\r
   }\r
 \r
   //\r
@@ -6061,7 +6119,6 @@ SaveBrowserContext (
   InsertHeadList (&gBrowserContextList, &Context->Link);\r
 }\r
 \r
-\r
 /**\r
   Restore globals used by previous call to SendForm().\r
 \r
@@ -6071,10 +6128,10 @@ RestoreBrowserContext (
   VOID\r
   )\r
 {\r
-  LIST_ENTRY       *Link;\r
-  BROWSER_CONTEXT  *Context;\r
-  FORM_ENTRY_INFO      *MenuList;\r
-  FORM_BROWSER_FORMSET *FormSet;\r
+  LIST_ENTRY            *Link;\r
+  BROWSER_CONTEXT       *Context;\r
+  FORM_ENTRY_INFO       *MenuList;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
 \r
   ASSERT (gBrowserContextCount != 0);\r
   gBrowserContextCount--;\r
@@ -6087,24 +6144,24 @@ RestoreBrowserContext (
 \r
   ASSERT (!IsListEmpty (&gBrowserContextList));\r
 \r
-  Link = GetFirstNode (&gBrowserContextList);\r
+  Link    = GetFirstNode (&gBrowserContextList);\r
   Context = BROWSER_CONTEXT_FROM_LINK (Link);\r
 \r
   //\r
   // Restore FormBrowser context\r
   //\r
-  gCurrentSelection     = Context->Selection;\r
+  gCurrentSelection       = Context->Selection;\r
   gResetRequiredFormLevel = Context->ResetRequired;\r
-  gFlagReconnect        = Context->FlagReconnect;\r
-  gCallbackReconnect    = Context->CallbackReconnect;\r
-  gExitRequired         = Context->ExitRequired;\r
-  mCurrentHiiHandle     = Context->HiiHandle;\r
-  mCurrentFormId        = Context->FormId;\r
+  gFlagReconnect          = Context->FlagReconnect;\r
+  gCallbackReconnect      = Context->CallbackReconnect;\r
+  gExitRequired           = Context->ExitRequired;\r
+  mCurrentHiiHandle       = Context->HiiHandle;\r
+  mCurrentFormId          = Context->FormId;\r
   CopyGuid (&mCurrentFormSetGuid, &Context->FormSetGuid);\r
-  mSystemLevelFormSet   = Context->SystemLevelFormSet;\r
-  mCurFakeQestId        = Context->CurFakeQestId;\r
+  mSystemLevelFormSet    = Context->SystemLevelFormSet;\r
+  mCurFakeQestId         = Context->CurFakeQestId;\r
   mHiiPackageListUpdated = Context->HiiPackageListUpdated;\r
-  mFinishRetrieveCall   = Context->FinishRetrieveCall;\r
+  mFinishRetrieveCall    = Context->FinishRetrieveCall;\r
 \r
   //\r
   // Restore the menu history data.\r
@@ -6113,7 +6170,7 @@ RestoreBrowserContext (
     MenuList = FORM_ENTRY_INFO_FROM_LINK (Context->FormHistoryList.ForwardLink);\r
     RemoveEntryList (&MenuList->Link);\r
 \r
-    InsertTailList(&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &MenuList->Link);\r
+    InsertTailList (&mPrivateData.FormBrowserEx2.FormViewHistoryHead, &MenuList->Link);\r
   }\r
 \r
   //\r
@@ -6123,7 +6180,7 @@ RestoreBrowserContext (
     FormSet = FORM_BROWSER_FORMSET_FROM_LINK (Context->FormSetList.ForwardLink);\r
     RemoveEntryList (&FormSet->Link);\r
 \r
-    InsertTailList(&gBrowserFormSetList, &FormSet->Link);\r
+    InsertTailList (&gBrowserFormSetList, &FormSet->Link);\r
   }\r
 \r
   //\r
@@ -6143,19 +6200,20 @@ RestoreBrowserContext (
 **/\r
 FORM_BROWSER_FORMSET *\r
 GetFormSetFromHiiHandle (\r
-  EFI_HII_HANDLE Handle\r
+  EFI_HII_HANDLE  Handle\r
   )\r
 {\r
-  LIST_ENTRY           *Link;\r
-  FORM_BROWSER_FORMSET *FormSet;\r
+  LIST_ENTRY            *Link;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
 \r
   Link = GetFirstNode (&gBrowserFormSetList);\r
   while (!IsNull (&gBrowserFormSetList, Link)) {\r
     FormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-    Link = GetNextNode (&gBrowserFormSetList, Link);\r
-    if (!ValidateFormSet(FormSet)) {\r
+    Link    = GetNextNode (&gBrowserFormSetList, Link);\r
+    if (!ValidateFormSet (FormSet)) {\r
       continue;\r
     }\r
+\r
     if (FormSet->HiiHandle == Handle) {\r
       return FormSet;\r
     }\r
@@ -6175,7 +6233,7 @@ GetFormSetFromHiiHandle (
 **/\r
 BOOLEAN\r
 IsHiiHandleInBrowserContext (\r
-  EFI_HII_HANDLE Handle\r
+  EFI_HII_HANDLE  Handle\r
   )\r
 {\r
   LIST_ENTRY       *Link;\r
@@ -6200,6 +6258,7 @@ IsHiiHandleInBrowserContext (
       //\r
       return TRUE;\r
     }\r
+\r
     Link = GetNextNode (&gBrowserContextList, Link);\r
   }\r
 \r
@@ -6221,9 +6280,9 @@ IsHiiHandleInBrowserContext (
 EFI_STATUS\r
 EFIAPI\r
 PasswordCheck (\r
-  IN FORM_DISPLAY_ENGINE_FORM      *Form,\r
-  IN FORM_DISPLAY_ENGINE_STATEMENT *Statement,\r
-  IN EFI_STRING                    PasswordString  OPTIONAL\r
+  IN FORM_DISPLAY_ENGINE_FORM       *Form,\r
+  IN FORM_DISPLAY_ENGINE_STATEMENT  *Statement,\r
+  IN EFI_STRING                     PasswordString  OPTIONAL\r
   )\r
 {\r
   EFI_STATUS                      Status;\r
@@ -6233,7 +6292,7 @@ PasswordCheck (
   FORM_BROWSER_STATEMENT          *Question;\r
 \r
   ConfigAccess = gCurrentSelection->FormSet->ConfigAccess;\r
-  Question = GetBrowserStatement(Statement);\r
+  Question     = GetBrowserStatement (Statement);\r
   ASSERT (Question != NULL);\r
 \r
   if ((Question->QuestionFlags & EFI_IFR_FLAG_CALLBACK) == EFI_IFR_FLAG_CALLBACK) {\r
@@ -6288,7 +6347,7 @@ PasswordCheck (
 **/\r
 BROWSER_HOT_KEY *\r
 GetHotKeyFromRegisterList (\r
-  IN EFI_INPUT_KEY *KeyData\r
+  IN EFI_INPUT_KEY  *KeyData\r
   )\r
 {\r
   LIST_ENTRY       *Link;\r
@@ -6300,6 +6359,7 @@ GetHotKeyFromRegisterList (
     if (HotKey->KeyData->ScanCode == KeyData->ScanCode) {\r
       return HotKey;\r
     }\r
+\r
     Link = GetNextNode (&gBrowserHotKeyList, Link);\r
   }\r
 \r
@@ -6322,7 +6382,7 @@ GetHotKeyFromRegisterList (
 EFI_STATUS\r
 EFIAPI\r
 SetScope (\r
-  IN BROWSER_SETTING_SCOPE Scope\r
+  IN BROWSER_SETTING_SCOPE  Scope\r
   )\r
 {\r
   if (Scope >= MaxLevel) {\r
@@ -6366,10 +6426,10 @@ SetScope (
 EFI_STATUS\r
 EFIAPI\r
 RegisterHotKey (\r
-  IN EFI_INPUT_KEY *KeyData,\r
-  IN UINT32        Action,\r
-  IN UINT16        DefaultId,\r
-  IN EFI_STRING    HelpString OPTIONAL\r
+  IN EFI_INPUT_KEY  *KeyData,\r
+  IN UINT32         Action,\r
+  IN UINT16         DefaultId,\r
+  IN EFI_STRING     HelpString OPTIONAL\r
   )\r
 {\r
   BROWSER_HOT_KEY  *HotKey;\r
@@ -6377,8 +6437,9 @@ RegisterHotKey (
   //\r
   // Check input parameters.\r
   //\r
-  if (KeyData == NULL || KeyData->UnicodeChar != CHAR_NULL ||\r
-     (Action != BROWSER_ACTION_UNREGISTER && HelpString == NULL)) {\r
+  if ((KeyData == NULL) || (KeyData->UnicodeChar != CHAR_NULL) ||\r
+      ((Action != BROWSER_ACTION_UNREGISTER) && (HelpString == NULL)))\r
+  {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -6424,11 +6485,12 @@ RegisterHotKey (
   //\r
   // Fill HotKey information.\r
   //\r
-  HotKey->Action     = Action;\r
-  HotKey->DefaultId  = DefaultId;\r
+  HotKey->Action    = Action;\r
+  HotKey->DefaultId = DefaultId;\r
   if (HotKey->HelpString != NULL) {\r
     FreePool (HotKey->HelpString);\r
   }\r
+\r
   HotKey->HelpString = AllocateCopyPool (StrSize (HelpString), HelpString);\r
 \r
   return EFI_SUCCESS;\r
@@ -6445,7 +6507,7 @@ RegisterHotKey (
 VOID\r
 EFIAPI\r
 RegiserExitHandler (\r
-  IN EXIT_HANDLER Handler\r
+  IN EXIT_HANDLER  Handler\r
   )\r
 {\r
   ExitHandlerFunction = Handler;\r
@@ -6465,35 +6527,39 @@ IsBrowserDataModified (
   VOID\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  FORM_BROWSER_FORMSET    *FormSet;\r
+  LIST_ENTRY            *Link;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
 \r
   switch (gBrowserSettingScope) {\r
     case FormLevel:\r
       if (gCurrentSelection == NULL) {\r
         return FALSE;\r
       }\r
+\r
       return IsNvUpdateRequiredForForm (gCurrentSelection->Form);\r
 \r
     case FormSetLevel:\r
       if (gCurrentSelection == NULL) {\r
         return FALSE;\r
       }\r
+\r
       return IsNvUpdateRequiredForFormSet (gCurrentSelection->FormSet);\r
 \r
     case SystemLevel:\r
       Link = GetFirstNode (&gBrowserFormSetList);\r
       while (!IsNull (&gBrowserFormSetList, Link)) {\r
         FormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-        if (!ValidateFormSet(FormSet)) {\r
+        if (!ValidateFormSet (FormSet)) {\r
           continue;\r
         }\r
 \r
         if (IsNvUpdateRequiredForFormSet (FormSet)) {\r
           return TRUE;\r
         }\r
+\r
         Link = GetNextNode (&gBrowserFormSetList, Link);\r
       }\r
+\r
       return FALSE;\r
 \r
     default:\r
@@ -6514,15 +6580,15 @@ IsBrowserDataModified (
 EFI_STATUS\r
 EFIAPI\r
 ExecuteAction (\r
-  IN UINT32        Action,\r
-  IN UINT16        DefaultId\r
+  IN UINT32  Action,\r
+  IN UINT16  DefaultId\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  FORM_BROWSER_FORMSET    *FormSet;\r
-  FORM_BROWSER_FORM       *Form;\r
+  EFI_STATUS            Status;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
+  FORM_BROWSER_FORM     *Form;\r
 \r
-  if (gBrowserSettingScope < SystemLevel && gCurrentSelection == NULL) {\r
+  if ((gBrowserSettingScope < SystemLevel) && (gCurrentSelection == NULL)) {\r
     return EFI_NOT_READY;\r
   }\r
 \r
@@ -6552,6 +6618,7 @@ ExecuteAction (
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
+\r
     UpdateStatementStatus (FormSet, Form, gBrowserSettingScope);\r
   }\r
 \r
@@ -6569,7 +6636,7 @@ ExecuteAction (
   // Executet the reset action.\r
   //\r
   if ((Action & BROWSER_ACTION_RESET) != 0) {\r
-    gResetRequiredFormLevel = TRUE;\r
+    gResetRequiredFormLevel   = TRUE;\r
     gResetRequiredSystemLevel = TRUE;\r
   }\r
 \r
@@ -6606,21 +6673,22 @@ SaveReminder (
   VOID\r
   )\r
 {\r
-  LIST_ENTRY              *Link;\r
-  FORM_BROWSER_FORMSET    *FormSet;\r
-  BOOLEAN                 IsDataChanged;\r
-  UINT32                  DataSavedAction;\r
-  UINT32                  ConfirmRet;\r
-\r
-  DataSavedAction  = BROWSER_NO_CHANGES;\r
-  IsDataChanged    = FALSE;\r
-  Link = GetFirstNode (&gBrowserFormSetList);\r
+  LIST_ENTRY            *Link;\r
+  FORM_BROWSER_FORMSET  *FormSet;\r
+  BOOLEAN               IsDataChanged;\r
+  UINT32                DataSavedAction;\r
+  UINT32                ConfirmRet;\r
+\r
+  DataSavedAction = BROWSER_NO_CHANGES;\r
+  IsDataChanged   = FALSE;\r
+  Link            = GetFirstNode (&gBrowserFormSetList);\r
   while (!IsNull (&gBrowserFormSetList, Link)) {\r
     FormSet = FORM_BROWSER_FORMSET_FROM_LINK (Link);\r
-    Link = GetNextNode (&gBrowserFormSetList, Link);\r
-    if (!ValidateFormSet(FormSet)) {\r
+    Link    = GetNextNode (&gBrowserFormSetList, Link);\r
+    if (!ValidateFormSet (FormSet)) {\r
       continue;\r
     }\r
+\r
     if (IsNvUpdateRequiredForFormSet (FormSet)) {\r
       IsDataChanged = TRUE;\r
       break;\r
@@ -6638,7 +6706,7 @@ SaveReminder (
   // If data is changed, prompt user to save or discard it.\r
   //\r
   do {\r
-    ConfirmRet = (UINT32) mFormDisplay->ConfirmDataChange();\r
+    ConfirmRet = (UINT32)mFormDisplay->ConfirmDataChange ();\r
 \r
     if (ConfirmRet == BROWSER_ACTION_SUBMIT) {\r
       SubmitForm (NULL, NULL, SystemLevel);\r
@@ -6672,4 +6740,3 @@ IsResetRequired (
 {\r
   return gResetRequiredSystemLevel;\r
 }\r
-\r