Re: [PATCH 2/2] MdePkg:IORT header update for IORT Rev E.e spec

[Swatisri Kantamsetti <swatisrik@nvidia.com>]

[-- Attachment #1: Type: text/plain, Size: 4319 bytes --]

Just checking to see when this patch could be merged?

Thanks,
Swati
________________________________
From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Saturday, January 28, 2023 10:59 PM
To: 'Name' <username@nvidia.com>; devel@edk2.groups.io <devel@edk2.groups.io>; Sami.Mujawar@arm.com <Sami.Mujawar@arm.com>; Alexei.Fedorov@arm.com <Alexei.Fedorov@arm.com>; pierre.gondois@arm.com <pierre.gondois@arm.com>; michael.d.kinney@intel.com <michael.d.kinney@intel.com>; zhiguang.liu@intel.com <zhiguang.liu@intel.com>
Cc: Swatisri Kantamsetti <swatisrik@nvidia.com>
Subject: 回复: [PATCH 2/2] MdePkg:IORT header update for IORT Rev E.e spec

External email: Use caution opening links or attachments


Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: Name <username@nvidia.com>
> 发送时间: 2023年1月26日 2:41
> 收件人: devel@edk2.groups.io; Sami.Mujawar@arm.com;
> Alexei.Fedorov@arm.com; pierre.gondois@arm.com;
> michael.d.kinney@intel.com; gaoliming@byosoft.com.cn;
> zhiguang.liu@intel.com
> 抄送: Swatisri Kantamsetti <swatisrik@nvidia.com>
> 主题: [PATCH 2/2] MdePkg:IORT header update for IORT Rev E.e spec
>
> From: Swatisri Kantamsetti <swatisrik@nvidia.com>
>
> The IO Remapping Table, Platform Design Document, Revision E.e,
> Sept 2022 (https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.arm.com%2Fdocumentation%2Fden0049%2Fee&data=05%7C01%7Cswatisrik%40nvidia.com%7C39d6cc086d3b45e32ea908db01bdfd93%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C638105687766743781%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Yfn%2BzXkA2rKS4Y44XkU65gMLQy8OL3iqPF2AmyYmRNM%3D&reserved=0)
> added flags in SMMUv3 node for validity of ID mappings for MSIs
> related to control interrupts.
>
> Therefore, update the IORT header file to:
> - increment IORT table revision to 6
> - add support for DeviceId valid flag
>
> Signed-off-by: Swatisri Kantamsetti <swatisrik@nvidia.com>
> ---
>  MdePkg/Include/IndustryStandard/IoRemappingTable.h | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> index f48a4a95cd..544aa67a05 100644
> --- a/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> +++ b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> @@ -9,6 +9,8 @@
>    @par Reference(s):
>    - IO Remapping Table, Platform Design Document, Revision E.d, Feb 2022
>      (https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.arm.com%2Fdocumentation%2Fden0049%2F&data=05%7C01%7Cswatisrik%40nvidia.com%7C39d6cc086d3b45e32ea908db01bdfd93%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C638105687766743781%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Yb238OnzfBZR8f9ad%2F5S0Ol3XetVptoMG24AO4Yj8To%3D&reserved=0)
> +  - IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022
> +      (https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdeveloper.arm.com%2Fdocumentation%2Fden0049%2F&data=05%7C01%7Cswatisrik%40nvidia.com%7C39d6cc086d3b45e32ea908db01bdfd93%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C638105687766743781%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Yb238OnzfBZR8f9ad%2F5S0Ol3XetVptoMG24AO4Yj8To%3D&reserved=0)
>
>    @par Glossary:
>    - Ref  : Reference
> @@ -24,6 +26,7 @@
>  #define EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00  0x0
>  #define EFI_ACPI_IO_REMAPPING_TABLE_REVISION_04  0x4   //
> Deprecated
>  #define EFI_ACPI_IO_REMAPPING_TABLE_REVISION_05  0x5
> +#define EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06  0x6
>
>  #define EFI_ACPI_IORT_TYPE_ITS_GROUP     0x0
>  #define EFI_ACPI_IORT_TYPE_NAMED_COMP    0x1
> @@ -59,6 +62,7 @@
>  #define EFI_ACPI_IORT_SMMUv3_FLAG_COHAC_OVERRIDE    BIT0
>  #define EFI_ACPI_IORT_SMMUv3_FLAG_HTTU_OVERRIDE     BIT1
>  #define EFI_ACPI_IORT_SMMUv3_FLAG_PROXIMITY_DOMAIN  BIT3
> +#define EFI_ACPI_IORT_SMMUv3_FLAG_DEVICEID_VALID    BIT4
>
>  #define EFI_ACPI_IORT_SMMUv3_MODEL_GENERIC           0x0
>  #define EFI_ACPI_IORT_SMMUv3_MODEL_HISILICON_HI161X  0x1
> --
> 2.17.1




[-- Attachment #2: Type: text/html, Size: 7920 bytes --]