[edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to OvmfPkg
Yao, Jiewen
jiewen.yao at intel.com
Thu Sep 30 02:39:56 UTC 2021
For OvmfPkg change, Acked-by: Jiewen Yao <Jiewen.yao at intel.com>
Need ARM/RISC-V expert to give Reviewed-by.
Thank you
Yao Jiewen
> -----Original Message-----
> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Abner
> Chang
> Sent: Thursday, September 30, 2021 8:46 AM
> To: devel at edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>; Leif Lindholm
> <leif at nuviainc.com>; Sami Mujawar <sami.mujawar at arm.com>; Yao, Jiewen
> <jiewen.yao at intel.com>; Justen, Jordan L <jordan.l.justen at intel.com>; Gerd
> Hoffmann <kraxel at redhat.com>; Schaefer, Daniel <daniel.schaefer at hpe.com>;
> Sunil V L <sunilvl at ventanamicro.com>; Liming Gao
> <gaoliming at byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu at intel.com>; Kinney,
> Michael D <michael.d.kinney at intel.com>
> Subject: [edk2-devel] [PATCH V3 00/12] Migrate ArmVirtPkg modules to
> OvmfPkg
>
> In V3: Address comments on V2.
> In V2: Remove HPE license on the files that just moved around or
> the changes in the file are just code removal.
>
> edk2 BZ #: 3665
> edk2 platform corresponding changes will be submitted after
> this pactch set is reviewed.
>
> This pacthes set is to migrate some modules from ArmVirtPkg
> to under OvmfPkg for the upcoming RiscVVirtPkg that can leverage
> those modules without the dependency with Arm*Pkg.
>
> The modules moved from ArmVirtPkg to OvmfPkg are,
> - FdtClientDxe
> - PciPcdProducerLib
> - HighMemDxe
> - QemuFwCfgLib
> - FdtPciHostBridgeLib
> - VirtioFdtDxe
>
> Below PCDs are moved to under MdePkg and leverage by RiscVVirtPkg.
> This change also remove the dependency on ArmPkg of OvmfPkg.
> - PcdPciIoTranslation
> - PcdPciIoTranslation
> - PcdPciMmio32(64)Translation
>
> Signed-off-by: Abner Chang <abner.chang at hpe.com>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Cc: Sami Mujawar <sami.mujawar at arm.com>
> Cc: Jiewen Yao <jiewen.yao at intel.com>
> Cc: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Daniel Schaefer <daniel.schaefer at hpe.com>
> Cc: Sunil V L <sunilvl at ventanamicro.com>
> Cc: Liming Gao <gaoliming at byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu at intel.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
>
> Abner Chang (12):
> ArmVirtPkg/FdtClintDxe: Move FdtClientDxe to EmbeddedPkg
> MdePkg: Add PcdPciIoTranslation PCD
> ArmPkg: Use PcdPciIoTranslation PCD from MdePkg
> ArmVirtPkg/FdtPciPcdProducerLib: Relocate PciPcdProducerLib to OvmfPkg
> ArmVirtPkg/HighMemDxe: Relocate HighMemDxe to OvmfPkg
> OvmfPkg/HighMemDxe: Add RISC-V in the supported arch.
> ArmVirtPkg/QemuFwCfgLib: Relocate QemuFwCfgLib to OvmfPkg
> OvmfPkg/QemuFwCfgLibMMIO: Add RISC-V arch support
> MdePkg: Add PcdPciMmio32(64)Translation PCDs
> ArmVirtPkg/FdtPciHostBridgeLib: Relocate FdtPciHostBridgeLib to
> OvmfPkg/Fdt
> OvmfPkg/FdtPciHostBridgeLib: Add RISC-V in the supported arch.
> ArmVirtPkg/VirtioFdtDxe: Relocate VirtioFdtDxe to OvmfPkg/Fdt
>
> ArmPkg/ArmPkg.dec | 15 ++++++--------
> ArmVirtPkg/ArmVirtPkg.dec | 3 ---
> EmbeddedPkg/EmbeddedPkg.dec | 1 +
> MdePkg/MdePkg.dec | 12 +++++++++++
> ArmVirtPkg/ArmVirtCloudHv.dsc | 18 ++++++++---------
> ArmVirtPkg/ArmVirtKvmTool.dsc | 18 ++++++++---------
> ArmVirtPkg/ArmVirtQemu.dsc | 20 +++++++++----------
> ArmVirtPkg/ArmVirtQemuKernel.dsc | 20 +++++++++----------
> ArmVirtPkg/ArmVirtXen.dsc | 2 +-
> EmbeddedPkg/EmbeddedPkg.dsc | 1 +
> ArmVirtPkg/ArmVirtCloudHv.fdf | 6 +++---
> ArmVirtPkg/ArmVirtKvmTool.fdf | 6 +++---
> ArmVirtPkg/ArmVirtXen.fdf | 2 +-
> ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 6 +++---
> .../ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf | 2 +-
> .../ArmVirtGicArchLib/ArmVirtGicArchLib.inf | 1 +
> .../ArmVirtPL031FdtClientLib.inf | 1 +
> .../ArmVirtPsciResetSystemLib.inf | 1 +
> .../ArmVirtTimerFdtClientLib.inf | 1 +
> .../KvmtoolRtcFdtClientLib.inf | 1 +
> .../NorFlashKvmtoolLib/NorFlashKvmtoolLib.inf | 1 +
> .../NorFlashQemuLib/NorFlashQemuLib.inf | 1 +
> .../XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf | 1 +
> ArmVirtPkg/XenioFdtDxe/XenioFdtDxe.inf | 1 +
> .../Drivers}/FdtClientDxe/FdtClientDxe.inf | 1 -
> .../FdtPciHostBridgeLib.inf | 11 +++++-----
> .../FdtPciPcdProducerLib.inf | 5 ++---
> .../Fdt}/HighMemDxe/HighMemDxe.inf | 7 ++++---
> .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf | 2 +-
> .../Library/QemuFwCfgLib/QemuFwCfgLibMmio.inf | 6 +++---
> .../Include/Protocol/FdtClient.h | 0
> .../Drivers}/FdtClientDxe/FdtClientDxe.c | 0
> .../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 0
> .../FdtPciPcdProducerLib.c | 0
> .../Fdt}/HighMemDxe/HighMemDxe.c | 3 ++-
> .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.c | 0
> .../Library/QemuFwCfgLib/QemuFwCfgLibMmio.c | 7 ++++---
> Maintainers.txt | 6 ++++++
> 38 files changed, 106 insertions(+), 83 deletions(-)
> rename {ArmVirtPkg => EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.inf
> (92%)
> rename {ArmVirtPkg/Library =>
> OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.inf (77%)
> rename {ArmVirtPkg/Library =>
> OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf (87%)
> rename {ArmVirtPkg => OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.inf (83%)
> rename {ArmVirtPkg => OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf (92%)
> rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.inf =>
> OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.inf (86%)
> rename {ArmVirtPkg => EmbeddedPkg}/Include/Protocol/FdtClient.h (100%)
> rename {ArmVirtPkg => EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.c
> (100%)
> rename {ArmVirtPkg/Library =>
> OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.c (100%)
> rename {ArmVirtPkg/Library =>
> OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.c (100%)
> rename {ArmVirtPkg => OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.c (95%)
> rename {ArmVirtPkg => OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.c (100%)
> rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c =>
> OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMmio.c (93%)
>
> --
> 2.17.1
>
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#81312): https://edk2.groups.io/g/devel/message/81312
Mute This Topic: https://groups.io/mt/85964669/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list