Liming,

Sounds like a good thing to get into the stable tag.

Thanks,

Andrew Fish

On Aug 27, 2020, at 5:57 PM, gaoliming <gaoliming@byosoft.com.cn> wrote:

Paul:
 This is a clear issue. Thanks for your reporting it. I would like to catch
the fix into this stable tag 202008. 
 Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

Thanks
Liming
-----邮件原件-----
发件人: bounce+27952+64705+4905953+8761045@groups.io
<bounce+27952+64705+4905953+8761045@groups.io> 代表 Paul
发送时间: 2020年8月28日 4:41
收件人: devel@edk2.groups.io
抄送: Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao
<gaoliming@byosoft.com.cn>; Zhiguang Liu <zhiguang.liu@intel.com>
主题: [edk2-devel] [PATCH v1 1/1] MdePkg: Correcting
EFI_ACPI_DMA_TRANSFER_TYPE_16_BIT definition

In Acpi10.h, EFI_ACPI_DMA_TRANSFER_TYPE_16_BIT is defined as 0x10,
but should be 0x02 per the ACPI Specification.

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2937

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Signed-off-by: Paul G <paul.grimes@amd.com>
---
MdePkg/Include/IndustryStandard/Acpi10.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MdePkg/Include/IndustryStandard/Acpi10.h
b/MdePkg/Include/IndustryStandard/Acpi10.h
index fa06eefbb6e6..adeb5ae8c219 100644
--- a/MdePkg/Include/IndustryStandard/Acpi10.h
+++ b/MdePkg/Include/IndustryStandard/Acpi10.h
@@ -358,7 +358,7 @@ typedef struct {
#define EFI_ACPI_DMA_TRANSFER_TYPE_MASK                 0x03


#define   EFI_ACPI_DMA_TRANSFER_TYPE_8_BIT              0x00


#define   EFI_ACPI_DMA_TRANSFER_TYPE_8_BIT_AND_16_BIT   0x01


-#define   EFI_ACPI_DMA_TRANSFER_TYPE_16_BIT             0x10


+#define   EFI_ACPI_DMA_TRANSFER_TYPE_16_BIT             0x02





//


// IO Information


--
2.21.0