Entry and initialization module for the browser.\r
\r
Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>\r
-This program and the accompanying materials\r
-are licensed and made available under the terms and conditions of the BSD License\r
-which accompanies this distribution. The full text of the license may be found at\r
-http://opensource.org/licenses/bsd-license.php\r
-\r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+(C) Copyright 2020 - 2022 Hewlett Packard Enterprise Development LP<BR>\r
+SPDX-License-Identifier: BSD-2-Clause-Patent\r
\r
**/\r
\r
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 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
**/\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
\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
**/\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
**/\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
} 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
**/\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
**/\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
**/\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
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
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
DestroyFormSet (LocalFormSet);\r
continue;\r
}\r
+\r
InitializeCurrentSetting (LocalFormSet);\r
\r
//\r
**/\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
gDisplayFormData.BrowserStatus = BrowserStatus;\r
\r
if (HiiHandle != NULL) {\r
- gDisplayFormData.HiiHandle = HiiHandle;\r
+ gDisplayFormData.HiiHandle = HiiHandle;\r
}\r
\r
mFormDisplay->FormDisplay (&gDisplayFormData, &UserInputData);\r
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
//\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
// 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
DestroyFormSet (FormSet);\r
break;\r
}\r
- Selection->FormSet = FormSet;\r
+\r
+ Selection->FormSet = FormSet;\r
mSystemLevelFormSet = FormSet;\r
\r
//\r
\r
Status = SetupBrowser (Selection);\r
\r
- gCurrentSelection = NULL;\r
+ gCurrentSelection = NULL;\r
mSystemLevelFormSet = NULL;\r
\r
- if (gFlagReconnect || gCallbackReconnect) {\r
+ //\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 ((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
// 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
}\r
}\r
\r
- mFormDisplay->ExitDisplay();\r
+ mFormDisplay->ExitDisplay ();\r
\r
//\r
// Clear the menu history data.\r
**/\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
//\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
//\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
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
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
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
}\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
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
}\r
\r
gBS->LocateProtocol (\r
- &gEdkiiFormDisplayEngineProtocolGuid,\r
- NULL,\r
- (VOID **) &mFormDisplay\r
- );\r
+ &gEdkiiFormDisplayEngineProtocolGuid,\r
+ NULL,\r
+ (VOID **)&mFormDisplay\r
+ );\r
}\r
\r
/**\r
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
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
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
Status = gBS->LocateProtocol (\r
&gEdkiiFormDisplayEngineProtocolGuid,\r
NULL,\r
- (VOID **) &mFormDisplay\r
+ (VOID **)&mFormDisplay\r
);\r
\r
if (EFI_ERROR (Status)) {\r
return EFI_SUCCESS;\r
}\r
\r
-\r
/**\r
Create a new string in HII Package List.\r
\r
**/\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
return StringId;\r
}\r
\r
-\r
/**\r
Delete a string from HII Package List.\r
\r
**/\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
return EFI_SUCCESS;\r
}\r
\r
-\r
/**\r
Get the string based on the StringId and HII Package List Handle.\r
\r
**/\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
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
**/\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
**/\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
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
**/\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
} else {\r
NewStringCpy (Value, Node->Value);\r
}\r
+\r
return EFI_SUCCESS;\r
}\r
\r
return EFI_NOT_FOUND;\r
}\r
\r
-\r
/**\r
Set Value of given Name in a NameValue Storage.\r
\r
**/\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
} 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
return EFI_NOT_FOUND;\r
}\r
\r
-\r
/**\r
Convert setting of Buffer Storage or NameValue Storage to <ConfigResp>.\r
\r
**/\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
**/\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
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
**/\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
// 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
**/\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
**/\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
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
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
// 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
**/\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
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
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
//\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
QuestionValue->time.Minute = 0xff;\r
QuestionValue->time.Second = 0xff;\r
}\r
+\r
return EFI_SUCCESS;\r
}\r
\r
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
//\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
}\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
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
// 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
FreePool (Result);\r
return EFI_NOT_FOUND;\r
}\r
+\r
//\r
// Skip "&VALUE"\r
//\r
} 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
return Status;\r
}\r
\r
-\r
/**\r
Save Question Value to edit copy(cached) or Storage(uncached).\r
\r
**/\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
//\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
//\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->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
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
//\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
}\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
} 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
//\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
//\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
// 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
// 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
return Status;\r
}\r
\r
-\r
/**\r
Perform nosubmitif check for a Form.\r
\r
**/\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
\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
// 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
**/\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
**/\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
\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
**/\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
}\r
\r
if (ConfigRequest != NULL) {\r
- Status = mHiiConfigRouting->BlockToConfig(\r
+ Status = mHiiConfigRouting->BlockToConfig (\r
mHiiConfigRouting,\r
ConfigRequest,\r
Src,\r
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
**/\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
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
//\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
**/\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
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
**/\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
**/\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
**/\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
**/\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
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
//\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
**/\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
**/\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
//\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
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
*EndStr = '&';\r
}\r
\r
- return (BOOLEAN) (*RetForm != NULL);\r
+ return (BOOLEAN)(*RetForm != NULL);\r
}\r
\r
/**\r
\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
//\r
ConfigHdrEndStr = StrStr (ConfigRequest, L"&OFFSET=");\r
}\r
+\r
//\r
// Find the first fail pair in the ConfigRequest.\r
//\r
//\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
//\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
**/\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
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
**/\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
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
**/\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
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
//\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
}\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
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
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
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
//\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
**/\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
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
// 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
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
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
// 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
//\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
**/\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
BOOLEAN SubmitFormSetFail;\r
BOOLEAN DiscardChange;\r
\r
- HasInserted = FALSE;\r
+ HasInserted = FALSE;\r
SubmitFormSetFail = FALSE;\r
DiscardChange = FALSE;\r
\r
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
return Status;\r
}\r
\r
- Form = NULL;\r
+ Form = NULL;\r
Question = NULL;\r
//\r
// Submit Buffer storage or Name/Value storage\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 (Storage->Type == EFI_HII_VARSTORE_EFI_VARIABLE) {\r
continue;\r
// 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
// 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
//\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
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
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
// 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
//\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
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
// 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
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
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
\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
}\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
// 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
**/\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
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
**/\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
//\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
**/\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
// 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
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
**/\r
INTN\r
GetDefaultIdForCallBack (\r
- UINTN DefaultId\r
+ UINTN DefaultId\r
)\r
{\r
if (DefaultId == EFI_HII_DEFAULT_CLASS_STANDARD) {\r
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
**/\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
**/\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
**/\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
// 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
return NULL;\r
}\r
\r
-\r
/**\r
Reset Question to its default value.\r
\r
**/\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
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
// 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
\r
FreePool (NewString);\r
}\r
+\r
return Status;\r
}\r
}\r
// 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
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
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
// 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
// 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
// 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
//\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
**/\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
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
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
// 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
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
**/\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
**/\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
// 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
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
\r
FormSetStorage->ConfigAltResp = ConfigResp;\r
}\r
-\r
}\r
\r
/**\r
**/\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
**/\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
//\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
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
// 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
//\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
// 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
// Clean the AltCfg String.\r
//\r
if (!SkipGetAltCfg && (GetDefaultValueScope != GetDefaultForNoStorage)) {\r
- CleanAltCfgForForm(Form);\r
+ CleanAltCfgForForm (Form);\r
}\r
} else if (SettingScope == FormSetLevel) {\r
//\r
//\r
// Preload all Hii formset.\r
//\r
- LoadAllHiiFormset();\r
+ LoadAllHiiFormset ();\r
\r
OldFormSet = mSystemLevelFormSet;\r
\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
return EFI_SUCCESS;\r
}\r
\r
-\r
/**\r
Validate whether this question's value has changed.\r
\r
**/\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
\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
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
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
**/\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
//\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
**/\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
**/\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
**/\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
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
**/\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
**/\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
CopyMem (NewStr, *ConfigRequest, StringSize);\r
FreePool (*ConfigRequest);\r
}\r
+\r
*ConfigRequest = NewStr;\r
*SpareStrLen = CONFIG_REQUEST_STRING_INCREMENTAL;\r
}\r
**/\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
}\r
\r
while (RequestElement != NULL) {\r
-\r
//\r
// +1 to avoid find header itself.\r
//\r
//\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
//\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
**/\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
\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
// 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
// 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
// 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
//\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
**/\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
//\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
**/\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
//\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
**/\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
RemoveEntryList (&OldFormSet->Link);\r
DestroyFormSet (OldFormSet);\r
}\r
+\r
InsertTailList (&gBrowserFormSetList, &FormSet->Link);\r
\r
//\r
}\r
}\r
\r
-\r
/**\r
Fetch the Ifr binary data of a FormSet.\r
\r
**/\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
EFI_GUID *ComparingGuid;\r
\r
OpCodeData = NULL;\r
- Package = NULL;\r
+ Package = NULL;\r
ZeroMem (&PackageHeader, sizeof (EFI_HII_PACKAGE_HEADER));\r
\r
//\r
//\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
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 (FormSetGuid) ||\r
- CompareGuid (ComparingGuid, (EFI_GUID *)(OpCodeData + sizeof (EFI_IFR_OP_HEADER)))) {\r
+ if (IsZeroGuid (ComparingGuid) ||\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
}\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
//\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
// 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
return EFI_SUCCESS;\r
}\r
\r
-\r
/**\r
Initialize the internal data structure of a FormSet.\r
\r
**/\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
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
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
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
//\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
InsertHeadList (&gBrowserContextList, &Context->Link);\r
}\r
\r
-\r
/**\r
Restore globals used by previous call to SendForm().\r
\r
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
\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
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
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
**/\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
**/\r
BOOLEAN\r
IsHiiHandleInBrowserContext (\r
- EFI_HII_HANDLE Handle\r
+ EFI_HII_HANDLE Handle\r
)\r
{\r
LIST_ENTRY *Link;\r
//\r
return TRUE;\r
}\r
+\r
Link = GetNextNode (&gBrowserContextList, Link);\r
}\r
\r
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
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
**/\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
if (HotKey->KeyData->ScanCode == KeyData->ScanCode) {\r
return HotKey;\r
}\r
+\r
Link = GetNextNode (&gBrowserHotKeyList, Link);\r
}\r
\r
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
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
//\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
//\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
VOID\r
EFIAPI\r
RegiserExitHandler (\r
- IN EXIT_HANDLER Handler\r
+ IN EXIT_HANDLER Handler\r
)\r
{\r
ExitHandlerFunction = Handler;\r
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
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
if (EFI_ERROR (Status)) {\r
return Status;\r
}\r
+\r
UpdateStatementStatus (FormSet, Form, gBrowserSettingScope);\r
}\r
\r
// Executet the reset action.\r
//\r
if ((Action & BROWSER_ACTION_RESET) != 0) {\r
- gResetRequiredFormLevel = TRUE;\r
+ gResetRequiredFormLevel = TRUE;\r
gResetRequiredSystemLevel = TRUE;\r
}\r
\r
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
// 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
{\r
return gResetRequiredSystemLevel;\r
}\r
-\r