Thank you for the review, Pierre! We will update the patches per your feedback and send v2 after validation.

Regards,
Kun

On 7/20/2022 6:38 AM, PierreGondois wrote:
Except for:
[PATCH v1 4/6] DynamicTablesPkg: DynamicTableManagerDxe: Added check for installed tables [PATCH v1 5/6] DynamicTablesPkg: AcpiSsdtPcieLibArm: Added function to reserve ECAM space
where I had some remarks,

Reviewed-by: Pierre Gondois <pierre.gond...@arm.com>

Thanks!

On 7/19/22 02:22, Kun Qin via groups.io wrote:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3996
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3997
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3998

Current DynamicTablesPkg provide great support for creating dynamic ACPI
tables during boot time.

However, there are some modules needs minor tweaks to expand support and
compatibility for OS requirements and platform needs.

This patch series proposes a few fixes to resolve minor issues discovered
in DynamicPlatRepoLib, AcpiSsdtPcieLibArm and DynamicTableManagerDxe.

Patch v1 branch: https://github.com/kuqin12/edk2/tree/dynamic_update

Cc: Sami Mujawar <sami.muja...@arm.com>
Cc: Alexei Fedorov <alexei.fedo...@arm.com>

Kun Qin (6):
   DynamicTablesPkg: DynamicPlatRepoLib: Added MemoryAllocationLib to inf
   DynamicTablesPkg: DynamicPlatRepoLib: Fix incorrect dereferencing
   DynamicTablesPkg: DynamicPlatRepoLib: Adding more token fixers
   DynamicTablesPkg: DynamicTableManagerDxe: Added check for installed
     tables
   DynamicTablesPkg: AcpiSsdtPcieLibArm: Added function to reserve ECAM
     space
   DynamicTablesPkg: AcpiSsdtPcieLibArm: Added case handling for PCI
     config

DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c | 200 ++++++++++++++++---- DynamicTablesPkg/Library/Acpi/Arm/AcpiSsdtPcieLibArm/SsdtPcieGenerator.c | 135 +++++++++++++ DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.c |  80 +++++++- DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.inf |   1 + DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoLib.inf |   1 +
  5 files changed, 379 insertions(+), 38 deletions(-)








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#91611): https://edk2.groups.io/g/devel/message/91611
Mute This Topic: https://groups.io/mt/92473108/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to