From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: Sami Mujawar <sami.mujawar@arm.com>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
Leif Lindholm <leif.lindholm@linaro.org>,
Matteo Carlini <Matteo.Carlini@arm.com>,
Stephanie Hughes-Fitt <Stephanie.Hughes-Fitt@arm.com>,
nd <nd@arm.com>,
"Kinney, Michael D" <michael.d.kinney@intel.com>,
"Gao, Liming" <liming.gao@intel.com>,
Evan Lloyd <evan.lloyd@arm.com>
Subject: Re: [PATCH v1 2/2] MdePkg: IORT Specification Rev D updates
Date: Thu, 26 Apr 2018 18:10:02 +0200 [thread overview]
Message-ID: <CAKv+Gu8h=m23ARZ0nRUwu-xxwH+1YitgUjyAb8PPSQCU9OXxhQ@mail.gmail.com> (raw)
In-Reply-To: <20180426154106.131664-3-sami.mujawar@arm.com>
On 26 April 2018 at 17:41, Sami Mujawar <sami.mujawar@arm.com> wrote:
> Updated IORT structure definitions to conform to the IO Remapping
> Table, Platform Design Document, Revision D, March 2018.
>
> The following structures have been updated:
> 1. SMMUv3 - fix on proximity node.
> 2. PMCG - added page 1 support.
> 3. Root complex node - added DMA mask (memory address size limit).
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
> Reviewed-by: Evan Lloyd <evan.lloyd@arm.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
> The changes can be seen at:
> https://github.com/samimujawar/edk2/tree/253_iort_update_v1
>
> Notes:
> v1:
> - Updated for IORT Specification Revision D. [SAMI]
>
> MdePkg/Include/IndustryStandard/IoRemappingTable.h | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/MdePkg/Include/IndustryStandard/IoRemappingTable.h b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> index 2e5cb45d7e2ffd4a0559ef706b71874843e3fdbd..389f352c2fbc725e5e9a881a73adc306035dc0de 100644
> --- a/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> +++ b/MdePkg/Include/IndustryStandard/IoRemappingTable.h
> @@ -1,7 +1,7 @@
> /** @file
> - ACPI IO Remapping Table (IORT) as specified in ARM spec DEN0049C
> + ACPI IO Remapping Table (IORT) as specified in ARM spec DEN0049D
>
> - http://infocenter.arm.com/help/topic/com.arm.doc.den0049c/DEN0049C_IO_Remapping_Table.pdf
> + http://infocenter.arm.com/help/topic/com.arm.doc.den0049d/DEN0049D_IO_Remapping_Table.pdf
>
> Copyright (c) 2017, Linaro Limited. All rights reserved.<BR>
> Copyright (c) 2018, ARM Limited. All rights reserved.<BR>
> @@ -123,6 +123,8 @@ typedef struct {
>
> UINT32 AtsAttribute;
> UINT32 PciSegmentNumber;
> + UINT8 MemoryAddressSize;
> + UINT8 Reserved1[3];
> } EFI_ACPI_6_0_IO_REMAPPING_RC_NODE;
>
> ///
> @@ -185,8 +187,7 @@ typedef struct {
> UINT32 Pri;
> UINT32 Gerr;
> UINT32 Sync;
> - UINT8 ProximityDomain;
> - UINT8 Reserved1[3];
> + UINT32 ProximityDomain;
> UINT32 DeviceIdMappingIndex;
> } EFI_ACPI_6_0_IO_REMAPPING_SMMU3_NODE;
>
> @@ -199,6 +200,7 @@ typedef struct {
> UINT64 Base;
> UINT32 OverflowInterruptGsiv;
> UINT32 NodeReference;
> + UINT64 Page1Base;
> //EFI_ACPI_6_0_IO_REMAPPING_ID_TABLE OverflowInterruptMsiMapping[1];
> } EFI_ACPI_6_0_IO_REMAPPING_PMCG_NODE;
>
> --
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
>
>
next prev parent reply other threads:[~2018-04-26 16:10 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-26 15:41 [PATCH v1 0/2] MdePkg: IORT Specification updates Sami Mujawar
2018-04-26 15:41 ` [PATCH v1 1/2] MdePkg: SMMUv3 updates for IORT table definitions Sami Mujawar
2018-04-26 15:41 ` [PATCH v1 2/2] MdePkg: IORT Specification Rev D updates Sami Mujawar
2018-04-26 16:10 ` Ard Biesheuvel [this message]
2018-04-27 2:19 ` [PATCH v1 0/2] MdePkg: IORT Specification updates Gao, Liming
2018-06-27 19:53 ` Sami Mujawar
2018-06-28 13:09 ` Gao, Liming
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAKv+Gu8h=m23ARZ0nRUwu-xxwH+1YitgUjyAb8PPSQCU9OXxhQ@mail.gmail.com' \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox