On Tue, Aug 25, 2020 at 19:09:50 +0530, Tanmay Jagdale wrote: > This patch series adds ACPI tables support for the SbsaQemu platform. > We are using a pseudo static approach to create the ACPI tables. > > The ACPI tables namely DBG2, DSDT, MCFG, SPCR, GTDT are created in a > static way at compile time because they hold a fixed configuration > and there are no changes at runtime. > > The MADT, SSDT and PPTT tables are dependant on the number of CPUs and > hence they are created at runtime based on the number of CPUs the user > has requested > > Changes in v3: > - Dropped ASSERT() in CountCpusFromFdt() function in patch 4. > - Changed EFI_D_ERROR to DEBUG_ERROR > > Changes in v2: > - Moved PcdCoreCount and Fdtlib related changes in SbsaQemu.dsc to a > separate patch (Patch 3). > - Removed Acpi6x.h header file includes and used IndustryStandard/Acpi.h > - Whitespace cleanups > - Added proper code comments
Thanks for quick respin. For the series: Reviewed-by: Leif Lindholm <l...@nuviainc.com> Pushed as 23863e9e9d40..90c3b3bc0c3e. > Tanmay Jagdale (8): > SbsaQemu: Initial support for static ACPI tables > SbsaQemu: AcpiTables: Add PCI support and MCFG Table > SbsaQemu: SbsaQemu.dsc: Move CoreCount and Fdtlib > SbsaQemu: Add new ACPI driver and FDT parser to count CPUs > SbsaQemu: AcpiDxe: Create MADT table at runtime > SbsaQemu: AcpiDxe: Create SSDT table at runtime > SbsaQemu: AcpiDxe: Create PPTT table at runtime > SbsaQemu: AcpiTables: Add DBG2 Table > > Silicon/Qemu/SbsaQemu/SbsaQemu.dec | 8 +- > Silicon/Qemu/SbsaQemu/Acpi.dsc.inc | 36 ++ > Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 12 +- > Platform/Qemu/SbsaQemu/SbsaQemu.fdf | 9 + > Silicon/Qemu/SbsaQemu/AcpiTables/AcpiTables.inf | 47 ++ > Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf | 67 +++ > Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h | 199 > ++++++++ > Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 485 > ++++++++++++++++++++ > Silicon/Qemu/SbsaQemu/AcpiTables/Dbg2.aslc | 67 +++ > Silicon/Qemu/SbsaQemu/AcpiTables/Dsdt.asl | 449 > ++++++++++++++++++ > Silicon/Qemu/SbsaQemu/AcpiTables/Fadt.aslc | 80 ++++ > Silicon/Qemu/SbsaQemu/AcpiTables/Gtdt.aslc | 67 +++ > Silicon/Qemu/SbsaQemu/AcpiTables/Mcfg.aslc | 43 ++ > Silicon/Qemu/SbsaQemu/AcpiTables/Spcr.aslc | 53 +++ > 14 files changed, 1618 insertions(+), 4 deletions(-) > create mode 100644 Silicon/Qemu/SbsaQemu/Acpi.dsc.inc > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/AcpiTables.inf > create mode 100644 > Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf > create mode 100644 > Silicon/Qemu/SbsaQemu/Include/IndustryStandard/SbsaQemuAcpi.h > create mode 100644 > Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Dbg2.aslc > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Dsdt.asl > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Fadt.aslc > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Gtdt.aslc > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Mcfg.aslc > create mode 100644 Silicon/Qemu/SbsaQemu/AcpiTables/Spcr.aslc > > -- > 2.28.0 > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#64620): https://edk2.groups.io/g/devel/message/64620 Mute This Topic: https://groups.io/mt/76406662/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-