Hi Pierre,

Thank you for this patch series.

These changes look good to me.

For this series,

Reviewed-by: Sami Mujawar <sami.muja...@arm.com>

Regards,

Sami Mujawar

On 10/06/2024, 10:34, "pierre.gond...@arm.com <mailto:pierre.gond...@arm.com>" 
<pierre.gond...@arm.com <mailto:pierre.gond...@arm.com>> wrote:


From: Pierre Gondois <pierre.gond...@arm.com <mailto:pierre.gond...@arm.com>>


Patches for the DynamicTablesPkg reorg branches are still sent
through the mailing list for now. A branch with the patches is
available at:
https://github.com/pierregondois/edk2-platforms/pull/new/pg/Morello_N1Sdp_NXP_plat_reorg
 
<https://github.com/pierregondois/edk2-platforms/pull/new/pg/Morello_N1Sdp_NXP_plat_reorg>
 


References:
- edk2 reorg branch:
https://github.com/tianocore/edk2-staging/tree/dynamictables-reorg 
<https://github.com/tianocore/edk2-staging/tree/dynamictables-reorg>
- edk2-platforms reorg branch:
https://github.com/tianocore/edk2-platforms/tree/devel-dynamictables-reorg 
<https://github.com/tianocore/edk2-platforms/tree/devel-dynamictables-reorg>
- Previous serie for FVP, Juno platforms:
https://edk2.groups.io/g/devel/message/117001 
<https://edk2.groups.io/g/devel/message/117001>


Other architectures than Arm aim to leverage the DynamicTablesPkg.
The package uses namespaces/domains to split objects IDs. Currently
most of the objects are part of the Arm namespace. Some objects are
generic enough to be re-used by other architectures.


Update the following platforms which use the DynamicTablesPkg
framework/objects:
- Platform/ARM/N1Sdp
- Platform/ARM/Morello
- Platform/NXP/ConfigurationManagerPkg
now that the following platforms have already been updated:
- Juno
- VExpress


Cc: Sami Mujawar <sami.muja...@arm.com <mailto:sami.muja...@arm.com>>
Cc: Pierre Gondois <pierre.gond...@arm.com <mailto:pierre.gond...@arm.com>>
Cc: Yeo Reum Yun <yeoreum....@arm.com <mailto:yeoreum....@arm.com>>
Cc: Sunil V L <suni...@ventanamicro.com <mailto:suni...@ventanamicro.com>>
Cc: AbdulLateef Attar <abdullateef.at...@amd.com 
<mailto:abdullateef.at...@amd.com>>
Cc: Jeshua Smith <jesh...@nvidia.com <mailto:jesh...@nvidia.com>>
Cc: Jeff Brasen <jbra...@nvidia.com <mailto:jbra...@nvidia.com>>
Cc: Girish Mahadevan <gmahade...@nvidia.com <mailto:gmahade...@nvidia.com>>
Cc: Leif Lindholm <quic_llind...@quicinc.com <mailto:quic_llind...@quicinc.com>>
Cc: Meenakshi Aggarwal <meenakshi.aggar...@nxp.com 
<mailto:meenakshi.aggar...@nxp.com>>


Pierre Gondois (14):
Platform/ARM: Morello,N1Sdp: Add Arch Common objects handler
Platform/ARM: Morello,N1Sdp: Move Power Mgmt Profile info to Arch
Common
Platform/ARM: Morello,N1Sdp: Move Serial Port info to Arch Common
Platform/ARM: Morello,N1Sdp: Move Fixed Feat Flags info to Arch Common
Platform/ARM: Morello,N1Sdp: Move CmRef info to Arch Common
Platform/ARM: Morello,N1Sdp: Move Pci Config Space info to Arch Common
Platform/ARM: Morello,N1Sdp: Rename GicCToken
Platform/ARM: Morello,N1Sdp: Move Proc Hierarchy info to Arch Common
Platform/ARM: Morello,N1Sdp: Move Cache info to Arch Common
Platform/ARM: N1Sdp: Move Memory Affinity info to Arch Common
Platform/NXP: Add Arch Common objects handler
Platform/NXP: Move Power Mgmt Profile info to Arch Common
Platform/NXP: Move Serial Port info to Arch Common
Platform/NXP: Move Pci Config Space info to Arch Common


.../ConfigurationManager.c | 216 +++++++++-----
.../ConfigurationManager.h | 20 +-
.../ConfigurationManagerFvp.c | 65 +++-
.../ConfigurationManagerFvp.h | 2 +-
.../ConfigurationManager.c | 281 +++++++++++-------
.../ConfigurationManager.h | 24 +-
.../ConfigurationManager.c | 120 ++++++--
.../ConfigurationManager.h | 7 +-
8 files changed, 473 insertions(+), 262 deletions(-)


-- 
2.25.1







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


Reply via email to