]> git.proxmox.com Git - mirror_edk2.git/blobdiff - IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h
Committing changes to the comments, to improve code documentation.
[mirror_edk2.git] / IntelFrameworkModulePkg / Include / Library / GenericBdsLib.h
index a57fe6c8b4447288392378e838b306230e60ca79..0692b9c50cace51b3d791cf93e5a3e3a261c07f0 100644 (file)
@@ -1,7 +1,10 @@
 /** @file\r
-  Generic BDS library definition, include the data structure and function.\r
+  Generic BDS library defines general interfaces for BDS driver including:\r
+    1) BDS boot policy interface\r
+    2) BDS boot device connect interface\r
+    3) BDS Misc interfaces for mainting boot variable, ouput string.\r
 \r
-Copyright (c) 2004 - 2008, Intel Corporation. <BR>\r
+Copyright (c) 2004 - 2009, Intel Corporation. <BR>\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -15,34 +18,30 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #ifndef _GENERIC_BDS_LIB_H_\r
 #define _GENERIC_BDS_LIB_H_\r
 \r
-#include <Protocol/HiiDatabase.h>\r
-#include <IndustryStandard/PeImage.h>\r
-\r
-\r
 extern EFI_HANDLE mBdsImageHandle;\r
 \r
-//\r
-// Constants which are variable names used to access variables\r
-//\r
+///\r
+/// Constants which are variable names used to access variables\r
+///\r
 #define VAR_LEGACY_DEV_ORDER L"LegacyDevOrder"\r
 \r
-//\r
-// Data structures and defines\r
-//\r
+///\r
+/// Data structures and defines\r
+///\r
 #define FRONT_PAGE_QUESTION_ID  0x0000\r
 #define FRONT_PAGE_DATA_WIDTH   0x01\r
 \r
-//\r
-// ConnectType\r
-//\r
+///\r
+/// ConnectType\r
+///\r
 #define CONSOLE_OUT 0x00000001\r
 #define STD_ERROR   0x00000002\r
 #define CONSOLE_IN  0x00000004\r
 #define CONSOLE_ALL (CONSOLE_OUT | CONSOLE_IN | STD_ERROR)\r
 \r
-//\r
-// Load Option Attributes defined in EFI Specification\r
-//\r
+///\r
+/// Load Option Attributes\r
+///\r
 #define LOAD_OPTION_ACTIVE              0x00000001\r
 #define LOAD_OPTION_FORCE_RECONNECT     0x00000002\r
 \r
@@ -57,22 +56,19 @@ extern EFI_HANDLE mBdsImageHandle;
 \r
 #define IS_LOAD_OPTION_TYPE(_c, _Mask)  (BOOLEAN) (((_c) & (_Mask)) != 0)\r
 \r
-//\r
-// Define Maxmim characters that will be accepted\r
-//\r
+///\r
+/// Define Maximum characters that will be accepted\r
+///\r
 #define MAX_CHAR            480\r
 #define MAX_CHAR_SIZE       (MAX_CHAR * 2)\r
 \r
-#define MIN_ALIGNMENT_SIZE  4\r
-#define ALIGN_SIZE(a)       ((a % MIN_ALIGNMENT_SIZE) ? MIN_ALIGNMENT_SIZE - (a % MIN_ALIGNMENT_SIZE) : 0)\r
-\r
-//\r
-// Define maximum characters for boot option variable "BootXXXX"\r
-//\r
+///\r
+/// Define maximum characters for boot option variable "BootXXXX"\r
+///\r
 #define BOOT_OPTION_MAX_CHAR 10\r
 \r
 //\r
-// This data structure is the part of BDS_CONNECT_ENTRY that we can hard code.\r
+// This data structure is the part of BDS_CONNECT_ENTRY\r
 //\r
 #define BDS_LOAD_OPTION_SIGNATURE SIGNATURE_32 ('B', 'd', 'C', 'O')\r
 \r
@@ -99,10 +95,6 @@ typedef struct {
   UINTN                     ConnectType;\r
 } BDS_CONSOLE_CONNECT_ENTRY;\r
 \r
-//\r
-// Lib Functions\r
-//\r
-\r
 //\r
 // Bds boot related lib functions\r
 //\r
@@ -120,15 +112,15 @@ BdsLibBootNext (
   Process the boot option follow the UEFI specification and\r
   special treat the legacy boot option with BBS_DEVICE_PATH.\r
 \r
-  @param  Option                 The boot option need to be processed\r
-  @param  DevicePath             The device path which describe where to load the\r
+  @param  Option                 The boot option to be processed\r
+  @param  DevicePath             The device path describing where to load the\r
                                  boot image or the legcy BBS device path to boot\r
                                  the legacy OS\r
   @param  ExitDataSize           The size of exit data.\r
   @param  ExitData               Data returned when Boot image failed.\r
 \r
-  @retval EFI_SUCCESS            Boot from the input boot option successfully.\r
-  @retval EFI_NOT_FOUND          If the Device Path is not found in the system\r
+  @retval EFI_SUCCESS            Boot from the input boot option succeeded.\r
+  @retval EFI_NOT_FOUND          The Device Path is not found in the system\r
 \r
 **/\r
 EFI_STATUS\r
@@ -142,15 +134,52 @@ BdsLibBootViaBootOption (
 \r
 \r
 /**\r
-  This function will enumerate all possible boot device in the system,\r
-  it will only excute once of every boot.\r
-\r
-  @param  BdsBootOptionList      The header of the link list which indexed all\r
+  This function will enumerate all possible boot devices in the system, and\r
+  automatically create boot options for Network, Shell, Removable BlockIo, \r
+  and Non-BlockIo Simplefile devices. \r
+  \r
+  BDS separates EFI boot options into six types:\r
+  1. Network - The boot option points to the SimpleNetworkProtocol device. \r
+               Bds will try to automatically create this type of boot option during enumeration.\r
+  2. Shell   - The boot option points to internal flash shell. \r
+               Bds will try to automatically create this type of boot option during enumeration.\r
+  3. Removable BlockIo      - The boot option points to a removable media\r
+                              device, such as a USB flash drive or DVD drive.\r
+                              These devices should contain a *removable* blockIo\r
+                              protocol in their device handle.\r
+                              Bds will try to automatically create this type boot option \r
+                              when enumerate.\r
+  4. Fixed BlockIo          - The boot option points to a Fixed blockIo device, \r
+                              such as a hard disk.\r
+                              These devices should contain a *fixed* blockIo\r
+                              protocol in their device handle.\r
+                              BDS will skip fixed blockIo devices, and not\r
+                              automatically create boot option for them. But BDS \r
+                              will help to delete those fixed blockIo boot options, \r
+                              whose description rules conflict with other auto-created\r
+                              boot options.\r
+  5. Non-BlockIo Simplefile - The boot option points to a device whose handle \r
+                              has SimpleFileSystem Protocol, but has no blockio\r
+                              protocol. These devices do not offer blockIo\r
+                              protocol, but BDS still can get the \r
+                              \EFI\BOOT\boot{machinename}.EFI by SimpleFileSystem\r
+                              Protocol.\r
+  6. File    - The boot option points to a file. These boot options are usually \r
+               created by the user, either manually or with an OS loader. BDS will not delete or modify\r
+               these boot options.        \r
+    \r
+  This function will enumerate all possible boot devices in the system, and\r
+  automatically create boot options for Network, Shell, Removable BlockIo, \r
+  and Non-BlockIo Simplefile devices.\r
+  It will excute once every boot.\r
+  \r
+  @param  BdsBootOptionList      The header of the linked list that indexed all\r
                                  current boot options\r
 \r
-  @retval EFI_SUCCESS            Finished all the boot device enumerate and create\r
-                                 the boot option base on that boot device\r
+  @retval EFI_SUCCESS            Finished all the boot device enumerations and created\r
+                                 the boot option based on the boot device\r
 \r
+  @retval EFI_OUT_OF_RESOURCES   Failed to enumerate the boot device and create the boot option list\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -161,7 +190,7 @@ BdsLibEnumerateAllBootOption (
 /**\r
   Build the boot option with the handle parsed in\r
 \r
-  @param  Handle                 The handle which present the device path to create\r
+  @param  Handle                 The handle representing the device path for which to create\r
                                  boot option\r
   @param  BdsBootOptionList      The header of the link list which indexed all\r
                                  current boot options\r
@@ -196,18 +225,6 @@ BdsLibBuildOptionFromShell (
 //\r
 // Bds misc lib functions\r
 //\r
-/**\r
-  Return the default value for system Timeout variable.\r
-\r
-  @return Timeout value.\r
-\r
-**/\r
-UINT16\r
-EFIAPI\r
-BdsLibGetTimeout (\r
-  VOID\r
-  );\r
-\r
 /**\r
   Get boot mode by looking up configuration table and parsing HOB list\r
 \r
@@ -224,8 +241,8 @@ BdsLibGetBootMode (
 \r
 \r
 /**\r
-  The function will go through the driver optoin link list, load and start\r
-  every driver the driver optoin device path point to.\r
+  The function will go through the driver option link list, and then load and start\r
+  every driver to which the driver option device path points.\r
 \r
   @param  BdsDriverLists        The header of the current driver option link list\r
 \r
@@ -238,19 +255,11 @@ BdsLibLoadDrivers (
 \r
 \r
 /**\r
-  Process BootOrder, or DriverOrder variables, by calling\r
+  This function processes BootOrder or DriverOrder variables, by calling\r
   BdsLibVariableToOption () for each UINT16 in the variables.\r
 \r
-  @param  BdsCommonOptionList   The header of the option list base on variable\r
-                                VariableName\r
-  @param  VariableName          EFI Variable name indicate the BootOrder or\r
-                                DriverOrder\r
-\r
-  @retval EFI_SUCCESS           Success create the boot option or driver option\r
-                                list\r
-  @retval EFI_OUT_OF_RESOURCES  Failed to get the boot option or driver option list\r
-\r
-**/\r
+  @param  BdsCommonOptionList   The header of the option list base on variable VariableName.  @param  VariableName          EFI Variable name indicate the BootOrder or DriverOrder.\r
+  @retval EFI_SUCCESS           Successfully created the boot option or driver option                                list  @retval EFI_OUT_OF_RESOURCES  Failed to get the boot option or driver option list**/\r
 EFI_STATUS\r
 EFIAPI\r
 BdsLibBuildOptionFromVar (\r
@@ -259,8 +268,8 @@ BdsLibBuildOptionFromVar (
   );\r
 \r
 /**\r
-  Read the EFI variable (VendorGuid/Name) and return a dynamically allocated\r
-  buffer, and the size of the buffer. If failure return NULL.\r
+  This function reads the EFI variable (VendorGuid/Name) and returns a dynamically allocated\r
+  buffer and the size of the buffer. If failure, return NULL.\r
 \r
   @param  Name                  String part of EFI variable name\r
   @param  VendorGuid            GUID part of EFI variable name\r
@@ -299,15 +308,15 @@ BdsLibOutputStrings (
   );\r
 \r
 /**\r
-  Build the boot#### or driver#### option from the VariableName, the\r
+  Build the boot#### or driver#### option from the VariableName. The\r
   build boot#### or driver#### will also be linked to BdsCommonOptionList.\r
 \r
   @param  BdsCommonOptionList   The header of the boot#### or driver#### option\r
                                 link list\r
-  @param  VariableName          EFI Variable name indicate if it is boot#### or\r
+  @param  VariableName          EFI Variable name, indicates if it is boot#### or\r
                                 driver####\r
 \r
-  @retval BDS_COMMON_OPTION     Get the option just been created\r
+  @retval BDS_COMMON_OPTION     The option that was created\r
   @retval NULL                  Failed to get the new option\r
 \r
 **/\r
@@ -319,7 +328,7 @@ BdsLibVariableToOption (
   );\r
 \r
 /**\r
-  This function will register the new boot#### or driver#### option base on\r
+  This function registers the new boot#### or driver#### option based on\r
   the VariableName. The new registered boot#### or driver#### will be linked\r
   to BdsOptionList and also update to the VariableName. After the boot#### or\r
   driver#### updated, the BootOrder or DriverOrder will also be updated.\r
@@ -345,13 +354,10 @@ BdsLibRegisterNewOption (
   );\r
 \r
 //\r
-// Bds connect or disconnect driver lib funcion\r
+// Bds connect and disconnect driver lib funcions\r
 //\r
 /**\r
-  Connects all drivers to all controllers.\r
-  This function make sure all the current system driver will manage\r
-  the correspoinding controllers if have. And at the same time, make\r
-  sure all the system controllers have driver to manage it if have.\r
+  This function connects all system drivers with the corresponding controllers. \r
 \r
 **/\r
 VOID\r
@@ -361,10 +367,7 @@ BdsLibConnectAllDriversToAllControllers (
   );\r
 \r
 /**\r
-  This function will connect all the system driver to controller\r
-  first, and then special connect the default console, this make\r
-  sure all the system controller available and the platform default\r
-  console connected.\r
+  This function connects all system drivers to controllers.\r
 \r
 **/\r
 VOID\r
@@ -374,19 +377,17 @@ BdsLibConnectAll (
   );\r
 \r
 /**\r
-  This function will create all handles associate with every device\r
-  path node. If the handle associate with one device path node can not\r
-  be created success, then still give one chance to do the dispatch,\r
-  which load the missing drivers if possible.\r
+  This function creates all handles associated with every device\r
+  path node. If the handle associated with one device path node can not\r
+  be created, then it tries to execute the dispatch to load the missing drivers.  \r
 \r
-  @param  DevicePathToConnect   The device path which will be connected, it can be\r
+  @param  DevicePathToConnect   The device path to be connected. Can be\r
                                 a multi-instance device path\r
 \r
-  @retval EFI_SUCCESS           All handles associate with every device path  node\r
-                                have been created\r
-  @retval EFI_OUT_OF_RESOURCES  There is no resource to create new handles\r
-  @retval EFI_NOT_FOUND         Create the handle associate with one device  path\r
-                                node failed\r
+  @retval EFI_SUCCESS           All handles associates with every device path node\r
+                                were created\r
+  @retval EFI_OUT_OF_RESOURCES  Not enough resources to create new handles\r
+  @retval EFI_NOT_FOUND         At least one handle could not be created\r
 \r
 **/\r
 EFI_STATUS\r
@@ -396,13 +397,8 @@ BdsLibConnectDevicePath (
   );\r
 \r
 /**\r
-  This function will connect all current system handles recursively. The\r
-  connection will finish until every handle's child handle created if it have.\r
-\r
-  @retval EFI_SUCCESS           All handles and it's child handle have been\r
-                                connected\r
-  @retval EFI_STATUS            Return the status of gBS->LocateHandleBuffer().\r
-\r
+  This function will connect all current system handles recursively.     gBS->ConnectController() service is invoked for each handle exist in system handler buffer.  If the handle is bus type handler, all childrens also will be connected recursively  by gBS->ConnectController().\r
+  @retval EFI_SUCCESS           All handles and child handles have been                                connected  @retval EFI_STATUS            Return the status of gBS->LocateHandleBuffer().\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -410,13 +406,10 @@ BdsLibConnectAllEfi (
   VOID\r
   );\r
 \r
-\r
 /**\r
-  This function will disconnect all current system handles. The disconnection\r
-  will finish until every handle have been disconnected.\r
-\r
+  This function will disconnect all current system handles.     gBS->DisconnectController() is invoked for each handle exists in system handle buffer.  If handle is a bus type handle, all childrens also are disconnected recursively by  gBS->DisconnectController().\r
   @retval EFI_SUCCESS           All handles have been disconnected\r
-  @retval EFI_STATUS            Return the status of gBS->LocateHandleBuffer().\r
+  @retval EFI_STATUS            Error status returned by of gBS->LocateHandleBuffer().\r
 \r
 **/\r
 EFI_STATUS\r
@@ -429,7 +422,7 @@ BdsLibDisconnectAllEfi (
 // Bds console related lib functions\r
 //\r
 /**\r
-  This function will search every simpletxt devive in current system,\r
+  This function will search every simpletxt device in the current system,\r
   and make every simpletxt device as pertantial console device.\r
 \r
 **/\r
@@ -441,11 +434,11 @@ BdsLibConnectAllConsoles (
 \r
 \r
 /**\r
-  This function will connect console device base on the console\r
+  This function will connect console device based on the console\r
   device variable ConIn, ConOut and ErrOut.\r
 \r
-  @retval EFI_SUCCESS              At least one of the ConIn and ConOut device have\r
-                                   been connected success.\r
+  @retval EFI_SUCCESS              At least one of the ConIn and ConOut devices have\r
+                                   been connected.\r
   @retval EFI_STATUS               Return the status of BdsLibConnectConsoleVariable ().\r
 \r
 **/\r
@@ -456,20 +449,18 @@ BdsLibConnectAllDefaultConsoles (
   );\r
 \r
 /**\r
-  This function update console variable based on ConVarName, it can\r
+  This function updates the console variable based on ConVarName. It can\r
   add or remove one specific console device path from the variable\r
 \r
-  @param  ConVarName               Console related variable name, ConIn, ConOut,\r
+  @param  ConVarName               Console-related variable name: ConIn, ConOut,\r
                                    ErrOut.\r
-  @param  CustomizedConDevicePath  The console device path which will be added to\r
-                                   the console variable ConVarName, this parameter\r
-                                   can not be multi-instance.\r
-  @param  ExclusiveDevicePath      The console device path which will be removed\r
-                                   from the console variable ConVarName, this\r
-                                   parameter can not be multi-instance.\r
-\r
-  @retval EFI_UNSUPPORTED          The added device path is same to the removed one.\r
-  @retval EFI_SUCCESS              Success add or remove the device path from  the\r
+  @param  CustomizedConDevicePath  The console device path to be added to\r
+                                   the console variable ConVarName. Can not be multi-instance.\r
+  @param  ExclusiveDevicePath      The console device path to be removed\r
+                                   from the console variable ConVarName. Can not be multi-instance.\r
+\r
+  @retval EFI_UNSUPPORTED          The added device path is the same as a removed one.\r
+  @retval EFI_SUCCESS              Successfully added or removed the device path from the\r
                                    console variable.\r
 \r
 **/\r
@@ -482,18 +473,17 @@ BdsLibUpdateConsoleVariable (
   );\r
 \r
 /**\r
-  Connect the console device base on the variable ConVarName, if\r
-  device path of the ConVarName is multi-instance device path, if\r
-  anyone of the instances is connected success, then this function\r
+  Connect the console device base on the variable ConVarName. If\r
+  ConVarName is a multi-instance device path, and at least one\r
+  instance connects successfully, then this function\r
   will return success.\r
 \r
   @param  ConVarName               Console related variable name, ConIn, ConOut,\r
                                    ErrOut.\r
 \r
-  @retval EFI_NOT_FOUND            There is not any console devices connected\r
-                                   success\r
-  @retval EFI_SUCCESS              Success connect any one instance of the console\r
-                                   device path base on the variable ConVarName.\r
+  @retval EFI_NOT_FOUND            No console devices were connected successfully\r
+  @retval EFI_SUCCESS              Connected at least one instance of the console\r
+                                   device path based on the variable ConVarName.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -506,15 +496,15 @@ BdsLibConnectConsoleVariable (
 // Bds device path related lib functions\r
 //\r
 /**\r
-  Delete the instance in Multi which matches partly with Single instance\r
+  Delete the instance in Multi that overlaps with Single \r
 \r
   @param  Multi                 A pointer to a multi-instance device path data\r
                                 structure.\r
   @param  Single                A pointer to a single-instance device path data\r
                                 structure.\r
 \r
-  @return This function will remove the device path instances in Multi which partly\r
-          match with the Single, and return the result device path. If there is no\r
+  @return This function removes the device path instances in Multi that overlap\r
+   Single, and returns the resulting device path. If there is no\r
           remaining device path as a result, this function will return NULL.\r
 \r
 **/\r
@@ -535,7 +525,7 @@ BdsLibDelPartMatchInstance (
                                 structure.\r
 \r
   @retval TRUE                  If the Single device path is contained within Multi device path.\r
-  @retval FALSE                 The Single device path is not match within Multi device path.\r
+  @retval FALSE                 The Single device path is not contained within Multi device path.\r
 \r
 **/\r
 BOOLEAN\r
@@ -564,9 +554,9 @@ DevicePathToStr (
 // Internal definitions\r
 //\r
 typedef struct {\r
-  CHAR16  *str;\r
-  UINTN   len;\r
-  UINTN   maxlen;\r
+  CHAR16  *Str;\r
+  UINTN   Len;\r
+  UINTN   Maxlen;\r
 } POOL_PRINT;\r
 \r
 typedef struct {\r
@@ -592,7 +582,7 @@ typedef struct {
   UINT16                    LoginOption;\r
   UINT64                    Lun;\r
   UINT16                    TargetPortalGroupTag;\r
-  CHAR16                    iSCSITargetName[1];\r
+  CHAR16                    TargetName[1];\r
 } ISCSI_DEVICE_PATH_WITH_NAME;\r
 \r
 \r
@@ -633,7 +623,7 @@ ShadowAllOptionRom (
 \r
   @retval EFI_SUCCESS             All invalide legacy boot options are deleted.\r
   @retval EFI_OUT_OF_RESOURCES    Fail to allocate necessary memory.\r
-  @retval EFI_NOT_FOUND           Fail to retrive variable of boot order.\r
+  @retval EFI_NOT_FOUND           Fail to retrieve variable of boot order.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -643,12 +633,13 @@ BdsDeleteAllInvalidLegacyBootOptions (
   );\r
 \r
 /**\r
-\r
   Add the legacy boot options from BBS table if they do not exist.\r
 \r
-  @retval EFI_SUCCESS       The boot options are added successfully \r
-                            or they are already in boot options.\r
-\r
+  @retval EFI_SUCCESS          The boot options are added successfully \r
+                               or they are already in boot options.\r
+  @retval EFI_NOT_FOUND        No legacy boot options is found.\r
+  @retval EFI_OUT_OF_RESOURCE  No enough memory.\r
+  @return Other value          LegacyBoot options are not added.\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -657,12 +648,14 @@ BdsAddNonExistingLegacyBootOptions (
   );\r
 \r
 /**\r
-\r
   Add the legacy boot devices from BBS table into \r
   the legacy device boot order.\r
 \r
-  @retval EFI_SUCCESS       The boot devices are added successfully.\r
-\r
+  @retval EFI_SUCCESS           The boot devices are added successfully.\r
+  @retval EFI_NOT_FOUND         The legacy boot devices are not found.\r
+  @retval EFI_OUT_OF_RESOURCES  Memmory or storage is not enough.\r
+  @retval EFI_DEVICE_ERROR      Fail to add the legacy device boot order into EFI variable\r
+                                because of hardware error.\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -671,13 +664,13 @@ BdsUpdateLegacyDevOrder (
   );\r
 \r
 /**\r
-\r
   Set the boot priority for BBS entries based on boot option entry and boot order.\r
 \r
   @param  Entry             The boot option is to be checked for refresh BBS table.\r
   \r
-  @retval EFI_SUCCESS       The boot priority for BBS entries is refreshed successfully.\r
-\r
+  @retval EFI_SUCCESS           The boot priority for BBS entries is refreshed successfully.\r
+  @retval EFI_NOT_FOUND         BBS entries can't be found.\r
+  @retval EFI_OUT_OF_RESOURCES  Failed to get the legacy device boot order.\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -686,15 +679,16 @@ BdsRefreshBbsTableForBoot (
   );\r
 \r
 /**\r
+  Deletete the Boot Option from EFI Variable. The Boot Order Arrray\r
+  is also updated.\r
 \r
-  Delete boot option specified by OptionNumber and adjust the boot order.\r
-\r
-  @param  OptionNumber      The boot option to be deleted.\r
-  @param  BootOrder         Boot order list to be adjusted by remove this boot option.\r
-  @param  BootOrderSize     The size of Boot order list will be modified.\r
-  \r
-  @retval EFI_SUCCESS       The boot option is deleted successfully.\r
+  @param OptionNumber    The number of Boot option want to be deleted.\r
+  @param BootOrder       The Boot Order array.\r
+  @param BootOrderSize   The size of the Boot Order Array.\r
 \r
+  @retval  EFI_SUCCESS           The Boot Option Variable was found and removed\r
+  @retval  EFI_UNSUPPORTED       The Boot Option Variable store was inaccessible\r
+  @retval  EFI_NOT_FOUND         The Boot Option Variable was not found\r
 **/\r
 EFI_STATUS\r
 EFIAPI\r
@@ -705,11 +699,11 @@ BdsDeleteBootOption (
   );\r
 \r
 //\r
-//The interface functions relate with Setup Browser Reset Reminder feature\r
+//The interface functions related to the Setup Browser Reset Reminder feature\r
 //\r
 /**\r
   Enable the setup browser reset reminder feature.\r
-  This routine is used in platform tip. If the platform policy need the feature, use the routine to enable it.\r
+  This routine is used in platform tip. If the platform policy needs the feature, use the routine to enable it.\r
 \r
 **/\r
 VOID\r
@@ -720,7 +714,7 @@ EnableResetReminderFeature (
 \r
 /**\r
   Disable the setup browser reset reminder feature.\r
-  This routine is used in platform tip. If the platform policy do not want the feature, use the routine to disable it.\r
+  This routine is used in platform tip. If the platform policy does not want the feature, use the routine to disable it.\r
 \r
 **/\r
 VOID\r
@@ -730,8 +724,8 @@ DisableResetReminderFeature (
   );\r
 \r
 /**\r
-  Record the info that  a reset is required.\r
-  A  module boolean variable is used to record whether a reset is required.\r
+  Record the info that a reset is required.\r
+  A module boolean variable is used to record whether a reset is required.\r
 \r
 **/\r
 VOID\r
@@ -742,8 +736,8 @@ EnableResetRequired (
 \r
 \r
 /**\r
-  Record the info that  no reset is required.\r
-  A  module boolean variable is used to record whether a reset is required.\r
+  Record the info that no reset is required.\r
+  A module boolean variable is used to record whether a reset is required.\r
 \r
 **/\r
 VOID\r
@@ -753,7 +747,7 @@ DisableResetRequired (
   );\r
 \r
 /**\r
-  Check whether platform policy enable the reset reminder feature. The default is enabled.\r
+  Check whether platform policy enables the reset reminder feature. The default is enabled.\r
 \r
 **/\r
 BOOLEAN\r
@@ -763,7 +757,7 @@ IsResetReminderFeatureEnable (
   );\r
 \r
 /**\r
-  Check if  user changed any option setting which needs a system reset to be effective.\r
+  Check if the user changed any option setting that needs a system reset to be effective.\r
 \r
 **/\r
 BOOLEAN\r
@@ -774,7 +768,7 @@ IsResetRequired (
 \r
 /**\r
   Check whether a reset is needed, and finish the reset reminder feature.\r
-  If a reset is needed, Popup a menu to notice user, and finish the feature\r
+  If a reset is needed, pop up a menu to notice user, and finish the feature\r
   according to the user selection.\r
 \r
 **/\r
@@ -785,72 +779,69 @@ SetupResetReminder (
   );\r
 \r
 \r
-/**\r
-  Get the headers (dos, image, optional header) from an image\r
-\r
-  @param  Device                SimpleFileSystem device handle\r
-  @param  FileName              File name for the image\r
-  @param  DosHeader             Pointer to dos header\r
-  @param  Hdr                   The buffer in which to return the PE32, PE32+, or TE header.\r
-\r
-  @retval EFI_SUCCESS           Successfully get the machine type.\r
-  @retval EFI_NOT_FOUND         The file is not found.\r
-  @retval EFI_LOAD_ERROR        File is not a valid image file.\r
-\r
-**/\r
-EFI_STATUS\r
-EFIAPI\r
-BdsLibGetImageHeader (\r
-  IN  EFI_HANDLE                  Device,\r
-  IN  CHAR16                      *FileName,\r
-  OUT EFI_IMAGE_DOS_HEADER        *DosHeader,\r
-  OUT EFI_IMAGE_OPTIONAL_HEADER_PTR_UNION   Hdr\r
-  );\r
-\r
-//\r
-// Define the boot type which to classify the boot option type\r
-// Different boot option type could have different boot behavior\r
-// Use their device path node (Type + SubType) as type value\r
-// The boot type here can be added according to requirement\r
-//\r
-//\r
-// ACPI boot type. For ACPI device, cannot use sub-type to distinguish device, so hardcode their value\r
-//\r
+///\r
+/// Define the boot option default description \r
+///\r
+#define DESCRIPTION_FLOPPY        L"EFI Floppy"\r
+#define DESCRIPTION_FLOPPY_NUM    L"EFI Floppy %d"\r
+#define DESCRIPTION_DVD           L"EFI DVD/CDROM"\r
+#define DESCRIPTION_DVD_NUM       L"EFI DVD/CDROM %d"\r
+#define DESCRIPTION_USB           L"EFI USB Device"\r
+#define DESCRIPTION_USB_NUM       L"EFI USB Device %d"\r
+#define DESCRIPTION_SCSI          L"EFI SCSI Device"\r
+#define DESCRIPTION_SCSI_NUM      L"EFI SCSI Device %d"\r
+#define DESCRIPTION_MISC          L"EFI Misc Device"\r
+#define DESCRIPTION_MISC_NUM      L"EFI Misc Device %d"\r
+#define DESCRIPTION_NETWORK       L"EFI Network"\r
+#define DESCRIPTION_NETWORK_NUM   L"EFI Network %d"       \r
+#define DESCRIPTION_NON_BLOCK     L"EFI Non-Block Boot Device"\r
+#define DESCRIPTION_NON_BLOCK_NUM L"EFI Non-Block Boot Device %d"\r
+\r
+///\r
+/// Define the boot type which to classify the boot option type\r
+/// Different boot option type could have different boot behavior\r
+/// Use their device path node (Type + SubType) as type value\r
+/// The boot type here can be added according to requirement\r
+///\r
+\r
+///\r
+/// ACPI boot type. For ACPI device, cannot use sub-type to distinguish device, so hardcode their value\r
+///\r
 #define  BDS_EFI_ACPI_FLOPPY_BOOT         0x0201\r
-//\r
-// Message boot type\r
-// If a device path of boot option only point to a message node, the boot option is message boot type\r
-//\r
+///\r
+/// Message boot type\r
+/// If a device path of boot option only point to a message node, the boot option is message boot type\r
+///\r
 #define  BDS_EFI_MESSAGE_ATAPI_BOOT       0x0301 // Type 03; Sub-Type 01\r
 #define  BDS_EFI_MESSAGE_SCSI_BOOT        0x0302 // Type 03; Sub-Type 02\r
 #define  BDS_EFI_MESSAGE_USB_DEVICE_BOOT  0x0305 // Type 03; Sub-Type 05\r
+#define  BDS_EFI_MESSAGE_SATA_BOOT        0x0318 // Type 03; Sub-Type 18\r
 #define  BDS_EFI_MESSAGE_MISC_BOOT        0x03FF\r
-//\r
-// Media boot type\r
-// If a device path of boot option contain a media node, the boot option is media boot type\r
-//\r
+///\r
+/// Media boot type\r
+/// If a device path of boot option contain a media node, the boot option is media boot type\r
+///\r
 #define  BDS_EFI_MEDIA_HD_BOOT            0x0401 // Type 04; Sub-Type 01\r
 #define  BDS_EFI_MEDIA_CDROM_BOOT         0x0402 // Type 04; Sub-Type 02\r
-//\r
-// BBS boot type\r
-// If a device path of boot option contain a BBS node, the boot option is BBS boot type\r
-//\r
+///\r
+/// BBS boot type\r
+/// If a device path of boot option contain a BBS node, the boot option is BBS boot type\r
+///\r
 #define  BDS_LEGACY_BBS_BOOT              0x0501 //  Type 05; Sub-Type 01\r
 \r
 #define  BDS_EFI_UNSUPPORT                0xFFFF\r
 \r
-//\r
-// USB host controller Programming Interface.\r
-//\r
+///\r
+/// USB host controller Programming Interface.\r
+///\r
 #define  PCI_CLASSC_PI_UHCI               0x00\r
 #define  PCI_CLASSC_PI_EHCI               0x20\r
 \r
 /**\r
-  Check whether there is a instance in BlockIoDevicePath, which contain multi device path\r
-  instances, has the same partition node with HardDriveDevicePath device path\r
+  Check whether an instance in BlockIoDevicePath has the same partition node as the HardDriveDevicePath device path\r
 \r
-  @param  BlockIoDevicePath      Multi device path instances which need to check\r
-  @param  HardDriveDevicePath    A device path which starts with a hard drive media\r
+  @param  BlockIoDevicePath      Multi device path instances to check\r
+  @param  HardDriveDevicePath    A device path starting with a hard drive media\r
                                  device path.\r
 \r
   @retval TRUE                   There is a matched device path instance.\r
@@ -887,13 +878,13 @@ BdsExpandPartitionPartialDevicePathToFull (
   \r
 /**\r
   Return the bootable media handle.\r
-  First, check the device is connected\r
-  Second, check whether the device path point to a device which support SimpleFileSystemProtocol,\r
+  First, check whether the device is connected.\r
+  Second, check whether the device path points to a device that supports SimpleFileSystemProtocol.\r
   Third, detect the the default boot file in the Media, and return the removable Media handle.\r
 \r
   @param  DevicePath             Device Path to a  bootable device\r
 \r
-  @retval NULL                   The media on the DevicePath is not bootable\r
+  @return  The bootable media handle. If the media on the DevicePath is not bootable, NULL will return.\r
 \r
 **/\r
 EFI_HANDLE\r
@@ -904,14 +895,14 @@ BdsLibGetBootableHandle (
   \r
 \r
 /**\r
-  Check whether the Device path in a boot option point to a valide bootable device,\r
-  And if CheckMedia is true, check the device is ready to boot now.\r
+  Checks whether the Device path in a boot option points to a valid bootable device, and if the device\r
+  is ready to boot now.\r
 \r
   @param  DevPath     the Device path in a boot option\r
-  @param  CheckMedia  if true, check the device is ready to boot now.\r
+  @param  CheckMedia  if true, check whether the device is ready to boot now.\r
 \r
-  @retval TRUE        the Device path  is valide\r
-  @retval FALSE       the Device path  is invalide .\r
+  @retval TRUE        the Device path is valid\r
+  @retval FALSE       the Device path is invalid\r
 \r
 **/\r
 BOOLEAN\r
@@ -920,30 +911,36 @@ BdsLibIsValidEFIBootOptDevicePath (
   IN EFI_DEVICE_PATH_PROTOCOL     *DevPath,\r
   IN BOOLEAN                      CheckMedia\r
   );\r
-  \r
+\r
+/**\r
+  Checks whether the Device path in a boot option points to a valid bootable device, and if the device\r
+  is ready to boot now.\r
+  If Description is not NULL and the device path points to a fixed BlockIo\r
+  device, this function checks whether the description conflicts with other auto-created\r
+  boot options.\r
+\r
+  @param  DevPath     the Device path in a boot option\r
+  @param  CheckMedia  if true, checks if the device is ready to boot now.\r
+  @param  Description the description of a boot option\r
+\r
+  @retval TRUE        the Device path is valid\r
+  @retval FALSE       the Device path is invalid\r
+\r
+**/\r
+BOOLEAN\r
+EFIAPI\r
+BdsLibIsValidEFIBootOptDevicePathExt (\r
+  IN EFI_DEVICE_PATH_PROTOCOL     *DevPath,\r
+  IN BOOLEAN                      CheckMedia,\r
+  IN CHAR16                       *Description\r
+  );\r
+\r
 /**\r
   For a bootable Device path, return its boot type.\r
 \r
   @param  DevicePath                      The bootable device Path to check\r
 \r
-  @retval BDS_EFI_MEDIA_HD_BOOT           If the device path contains any media deviec path node, it is media boot type\r
-                                          For the floppy node, handle it as media node\r
-  @retval BDS_EFI_MEDIA_CDROM_BOOT        If the device path contains any media deviec path node, it is media boot type\r
-                                          For the floppy node, handle it as media node\r
-  @retval BDS_EFI_ACPI_FLOPPY_BOOT        If the device path contains any media deviec path node, it is media boot type\r
-                                          For the floppy node, handle it as media node\r
-  @retval BDS_EFI_MESSAGE_ATAPI_BOOT      If the device path not contains any media deviec path node,  and\r
-                                          its last device path node point to a message device path node, it is\r
-  \r
-  @retval BDS_EFI_MESSAGE_SCSI_BOOT       If the device path not contains any media deviec path node,  and\r
-                                          its last device path node point to a message device path node, it is\r
-  @retval BDS_EFI_MESSAGE_USB_DEVICE_BOOT If the device path not contains any media deviec path node,  and\r
-                                          its last device path node point to a message device path node, it is\r
-  @retval BDS_EFI_MESSAGE_MISC_BOOT       If the device path not contains any media deviec path node,  and\r
-                                          its last device path node point to a message device path node, it is\r
-  @retval BDS_LEGACY_BBS_BOOT             Legacy boot type\r
-  @retval BDS_EFI_UNSUPPORT               An EFI Removable BlockIO device path not point to a media and message devie,   \r
-\r
+  @retval BDS_EFI_MEDIA_HD_BOOT           If given device path contains MEDIA_DEVICE_PATH type device path node                                           which subtype is MEDIA_HARDDRIVE_DP  @retval BDS_EFI_MEDIA_CDROM_BOOT        If given device path contains MEDIA_DEVICE_PATH type device path node                                          which subtype is MEDIA_CDROM_DP  @retval BDS_EFI_ACPI_FLOPPY_BOOT        If given device path contains ACPI_DEVICE_PATH type device path node                                          which HID is floppy device.  @retval BDS_EFI_MESSAGE_ATAPI_BOOT      If given device path contains MESSAGING_DEVICE_PATH type device path node                                          and its last device path node's subtype is MSG_ATAPI_DP.  @retval BDS_EFI_MESSAGE_SCSI_BOOT       If given device path contains MESSAGING_DEVICE_PATH type device path node                                          and its last device path node's subtype is MSG_SCSI_DP.  @retval BDS_EFI_MESSAGE_USB_DEVICE_BOOT If given device path contains MESSAGING_DEVICE_PATH type device path node                                          and its last device path node's subtype is MSG_USB_DP.  @retval BDS_EFI_MESSAGE_MISC_BOOT       If the device path not contains any media device path node,  and                                          its last device path node point to a message device path node.  @retval BDS_LEGACY_BBS_BOOT             If given device path contains BBS_DEVICE_PATH type device path node.  @retval BDS_EFI_UNSUPPORT               An EFI Removable BlockIO device path not point to a media and message device,   \r
 **/\r
 UINT32\r
 EFIAPI\r
@@ -953,8 +950,8 @@ BdsGetBootTypeFromDevicePath (
 \r
 \r
 /**\r
-  This routine register a function to adjust the different type memory page number\r
-  just before booting and save the updated info into the variable for next boot to use.\r
+  This routine registers a function to adjust the different types of memory page numbers\r
+  just before booting, and saves the updated info into the variable for the next boot to use.\r
 \r
 **/\r
 VOID\r
@@ -965,23 +962,22 @@ BdsLibSaveMemoryTypeInformation (
   \r
 \r
 /**\r
-  According to a file guild, check a Fv file device path is valid. If it is invalid,\r
-  try to return the valid device path.\r
+  This function checks if a Fv file device path is valid, according to a file GUID. If it is invalid,\r
+  it tries to return the valid device path.\r
   FV address maybe changes for memory layout adjust from time to time, use this funciton\r
   could promise the Fv file device path is right.\r
 \r
-  @param  DevicePath             on input, the Fv file device path need to check on\r
+  @param  DevicePath             on input, the Fv file device path to check. On\r
                                  output, the updated valid Fv file device path\r
-  @param  FileGuid               the Fv file guild\r
+  @param  FileGuid               the Fv file GUID\r
 \r
   @retval EFI_INVALID_PARAMETER  the input DevicePath or FileGuid is invalid\r
-                                 parameter\r
-  @retval EFI_UNSUPPORTED        the input DevicePath does not contain Fv file\r
-                                 guild at all\r
-  @retval EFI_ALREADY_STARTED    the input DevicePath has pointed to Fv file, it is\r
+  @retval EFI_UNSUPPORTED        the input DevicePath does not contain an Fv file\r
+                                 GUID at all\r
+  @retval EFI_ALREADY_STARTED    the input DevicePath has pointed to the Fv file and is\r
                                  valid\r
-  @retval EFI_SUCCESS            has successfully updated the invalid DevicePath,\r
-                                 and return the updated device path in DevicePath\r
+  @retval EFI_SUCCESS            successfully updated the invalid DevicePath\r
+                                 and returned the updated device path in DevicePath\r
 \r
 **/\r
 EFI_STATUS\r
@@ -993,7 +989,7 @@ BdsLibUpdateFvFileDevicePath (
 \r
 \r
 /**\r
-  Connect the specific Usb device which match the short form device path,\r
+  Connect the specific USB device that matches the RemainingDevicePath,\r
   and whose bus is determined by Host Controller (Uhci or Ehci)\r
 \r
   @param  HostControllerPI      Uhci (0x00) or Ehci (0x20) or Both uhci and ehci\r
@@ -1020,14 +1016,13 @@ BdsLibConnectUsbDevByShortFormDP(
 // The implementation of this function is provided by Platform code.\r
 //\r
 /**\r
-  Convert Vendor device path to device name.\r
+  Convert Vendor device path to device name.\r
 \r
-  @param  Str      The buffer store device name\r
+  @param  Str      The buffer storing device name\r
   @param  DevPath  Pointer to vendor device path\r
 \r
 **/\r
 VOID\r
-EFIAPI\r
 DevPathVendor (\r
   IN OUT POOL_PRINT       *Str,\r
   IN VOID                 *DevPath\r
@@ -1038,7 +1033,7 @@ DevPathVendor (
   The caller must free the resulting buffer.\r
 \r
   @param  Str      Tracks the allocated pool, size in use, and amount of pool allocated.\r
-  @param  fmt      The format string\r
+  @param  Fmt      The format string\r
   @param  ...      The data will be printed.\r
 \r
   @return Allocated buffer with the formatted string printed in it.\r
@@ -1050,13 +1045,13 @@ CHAR16 *
 EFIAPI\r
 CatPrint (\r
   IN OUT POOL_PRINT   *Str,\r
-  IN CHAR16           *fmt,\r
+  IN CHAR16           *Fmt,\r
   ...\r
   );\r
 \r
 /**\r
-  Use Console Control to turn off UGA based Simple Text Out consoles from going\r
-  to the UGA device. Put up LogoFile on every UGA device that is a console\r
+  Use Console Control to stop UGA-based Simple Text Out consoles from going\r
+  to the UGA device. Put up LogoFile on every UGA device that is a console.\r
 \r
   @param[in]  LogoFile   File name of logo to display on the center of the screen.\r
 \r