]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Include/Protocol/FirmwareManagement.h
MdePkg FirmwareManagement.h: Fix typo EFI_SECURITY_VIOLATIO
[mirror_edk2.git] / MdePkg / Include / Protocol / FirmwareManagement.h
index ae840268a61de7db83fac6444a7b4036a859b659..7c8f1ce65dede9583b6a6d92393a1b2aa1ea0ede 100644 (file)
@@ -1,24 +1,25 @@
 /** @file\r
   UEFI Firmware Management Protocol definition\r
-  Firmware Management Protocol provides an abstraction for device to provide firmware \r
-  management support. The base requirements for managing device firmware images include \r
+  Firmware Management Protocol provides an abstraction for device to provide firmware\r
+  management support. The base requirements for managing device firmware images include\r
   identifying firmware image revision level and programming the image into the device.\r
 \r
-  GetImageInfo() is the only required function. GetImage(), SetImage(), \r
-  CheckImage(), GetPackageInfo(), and SetPackageInfo() shall return \r
+  GetImageInfo() is the only required function. GetImage(), SetImage(),\r
+  CheckImage(), GetPackageInfo(), and SetPackageInfo() shall return\r
   EFI_UNSUPPORTED if not supported by the driver.\r
-  \r
-  Copyright (c) 2009 - 2013, 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
+  Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2013 - 2014, Hewlett-Packard Development Company, L.P.<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
-  @par Revision Reference:          \r
-  This Protocol is introduced in UEFI Specification 2.3      \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
+\r
+  @par Revision Reference:\r
+  This Protocol is introduced in UEFI Specification 2.3\r
 \r
 **/\r
 \r
@@ -37,59 +38,85 @@ typedef struct _EFI_FIRMWARE_MANAGEMENT_PROTOCOL EFI_FIRMWARE_MANAGEMENT_PROTOCO
 /// EFI_FIRMWARE_IMAGE_DESCRIPTOR\r
 ///\r
 typedef struct {\r
-  /// \r
-  /// A unique number identifying the firmware image within the device.  The number is \r
+  ///\r
+  /// A unique number identifying the firmware image within the device. The number is\r
   /// between 1 and DescriptorCount.\r
-  /// \r
+  ///\r
   UINT8                            ImageIndex;\r
   ///\r
-  /// A unique number identifying the firmware image type.  \r
-  /// \r
+  /// A unique GUID identifying the firmware image type.\r
+  ///\r
   EFI_GUID                         ImageTypeId;\r
-  /// \r
-  /// A unique number identifying the firmware image. \r
-  /// \r
+  ///\r
+  /// A unique number identifying the firmware image.\r
+  ///\r
   UINT64                           ImageId;\r
-  /// \r
-  /// A pointer to a null-terminated string representing the firmware image name.  \r
-  /// \r
+  ///\r
+  /// A pointer to a null-terminated string representing the firmware image name.\r
+  ///\r
   CHAR16                           *ImageIdName;\r
-  /// \r
-  /// Identifies the version of the device firmware. The format is vendor specific and new \r
+  ///\r
+  /// Identifies the version of the device firmware. The format is vendor specific and new\r
   /// version must have a greater value than an old version.\r
-  /// \r
+  ///\r
   UINT32                           Version;\r
-  /// \r
+  ///\r
   /// A pointer to a null-terminated string representing the firmware image version name.\r
-  /// \r
+  ///\r
   CHAR16                           *VersionName;\r
-  /// \r
-  /// Size of the image in bytes.  If size=0, then only ImageIndex and ImageTypeId are valid.\r
-  /// \r
+  ///\r
+  /// Size of the image in bytes. If size=0, then only ImageIndex and ImageTypeId are valid.\r
+  ///\r
   UINTN                            Size;\r
-  /// \r
-  /// Image attributes that are supported by this device.  See 'Image Attribute Definitions' \r
-  /// for possible returned values of this parameter.  A value of 1 indicates the attribute is \r
-  /// supported and the current setting value is indicated in AttributesSetting.  A \r
-  /// value of 0 indicates the attribute is not supported and the current setting value in \r
+  ///\r
+  /// Image attributes that are supported by this device. See 'Image Attribute Definitions'\r
+  /// for possible returned values of this parameter. A value of 1 indicates the attribute is\r
+  /// supported and the current setting value is indicated in AttributesSetting. A\r
+  /// value of 0 indicates the attribute is not supported and the current setting value in\r
   /// AttributesSetting is meaningless.\r
-  /// \r
+  ///\r
   UINT64                           AttributesSupported;\r
-  /// \r
-  /// Image attributes.  See 'Image Attribute Definitions' for possible returned values of \r
+  ///\r
+  /// Image attributes. See 'Image Attribute Definitions' for possible returned values of\r
   /// this parameter.\r
-  /// \r
+  ///\r
   UINT64                           AttributesSetting;\r
-  /// \r
-  /// Image compatibilities.  See 'Image Compatibility Definitions' for possible returned \r
+  ///\r
+  /// Image compatibilities. See 'Image Compatibility Definitions' for possible returned\r
   /// values of this parameter.\r
-  /// \r
+  ///\r
   UINT64                           Compatibilities;\r
   ///\r
   /// Describes the lowest ImageDescriptor version that the device will accept. Only\r
   /// present in version 2 or higher.\r
   ///\r
   UINT32                           LowestSupportedImageVersion;\r
+  ///\r
+  /// Describes the version that was last attempted to update. If no update attempted the\r
+  /// value will be 0. If the update attempted was improperly formatted and no version\r
+  /// number was available then the value will be zero. Only present in version 3 or higher.\r
+  UINT32                           LastAttemptVersion;\r
+  ///\r
+  /// Describes the status that was last attempted to update. If no update has been attempted\r
+  /// the value will be LAST_ATTEMPT_STATUS_SUCCESS. Only present in version 3 or higher.\r
+  ///\r
+  UINT32                           LastAttemptStatus;\r
+  ///\r
+  /// An optional number to identify the unique hardware instance within the system for\r
+  /// devices that may have multiple instances (Example: a plug in pci network card). This\r
+  /// number must be unique within the namespace of the ImageTypeId GUID and\r
+  /// ImageIndex. For FMP instances that have multiple descriptors for a single\r
+  /// hardware instance, all descriptors must have the same HardwareInstance value.\r
+  /// This number must be consistent between boots and should be based on some sort of\r
+  /// hardware identified unique id (serial number, etc) whenever possible. If a hardware\r
+  /// based number is not available the FMP provider may use some other characteristic\r
+  /// such as device path, bus/dev/function, slot num, etc for generating the\r
+  /// HardwareInstance. For implementations that will never have more than one\r
+  /// instance a zero can be used. A zero means the FMP provider is not able to determine a\r
+  /// unique hardware instance number or a hardware instance number is not needed. Only\r
+  /// present in version 3 or higher.\r
+  ///\r
+  UINT64                           HardwareInstance;\r
 } EFI_FIRMWARE_IMAGE_DESCRIPTOR;\r
 \r
 \r
@@ -97,57 +124,64 @@ typedef struct {
 // Image Attribute Definitions\r
 //\r
 ///\r
-/// The attribute IMAGE_ATTRIBUTE_IMAGE_UPDATABLE indicates this device supports firmware \r
+/// The attribute IMAGE_ATTRIBUTE_IMAGE_UPDATABLE indicates this device supports firmware\r
 /// image update.\r
 ///\r
 #define    IMAGE_ATTRIBUTE_IMAGE_UPDATABLE         0x0000000000000001\r
 ///\r
-/// The attribute IMAGE_ATTRIBUTE_RESET_REQUIRED indicates a reset of the device is required \r
-/// for the new firmware image to take effect after a firmware update.  The device is the device hosting \r
+/// The attribute IMAGE_ATTRIBUTE_RESET_REQUIRED indicates a reset of the device is required\r
+/// for the new firmware image to take effect after a firmware update. The device is the device hosting\r
 /// the firmware image.\r
-/// \r
+///\r
 #define    IMAGE_ATTRIBUTE_RESET_REQUIRED          0x0000000000000002\r
 ///\r
-/// The attribute IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED indicates authentication is \r
-/// required to perform the following image operations: GetImage(), SetImage(), and \r
+/// The attribute IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED indicates authentication is\r
+/// required to perform the following image operations: GetImage(), SetImage(), and\r
 /// CheckImage(). See 'Image Attribute - Authentication'.\r
-/// \r
+///\r
 #define    IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED 0x0000000000000004\r
 ///\r
-/// The attribute IMAGE_ATTRIBUTE_IN_USE indicates the current state of the firmware image.  \r
-/// This distinguishes firmware images in a device that supports redundant images.  \r
-/// \r
+/// The attribute IMAGE_ATTRIBUTE_IN_USE indicates the current state of the firmware image.\r
+/// This distinguishes firmware images in a device that supports redundant images.\r
+///\r
 #define    IMAGE_ATTRIBUTE_IN_USE                  0x0000000000000008\r
+///\r
+/// The attribute IMAGE_ATTRIBUTE_UEFI_IMAGE indicates that this image is an EFI compatible image.\r
+///\r
+#define    IMAGE_ATTRIBUTE_UEFI_IMAGE              0x0000000000000010\r
 \r
 \r
 //\r
 // Image Compatibility Definitions\r
 //\r
+///\r
 /// Values from 0x0000000000000002 thru 0x000000000000FFFF are reserved for future assignments.\r
-/// Values from 0x0000000000010000 thru 0xFFFFFFFFFFFFFFFF are used by firmware vendor for \r
+/// Values from 0x0000000000010000 thru 0xFFFFFFFFFFFFFFFF are used by firmware vendor for\r
 /// compatibility check.\r
 ///\r
 #define   IMAGE_COMPATIBILITY_CHECK_SUPPORTED      0x0000000000000001\r
 \r
 ///\r
-/// Descriptor Version\r
+/// Descriptor Version exposed by GetImageInfo() function\r
 ///\r
-#define   EFI_FIRMWARE_IMAGE_DESCRIPTOR_VERSION   2\r
+#define   EFI_FIRMWARE_IMAGE_DESCRIPTOR_VERSION   3\r
 \r
 \r
 ///\r
-/// Image Attribute -Authentication Required\r
+/// Image Attribute - Authentication Required\r
 ///\r
 typedef struct {\r
   ///\r
-  /// It is included in the signature of AuthInfo.  It is used to ensure freshness/no replay.   \r
-  /// It is incremented during each firmware image operation.  \r
-  ///                                \r
+  /// It is included in the signature of AuthInfo. It is used to ensure freshness/no replay.\r
+  /// It is incremented during each firmware image operation.\r
+  ///\r
   UINT64                                  MonotonicCount;\r
   ///\r
-  /// Provides the authorization for the firmware image operations.  It is a signature across \r
-  /// the image data and the Monotonic Count value.  Caller uses the private key that is \r
-  /// associated with a public key that has been provisioned via the key exchange.  \r
+  /// Provides the authorization for the firmware image operations. It is a signature across\r
+  /// the image data and the Monotonic Count value. Caller uses the private key that is\r
+  /// associated with a public key that has been provisioned via the key exchange.\r
+  /// Because this is defined as a signature, WIN_CERTIFICATE_UEFI_GUID.CertType must\r
+  /// be EFI_CERT_TYPE_PKCS7_GUID.\r
   ///\r
   WIN_CERTIFICATE_UEFI_GUID               AuthInfo;\r
 } EFI_FIRMWARE_IMAGE_AUTHENTICATION;\r
@@ -157,53 +191,61 @@ typedef struct {
 // ImageUpdatable Definitions\r
 //\r
 ///\r
-/// IMAGE_UPDATABLE_VALID indicates SetImage() will accept the new image and update the \r
-/// device with the new image.\r
+/// IMAGE_UPDATABLE_VALID indicates SetImage() will accept the new image and update the\r
+/// device with the new image. The version of the new image could be higher or lower than\r
+/// the current image. SetImage VendorCode is optional but can be used for vendor\r
+/// specific action.\r
 ///\r
 #define  IMAGE_UPDATABLE_VALID                     0x0000000000000001\r
 ///\r
-/// IMAGE_UPDATABLE_INVALID indicates SetImage() will reject the new image. No additional \r
+/// IMAGE_UPDATABLE_INVALID indicates SetImage() will reject the new image. No additional\r
 /// information is provided for the rejection.\r
 ///\r
 #define  IMAGE_UPDATABLE_INVALID                   0x0000000000000002\r
 ///\r
-/// IMAGE_UPDATABLE_INVALID_TYPE indicates SetImage() will reject the new image. The \r
+/// IMAGE_UPDATABLE_INVALID_TYPE indicates SetImage() will reject the new image. The\r
 /// rejection is due to the new image is not a firmware image recognized for this device.\r
 ///\r
 #define  IMAGE_UPDATABLE_INVALID_TYPE              0x0000000000000004\r
 ///\r
-/// IMAGE_UPDATABLE_INVALID_OLD indicates SetImage() will reject the new image. The \r
+/// IMAGE_UPDATABLE_INVALID_OLD indicates SetImage() will reject the new image. The\r
 /// rejection is due to the new image version is older than the current firmware image\r
-/// version in the device. The device firmware update policy does not support firmware \r
+/// version in the device. The device firmware update policy does not support firmware\r
 /// version downgrade.\r
 ///\r
 #define  IMAGE_UPDATABLE_INVALID_OLD               0x0000000000000008\r
+///\r
+/// IMAGE_UPDATABLE_VALID_WITH_VENDOR_CODE indicates SetImage() will accept and update\r
+/// the new image only if a correct VendorCode is provided or else image would be\r
+/// rejected and SetImage will return appropriate error.\r
+///\r
+#define IMAGE_UPDATABLE_VALID_WITH_VENDOR_CODE     0x0000000000000010\r
 \r
 \r
 //\r
 // Package Attribute Definitions\r
 //\r
 ///\r
-/// The attribute PACKAGE_ATTRIBUTE_VERSION_UPDATABLE indicates this device supports the \r
+/// The attribute PACKAGE_ATTRIBUTE_VERSION_UPDATABLE indicates this device supports the\r
 /// update of the firmware package version.\r
 ///\r
 #define  PACKAGE_ATTRIBUTE_VERSION_UPDATABLE       0x0000000000000001\r
 ///\r
-/// The attribute PACKAGE_ATTRIBUTE_RESET_REQUIRED indicates a reset of the device is \r
+/// The attribute PACKAGE_ATTRIBUTE_RESET_REQUIRED indicates a reset of the device is\r
 /// required for the new package info to take effect after an update.\r
 ///\r
 #define  PACKAGE_ATTRIBUTE_RESET_REQUIRED          0x0000000000000002\r
 ///\r
-/// The attribute PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED indicates authentication \r
+/// The attribute PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED indicates authentication\r
 /// is required to update the package info.\r
 ///\r
 #define  PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED 0x0000000000000004\r
 \r
-/** \r
-  Callback funtion to report the process of the firmware updating.\r
+/**\r
+  Callback function to report the process of the firmware updating.\r
 \r
   @param[in]  Completion    A value between 1 and 100 indicating the current completion\r
-                            progress of the firmware update. Completion progress is \r
+                            progress of the firmware update. Completion progress is\r
                             reported as from 1 to 100 percent. A value of 0 is used by\r
                             the driver to indicate that progress reporting is not supported.\r
 \r
@@ -225,15 +267,15 @@ EFI_STATUS
   The saved copy could later been used, for example, in firmware image recovery or rollback.\r
 \r
   @param[in]      This               A pointer to the EFI_FIRMWARE_MANAGEMENT_PROTOCOL instance.\r
-  @param[in, out] ImageInfoSize      A pointer to the size, in bytes, of the ImageInfo buffer. \r
+  @param[in, out] ImageInfoSize      A pointer to the size, in bytes, of the ImageInfo buffer.\r
                                      On input, this is the size of the buffer allocated by the caller.\r
-                                     On output, it is the size of the buffer returned by the firmware \r
+                                     On output, it is the size of the buffer returned by the firmware\r
                                      if the buffer was large enough, or the size of the buffer needed\r
                                      to contain the image(s) information if the buffer was too small.\r
   @param[in, out] ImageInfo          A pointer to the buffer in which firmware places the current image(s)\r
                                      information. The information is an array of EFI_FIRMWARE_IMAGE_DESCRIPTORs.\r
   @param[out]     DescriptorVersion  A pointer to the location in which firmware returns the version number\r
-                                     associated with the EFI_FIRMWARE_IMAGE_DESCRIPTOR. \r
+                                     associated with the EFI_FIRMWARE_IMAGE_DESCRIPTOR.\r
   @param[out]     DescriptorCount    A pointer to the location in which firmware returns the number of\r
                                      descriptors or firmware images within this device.\r
   @param[out]     DescriptorSize     A pointer to the location in which firmware returns the size, in bytes,\r
@@ -245,17 +287,17 @@ EFI_STATUS
                                      is to be performed using PackageVersionName. A value of 0xFFFFFFFD indicates\r
                                      that package version update is in progress.\r
   @param[out]     PackageVersionName A pointer to a pointer to a null-terminated string representing the\r
-                                     package version name. The buffer is allocated by this function with \r
+                                     package version name. The buffer is allocated by this function with\r
                                      AllocatePool(), and it is the caller's responsibility to free it with a call\r
                                      to FreePool().\r
 \r
   @retval EFI_SUCCESS                The device was successfully updated with the new image.\r
-  @retval EFI_BUFFER_TOO_SMALL       The ImageInfo buffer was too small. The current buffer size \r
-                                     needed to hold the image(s) information is returned in ImageInfoSize.                                               \r
+  @retval EFI_BUFFER_TOO_SMALL       The ImageInfo buffer was too small. The current buffer size\r
+                                     needed to hold the image(s) information is returned in ImageInfoSize.\r
   @retval EFI_INVALID_PARAMETER      ImageInfoSize is NULL.\r
   @retval EFI_DEVICE_ERROR           Valid information could not be returned. Possible corrupted image.\r
 \r
-**/                         \r
+**/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_FIRMWARE_MANAGEMENT_PROTOCOL_GET_IMAGE_INFO)(\r
@@ -270,7 +312,7 @@ EFI_STATUS
   );\r
 \r
 /**\r
-  Retrieves a copy of the current firmware image of the device. \r
+  Retrieves a copy of the current firmware image of the device.\r
 \r
   This function allows a copy of the current firmware image to be created and saved.\r
   The saved copy could later been used, for example, in firmware image recovery or rollback.\r
@@ -283,15 +325,15 @@ EFI_STATUS
                                  On return, points to the length of the image, in bytes.\r
 \r
   @retval EFI_SUCCESS            The device was successfully updated with the new image.\r
-  @retval EFI_BUFFER_TOO_SMALL   The buffer specified by ImageSize is too small to hold the \r
-                                 image. The current buffer size needed to hold the image is returned \r
+  @retval EFI_BUFFER_TOO_SMALL   The buffer specified by ImageSize is too small to hold the\r
+                                 image. The current buffer size needed to hold the image is returned\r
                                  in ImageSize.\r
   @retval EFI_INVALID_PARAMETER  The Image was NULL.\r
   @retval EFI_NOT_FOUND          The current image is not copied to the buffer.\r
   @retval EFI_UNSUPPORTED        The operation is not supported.\r
-  @retval EFI_SECURITY_VIOLATIO  The operation could not be performed due to an authentication failure.\r
+  @retval EFI_SECURITY_VIOLATION The operation could not be performed due to an authentication failure.\r
 \r
-**/                         \r
+**/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_FIRMWARE_MANAGEMENT_PROTOCOL_GET_IMAGE)(\r
@@ -308,20 +350,20 @@ EFI_STATUS
   This function returns EFI_UNSUPPORTED if the firmware image is not updatable.\r
   If the firmware image is updatable, the function should perform the following minimal validations\r
   before proceeding to do the firmware image update.\r
-  - Validate the image authentication if image has attribute \r
-    IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED.  The function returns \r
+  - Validate the image authentication if image has attribute\r
+    IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED. The function returns\r
     EFI_SECURITY_VIOLATION if the validation fails.\r
-  - Validate the image is a supported image for this device.  The function returns EFI_ABORTED if \r
-    the image is unsupported.  The function can optionally provide more detailed information on \r
+  - Validate the image is a supported image for this device. The function returns EFI_ABORTED if\r
+    the image is unsupported. The function can optionally provide more detailed information on\r
     why the image is not a supported image.\r
-  - Validate the data from VendorCode if not null.  Image validation must be performed before \r
-    VendorCode data validation.  VendorCode data is ignored or considered invalid if image \r
-    validation failed.  The function returns EFI_ABORTED if the data is invalid.\r
-  \r
-  VendorCode enables vendor to implement vendor-specific firmware image update policy.  Null if \r
-  the caller did not specify the policy or use the default policy.  As an example, vendor can implement \r
-  a policy to allow an option to force a firmware image update when the abort reason is due to the new \r
-  firmware image version is older than the current firmware image version or bad image checksum.  \r
+  - Validate the data from VendorCode if not null. Image validation must be performed before\r
+    VendorCode data validation. VendorCode data is ignored or considered invalid if image\r
+    validation failed. The function returns EFI_ABORTED if the data is invalid.\r
+\r
+  VendorCode enables vendor to implement vendor-specific firmware image update policy. Null if\r
+  the caller did not specify the policy or use the default policy. As an example, vendor can implement\r
+  a policy to allow an option to force a firmware image update when the abort reason is due to the new\r
+  firmware image version is older than the current firmware image version or bad image checksum.\r
   Sensitive operations such as those wiping the entire firmware image and render the device to be\r
   non-functional should be encoded in the image itself rather than passed with the VendorCode.\r
   AbortReason enables vendor to have the option to provide a more detailed description of the abort\r
@@ -332,11 +374,11 @@ EFI_STATUS
                                  The number is between 1 and DescriptorCount.\r
   @param[in]  Image              Points to the new image.\r
   @param[in]  ImageSize          Size of the new image in bytes.\r
-  @param[in]  VendorCode         This enables vendor to implement vendor-specific firmware image update policy.  \r
+  @param[in]  VendorCode         This enables vendor to implement vendor-specific firmware image update policy.\r
                                  Null indicates the caller did not specify the policy or use the default policy.\r
   @param[in]  Progress           A function used by the driver to report the progress of the firmware update.\r
   @param[out] AbortReason        A pointer to a pointer to a null-terminated string providing more\r
-                                 details for the aborted operation. The buffer is allocated by this function \r
+                                 details for the aborted operation. The buffer is allocated by this function\r
                                  with AllocatePool(), and it is the caller's responsibility to free it with a\r
                                  call to FreePool().\r
 \r
@@ -344,9 +386,9 @@ EFI_STATUS
   @retval EFI_ABORTED            The operation is aborted.\r
   @retval EFI_INVALID_PARAMETER  The Image was NULL.\r
   @retval EFI_UNSUPPORTED        The operation is not supported.\r
-  @retval EFI_SECURITY_VIOLATIO  The operation could not be performed due to an authentication failure.\r
+  @retval EFI_SECURITY_VIOLATION The operation could not be performed due to an authentication failure.\r
 \r
-**/                         \r
+**/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_FIRMWARE_MANAGEMENT_PROTOCOL_SET_IMAGE)(\r
@@ -362,7 +404,7 @@ EFI_STATUS
 /**\r
   Checks if the firmware image is valid for the device.\r
 \r
-  This function allows firmware update application to validate the firmware image without \r
+  This function allows firmware update application to validate the firmware image without\r
   invoking the SetImage() first.\r
 \r
   @param[in]  This               A pointer to the EFI_FIRMWARE_MANAGEMENT_PROTOCOL instance.\r
@@ -371,14 +413,14 @@ EFI_STATUS
   @param[in]  Image              Points to the new image.\r
   @param[in]  ImageSize          Size of the new image in bytes.\r
   @param[out] ImageUpdatable     Indicates if the new image is valid for update. It also provides,\r
-                                 if available, additional information if the image is invalid. \r
+                                 if available, additional information if the image is invalid.\r
 \r
   @retval EFI_SUCCESS            The image was successfully checked.\r
   @retval EFI_INVALID_PARAMETER  The Image was NULL.\r
   @retval EFI_UNSUPPORTED        The operation is not supported.\r
-  @retval EFI_SECURITY_VIOLATIO  The operation could not be performed due to an authentication failure.\r
+  @retval EFI_SECURITY_VIOLATION The operation could not be performed due to an authentication failure.\r
 \r
-**/                         \r
+**/\r
 typedef\r
 EFI_STATUS\r
 (EFIAPI *EFI_FIRMWARE_MANAGEMENT_PROTOCOL_CHECK_IMAGE)(\r
@@ -390,7 +432,7 @@ EFI_STATUS
   );\r
 \r
 /**\r
-  Returns information about the firmware package. \r
+  Returns information about the firmware package.\r
 \r
   This function returns package information.\r
 \r
@@ -409,8 +451,8 @@ EFI_STATUS
                                        package version name. A value of 0 indicates the device does not support\r
                                        update of package version name. Length is the number of Unicode characters,\r
                                        including the terminating null character.\r
-  @param[out] AttributesSupported      Package attributes that are supported by this device. See 'Package Attribute \r
-                                       Definitions' for possible returned values of this parameter. A value of 1 \r
+  @param[out] AttributesSupported      Package attributes that are supported by this device. See 'Package Attribute\r
+                                       Definitions' for possible returned values of this parameter. A value of 1\r
                                        indicates the attribute is supported and the current setting value is\r
                                        indicated in AttributesSetting. A value of 0 indicates the attribute is not\r
                                        supported and the current setting value in AttributesSetting is meaningless.\r
@@ -446,21 +488,21 @@ EFI_STATUS
   @param[in]  ImageSize          Size of the authentication image in bytes.\r
                                  0 if authentication is not required.\r
   @param[in]  VendorCode         This enables vendor to implement vendor-specific firmware\r
-                                 image update policy.  \r
+                                 image update policy.\r
                                  Null indicates the caller did not specify this policy or use\r
                                  the default policy.\r
   @param[in]  PackageVersion     The new package version.\r
   @param[in]  PackageVersionName A pointer to the new null-terminated Unicode string representing\r
-                                 the package version name. \r
-                                 The string length is equal to or less than the value returned in \r
+                                 the package version name.\r
+                                 The string length is equal to or less than the value returned in\r
                                  PackageVersionNameMaxLen.\r
 \r
   @retval EFI_SUCCESS            The device was successfully updated with the new package\r
                                  information.\r
-  @retval EFI_INVALID_PARAMETER  The PackageVersionName length is longer than the value \r
+  @retval EFI_INVALID_PARAMETER  The PackageVersionName length is longer than the value\r
                                  returned in PackageVersionNameMaxLen.\r
   @retval EFI_UNSUPPORTED        The operation is not supported.\r
-  @retval EFI_SECURITY_VIOLATIO  The operation could not be performed due to an authentication failure.\r
+  @retval EFI_SECURITY_VIOLATION The operation could not be performed due to an authentication failure.\r
 \r
 **/\r
 typedef\r
@@ -478,7 +520,7 @@ EFI_STATUS
 /// EFI_FIRMWARE_MANAGEMENT_PROTOCOL\r
 /// The protocol for managing firmware provides the following services.\r
 /// - Get the attributes of the current firmware image. Attributes include revision level.\r
-/// - Get a copy of the current firmware image. As an example, this service could be used by a \r
+/// - Get a copy of the current firmware image. As an example, this service could be used by a\r
 ///   management application to facilitate a firmware roll-back.\r
 /// - Program the device with a firmware image supplied by the user.\r
 /// - Label all the firmware images within a device with a single version.\r