Re: [edk2-devel] [platforms/devel-dynamictables-reorg PATCH 00/14] Platform: N1Sdp,Morello,NXP: Move DynamicTablesPkg obj to Arch Common

["Sami Mujawar" <sami.mujawar@arm.com>]

Hi Pierre,

Thank you for this patch series.

These changes look good to me.

For this series,

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

Regards,

Sami Mujawar

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


From: Pierre Gondois <pierre.gondois@arm.com <mailto:pierre.gondois@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.mujawar@arm.com <mailto:sami.mujawar@arm.com>>
Cc: Pierre Gondois <Pierre.Gondois@arm.com <mailto:Pierre.Gondois@arm.com>>
Cc: Yeo Reum Yun <YeoReum.Yun@arm.com <mailto:YeoReum.Yun@arm.com>>
Cc: Sunil V L <sunilvl@ventanamicro.com <mailto:sunilvl@ventanamicro.com>>
Cc: AbdulLateef Attar <AbdulLateef.Attar@amd.com <mailto:AbdulLateef.Attar@amd.com>>
Cc: Jeshua Smith <jeshuas@nvidia.com <mailto:jeshuas@nvidia.com>>
Cc: Jeff Brasen <jbrasen@nvidia.com <mailto:jbrasen@nvidia.com>>
Cc: Girish Mahadevan <gmahadevan@nvidia.com <mailto:gmahadevan@nvidia.com>>
Cc: Leif Lindholm <quic_llindhol@quicinc.com <mailto:quic_llindhol@quicinc.com>>
Cc: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com <mailto:meenakshi.aggarwal@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/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-