From: "Thomas Abraham" <thomas.abraham@arm.com>
To: devel@edk2.groups.io, aditya.angadi@arm.com
Subject: Re: [edk2-devel] [edk2-platforms][PATCH v3 0/8] Add platform support for RD-Daniel
Date: Sun, 29 Mar 2020 22:11:45 +0530 [thread overview]
Message-ID: <CAJuA9ahoz-TzboGGS9-C7bjxKKo3V3xo6DivLjZvCdsiNJME7Q@mail.gmail.com> (raw)
In-Reply-To: <20200325105252.13905-1-aditya.angadi@arm.com>
Hi Aditya,
On Wed, Mar 25, 2020 at 4:23 PM Aditya Angadi <aditya.angadi@arm.com> wrote:
>
> This patch series adds support for Arm's RD-Daniel platform. There are two
> configurations of this platform being added in this series - Config-M and
> Config-XLR. RD-Daniel is the next Arm's reference design subsystem.
>
> The first four patches consolidate the code in SgiPkg in preparation for
> adding support for the RD-Daniel platform. The next four patches add support
> for the RD-Daniel platforms.
>
> Aditya Angadi (7):
> Platform/ARM/SgiPkg: move the GIC related ACPI helper macros
> Platform/ARM/SgiPkg: move common platform description to SSDT
> Platform/ARM/SgiPkg: remove PcdCoreCount and PcdClusterCount
> Platform/ARM/SgiPkg: add ACPI tables for Rd-Daniel Config-M
> Platform/ARM/Sgi: add initial support for RD-Daniel Config-M platform
> Platform/ARM/SgiPkg: add ACPI tables for RD-Daniel Config-XLR
> Platform/ARM/Sgi: add initial support for RD-Daniel Config-XLR
> platform
>
> Thomas Abraham (1):
> Maintainers.txt: Update Arm platform reviewers list
>
> Vijayenthiran Subramaniam (1):
> Platform/ARM/SgiPkg: create individual Platform Description File
Few comments on this patch series.
- The patch subject of all the patches seem to be have been truncated.
- Copyright message in patches that add new files is incorrect.
- Patches 2/9 and 4/9 are missing on the list.
Could you check about these and post the next version.
Thanks,
Thomas.
>
> Maintainers.txt | 1 +
> .../SgiPkg/AcpiTables/RdDanielCfgM/Dsdt.asl | 118 ++++++++++++++
> .../SgiPkg/AcpiTables/RdDanielCfgM/Madt.aslc | 134 ++++++++++++++++
> ...iTables.inf => RdDanielCfgMAcpiTables.inf} | 20 ++-
> .../SgiPkg/AcpiTables/RdDanielCfgXlr/Dsdt.asl | 125 +++++++++++++++
> .../AcpiTables/RdDanielCfgXlr/Madt.aslc | 150 ++++++++++++++++++
> ...ables.inf => RdDanielCfgXlrAcpiTables.inf} | 20 ++-
> .../ARM/SgiPkg/AcpiTables/RdE1Edge/Dsdt.asl | 70 +-------
> .../ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc | 73 +--------
> .../SgiPkg/AcpiTables/RdE1EdgeAcpiTables.inf | 5 +-
> .../ARM/SgiPkg/AcpiTables/RdN1Edge/Dsdt.asl | 69 +-------
> .../ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc | 75 +--------
> .../SgiPkg/AcpiTables/RdN1EdgeAcpiTables.inf | 5 +-
> .../SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc | 69 +-------
> .../AcpiTables/RdN1EdgeX2AcpiTables.inf | 3 +-
> .../ARM/SgiPkg/AcpiTables/Sgi575/Madt.aslc | 9 +-
> .../SgiPkg/AcpiTables/Sgi575AcpiTables.inf | 4 +-
> .../{RdN1Edge/Dsdt.asl => SsdtRos.asl} | 67 ++------
> .../SgiPkg/Drivers/PlatformDxe/PlatformDxe.c | 10 ++
> .../Drivers/PlatformDxe/PlatformDxe.inf | 2 +
> Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h | 70 +++++++-
> Platform/ARM/SgiPkg/Include/SgiPlatform.h | 12 +-
> .../Library/PlatformLib/PlatformLib.inf | 6 +-
> .../Library/PlatformLib/PlatformLibMem.c | 8 +-
> Platform/ARM/SgiPkg/RdDaniel.dsc | 37 +++++
> Platform/ARM/SgiPkg/RdE1Edge.dsc | 37 +++++
> Platform/ARM/SgiPkg/RdN1Edge.dsc | 37 +++++
> Platform/ARM/SgiPkg/Sgi575.dsc | 37 +++++
> Platform/ARM/SgiPkg/SgiPlatform.dec | 7 +-
> Platform/ARM/SgiPkg/SgiPlatform.dsc | 31 +---
> Platform/ARM/SgiPkg/SgiPlatform.fdf | 2 +
> 31 files changed, 848 insertions(+), 465 deletions(-)
> create mode 100644 Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgM/Dsdt.asl
> create mode 100644 Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgM/Madt.aslc
> copy Platform/ARM/SgiPkg/AcpiTables/{RdN1EdgeX2AcpiTables.inf => RdDanielCfgMAcpiTables.inf} (67%)
> create mode 100644 Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlr/Dsdt.asl
> create mode 100644 Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlr/Madt.aslc
> copy Platform/ARM/SgiPkg/AcpiTables/{RdN1EdgeX2AcpiTables.inf => RdDanielCfgXlrAcpiTables.inf} (67%)
> copy Platform/ARM/SgiPkg/AcpiTables/{RdN1Edge/Dsdt.asl => SsdtRos.asl} (57%)
> create mode 100644 Platform/ARM/SgiPkg/RdDaniel.dsc
> create mode 100644 Platform/ARM/SgiPkg/RdE1Edge.dsc
> create mode 100644 Platform/ARM/SgiPkg/RdN1Edge.dsc
> create mode 100644 Platform/ARM/SgiPkg/Sgi575.dsc
>
> --
> 2.17.1
>
>
> ------------
> Groups.io Links: You receive all messages sent to this group.
>
> View/Reply Online (#56274): https://edk2.groups.io/g/devel/message/56274
> Mute This Topic: https://groups.io/mt/72538017/1785013
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [ta.omasab@gmail.com]
> ------------
>
prev parent reply other threads:[~2020-03-29 16:42 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-25 10:52 [edk2-platforms][PATCH v3 0/8] Add platform support for RD-Daniel Aditya Angadi
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 1/9] Platform/ARM/SgiPkg: create individual Aditya Angadi
2020-03-31 10:33 ` Ard Biesheuvel
2020-03-31 10:42 ` Ard Biesheuvel
2020-04-13 12:31 ` [edk2-devel] " Aditya Angadi
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 2/9] Platform/ARM/SgiPkg: move the GIC Aditya Angadi
2020-03-31 10:35 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 3/9] Platform/ARM/SgiPkg: move common Aditya Angadi
2020-03-31 10:36 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 4/9] Platform/ARM/SgiPkg: remove Aditya Angadi
2020-03-31 10:37 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 5/9] Platform/ARM/SgiPkg: add ACPI tables Aditya Angadi
2020-03-31 10:40 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 6/9] Platform/ARM/Sgi: add initial support Aditya Angadi
2020-03-31 10:39 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 7/9] Platform/ARM/SgiPkg: add ACPI tables Aditya Angadi
2020-03-31 10:45 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 8/9] Platform/ARM/Sgi: add initial support Aditya Angadi
2020-03-31 10:46 ` Ard Biesheuvel
2020-03-25 10:52 ` [edk2-platforms][PATCH v3 9/9] Maintainers.txt: Update Arm platform Aditya Angadi
2020-03-31 10:48 ` Ard Biesheuvel
2020-03-29 16:41 ` Thomas Abraham [this message]
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=CAJuA9ahoz-TzboGGS9-C7bjxKKo3V3xo6DivLjZvCdsiNJME7Q@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