Correct Minor Comments in M3 to M4 review.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@6842 6f19259b-4bc3-4df7-8a09-765794883524

4 files changed, 17 insertions, 19 deletions

diff --git a/MdePkg/Include/IndustryStandard/PeImage.h b/MdePkg/Include/IndustryStandard/PeImage.h
index 97a17e5eb5..310f16b8e3 100644
--- a/MdePkg/Include/IndustryStandard/PeImage.h
+++ b/MdePkg/Include/IndustryStandard/PeImage.h
@@ -134,8 +134,6 @@ typedef struct {
   UINT32  Size;

 } EFI_IMAGE_DATA_DIRECTORY;

 

-#define EFI_IMAGE_ROM_OPTIONAL_HDR_MAGIC      0x107

-

 //

 // Directory Entries

 //

@@ -386,7 +384,7 @@ typedef struct {
                                                    

 #define EFI_IMAGE_SCN_LNK_OTHER                    BIT8   ///< 0x00000100  ///< Reserved.

 #define EFI_IMAGE_SCN_LNK_INFO                     BIT9   ///< 0x00000200  ///< Section contains comments or some other type of information.

-#define EFI_IMAGE_SCN_LNK_REMOVE                   BIT10  ///< 0x00000800  ///< Section contents will not become part of image.

+#define EFI_IMAGE_SCN_LNK_REMOVE                   BIT11  ///< 0x00000800  ///< Section contents will not become part of image.

 #define EFI_IMAGE_SCN_LNK_COMDAT                   BIT12  ///< 0x00001000

                                                    

 #define EFI_IMAGE_SCN_ALIGN_1BYTES                 BIT20  ///< 0x00100000

@@ -566,15 +564,15 @@ typedef struct {
 //

 // Based relocation types.

 //

-#define EFI_IMAGE_REL_BASED_ABSOLUTE      0

-#define EFI_IMAGE_REL_BASED_HIGH          1

-#define EFI_IMAGE_REL_BASED_LOW           2

-#define EFI_IMAGE_REL_BASED_HIGHLOW       3

-#define EFI_IMAGE_REL_BASED_HIGHADJ       4

-#define EFI_IMAGE_REL_BASED_MIPS_JMPADDR  5

-#define EFI_IMAGE_REL_BASED_IA64_IMM64    9

-#define IMAGE_REL_BASED_MIPS_JMPADDR16    9

-#define EFI_IMAGE_REL_BASED_DIR64         10

+#define EFI_IMAGE_REL_BASED_ABSOLUTE        0

+#define EFI_IMAGE_REL_BASED_HIGH            1

+#define EFI_IMAGE_REL_BASED_LOW             2

+#define EFI_IMAGE_REL_BASED_HIGHLOW         3

+#define EFI_IMAGE_REL_BASED_HIGHADJ         4

+#define EFI_IMAGE_REL_BASED_MIPS_JMPADDR    5

+#define EFI_IMAGE_REL_BASED_IA64_IMM64      9

+#define EFI_IMAGE_REL_BASED_MIPS_JMPADDR16  9

+#define EFI_IMAGE_REL_BASED_DIR64           10

 

 ///

 /// Line number format.

diff --git a/MdePkg/Include/IndustryStandard/Sal.h b/MdePkg/Include/IndustryStandard/Sal.h
index ca1bfb2c29..bf6c54b130 100644
--- a/MdePkg/Include/IndustryStandard/Sal.h
+++ b/MdePkg/Include/IndustryStandard/Sal.h
@@ -87,7 +87,7 @@ typedef struct {
 /**

   Prototype of SAL procedures.

 

-  @param  Arg0               Functional identifier.

+  @param  FunctionId         Functional identifier.

                              The upper 32 bits are ignored and only the lower 32 bits

                              are used. The following functional identifiers are defined:

                              0x01XXXXXX - Architected SAL functional group.

@@ -113,13 +113,13 @@ typedef
 SAL_RETURN_REGS

 (EFIAPI *SAL_PROC) (

   IN UINT64 FunctionId,

+  IN UINT64 Arg1,

   IN UINT64 Arg2,

   IN UINT64 Arg3,

   IN UINT64 Arg4,

   IN UINT64 Arg5,

   IN UINT64 Arg6,

-  IN UINT64 Arg7,

-  IN UINT64 Arg8

+  IN UINT64 Arg7

   );

 

 //

diff --git a/MdePkg/Include/IndustryStandard/Scsi.h b/MdePkg/Include/IndustryStandard/Scsi.h
index ccb3590bdf..c86c452870 100644
--- a/MdePkg/Include/IndustryStandard/Scsi.h
+++ b/MdePkg/Include/IndustryStandard/Scsi.h
@@ -265,7 +265,7 @@ typedef struct {
 // defined here. Please refer to SCSI standard for full value definition.

 //

 #define EFI_SCSI_ASC_NOT_READY                    (0x04)

-#define   EFI_SCSI_ASCQ_IN_PROGRESS                 (0x01)

+#define EFI_SCSI_ASCQ_IN_PROGRESS                 (0x01)

 

 #define EFI_SCSI_ASC_MEDIA_ERR1                   (0x10)

 #define EFI_SCSI_ASC_MEDIA_ERR2                   (0x11)

diff --git a/MdePkg/Include/IndustryStandard/Tpm12.h b/MdePkg/Include/IndustryStandard/Tpm12.h
index eca5eec327..77b2e9ad6b 100644
--- a/MdePkg/Include/IndustryStandard/Tpm12.h
+++ b/MdePkg/Include/IndustryStandard/Tpm12.h
@@ -1052,9 +1052,9 @@ typedef struct tdTPM_TRANSPORT_PUBLIC {
 //

 // Part 2, section 13.1.1 TPM_TRANSPORT_ATTRIBUTES Definitions

 //

-#define TPM_TRANSPORT_ENCRYPT       ((UINT32)0x00000001)

-#define TPM_TRANSPORT_LOG           ((UINT32)0x00000002)

-#define TPM_TRANSPORT_EXCLUSIVE     ((UINT32)0x00000004)

+#define TPM_TRANSPORT_ENCRYPT       ((UINT32)BIT0)

+#define TPM_TRANSPORT_LOG           ((UINT32)BIT1)

+#define TPM_TRANSPORT_EXCLUSIVE     ((UINT32)BIT2)

 

 ///

 /// Part 2, section 13.2 TPM_TRANSPORT_INTERNAL