]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Include/Pi/PiFirmwareFile.h
MdePkg: Apply uncrustify changes
[mirror_edk2.git] / MdePkg / Include / Pi / PiFirmwareFile.h
index ec7729e9c36e5b2910d07e32999a284095cde394..f740f9cd507c59410f95c8b18eae79682085bca1 100644 (file)
@@ -9,7 +9,6 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 \r
 **/\r
 \r
-\r
 #ifndef __PI_FIRMWARE_FILE_H__\r
 #define __PI_FIRMWARE_FILE_H__\r
 \r
@@ -24,7 +23,7 @@ typedef union {
     /// header. The State and IntegrityCheck.Checksum.File fields are assumed\r
     /// to be zero and the checksum is calculated such that the entire header sums to zero.\r
     ///\r
-    UINT8   Header;\r
+    UINT8    Header;\r
     ///\r
     /// If the FFS_ATTRIB_CHECKSUM (see definition below) bit of the Attributes\r
     /// field is set to one, the IntegrityCheck.Checksum.File field is an 8-bit\r
@@ -34,7 +33,7 @@ typedef union {
     /// 0xAA. The IntegrityCheck.Checksum.File field is valid any time the\r
     /// EFI_FILE_DATA_VALID bit is set in the State field.\r
     ///\r
-    UINT8   File;\r
+    UINT8    File;\r
   } Checksum;\r
   ///\r
   /// This is the full 16 bits of the IntegrityCheck field.\r
@@ -48,47 +47,47 @@ typedef union {
 ///\r
 #define FFS_FIXED_CHECKSUM  0xAA\r
 \r
-typedef UINT8 EFI_FV_FILETYPE;\r
-typedef UINT8 EFI_FFS_FILE_ATTRIBUTES;\r
-typedef UINT8 EFI_FFS_FILE_STATE;\r
+typedef UINT8  EFI_FV_FILETYPE;\r
+typedef UINT8  EFI_FFS_FILE_ATTRIBUTES;\r
+typedef UINT8  EFI_FFS_FILE_STATE;\r
 \r
 ///\r
 /// File Types Definitions\r
 ///\r
-#define EFI_FV_FILETYPE_ALL                   0x00\r
-#define EFI_FV_FILETYPE_RAW                   0x01\r
-#define EFI_FV_FILETYPE_FREEFORM              0x02\r
-#define EFI_FV_FILETYPE_SECURITY_CORE         0x03\r
-#define EFI_FV_FILETYPE_PEI_CORE              0x04\r
-#define EFI_FV_FILETYPE_DXE_CORE              0x05\r
-#define EFI_FV_FILETYPE_PEIM                  0x06\r
-#define EFI_FV_FILETYPE_DRIVER                0x07\r
-#define EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER  0x08\r
-#define EFI_FV_FILETYPE_APPLICATION           0x09\r
-#define EFI_FV_FILETYPE_MM                    0x0A\r
-#define EFI_FV_FILETYPE_SMM                   EFI_FV_FILETYPE_MM\r
-#define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE 0x0B\r
-#define EFI_FV_FILETYPE_COMBINED_MM_DXE       0x0C\r
-#define EFI_FV_FILETYPE_COMBINED_SMM_DXE      EFI_FV_FILETYPE_COMBINED_MM_DXE\r
-#define EFI_FV_FILETYPE_MM_CORE               0x0D\r
-#define EFI_FV_FILETYPE_SMM_CORE              EFI_FV_FILETYPE_MM_CORE\r
-#define EFI_FV_FILETYPE_MM_STANDALONE         0x0E\r
-#define EFI_FV_FILETYPE_MM_CORE_STANDALONE    0x0F\r
-#define EFI_FV_FILETYPE_OEM_MIN               0xc0\r
-#define EFI_FV_FILETYPE_OEM_MAX               0xdf\r
-#define EFI_FV_FILETYPE_DEBUG_MIN             0xe0\r
-#define EFI_FV_FILETYPE_DEBUG_MAX             0xef\r
-#define EFI_FV_FILETYPE_FFS_MIN               0xf0\r
-#define EFI_FV_FILETYPE_FFS_MAX               0xff\r
-#define EFI_FV_FILETYPE_FFS_PAD               0xf0\r
+#define EFI_FV_FILETYPE_ALL                    0x00\r
+#define EFI_FV_FILETYPE_RAW                    0x01\r
+#define EFI_FV_FILETYPE_FREEFORM               0x02\r
+#define EFI_FV_FILETYPE_SECURITY_CORE          0x03\r
+#define EFI_FV_FILETYPE_PEI_CORE               0x04\r
+#define EFI_FV_FILETYPE_DXE_CORE               0x05\r
+#define EFI_FV_FILETYPE_PEIM                   0x06\r
+#define EFI_FV_FILETYPE_DRIVER                 0x07\r
+#define EFI_FV_FILETYPE_COMBINED_PEIM_DRIVER   0x08\r
+#define EFI_FV_FILETYPE_APPLICATION            0x09\r
+#define EFI_FV_FILETYPE_MM                     0x0A\r
+#define EFI_FV_FILETYPE_SMM                    EFI_FV_FILETYPE_MM\r
+#define EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE  0x0B\r
+#define EFI_FV_FILETYPE_COMBINED_MM_DXE        0x0C\r
+#define EFI_FV_FILETYPE_COMBINED_SMM_DXE       EFI_FV_FILETYPE_COMBINED_MM_DXE\r
+#define EFI_FV_FILETYPE_MM_CORE                0x0D\r
+#define EFI_FV_FILETYPE_SMM_CORE               EFI_FV_FILETYPE_MM_CORE\r
+#define EFI_FV_FILETYPE_MM_STANDALONE          0x0E\r
+#define EFI_FV_FILETYPE_MM_CORE_STANDALONE     0x0F\r
+#define EFI_FV_FILETYPE_OEM_MIN                0xc0\r
+#define EFI_FV_FILETYPE_OEM_MAX                0xdf\r
+#define EFI_FV_FILETYPE_DEBUG_MIN              0xe0\r
+#define EFI_FV_FILETYPE_DEBUG_MAX              0xef\r
+#define EFI_FV_FILETYPE_FFS_MIN                0xf0\r
+#define EFI_FV_FILETYPE_FFS_MAX                0xff\r
+#define EFI_FV_FILETYPE_FFS_PAD                0xf0\r
 ///\r
 /// FFS File Attributes.\r
 ///\r
-#define FFS_ATTRIB_LARGE_FILE         0x01\r
-#define FFS_ATTRIB_DATA_ALIGNMENT_2   0x02\r
-#define FFS_ATTRIB_FIXED              0x04\r
-#define FFS_ATTRIB_DATA_ALIGNMENT     0x38\r
-#define FFS_ATTRIB_CHECKSUM           0x40\r
+#define FFS_ATTRIB_LARGE_FILE        0x01\r
+#define FFS_ATTRIB_DATA_ALIGNMENT_2  0x02\r
+#define FFS_ATTRIB_FIXED             0x04\r
+#define FFS_ATTRIB_DATA_ALIGNMENT    0x38\r
+#define FFS_ATTRIB_CHECKSUM          0x40\r
 \r
 ///\r
 /// FFS File State Bits.\r
@@ -100,7 +99,6 @@ typedef UINT8 EFI_FFS_FILE_STATE;
 #define EFI_FILE_DELETED              0x10\r
 #define EFI_FILE_HEADER_INVALID       0x20\r
 \r
-\r
 ///\r
 /// Each file begins with the header that describe the\r
 /// contents and state of the files.\r
@@ -109,27 +107,27 @@ typedef struct {
   ///\r
   /// This GUID is the file name. It is used to uniquely identify the file.\r
   ///\r
-  EFI_GUID                Name;\r
+  EFI_GUID                   Name;\r
   ///\r
   /// Used to verify the integrity of the file.\r
   ///\r
-  EFI_FFS_INTEGRITY_CHECK IntegrityCheck;\r
+  EFI_FFS_INTEGRITY_CHECK    IntegrityCheck;\r
   ///\r
   /// Identifies the type of file.\r
   ///\r
-  EFI_FV_FILETYPE         Type;\r
+  EFI_FV_FILETYPE            Type;\r
   ///\r
   /// Declares various file attribute bits.\r
   ///\r
-  EFI_FFS_FILE_ATTRIBUTES Attributes;\r
+  EFI_FFS_FILE_ATTRIBUTES    Attributes;\r
   ///\r
   /// The length of the file in bytes, including the FFS header.\r
   ///\r
-  UINT8                   Size[3];\r
+  UINT8                      Size[3];\r
   ///\r
   /// Used to track the state of the file throughout the life of the file from creation to deletion.\r
   ///\r
-  EFI_FFS_FILE_STATE      State;\r
+  EFI_FFS_FILE_STATE         State;\r
 } EFI_FFS_FILE_HEADER;\r
 \r
 typedef struct {\r
@@ -138,22 +136,22 @@ typedef struct {
   /// one instance of a file with the file name GUID of Name in any given firmware\r
   /// volume, except if the file type is EFI_FV_FILETYPE_FFS_PAD.\r
   ///\r
-  EFI_GUID                  Name;\r
+  EFI_GUID                   Name;\r
 \r
   ///\r
   /// Used to verify the integrity of the file.\r
   ///\r
-  EFI_FFS_INTEGRITY_CHECK   IntegrityCheck;\r
+  EFI_FFS_INTEGRITY_CHECK    IntegrityCheck;\r
 \r
   ///\r
   /// Identifies the type of file.\r
   ///\r
-  EFI_FV_FILETYPE           Type;\r
+  EFI_FV_FILETYPE            Type;\r
 \r
   ///\r
   /// Declares various file attribute bits.\r
   ///\r
-  EFI_FFS_FILE_ATTRIBUTES   Attributes;\r
+  EFI_FFS_FILE_ATTRIBUTES    Attributes;\r
 \r
   ///\r
   /// The length of the file in bytes, including the FFS header.\r
@@ -162,18 +160,18 @@ typedef struct {
   /// Size is not required to be a multiple of 8 bytes. Given a file F, the next file header is\r
   /// located at the next 8-byte aligned firmware volume offset following the last byte of the file F.\r
   ///\r
-  UINT8                     Size[3];\r
+  UINT8                 Size[3];\r
 \r
   ///\r
   /// Used to track the state of the file throughout the life of the file from creation to deletion.\r
   ///\r
-  EFI_FFS_FILE_STATE        State;\r
+  EFI_FFS_FILE_STATE    State;\r
 \r
   ///\r
   /// If FFS_ATTRIB_LARGE_FILE is set in Attributes, then ExtendedSize exists and Size must be set to zero.\r
   /// If FFS_ATTRIB_LARGE_FILE is not set then EFI_FFS_FILE_HEADER is used.\r
   ///\r
-  UINT64                    ExtendedSize;\r
+  UINT64                ExtendedSize;\r
 } EFI_FFS_FILE_HEADER2;\r
 \r
 #define IS_FFS_FILE2(FfsFileHeaderPtr) \\r
@@ -184,7 +182,7 @@ typedef struct {
 /// FFS_FILE_SIZE() function-like macro below must not have side effects:\r
 /// FfsFileHeaderPtr is evaluated multiple times.\r
 ///\r
-#define FFS_FILE_SIZE(FfsFileHeaderPtr) ((UINT32) ( \\r
+#define FFS_FILE_SIZE(FfsFileHeaderPtr)  ((UINT32) (\\r
     (((EFI_FFS_FILE_HEADER *) (UINTN) (FfsFileHeaderPtr))->Size[0]      ) | \\r
     (((EFI_FFS_FILE_HEADER *) (UINTN) (FfsFileHeaderPtr))->Size[1] <<  8) | \\r
     (((EFI_FFS_FILE_HEADER *) (UINTN) (FfsFileHeaderPtr))->Size[2] << 16)))\r
@@ -198,33 +196,33 @@ typedef UINT8 EFI_SECTION_TYPE;
 /// Pseudo type. It is used as a wild card when retrieving sections.\r
 ///  The section type EFI_SECTION_ALL matches all section types.\r
 ///\r
-#define EFI_SECTION_ALL                   0x00\r
+#define EFI_SECTION_ALL  0x00\r
 \r
 ///\r
 /// Encapsulation section Type values.\r
 ///\r
-#define EFI_SECTION_COMPRESSION           0x01\r
+#define EFI_SECTION_COMPRESSION  0x01\r
 \r
-#define EFI_SECTION_GUID_DEFINED          0x02\r
+#define EFI_SECTION_GUID_DEFINED  0x02\r
 \r
-#define EFI_SECTION_DISPOSABLE            0x03\r
+#define EFI_SECTION_DISPOSABLE  0x03\r
 \r
 ///\r
 /// Leaf section Type values.\r
 ///\r
-#define EFI_SECTION_PE32                  0x10\r
-#define EFI_SECTION_PIC                   0x11\r
-#define EFI_SECTION_TE                    0x12\r
-#define EFI_SECTION_DXE_DEPEX             0x13\r
-#define EFI_SECTION_VERSION               0x14\r
-#define EFI_SECTION_USER_INTERFACE        0x15\r
-#define EFI_SECTION_COMPATIBILITY16       0x16\r
-#define EFI_SECTION_FIRMWARE_VOLUME_IMAGE 0x17\r
-#define EFI_SECTION_FREEFORM_SUBTYPE_GUID 0x18\r
-#define EFI_SECTION_RAW                   0x19\r
-#define EFI_SECTION_PEI_DEPEX             0x1B\r
-#define EFI_SECTION_MM_DEPEX              0x1C\r
-#define EFI_SECTION_SMM_DEPEX             EFI_SECTION_MM_DEPEX\r
+#define EFI_SECTION_PE32                   0x10\r
+#define EFI_SECTION_PIC                    0x11\r
+#define EFI_SECTION_TE                     0x12\r
+#define EFI_SECTION_DXE_DEPEX              0x13\r
+#define EFI_SECTION_VERSION                0x14\r
+#define EFI_SECTION_USER_INTERFACE         0x15\r
+#define EFI_SECTION_COMPATIBILITY16        0x16\r
+#define EFI_SECTION_FIRMWARE_VOLUME_IMAGE  0x17\r
+#define EFI_SECTION_FREEFORM_SUBTYPE_GUID  0x18\r
+#define EFI_SECTION_RAW                    0x19\r
+#define EFI_SECTION_PEI_DEPEX              0x1B\r
+#define EFI_SECTION_MM_DEPEX               0x1C\r
+#define EFI_SECTION_SMM_DEPEX              EFI_SECTION_MM_DEPEX\r
 \r
 ///\r
 /// Common section header.\r
@@ -234,8 +232,8 @@ typedef struct {
   /// A 24-bit unsigned integer that contains the total size of the section in bytes,\r
   /// including the EFI_COMMON_SECTION_HEADER.\r
   ///\r
-  UINT8             Size[3];\r
-  EFI_SECTION_TYPE  Type;\r
+  UINT8               Size[3];\r
+  EFI_SECTION_TYPE    Type;\r
   ///\r
   /// Declares the section type.\r
   ///\r
@@ -246,15 +244,15 @@ typedef struct {
   /// A 24-bit unsigned integer that contains the total size of the section in bytes,\r
   /// including the EFI_COMMON_SECTION_HEADER.\r
   ///\r
-  UINT8             Size[3];\r
+  UINT8               Size[3];\r
 \r
-  EFI_SECTION_TYPE  Type;\r
+  EFI_SECTION_TYPE    Type;\r
 \r
   ///\r
   /// If Size is 0xFFFFFF, then ExtendedSize contains the size of the section. If\r
   /// Size is not equal to 0xFFFFFF, then this field does not exist.\r
   ///\r
-  UINT32            ExtendedSize;\r
+  UINT32              ExtendedSize;\r
 } EFI_COMMON_SECTION_HEADER2;\r
 \r
 ///\r
@@ -277,15 +275,15 @@ typedef struct {
   ///\r
   /// Usual common section header. CommonHeader.Type = EFI_SECTION_COMPRESSION.\r
   ///\r
-  EFI_COMMON_SECTION_HEADER   CommonHeader;\r
+  EFI_COMMON_SECTION_HEADER    CommonHeader;\r
   ///\r
   /// The UINT32 that indicates the size of the section data after decompression.\r
   ///\r
-  UINT32                      UncompressedLength;\r
+  UINT32                       UncompressedLength;\r
   ///\r
   /// Indicates which compression algorithm is used.\r
   ///\r
-  UINT8                       CompressionType;\r
+  UINT8                        CompressionType;\r
 } EFI_COMPRESSION_SECTION;\r
 \r
 typedef struct {\r
@@ -312,20 +310,20 @@ typedef struct {
 /// order to conserve space. The contents of this section are implementation specific, but might contain\r
 /// debug data or detailed integration instructions.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_DISPOSABLE_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_DISPOSABLE_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_DISPOSABLE_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_DISPOSABLE_SECTION2;\r
 \r
 ///\r
 /// The leaf section which could be used to determine the dispatch order of DXEs.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_DXE_DEPEX_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_DXE_DEPEX_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_DXE_DEPEX_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_DXE_DEPEX_SECTION2;\r
 \r
 ///\r
 /// The leaf section which contains a PI FV.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_FIRMWARE_VOLUME_IMAGE_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_FIRMWARE_VOLUME_IMAGE_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_FIRMWARE_VOLUME_IMAGE_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_FIRMWARE_VOLUME_IMAGE_SECTION2;\r
 \r
 ///\r
 /// The leaf section which contains a single GUID.\r
@@ -334,11 +332,11 @@ typedef struct {
   ///\r
   /// Common section header. CommonHeader.Type = EFI_SECTION_FREEFORM_SUBTYPE_GUID.\r
   ///\r
-  EFI_COMMON_SECTION_HEADER   CommonHeader;\r
+  EFI_COMMON_SECTION_HEADER    CommonHeader;\r
   ///\r
   /// This GUID is defined by the creator of the file. It is a vendor-defined file type.\r
   ///\r
-  EFI_GUID                    SubTypeGuid;\r
+  EFI_GUID                     SubTypeGuid;\r
 } EFI_FREEFORM_SUBTYPE_GUID_SECTION;\r
 \r
 typedef struct {\r
@@ -364,19 +362,19 @@ typedef struct {
   ///\r
   /// The common section header. CommonHeader.Type = EFI_SECTION_GUID_DEFINED.\r
   ///\r
-  EFI_COMMON_SECTION_HEADER   CommonHeader;\r
+  EFI_COMMON_SECTION_HEADER    CommonHeader;\r
   ///\r
   /// The GUID that defines the format of the data that follows. It is a vendor-defined section type.\r
   ///\r
-  EFI_GUID                    SectionDefinitionGuid;\r
+  EFI_GUID                     SectionDefinitionGuid;\r
   ///\r
   /// Contains the offset in bytes from the beginning of the common header to the first byte of the data.\r
   ///\r
-  UINT16                      DataOffset;\r
+  UINT16                       DataOffset;\r
   ///\r
   /// The bit field that declares some specific characteristics of the section contents.\r
   ///\r
-  UINT16                      Attributes;\r
+  UINT16                       Attributes;\r
 } EFI_GUID_DEFINED_SECTION;\r
 \r
 typedef struct {\r
@@ -401,14 +399,14 @@ typedef struct {
 ///\r
 /// The leaf section which contains PE32+ image.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_PE32_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_PE32_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_PE32_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_PE32_SECTION2;\r
 \r
 ///\r
 /// The leaf section used to determine the dispatch order of PEIMs.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_PEI_DEPEX_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_PEI_DEPEX_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_PEI_DEPEX_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_PEI_DEPEX_SECTION2;\r
 \r
 ///\r
 /// A leaf section type that contains a position-independent-code (PIC) image.\r
@@ -419,20 +417,20 @@ typedef EFI_COMMON_SECTION_HEADER2  EFI_PEI_DEPEX_SECTION2;
 /// execute correctly without performing any relocation or other fix-ups. EFI_PIC_SECTION2 must\r
 /// be used if the section is 16MB or larger.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_PIC_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_PIC_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_PIC_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_PIC_SECTION2;\r
 \r
 ///\r
 /// The leaf section which constains the position-independent-code image.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_TE_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_TE_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_TE_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_TE_SECTION2;\r
 \r
 ///\r
 /// The leaf section which contains an array of zero or more bytes.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER   EFI_RAW_SECTION;\r
-typedef EFI_COMMON_SECTION_HEADER2  EFI_RAW_SECTION2;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_RAW_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER2 EFI_RAW_SECTION2;\r
 \r
 ///\r
 /// The SMM dependency expression section is a leaf section that contains a dependency expression that\r
@@ -442,7 +440,7 @@ typedef EFI_COMMON_SECTION_HEADER2  EFI_RAW_SECTION2;
 /// The dependency expression may refer to protocols installed in either the UEFI or the SMM protocol\r
 /// database. EFI_SMM_DEPEX_SECTION2 must be used if the section is 16MB or larger.\r
 ///\r
-typedef EFI_COMMON_SECTION_HEADER EFI_SMM_DEPEX_SECTION;\r
+typedef EFI_COMMON_SECTION_HEADER  EFI_SMM_DEPEX_SECTION;\r
 typedef EFI_COMMON_SECTION_HEADER2 EFI_SMM_DEPEX_SECTION2;\r
 \r
 ///\r
@@ -450,12 +448,12 @@ typedef EFI_COMMON_SECTION_HEADER2 EFI_SMM_DEPEX_SECTION2;
 /// is human readable file name.\r
 ///\r
 typedef struct {\r
-  EFI_COMMON_SECTION_HEADER   CommonHeader;\r
+  EFI_COMMON_SECTION_HEADER    CommonHeader;\r
 \r
   ///\r
   /// Array of unicode string.\r
   ///\r
-  CHAR16                      FileNameString[1];\r
+  CHAR16                       FileNameString[1];\r
 } EFI_USER_INTERFACE_SECTION;\r
 \r
 typedef struct {\r
@@ -468,13 +466,13 @@ typedef struct {
 /// an optional unicode string that represents the file revision.\r
 ///\r
 typedef struct {\r
-  EFI_COMMON_SECTION_HEADER   CommonHeader;\r
-  UINT16                      BuildNumber;\r
+  EFI_COMMON_SECTION_HEADER    CommonHeader;\r
+  UINT16                       BuildNumber;\r
 \r
   ///\r
   /// Array of unicode string.\r
   ///\r
-  CHAR16                      VersionString[1];\r
+  CHAR16                       VersionString[1];\r
 } EFI_VERSION_SECTION;\r
 \r
 typedef struct {\r
@@ -492,7 +490,7 @@ typedef struct {
 /// and IS_SECTION2() function-like macros below must not have side effects:\r
 /// SectionHeaderPtr is evaluated multiple times.\r
 ///\r
-#define SECTION_SIZE(SectionHeaderPtr) ((UINT32) ( \\r
+#define SECTION_SIZE(SectionHeaderPtr)  ((UINT32) (\\r
     (((EFI_COMMON_SECTION_HEADER *) (UINTN) (SectionHeaderPtr))->Size[0]      ) | \\r
     (((EFI_COMMON_SECTION_HEADER *) (UINTN) (SectionHeaderPtr))->Size[1] <<  8) | \\r
     (((EFI_COMMON_SECTION_HEADER *) (UINTN) (SectionHeaderPtr))->Size[2] << 16)))\r
@@ -506,4 +504,3 @@ typedef struct {
 #pragma pack()\r
 \r
 #endif\r
-\r