Committing changes to the comments, after review with engineers.
authorpkandel <pkandel@6f19259b-4bc3-4df7-8a09-765794883524>
Wed, 15 Jul 2009 13:32:34 +0000 (13:32 +0000)
committerpkandel <pkandel@6f19259b-4bc3-4df7-8a09-765794883524>
Wed, 15 Jul 2009 13:32:34 +0000 (13:32 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8950 6f19259b-4bc3-4df7-8a09-765794883524

IntelFrameworkModulePkg/Include/Library/GenericBdsLib.h
IntelFrameworkModulePkg/Include/Library/OemHookStatusCodeLib.h
IntelFrameworkModulePkg/Include/Library/PlatformBdsLib.h
IntelFrameworkModulePkg/Include/Protocol/IsaAcpi.h
IntelFrameworkModulePkg/Include/Protocol/OEMBadging.h
IntelFrameworkModulePkg/Include/Protocol/VgaMiniPort.h

index e2142db..b3c748c 100644 (file)
@@ -109,8 +109,7 @@ BdsLibBootNext (
   );\r
 \r
 /**\r
-  Process the boot option follow the UEFI specification and\r
-  special treat the legacy boot option with BBS_DEVICE_PATH.\r
+  Process the boot option according to the UEFI specification. The legacy boot option device path includes BBS_DEVICE_PATH.\r
 \r
   @param  Option                 The boot option to be processed\r
   @param  DevicePath             The device path describing where to load the\r
@@ -255,7 +254,8 @@ BdsLibLoadDrivers (
 \r
 \r
 /**\r
-  Process BootOrder, or DriverOrder variables, by calling\r
+  This function processes BootOrder or DriverOrder variables, by calling\r
+\r
   BdsLibVariableToOption () for each UINT16 in the variables.\r
 \r
   @param  BdsCommonOptionList   The header of the option list base on variable\r
@@ -384,7 +384,7 @@ BdsLibConnectAll (
   );\r
 \r
 /**\r
-  This function creates all handles associated with every device\r
+  This function creates all handles associated with the given 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
@@ -430,7 +430,7 @@ BdsLibDisconnectAllEfi (
 //\r
 /**\r
   This function will search every simpletxt device in the current system,\r
-  and make every simpletxt device as pertantial console device.\r
+  and make every simpletxt device a potential console device.\r
 \r
 **/\r
 VOID\r
@@ -671,7 +671,7 @@ BdsUpdateLegacyDevOrder (
   );\r
 \r
 /**\r
-  Set the boot priority for BBS entries based on boot option entry and boot order.\r
+  Refresh 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
@@ -686,7 +686,7 @@ BdsRefreshBbsTableForBoot (
   );\r
 \r
 /**\r
-  Deletete the Boot Option from EFI Variable. The Boot Order Arrray\r
+  Delete the Boot Option from EFI Variable. The Boot Order Arrray\r
   is also updated.\r
 \r
   @param OptionNumber    The number of Boot option want to be deleted.\r
@@ -849,11 +849,11 @@ MatchPartitionDevicePathNode (
 \r
 /**\r
   Expand a device path that starts with a hard drive media device path node to be a\r
-  full device path that includes the full hardware path to the device. We need\r
-  to do this so it can be booted. As an optimizaiton the front match (the part point\r
-  to the partition node. E.g. ACPI() /PCI()/ATA()/Partition() ) is saved in a variable\r
-  so a connect all is not required on every boot. All successful history device path\r
-  which point to partition node (the front part) will be saved.\r
+  full device path that includes the full hardware path to the device. This function enables the device to boot. \r
+  To avoid requiring a connect on every boot, the front match is saved in a variable (the part point\r
+  to the partition node. E.g. ACPI() /PCI()/ATA()/Partition() ).\r
+  All successful history device paths\r
+  that point to the front part of the partition node will be saved.\r
 \r
   @param  HardDriveDevicePath    EFI Device Path to boot, if it starts with a hard\r
                                  drive media device path.\r
index 63387f8..8b09e78 100644 (file)
@@ -33,14 +33,13 @@ OemHookStatusCodeInitialize (
 /**\r
   Report status code to OEM device.\r
  \r
-  @param  CodeType      Indicates the type of status code being reported.  Type EFI_STATUS_CODE_TYPE is defined in "Related Definitions" below.\r
+  @param  CodeType      Indicates the type of status code being reported.  \r
  \r
   @param  Value         Describes the current status of a hardware or software entity.  \r
                         This includes both an operation and classification information about the class and subclass.\r
                         For progress codes, the operation is the current activity. \r
                         For error codes, it is the exception.  For debug codes, it is not defined at this time. \r
-                        Type EFI_STATUS_CODE_VALUE is defined in "Related Definitions" below.  \r
-                        Specific values are discussed in the Intel? Platform Innovation Framework for EFI Status Code Specification.\r
+                        Specific values are discussed in the Intel Platform Innovation Framework for EFI Status Code Specification.\r
  \r
   @param  Instance      The enumeration of a hardware or software entity within the system.  \r
                         A system may contain multiple entities that match a class/subclass pairing. \r
index 96cd87d..d2099f1 100644 (file)
@@ -92,7 +92,7 @@ PlatformBdsPolicyBehavior (
   );\r
 \r
 /**\r
-  Hook point after a boot attempt fails.\r
+  Hook point for a user-provided function, for after a boot attempt fails. \r
 \r
   @param  Option                  Pointer to Boot Option that failed to boot.\r
   @param  Status                  Status returned from failed boot.\r
index 819d5b4..4445d98 100644 (file)
@@ -271,7 +271,7 @@ EFI_STATUS
  \r
   @param[in] This  Pointer to the EFI_ISA_ACPI_PROTOCOL instance.\r
 \r
-  @retval EFI_SUCCESS   Successfully initialized the ISA interface.\r
+  @retval EFI_SUCCESS   Successfully initialized all required hardware states.\r
   @retval Other         The ISA interface could not be initialized.\r
 \r
 **/\r
index a0d3808..147916e 100644 (file)
@@ -53,7 +53,7 @@ typedef enum {
   @param This              Pointer to this protocol instance.\r
   @param Instance          The visible image instance is found.\r
   @param Format            Format of the image. Examples: BMP, JPEG.\r
-  @param ImageData         Image data returned.\r
+  @param ImageData         Image data for the badge file. Currently only supports the .bmp file format. \r
   @param ImageSize         Size of the image returned.\r
   @param Attribute         Display attributes of the image returned.\r
   @param CoordinateX       X coordinate of the image.\r
index f8a7387..d5ee730 100644 (file)
@@ -61,7 +61,7 @@ struct _EFI_VGA_MINI_PORT_PROTOCOL {
   ///\r
   UINT64                      VgaMemoryOffset;\r
   ///\r
-  /// I/O Port address for the VGA CRTC address register  Typically set to 0x3D4.\r
+  /// I/O Port address for the VGA CRTC address register. Typically set to 0x3D4.\r
   ///\r
   UINT64                      CrtcAddressRegisterOffset;\r
   ///\r