* [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec @ 2023-09-22 21:40 Swatisri Kantamsetti via groups.io 2023-09-25 9:19 ` Sami Mujawar 2023-09-25 9:40 ` Leif Lindholm 0 siblings, 2 replies; 4+ messages in thread From: Swatisri Kantamsetti via groups.io @ 2023-09-22 21:40 UTC (permalink / raw) To: devel, Sami.Mujawar, pierre.gondois; +Cc: Swatisri Kantamsetti The IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022 (https://developer.arm.com/documentation/den0049/ee) added flags in SMMUv3 node for validity of ID mappings for MSIs related to control interrupts. It makes one small addition to SMMUv3 nodes to describe MSI support independently of wired GSIV support Therefore, update the IORT generator to: - increment IORT table revision to 6 - increment SMMUV3 node revision to 5 - for SMMUV3 node revision >=5 check the DeviceID mapping index valid flag to populate DeviceIdMappingIndex field Signed-off-by: Swatisri Kantamsetti <swatisrik@nvidia.com> --- .../Acpi/Arm/AcpiIortLibArm/IortGenerator.c | 35 +++++++++++++++---- 1 file changed, 28 insertions(+), 7 deletions(-) diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c index f28973c1a8..a6e4b49cb1 100644 --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c @@ -5,7 +5,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent @par Reference(s): - - IO Remapping Table, Platform Design Document, Revision E.d, Feb 2022 + - IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022 (https://developer.arm.com/documentation/den0049/) **/ @@ -1554,9 +1554,14 @@ AddSmmuV3Nodes ( { SmmuV3Node->Node.Revision = 2; SmmuV3Node->Node.Identifier = EFI_ACPI_RESERVED_DWORD; - } else { + } else if (AcpiTableInfo->AcpiTableRevision < + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06) + { SmmuV3Node->Node.Revision = 4; SmmuV3Node->Node.Identifier = NodeList->Identifier; + } else { + SmmuV3Node->Node.Revision = 5; + SmmuV3Node->Node.Identifier = NodeList->Identifier; } // SMMUv3 specific data @@ -1577,11 +1582,27 @@ AddSmmuV3Nodes ( SmmuV3Node->ProximityDomain = 0; } - if ((SmmuV3Node->Event != 0) && (SmmuV3Node->Pri != 0) && - (SmmuV3Node->Gerr != 0) && (SmmuV3Node->Sync != 0)) + /* For older SMMUV3 nodes rev. < 5. + If all the SMMU control interrupts are GSIV based, + the DeviceID mapping index field is ignored. + DeviceID mapping valid flag was introduced in IORT rev E.e + for SMMUV3 nodes rev. > 5. + If the DeviceID mapping index valid flag is set to 0, + DeviceID mapping index field must be ignored. + Where the SMMU uses message signaled interrupts for + its control interrupts, DeviceId Mapping Index contains an + index into the array of ID mapping. + */ + if (((SmmuV3Node->Node.Revision < 5) && + (SmmuV3Node->Event != 0) && + (SmmuV3Node->Pri != 0) && + (SmmuV3Node->Gerr != 0) && + (SmmuV3Node->Sync != 0) + ) || + ((SmmuV3Node->Node.Revision >= 5) && + ((SmmuV3Node->Flags & EFI_ACPI_IORT_SMMUv3_FLAG_DEVICEID_VALID) == 0)) + ) { - // If all the SMMU control interrupts are GSIV based, - // the DeviceID mapping index field is ignored. SmmuV3Node->DeviceIdMappingIndex = 0; } else { SmmuV3Node->DeviceIdMappingIndex = NodeList->DeviceIdMappingIndex; @@ -2819,7 +2840,7 @@ ACPI_IORT_GENERATOR IortGenerator = { // ACPI Table Signature EFI_ACPI_6_4_IO_REMAPPING_TABLE_SIGNATURE, // ACPI Table Revision supported by this Generator - EFI_ACPI_IO_REMAPPING_TABLE_REVISION_05, + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06, // Minimum supported ACPI Table Revision EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00, // Creator ID -- 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109023): https://edk2.groups.io/g/devel/message/109023 Mute This Topic: https://groups.io/mt/101535844/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec 2023-09-22 21:40 [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec Swatisri Kantamsetti via groups.io @ 2023-09-25 9:19 ` Sami Mujawar 2023-09-25 9:40 ` Leif Lindholm 1 sibling, 0 replies; 4+ messages in thread From: Sami Mujawar @ 2023-09-25 9:19 UTC (permalink / raw) To: Swatisri Kantamsetti, devel, pierre.gondois, nd@arm.com Hi Swatisri, Thank you for this patch. I believe this was reviewed intially at https://edk2.groups.io/g/devel/topic/96534672#99048 and I see you have addressed the feedback in this patch. I also see that patch 2/2 from your previous series was already merged. Can you increment the revision for the patch series (even if some patches get merged) in future, please? It helps keep track of the review history. Other than that this patch looks good to me and I will queue this for merging. Reviewed-by: Sami Mujawar <sami.mujawar@arm.com> Regards, Sami Mujawar On 22/09/2023 10:40 pm, Swatisri Kantamsetti wrote: > The IO Remapping Table, Platform Design Document, Revision E.e, > Sept 2022 (https://developer.arm.com/documentation/den0049/ee) > added flags in SMMUv3 node for validity of ID mappings for MSIs > related to control interrupts. > It makes one small addition to SMMUv3 nodes to > describe MSI support independently of wired GSIV support > > Therefore, update the IORT generator to: > - increment IORT table revision to 6 > - increment SMMUV3 node revision to 5 > - for SMMUV3 node revision >=5 check the DeviceID mapping index > valid flag to populate DeviceIdMappingIndex field > > Signed-off-by: Swatisri Kantamsetti <swatisrik@nvidia.com> > --- > .../Acpi/Arm/AcpiIortLibArm/IortGenerator.c | 35 +++++++++++++++---- > 1 file changed, 28 insertions(+), 7 deletions(-) > > diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > index f28973c1a8..a6e4b49cb1 100644 > --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > @@ -5,7 +5,7 @@ > SPDX-License-Identifier: BSD-2-Clause-Patent > > @par Reference(s): > - - IO Remapping Table, Platform Design Document, Revision E.d, Feb 2022 > + - IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022 > (https://developer.arm.com/documentation/den0049/) > > **/ > @@ -1554,9 +1554,14 @@ AddSmmuV3Nodes ( > { > SmmuV3Node->Node.Revision = 2; > SmmuV3Node->Node.Identifier = EFI_ACPI_RESERVED_DWORD; > - } else { > + } else if (AcpiTableInfo->AcpiTableRevision < > + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06) > + { > SmmuV3Node->Node.Revision = 4; > SmmuV3Node->Node.Identifier = NodeList->Identifier; > + } else { > + SmmuV3Node->Node.Revision = 5; > + SmmuV3Node->Node.Identifier = NodeList->Identifier; > } > > // SMMUv3 specific data > @@ -1577,11 +1582,27 @@ AddSmmuV3Nodes ( > SmmuV3Node->ProximityDomain = 0; > } > > - if ((SmmuV3Node->Event != 0) && (SmmuV3Node->Pri != 0) && > - (SmmuV3Node->Gerr != 0) && (SmmuV3Node->Sync != 0)) > + /* For older SMMUV3 nodes rev. < 5. > + If all the SMMU control interrupts are GSIV based, > + the DeviceID mapping index field is ignored. > + DeviceID mapping valid flag was introduced in IORT rev E.e > + for SMMUV3 nodes rev. > 5. > + If the DeviceID mapping index valid flag is set to 0, > + DeviceID mapping index field must be ignored. > + Where the SMMU uses message signaled interrupts for > + its control interrupts, DeviceId Mapping Index contains an > + index into the array of ID mapping. > + */ > + if (((SmmuV3Node->Node.Revision < 5) && > + (SmmuV3Node->Event != 0) && > + (SmmuV3Node->Pri != 0) && > + (SmmuV3Node->Gerr != 0) && > + (SmmuV3Node->Sync != 0) > + ) || > + ((SmmuV3Node->Node.Revision >= 5) && > + ((SmmuV3Node->Flags & EFI_ACPI_IORT_SMMUv3_FLAG_DEVICEID_VALID) == 0)) > + ) > { > - // If all the SMMU control interrupts are GSIV based, > - // the DeviceID mapping index field is ignored. > SmmuV3Node->DeviceIdMappingIndex = 0; > } else { > SmmuV3Node->DeviceIdMappingIndex = NodeList->DeviceIdMappingIndex; > @@ -2819,7 +2840,7 @@ ACPI_IORT_GENERATOR IortGenerator = { > // ACPI Table Signature > EFI_ACPI_6_4_IO_REMAPPING_TABLE_SIGNATURE, > // ACPI Table Revision supported by this Generator > - EFI_ACPI_IO_REMAPPING_TABLE_REVISION_05, > + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06, > // Minimum supported ACPI Table Revision > EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00, > // Creator ID IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109038): https://edk2.groups.io/g/devel/message/109038 Mute This Topic: https://groups.io/mt/101535844/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec 2023-09-22 21:40 [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec Swatisri Kantamsetti via groups.io 2023-09-25 9:19 ` Sami Mujawar @ 2023-09-25 9:40 ` Leif Lindholm 2023-09-25 9:47 ` Sami Mujawar 1 sibling, 1 reply; 4+ messages in thread From: Leif Lindholm @ 2023-09-25 9:40 UTC (permalink / raw) To: devel, swatisrik; +Cc: Sami.Mujawar, pierre.gondois On Fri, Sep 22, 2023 at 15:40:38 -0600, Swatisri Kantamsetti via groups.io wrote: > The IO Remapping Table, Platform Design Document, Revision E.e, > Sept 2022 (https://developer.arm.com/documentation/den0049/ee) > added flags in SMMUv3 node for validity of ID mappings for MSIs > related to control interrupts. > It makes one small addition to SMMUv3 nodes to > describe MSI support independently of wired GSIV support > > Therefore, update the IORT generator to: > - increment IORT table revision to 6 > - increment SMMUV3 node revision to 5 > - for SMMUV3 node revision >=5 check the DeviceID mapping index > valid flag to populate DeviceIdMappingIndex field > > Signed-off-by: Swatisri Kantamsetti <swatisrik@nvidia.com> > --- > .../Acpi/Arm/AcpiIortLibArm/IortGenerator.c | 35 +++++++++++++++---- > 1 file changed, 28 insertions(+), 7 deletions(-) > > diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > index f28973c1a8..a6e4b49cb1 100644 > --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c > @@ -5,7 +5,7 @@ > SPDX-License-Identifier: BSD-2-Clause-Patent > > @par Reference(s): > - - IO Remapping Table, Platform Design Document, Revision E.d, Feb 2022 > + - IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022 > (https://developer.arm.com/documentation/den0049/) > > **/ > @@ -1554,9 +1554,14 @@ AddSmmuV3Nodes ( > { > SmmuV3Node->Node.Revision = 2; > SmmuV3Node->Node.Identifier = EFI_ACPI_RESERVED_DWORD; > - } else { > + } else if (AcpiTableInfo->AcpiTableRevision < > + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06) > + { > SmmuV3Node->Node.Revision = 4; > SmmuV3Node->Node.Identifier = NodeList->Identifier; > + } else { > + SmmuV3Node->Node.Revision = 5; > + SmmuV3Node->Node.Identifier = NodeList->Identifier; This is borderline, but I think it would be worth breaking out into a helper function. It feels like this may have further cases added in future. > } > > // SMMUv3 specific data > @@ -1577,11 +1582,27 @@ AddSmmuV3Nodes ( > SmmuV3Node->ProximityDomain = 0; > } > > - if ((SmmuV3Node->Event != 0) && (SmmuV3Node->Pri != 0) && > - (SmmuV3Node->Gerr != 0) && (SmmuV3Node->Sync != 0)) > + /* For older SMMUV3 nodes rev. < 5. > + If all the SMMU control interrupts are GSIV based, > + the DeviceID mapping index field is ignored. > + DeviceID mapping valid flag was introduced in IORT rev E.e > + for SMMUV3 nodes rev. > 5. > + If the DeviceID mapping index valid flag is set to 0, > + DeviceID mapping index field must be ignored. > + Where the SMMU uses message signaled interrupts for > + its control interrupts, DeviceId Mapping Index contains an > + index into the array of ID mapping. > + */ > + if (((SmmuV3Node->Node.Revision < 5) && > + (SmmuV3Node->Event != 0) && > + (SmmuV3Node->Pri != 0) && > + (SmmuV3Node->Gerr != 0) && > + (SmmuV3Node->Sync != 0) > + ) || > + ((SmmuV3Node->Node.Revision >= 5) && > + ((SmmuV3Node->Flags & EFI_ACPI_IORT_SMMUv3_FLAG_DEVICEID_VALID) == 0)) > + ) This is not borderline. I'm sure all of this is correct, but it is no longer human readable. This type of information overload is better kept out of a top-level function. This also means we can give it an informative name. Best Regards, Leif p.s. Apologies Sami, but since I spotted this before it had been merged by CI, I dropped the "push" tag since I wanted to make a comment. You are the maintainer, and the above are my opinions only. If you disagree, feel free to add it back and push as is. (And feel free to do the same to push requests initiated by me in future.) > { > - // If all the SMMU control interrupts are GSIV based, > - // the DeviceID mapping index field is ignored. > SmmuV3Node->DeviceIdMappingIndex = 0; > } else { > SmmuV3Node->DeviceIdMappingIndex = NodeList->DeviceIdMappingIndex; > @@ -2819,7 +2840,7 @@ ACPI_IORT_GENERATOR IortGenerator = { > // ACPI Table Signature > EFI_ACPI_6_4_IO_REMAPPING_TABLE_SIGNATURE, > // ACPI Table Revision supported by this Generator > - EFI_ACPI_IO_REMAPPING_TABLE_REVISION_05, > + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06, > // Minimum supported ACPI Table Revision > EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00, > // Creator ID > -- > 2.17.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109039): https://edk2.groups.io/g/devel/message/109039 Mute This Topic: https://groups.io/mt/101535844/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/12367111/7686176/1913456212/xyzzy [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec 2023-09-25 9:40 ` Leif Lindholm @ 2023-09-25 9:47 ` Sami Mujawar 0 siblings, 0 replies; 4+ messages in thread From: Sami Mujawar @ 2023-09-25 9:47 UTC (permalink / raw) To: Leif Lindholm, devel, swatisrik; +Cc: pierre.gondois, nd@arm.com Hi Leif, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 25/09/2023 10:40 am, Leif Lindholm wrote: > On Fri, Sep 22, 2023 at 15:40:38 -0600, Swatisri Kantamsetti via groups.io wrote: >> The IO Remapping Table, Platform Design Document, Revision E.e, >> Sept 2022 (https://developer.arm.com/documentation/den0049/ee) >> added flags in SMMUv3 node for validity of ID mappings for MSIs >> related to control interrupts. >> It makes one small addition to SMMUv3 nodes to >> describe MSI support independently of wired GSIV support >> >> Therefore, update the IORT generator to: >> - increment IORT table revision to 6 >> - increment SMMUV3 node revision to 5 >> - for SMMUV3 node revision >=5 check the DeviceID mapping index >> valid flag to populate DeviceIdMappingIndex field >> >> Signed-off-by: Swatisri Kantamsetti <swatisrik@nvidia.com> >> --- >> .../Acpi/Arm/AcpiIortLibArm/IortGenerator.c | 35 +++++++++++++++---- >> 1 file changed, 28 insertions(+), 7 deletions(-) >> >> diff --git a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c >> index f28973c1a8..a6e4b49cb1 100644 >> --- a/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c >> +++ b/DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c >> @@ -5,7 +5,7 @@ >> SPDX-License-Identifier: BSD-2-Clause-Patent >> >> @par Reference(s): >> - - IO Remapping Table, Platform Design Document, Revision E.d, Feb 2022 >> + - IO Remapping Table, Platform Design Document, Revision E.e, Sept 2022 >> (https://developer.arm.com/documentation/den0049/) >> >> **/ >> @@ -1554,9 +1554,14 @@ AddSmmuV3Nodes ( >> { >> SmmuV3Node->Node.Revision = 2; >> SmmuV3Node->Node.Identifier = EFI_ACPI_RESERVED_DWORD; >> - } else { >> + } else if (AcpiTableInfo->AcpiTableRevision < >> + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06) >> + { >> SmmuV3Node->Node.Revision = 4; >> SmmuV3Node->Node.Identifier = NodeList->Identifier; >> + } else { >> + SmmuV3Node->Node.Revision = 5; >> + SmmuV3Node->Node.Identifier = NodeList->Identifier; > This is borderline, but I think it would be worth breaking out into a > helper function. It feels like this may have further cases added in > future. > >> } >> >> // SMMUv3 specific data >> @@ -1577,11 +1582,27 @@ AddSmmuV3Nodes ( >> SmmuV3Node->ProximityDomain = 0; >> } >> >> - if ((SmmuV3Node->Event != 0) && (SmmuV3Node->Pri != 0) && >> - (SmmuV3Node->Gerr != 0) && (SmmuV3Node->Sync != 0)) >> + /* For older SMMUV3 nodes rev. < 5. >> + If all the SMMU control interrupts are GSIV based, >> + the DeviceID mapping index field is ignored. >> + DeviceID mapping valid flag was introduced in IORT rev E.e >> + for SMMUV3 nodes rev. > 5. >> + If the DeviceID mapping index valid flag is set to 0, >> + DeviceID mapping index field must be ignored. >> + Where the SMMU uses message signaled interrupts for >> + its control interrupts, DeviceId Mapping Index contains an >> + index into the array of ID mapping. >> + */ >> + if (((SmmuV3Node->Node.Revision < 5) && >> + (SmmuV3Node->Event != 0) && >> + (SmmuV3Node->Pri != 0) && >> + (SmmuV3Node->Gerr != 0) && >> + (SmmuV3Node->Sync != 0) >> + ) || >> + ((SmmuV3Node->Node.Revision >= 5) && >> + ((SmmuV3Node->Flags & EFI_ACPI_IORT_SMMUv3_FLAG_DEVICEID_VALID) == 0)) >> + ) > This is not borderline. I'm sure all of this is correct, but it is no > longer human readable. This type of information overload is better > kept out of a top-level function. > This also means we can give it an informative name. > > Best Regards, > > Leif > > p.s. > Apologies Sami, but since I spotted this before it had been merged by > CI, I dropped the "push" tag since I wanted to make a comment. > You are the maintainer, and the above are my opinions only. If you > disagree, feel free to add it back and push as is. > (And feel free to do the same to push requests initiated by me in > future.) [SAMI] I agree, this is becoming complex to read. I think it would be good to have helper functions to make the code clearer. I have closed the pull request and look forward to a new patch from Swatisri. [/SAMI] >> { >> - // If all the SMMU control interrupts are GSIV based, >> - // the DeviceID mapping index field is ignored. >> SmmuV3Node->DeviceIdMappingIndex = 0; >> } else { >> SmmuV3Node->DeviceIdMappingIndex = NodeList->DeviceIdMappingIndex; >> @@ -2819,7 +2840,7 @@ ACPI_IORT_GENERATOR IortGenerator = { >> // ACPI Table Signature >> EFI_ACPI_6_4_IO_REMAPPING_TABLE_SIGNATURE, >> // ACPI Table Revision supported by this Generator >> - EFI_ACPI_IO_REMAPPING_TABLE_REVISION_05, >> + EFI_ACPI_IO_REMAPPING_TABLE_REVISION_06, >> // Minimum supported ACPI Table Revision >> EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00, >> // Creator ID >> -- >> 2.17.1 >> >> >> >> >> >> -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109040): https://edk2.groups.io/g/devel/message/109040 Mute This Topic: https://groups.io/mt/101535844/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=- ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-09-25 9:47 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-09-22 21:40 [edk2-devel] [PATCH] DynamicTablesPkg: IORT generator updates for Rev E.e spec Swatisri Kantamsetti via groups.io 2023-09-25 9:19 ` Sami Mujawar 2023-09-25 9:40 ` Leif Lindholm 2023-09-25 9:47 ` Sami Mujawar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox