]> git.proxmox.com Git - mirror_edk2.git/commitdiff
Refine function comments for BdsDxe module.
authorklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 26 Dec 2008 02:22:37 +0000 (02:22 +0000)
committerklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 26 Dec 2008 02:22:37 +0000 (02:22 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@7127 6f19259b-4bc3-4df7-8a09-765794883524

MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
MdeModulePkg/Universal/BdsDxe/BootMaint/FileExplorer.c
MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.c
MdeModulePkg/Universal/BdsDxe/Capsules.c
MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.c
MdeModulePkg/Universal/BdsDxe/FrontPage.c
MdeModulePkg/Universal/BdsDxe/Hotkey.c
MdeModulePkg/Universal/BdsDxe/HwErrRecSupport.c

index b9ffa056dacf372ab4aa06ff7811809e8588214a..0fc187aca39a796281ae7defc930d3837c0fa27d 100644 (file)
@@ -18,7 +18,6 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
   Function creates a device path data structure that identically matches the\r
   device path passed in.\r
 \r
-\r
   @param DevPath         A pointer to a device path data structure.\r
 \r
   @return        The new copy of DevPath is created to identically match the input.\r
@@ -33,7 +32,6 @@ DevicePathInstanceDup (
 /**\r
   Update Com Ports attributes from DevicePath\r
 \r
-\r
   @param DevicePath      DevicePath that contains Com ports\r
 \r
   @retval EFI_SUCCESS   The update is successful.\r
index 8a946d3e42256bd02757e6b7a6405d602210f743..8253ded9882ba3ef489d7ae5696c7c9f19708442 100644 (file)
@@ -232,7 +232,7 @@ UpdateFileExplorer (
   @retval  EFI_OUT_OF_RESOURCES  Not enough storage is available to hold the variable and its data.\r
   @retval  EFI_DEVICE_ERROR      The variable could not be saved.\r
   @retval  EFI_UNSUPPORTED       The specified Action is not supported by the callback.\r
-\r
+  @retval  EFI_INVALID_PARAMETER If paramter Value or ActionRequest is NULL.\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -259,7 +259,7 @@ FileExplorerCallback (
   *ActionRequest = EFI_BROWSER_ACTION_REQUEST_NONE;\r
 \r
   //\r
-  // Retrive uncommitted data from Form Browser\r
+  // Retrieve uncommitted data from Form Browser\r
   //\r
   NvRamMap = &Private->FeFakeNvData;\r
   BufferSize = sizeof (FILE_EXPLORER_NV_DATA);\r
index 2ffb1ff037a7259751f37598e72dff5ade905175..486f2fdcfc52ba786da688c3878cc343475be322 100644 (file)
@@ -114,11 +114,9 @@ UpdatePageEnd (
 }\r
 \r
 /**\r
-  Clean up the dynamic opcode at label and form specified by\r
-  both LabelId. \r
+  Clean up the dynamic opcode at label and form specified by both LabelId. \r
 \r
-  @param LabelId         It is both the Form ID and Label ID for\r
-                         opcode deletion.\r
+  @param LabelId         It is both the Form ID and Label ID for opcode deletion.\r
   @param CallbackData    The BMM context data.\r
 \r
 **/\r
@@ -150,7 +148,7 @@ CleanUpPage (
                          of the file to be boot from.\r
 \r
   @retval EFI_SUCCESS    The function completed successfull.\r
-  @return                 Other value if the boot from the file fails.\r
+  @return Other value if the boot from the file fails.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -163,7 +161,7 @@ BootThisFile (
   CHAR16            *ExitData;\r
   BDS_COMMON_OPTION *Option;\r
 \r
-  Status                  = gBS->AllocatePool (EfiBootServicesData, sizeof (BDS_COMMON_OPTION), (VOID **) &Option);\r
+  Option = (BDS_COMMON_OPTION *) AllocatePool (sizeof (BDS_COMMON_OPTION));\r
   Option->Description     = FileContext->FileName;\r
   Option->DevicePath      = FileContext->DevicePath;\r
   Option->LoadOptionsSize = 0;\r
@@ -562,7 +560,7 @@ UpdateOrderPage (
 \r
 /**\r
   Create the dynamic page to allow user to set\r
-  the "BootNext" vaule.\r
+  the "BootNext" value.\r
 \r
   @param CallbackData    The BMM context data.\r
 \r
@@ -633,8 +631,7 @@ UpdateBootNextPage (
 }\r
 \r
 /**\r
-  Create the dynamic page to allow user to set\r
-  the "TimeOut" vaule.\r
+  Create the dynamic page to allow user to set the "TimeOut" value.\r
 \r
   @param CallbackData    The BMM context data.\r
 \r
@@ -673,7 +670,7 @@ UpdateTimeOutPage (
 }\r
 \r
 /**\r
-  Refresh the text mode page\r
+  Refresh the text mode page.\r
 \r
   @param CallbackData    The BMM context data.\r
 \r
@@ -778,8 +775,7 @@ UpdateConModePage (
 }\r
 \r
 /**\r
-  Create the dynamic page which allows user to \r
-  set the property such as Baud Rate, Data Bits,\r
+  Create the dynamic page which allows user to set the property such as Baud Rate, Data Bits,\r
   Parity, Stop Bits, Terminal Type.\r
 \r
   @param CallbackData    The BMM context data.\r
index 8576e41f0fe99a6a0b05978e752d159495ab9f42..55af813d6892dde83b27795a8d3ca7c441aa4ffa 100644 (file)
@@ -20,7 +20,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
   make sure BootOrder is in valid state.\r
 \r
   @retval EFI_SUCCESS   If all boot load option EFI Variables corresponding to  \r
-                        BM_LOAD_CONTEXT marked for deletion is deleted\r
+                        BM_LOAD_CONTEXT marked for deletion is deleted.\r
+  @retval EFI_NOT_FOUND If can not find the boot option want to be deleted.\r
   @return Others        If failed to update the "BootOrder" variable after deletion. \r
 \r
 **/\r
@@ -176,7 +177,8 @@ Var_ChangeBootOrder (
   After deleting this Driver option, call Var_ChangeDriverOrder to\r
   make sure DriverOrder is in valid state.\r
 \r
-  @retval EFI_SUCCESS Load Option is successfully updated.\r
+  @retval EFI_SUCCESS       Load Option is successfully updated.\r
+  @retval EFI_NOT_FOUND     Fail to find the driver option want to be deleted.\r
   @return Other value than EFI_SUCCESS if failed to update "Driver Order" EFI\r
           Variable.\r
 \r
@@ -476,8 +478,7 @@ Var_UpdateConsoleOption (
   console device.\r
 \r
   @retval EFI_SUCCESS    The function complete successfully.\r
-  @return                The EFI variable can be saved. See gRT->SetVariable \r
-                         for detail return information.\r
+  @return The EFI variable can not be saved. See gRT->SetVariable for detail return information.\r
 **/\r
 EFI_STATUS\r
 Var_UpdateConsoleInpOption (\r
@@ -492,8 +493,7 @@ Var_UpdateConsoleInpOption (
   console device.\r
 \r
   @retval EFI_SUCCESS    The function complete successfully.\r
-  @return                The EFI variable can be saved. See gRT->SetVariable \r
-                         for detail return information.\r
+  @return The EFI variable can not be saved. See gRT->SetVariable for detail return information.\r
 **/\r
 EFI_STATUS\r
 Var_UpdateConsoleOutOption (\r
@@ -508,8 +508,7 @@ Var_UpdateConsoleOutOption (
   console device.\r
 \r
   @retval EFI_SUCCESS    The function complete successfully.\r
-  @return                The EFI variable can be saved. See gRT->SetVariable \r
-                         for detail return information.\r
+  @return The EFI variable can not be saved. See gRT->SetVariable for detail return information.  \r
 **/\r
 EFI_STATUS\r
 Var_UpdateErrorOutOption (\r
@@ -586,6 +585,7 @@ Var_UpdateDriverOption (
 \r
   NewMenuEntry = BOpt_CreateMenuEntry (BM_LOAD_CONTEXT_SELECT);\r
   if (NULL == NewMenuEntry) {\r
+    FreePool (Buffer);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
@@ -1007,10 +1007,9 @@ Var_UpdateBootOrder (
 \r
   @param CallbackData    The BMM context data.\r
 \r
-  @retval EFI_SUCCESS    The function complete successfully.\r
-  @retval EFI_SUCCESS    Not enough memory to complete the function.\r
-  @return                The EFI variable can be saved. See gRT->SetVariable \r
-                         for detail return information.\r
+  @retval EFI_SUCCESS           The function complete successfully.\r
+  @retval EFI_OUT_OF_RESOURCES  Not enough memory to complete the function.\r
+  @return The EFI variable can not be saved. See gRT->SetVariable for detail return information.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1076,9 +1075,9 @@ Var_UpdateDriverOrder (
 \r
   @param CallbackData    The context data for BMM.\r
 \r
-  @return EFI_SUCCESS    The function completed successfully.\r
-  @retval EFI_NOT_FOUND  If L"LegacyDevOrder" and EfiLegacyDevOrderGuid EFI Variable can be found.\r
-\r
+  @return EFI_SUCCESS           The function completed successfully.\r
+  @retval EFI_NOT_FOUND         If L"LegacyDevOrder" and EfiLegacyDevOrderGuid EFI Variable can be found.\r
+  @retval EFI_OUT_OF_RESOURCES  Fail to allocate memory resource\r
 **/\r
 EFI_STATUS\r
 Var_UpdateBBSOption (\r
@@ -1169,7 +1168,7 @@ Var_UpdateBBSOption (
                         &VarSize\r
                         );\r
 \r
-  if (NULL == VarData) {\r
+  if (VarData == NULL) {\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
@@ -1192,7 +1191,7 @@ Var_UpdateBBSOption (
   }\r
 \r
   NewOrder = (UINT16 *) AllocateZeroPool (DevOrder->Length - sizeof (UINT16));\r
-  if (NULL == NewOrder) {\r
+  if (NewOrder == NULL) {\r
     FreePool (VarData);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -1243,7 +1242,7 @@ Var_UpdateBBSOption (
   //\r
   BootOptionVar = GetLegacyBootOptionVar (CallbackData->BbsType, &Index, &OptionSize);\r
 \r
-  if (NULL != BootOptionVar) {\r
+  if (BootOptionVar != NULL) {\r
     CopyMem (\r
       DescString,\r
       LegacyDeviceContext->Description,\r
@@ -1263,7 +1262,7 @@ Var_UpdateBBSOption (
 \r
     Attribute = (UINT32 *) Ptr;\r
     *Attribute |= LOAD_OPTION_ACTIVE;\r
-    if (0xFF == LegacyDev[0]) {\r
+    if (LegacyDev[0] == 0xFF) {\r
       //\r
       // Disable this legacy boot option\r
       //\r
@@ -1276,7 +1275,7 @@ Var_UpdateBBSOption (
     Ptr += StrSize ((CHAR16 *) Ptr);\r
 \r
     NewOptionPtr = AllocateZeroPool (NewOptionSize);\r
-    if (NULL == NewOptionPtr) {\r
+    if (NewOptionPtr == NULL) {\r
       return EFI_OUT_OF_RESOURCES;\r
     }\r
 \r
index 459a396a697419352f06eb0a7d103a4bdb168d6a..d0ac2fc1dd8b0d91b03bdc2e8798cad19d726889 100644 (file)
@@ -1,5 +1,5 @@
 /** @file\r
-  The platform boot manager reference implement\r
+  The platform boot manager reference implementation\r
 \r
 Copyright (c) 2004 - 2008, Intel Corporation. <BR>\r
 All rights reserved. This program and the accompanying materials\r
index 0ecf3a6f037ec1409d791143062e78361d1654bb..904a775493a7b61ad8e4fe21498ec858885ade0e 100644 (file)
@@ -70,17 +70,16 @@ BdsLockFv (
   Then call the dispatcher to dispatch drivers from them. Finally, check\r
   the status of the updates.\r
 \r
+  This function should be called by BDS in case we need to do some\r
+  sort of processing even if there is no capsule to process. We\r
+  need to do this if an earlier update went away and we need to\r
+  clear the capsule variable so on the next reset PEI does not see it and\r
+  think there is a capsule available.\r
 \r
-  @param BootMode        - the current boot mode\r
+  @param BootMode                 the current boot mode\r
 \r
-  @retval  EFI_INVALID_PARAMETER  boot mode is not correct for an update\r
-                                  Note:\r
-                                  This function should be called by BDS in case we need to do some\r
-                                  sort of processing even if there is no capsule to process. We\r
-                                  need to do this if an earlier update went awry and we need to\r
-                                  clear the capsule variable so on the next reset PEI does not see it and\r
-                                  think there is a capsule available.\r
-  @retval EFI_SUCCESS There is no error when processing capsule\r
+  @retval EFI_INVALID_PARAMETER   boot mode is not correct for an update\r
+  @retval EFI_SUCCESS             There is no error when processing capsule\r
 \r
 **/\r
 EFI_STATUS\r
index 5621efde6f073d6f3d8f6c4afb669381121dbb9b..f0e07ecc212765d533555ea0c9c08a4b24714b6b 100644 (file)
@@ -1,5 +1,5 @@
 /** @file\r
-  The platform device manager reference implement\r
+  The platform device manager reference implementation\r
 \r
 Copyright (c) 2004 - 2008, Intel Corporation. <BR>\r
 All rights reserved. This program and the accompanying materials\r
@@ -44,7 +44,6 @@ DEVICE_MANAGER_MENU_ITEM  mDeviceManagerMenuItemTable[] = {
   Formset. The decision by user is saved to gCallbackKey for later processing. If\r
   user set VBIOS, the new value is saved to EFI variable.\r
 \r
-\r
   @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
   @param Action          Specifies the type of action taken by the browser.\r
   @param QuestionId      A unique value which is sent to the original exporting driver\r
@@ -163,7 +162,6 @@ InitializeDeviceManager (
 }\r
 \r
 /**\r
-\r
   Call the browser and display the device manager to allow user\r
   to configure the platform.\r
 \r
@@ -171,7 +169,7 @@ InitializeDeviceManager (
   section header for all class of devices, one-of opcode to set VBIOS.\r
   \r
   @retval  EFI_SUCCESS             Operation is successful.\r
-  @retval  Other values if failed to clean up the dynamic content from HII\r
+  @return  Other values if failed to clean up the dynamic content from HII\r
            database.\r
 \r
 **/\r
@@ -306,7 +304,7 @@ CallDeviceManager (
                   &gEfiGenericPlatformVariableGuid,\r
                   &VideoOptionSize\r
                   );\r
-  if (NULL == VideoOption) {\r
+  if (VideoOption == NULL) {\r
     gDeviceManagerPrivate.VideoBios = 0;\r
   } else {\r
     gDeviceManagerPrivate.VideoBios = VideoOption[0];\r
index 570dcd8754e4086adef48dfa4a04a8e77836a42f..44a1e0868932ac6b7d0d830cbf5c0142b255ff76 100644 (file)
@@ -42,14 +42,14 @@ FRONT_PAGE_CALLBACK_DATA  gFrontPagePrivate = {
   or more named elements from the target driver.\r
 \r
 \r
-  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
-  @param Request         A null-terminated Unicode string in <ConfigRequest> format.\r
-  @param Progress        On return, points to a character in the Request string.\r
+  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
+  @param Request         A null-terminated Unicode string in <ConfigRequest> format.\r
+  @param Progress        On return, points to a character in the Request string.\r
                          Points to the string's null terminator if request was successful.\r
                          Points to the most recent '&' before the first failing name/value\r
                          pair (or the beginning of the string if the failure is in the\r
                          first name/value pair) if the request was not successful.\r
-  @param Results         A null-terminated Unicode string in <ConfigAltResp> format which\r
+  @param Results         A null-terminated Unicode string in <ConfigAltResp> format which\r
                          has all values filled in for the names in the Request string.\r
                          String to be allocated by the called function.\r
 \r
@@ -75,9 +75,9 @@ FakeExtractConfig (
   This function processes the results of changes in configuration.\r
 \r
 \r
-  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
-  @param Configuration   A null-terminated Unicode string in <ConfigResp> format.\r
-  @param Progress        A pointer to a string filled in with the offset of the most\r
+  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
+  @param Configuration   A null-terminated Unicode string in <ConfigResp> format.\r
+  @param Progress        A pointer to a string filled in with the offset of the most\r
                          recent '&' before the first failing name/value pair (or the\r
                          beginning of the string if the failure is in the first\r
                          name/value pair) or the terminating NULL if all was successful.\r
@@ -102,13 +102,13 @@ FakeRouteConfig (
   This function processes the results of changes in configuration.\r
 \r
 \r
-  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
-  @param Action          Specifies the type of action taken by the browser.\r
-  @param QuestionId      A unique value which is sent to the original exporting driver\r
+  @param This            Points to the EFI_HII_CONFIG_ACCESS_PROTOCOL.\r
+  @param Action          Specifies the type of action taken by the browser.\r
+  @param QuestionId      A unique value which is sent to the original exporting driver\r
                          so that it can identify the type of data to expect.\r
-  @param Type            The type of value for the question.\r
-  @param Value           A pointer to the data being sent to the original exporting driver.\r
-  @param ActionRequest   On return, points to the action requested by the callback function.\r
+  @param Type            The type of value for the question.\r
+  @param Value           A pointer to the data being sent to the original exporting driver.\r
+  @param ActionRequest   On return, points to the action requested by the callback function.\r
 \r
   @retval  EFI_SUCCESS           The callback successfully handled the action.\r
   @retval  EFI_OUT_OF_RESOURCES  Not enough storage is available to hold the variable and its data.\r
@@ -491,9 +491,9 @@ CallFrontPage (
   Acquire the string associated with the ProducerGuid and return it.\r
 \r
 \r
-  @param ProducerGuid    The Guid to search the HII database for\r
-  @param Token           The token value of the string to extract\r
-  @param String          The string that is extracted\r
+  @param ProducerGuid    The Guid to search the HII database for\r
+  @param Token           The token value of the string to extract\r
+  @param String          The string that is extracted\r
 \r
   @retval  EFI_SUCCESS  The function returns EFI_SUCCESS always.\r
 \r
@@ -518,11 +518,8 @@ GetProducerString (
 /**\r
   Convert Processor Frequency Data to a string\r
 \r
-\r
-  @param ProcessorFrequency - The frequency data to process\r
-  @param String          - The string that is created\r
-\r
-  @return VOID.\r
+  @param ProcessorFrequency The frequency data to process\r
+  @param String             The string that is created\r
 \r
 **/\r
 VOID\r
@@ -559,9 +556,8 @@ ConvertProcessorToString (
 /**\r
   Convert Memory Size to a string.\r
 \r
-\r
-  @param MemorySize      - The size of the memory to process\r
-  @param String          - The string that is created\r
+  @param MemorySize      The size of the memory to process\r
+  @param String          The string that is created\r
 \r
 **/\r
 VOID\r
@@ -689,13 +685,11 @@ UpdateFrontPageStrings (
 /**\r
   Function waits for a given event to fire, or for an optional timeout to expire.\r
 \r
+  @param Event              The event to wait for\r
+  @param Timeout            An optional timeout value in 100 ns units.\r
 \r
-  @param Event           The event to wait for\r
-                        \r
-  @param Timeout         An optional timeout value in 100 ns units.\r
-\r
-  @retval  EFI_SUCCESS       Event fired before Timeout expired.\r
-  @retval  EFI_TIME_OUT      Timout expired before Event fired..\r
+  @retval  EFI_SUCCESS      Event fired before Timeout expired.\r
+  @retval  EFI_TIME_OUT     Timout expired before Event fired..\r
 \r
 **/\r
 EFI_STATUS\r
@@ -755,8 +749,7 @@ WaitForSingleEvent (
   Function show progress bar to wait for user input.\r
 \r
 \r
-  @param TimeoutDefault  - The fault time out value before the system\r
-                         continue to boot.\r
+  @param TimeoutDefault  The fault time out value before the system continue to boot.\r
 \r
   @retval  EFI_SUCCESS       User pressed some key except "Enter"\r
   @retval  EFI_TIME_OUT      Timout expired or user press "Enter"\r
@@ -850,12 +843,10 @@ ShowProgress (
   this is the platform reference part and can be customize.\r
 \r
 \r
-  @param TimeoutDefault  - The fault time out value before the system\r
-                         continue to boot.\r
-  @param ConnectAllHappened - The indicater to check if the connect all have\r
-                         already happended.\r
-\r
-  @return VOID.\r
+  @param TimeoutDefault     The fault time out value before the system\r
+                            continue to boot.\r
+  @param ConnectAllHappened The indicater to check if the connect all have\r
+                            already happended.\r
 \r
 **/\r
 VOID\r
index 96ce6d7238fb5ea2c8071a4f572fbee3edbed245..9efba6d730cbeb209d5ff7ee93a5de6f436d3205 100644 (file)
@@ -23,10 +23,8 @@ VOID            *mHotkeyRegistration;
 \r
 \r
 /**\r
-\r
   Check if the Key Option is valid or not.\r
 \r
-\r
   @param KeyOption       The Hot Key Option to be checked.\r
 \r
   @retval  TRUE          The Hot Key Option is valid.\r
@@ -67,15 +65,14 @@ IsKeyOptionValid (
 }\r
 \r
 /**\r
-\r
   Create Key#### for the given hotkey.\r
 \r
-\r
   @param KeyOption       The Hot Key Option to be added.\r
   @param KeyOptionNumber The key option number for Key#### (optional).\r
 \r
   @retval  EFI_SUCCESS            Register hotkey successfully.\r
   @retval  EFI_INVALID_PARAMETER  The hotkey option is invalid.\r
+  @retval  EFI_OUT_OF_RESOURCES   Fail to allocate memory resource.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -110,7 +107,6 @@ RegisterHotkey (
   //\r
   // check whether HotKey conflict with keys used by Setup Browser\r
   //\r
-\r
   KeyOrder = BdsLibGetVariableAndSize (\r
                VAR_KEY_ORDER,\r
                &gEfiGlobalVariableGuid,\r
@@ -187,7 +183,7 @@ RegisterHotkey (
                   KeyOption\r
                   );\r
   if (EFI_ERROR (Status)) {\r
-    gBS->FreePool (KeyOrder);\r
+    FreePool (KeyOrder);\r
     return Status;\r
   }\r
 \r
@@ -201,6 +197,7 @@ RegisterHotkey (
 \r
   NewKeyOrder = AllocatePool (KeyOrderSize);\r
   if (NewKeyOrder == NULL) {\r
+    FreePool (KeyOrder);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
 \r
@@ -307,11 +304,10 @@ UnregisterHotkey (
   This is the common notification function for HotKeys, it will be registered\r
   with SimpleTextInEx protocol interface - RegisterKeyNotify() of ConIn handle.\r
 \r
-\r
   @param KeyData         A pointer to a buffer that is filled in with the keystroke\r
                          information for the key that was pressed.\r
 \r
-  @retval  EFI_SUCCESS            KeyData is successfully processed.\r
+  @retval  EFI_SUCCESS   KeyData is successfully processed.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -427,10 +423,8 @@ HotkeyCallback (
 }\r
 \r
 /**\r
-\r
   Register the common HotKey notify function to given SimpleTextInEx protocol instance.\r
 \r
-\r
   @param SimpleTextInEx  Simple Text Input Ex protocol instance\r
 \r
   @retval  EFI_SUCCESS            Register hotkey notification function successfully.\r
@@ -481,12 +475,9 @@ HotkeyRegisterNotify (
 /**\r
   Callback function for SimpleTextInEx protocol install events\r
 \r
-\r
   @param Event           the event that is signaled.\r
   @param Context         not used here.\r
 \r
-  @return VOID\r
-\r
 **/\r
 VOID\r
 EFIAPI\r
@@ -528,14 +519,12 @@ HotkeyEvent (
 }\r
 \r
 /**\r
-\r
   Insert Key Option to hotkey list.\r
 \r
-\r
   @param KeyOption       The Hot Key Option to be added to hotkey list.\r
 \r
-  @retval  EFI_SUCCESS  Add to hotkey list success.\r
-\r
+  @retval EFI_SUCCESS           Add to hotkey list success.\r
+  @retval EFI_OUT_OF_RESOURCES  Fail to allocate memory resource.\r
 **/\r
 EFI_STATUS\r
 HotkeyInsertList (\r
index 9ea6dbc554012e241c53e3d8d02ba07ef46d5b1c..3b48c370ce37beb8134ee556f555ada8b8c55651 100644 (file)
@@ -22,10 +22,10 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 \r
   @param HwErrRecSupportLevel\r
-                         zero value      - Indicates that the platform implements no support for\r
-                         Hardware Error Record Persistence.\r
-                         non-zero value  - Indicates that the platform implements Hardware Error\r
-                         Record Persistence.\r
+                         zero value:      Indicates that the platform implements no support for\r
+                                          Hardware Error Record Persistence.\r
+                         non-zero value Indicates that the platform implements Hardware Error\r
+                                          Record Persistence.\r
 \r
 **/\r
 VOID\r