Re: [edk2-devel] [edk2-platforms][PATCH v4 0/9]Platform/Arm/Sgi: Add platform support for RD-Daniel

["Aditya Angadi" <aditya.angadi@arm.com>]

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Thomas
> Abraham via groups.io
> Sent: 15 April 2020 10:05
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.com; Aditya
> Angadi <Aditya.Angadi@arm.com>
> Subject: Re: [edk2-devel] [edk2-platforms][PATCH v4 0/9]Platform/Arm/Sgi:
> Add platform support for RD-Daniel
>
> On Tue, Apr 14, 2020 at 6:22 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.
> > Config XLR is a multi chip platform.
> >
> > The first four patches consolidate the code in SgiPkg in preparation
> > for adding support for numa memory nodes and the RD-Daniel platform.
> > The next four patches add support for the RD-Daniel platforms and the
> > last patch adds SRAT table for dual chip RdN1Edge platform.
> >
> >
> > Aditya Angadi (6):
> >   Platform/ARM/Sgi: Move the GIC related ACPI helper macros
> >   Platform/ARM/Sgi: Move common platform description to SSDT
> >   Platform/ARM/Sgi: Add ACPI tables for Rd-Daniel Config-M
> >   Platform/ARM/Sgi: Add initial support for RD-Daniel Config-M platform
> >   Platform/ARM/Sgi: Add ACPI tables for RD-Daniel Config-XLR
> >   Platform/ARM/Sgi: Add initial support for RD-Daniel Config-XLR
> >     platform
> >
> > Vijayenthiran Subramaniam (3):
> >   Platform/ARM/Sgi: Create individual Platform Description File
> >   Platform/ARM/Sgi: Add support for remote numa memory nodes
> >   Platform/ARM/Sgi: Add SRAT table for RdN1Edge dual-chip platform
>
> For the series:
> Reviewed-by: Thomas Abraham <thomas.abraham@arm.com>

Any other comments to address in this patch series

Thank You
Aditya

>
> >
> >  .../SgiPkg/AcpiTables/RdDanielCfgM/Dsdt.asl   | 118 +++++++++++++
> >  .../SgiPkg/AcpiTables/RdDanielCfgM/Madt.aslc  | 141 ++++++++++++++++
> > ...iTables.inf => RdDanielCfgMAcpiTables.inf} |  24 ++-
> > .../SgiPkg/AcpiTables/RdDanielCfgXlr/Dsdt.asl | 125 ++++++++++++++
> >  .../AcpiTables/RdDanielCfgXlr/Madt.aslc       | 158 ++++++++++++++++++
> >  .../AcpiTables/RdDanielCfgXlr/Srat.aslc       | 102 +++++++++++
> >  .../AcpiTables/RdDanielCfgXlrAcpiTables.inf   |  87 ++++++++++
> >  .../ARM/SgiPkg/AcpiTables/RdE1Edge/Dsdt.asl   |  70 +-------
> >  .../ARM/SgiPkg/AcpiTables/RdE1Edge/Madt.aslc  |  71 +-------
> >  .../SgiPkg/AcpiTables/RdE1EdgeAcpiTables.inf  |   5 +-
> >  .../ARM/SgiPkg/AcpiTables/RdN1Edge/Dsdt.asl   |  69 +-------
> >  .../ARM/SgiPkg/AcpiTables/RdN1Edge/Madt.aslc  |  68 +-------
> >  .../SgiPkg/AcpiTables/RdN1EdgeAcpiTables.inf  |   5 +-
> >  .../SgiPkg/AcpiTables/RdN1EdgeX2/Madt.aslc    |  57 +------
> >  .../SgiPkg/AcpiTables/RdN1EdgeX2/Srat.aslc    |  90 ++++++++++
> >  .../AcpiTables/RdN1EdgeX2AcpiTables.inf       |  14 +-
> >  .../ARM/SgiPkg/AcpiTables/Sgi575/Madt.aslc    | 130 ++++----------
> >  .../SgiPkg/AcpiTables/Sgi575AcpiTables.inf    |   2 +-
> >  .../{RdN1Edge/Dsdt.asl => SsdtRos.asl}        |  63 ++-----
> >  .../SgiPkg/Drivers/PlatformDxe/PlatformDxe.c  |  69 +-------
> >  .../Drivers/PlatformDxe/PlatformDxe.inf       |   5 +-
> >  Platform/ARM/SgiPkg/Include/SgiAcpiHeader.h   |  89 +++++++++-
> >  Platform/ARM/SgiPkg/Include/SgiPlatform.h     |  12 +-
> >  .../Library/PlatformLib/PlatformLib.inf       |  21 ++-
> >  .../Library/PlatformLib/PlatformLibMem.c      |  95 ++++++++++-
> >  .../ARM/SgiPkg/RdDanielCfgM/RdDanielCfgM.dsc  |  51 ++++++
> > .../SgiPkg/RdDanielCfgM/RdDanielCfgM.fdf.inc  |  10 ++
> > .../SgiPkg/RdDanielCfgXlr/RdDanielCfgXlr.dsc  |  70 ++++++++
> >  .../RdDanielCfgXlr/RdDanielCfgXlr.fdf.inc     |  10 ++
> >  Platform/ARM/SgiPkg/RdE1Edge/RdE1Edge.dsc     |  51 ++++++
> >  Platform/ARM/SgiPkg/RdE1Edge/RdE1Edge.fdf.inc |  10 ++
> >  Platform/ARM/SgiPkg/RdN1Edge/RdN1Edge.dsc     |  51 ++++++
> >  Platform/ARM/SgiPkg/RdN1Edge/RdN1Edge.fdf.inc |  10 ++
> > Platform/ARM/SgiPkg/RdN1EdgeX2/RdN1EdgeX2.dsc |  60 +++++++
> > .../ARM/SgiPkg/RdN1EdgeX2/RdN1EdgeX2.fdf.inc  |  10 ++
> >  Platform/ARM/SgiPkg/Sgi575/Sgi575.dsc         |  51 ++++++
> >  Platform/ARM/SgiPkg/Sgi575/Sgi575.fdf.inc     |  10 ++
> >  Platform/ARM/SgiPkg/SgiPlatform.dec           |  29 +++-
> >  .../{SgiPlatform.dsc => SgiPlatform.dsc.inc}  |  33 +---
> >  Platform/ARM/SgiPkg/SgiPlatform.fdf           |   5 +-
> >  40 files changed, 1532 insertions(+), 619 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/{Sgi575AcpiTables.inf =>
> > RdDanielCfgMAcpiTables.inf} (68%)  create mode 100644
> > Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlr/Dsdt.asl
> >  create mode 100644
> > Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlr/Madt.aslc
> >  create mode 100644
> > Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlr/Srat.aslc
> >  create mode 100644
> > Platform/ARM/SgiPkg/AcpiTables/RdDanielCfgXlrAcpiTables.inf
> >  create mode 100644
> > Platform/ARM/SgiPkg/AcpiTables/RdN1EdgeX2/Srat.aslc
> >  copy Platform/ARM/SgiPkg/AcpiTables/{RdN1Edge/Dsdt.asl =>
> > SsdtRos.asl} (58%)  create mode 100644
> > Platform/ARM/SgiPkg/RdDanielCfgM/RdDanielCfgM.dsc
> >  create mode 100644
> > Platform/ARM/SgiPkg/RdDanielCfgM/RdDanielCfgM.fdf.inc
> >  create mode 100644
> > Platform/ARM/SgiPkg/RdDanielCfgXlr/RdDanielCfgXlr.dsc
> >  create mode 100644
> > Platform/ARM/SgiPkg/RdDanielCfgXlr/RdDanielCfgXlr.fdf.inc
> >  create mode 100644 Platform/ARM/SgiPkg/RdE1Edge/RdE1Edge.dsc
> >  create mode 100644 Platform/ARM/SgiPkg/RdE1Edge/RdE1Edge.fdf.inc
> >  create mode 100644 Platform/ARM/SgiPkg/RdN1Edge/RdN1Edge.dsc
> >  create mode 100644 Platform/ARM/SgiPkg/RdN1Edge/RdN1Edge.fdf.inc
> >  create mode 100644 Platform/ARM/SgiPkg/RdN1EdgeX2/RdN1EdgeX2.dsc
> >  create mode 100644
> Platform/ARM/SgiPkg/RdN1EdgeX2/RdN1EdgeX2.fdf.inc
> >  create mode 100644 Platform/ARM/SgiPkg/Sgi575/Sgi575.dsc
> >  create mode 100644 Platform/ARM/SgiPkg/Sgi575/Sgi575.fdf.inc
> >  rename Platform/ARM/SgiPkg/{SgiPlatform.dsc => SgiPlatform.dsc.inc}
> > (87%)
> >
> > --
> > 2.17.1
> >
> >
> > ------------
> > Groups.io Links: You receive all messages sent to this group.
> >
> > View/Reply Online (#57334):
> > https://edk2.groups.io/g/devel/message/57334
> > Mute This Topic: https://groups.io/mt/73009389/1785013
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > [ta.omasab@gmail.com]
> > ------------
> >
>
> 

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.