> -----Original Message----- > From: Schaefer, Daniel > Sent: Wednesday, September 29, 2021 7:12 AM > To: Chang, Abner (HPS SW/FW Technologist) <abner.ch...@hpe.com>; > devel@edk2.groups.io > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>; Leif Lindholm > <l...@nuviainc.com>; Sami Mujawar <sami.muja...@arm.com>; Jiewen Yao > <jiewen....@intel.com>; Jordan Justen <jordan.l.jus...@intel.com>; Gerd > Hoffmann <kra...@redhat.com>; Sunil V L <suni...@ventanamicro.com>; > Liming Gao <gaolim...@byosoft.com.cn>; Zhiguang Liu > <zhiguang....@intel.com>; Michael D Kinney <michael.d.kin...@intel.com> > Subject: Re: [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg > > Is there CI to check that the ArmVirtPkg platforms still builds with this? > I assume you haven't checked, Abner? Yes, this patch set passed the CI before I sending it out :). Abner
> > On 9/28/21 16:30, Abner Chang wrote: > > In V2: Remove HPE license on the files that just moved around or > > the changes in the file are just code removal. > > > > 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.ch...@hpe.com> > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > > Cc: Leif Lindholm <l...@nuviainc.com> > > Cc: Sami Mujawar <sami.muja...@arm.com> > > Cc: Jiewen Yao <jiewen....@intel.com> > > Cc: Jordan Justen <jordan.l.jus...@intel.com> > > Cc: Gerd Hoffmann <kra...@redhat.com> > > Cc: Daniel Schaefer <daniel.schae...@hpe.com> > > Cc: Sunil V L <suni...@ventanamicro.com> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Zhiguang Liu <zhiguang....@intel.com> > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > > > Abner Chang (9): > > 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 > > ArmVirtPkg/QemuFwCfgLib: Relocate QemuFwCfgLib to OvmfPkg > > MdePkg: Add PcdPciMmio32(64)Translation PCDs > > ArmVirtPkg/FdtPciHostBridgeLib: Relocate FdtPciHostBridgeLib to > > OvmfPkg/Fdt > > 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 | 4 ++-- > > .../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 | 0 > > .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.c | 0 > > .../Library/QemuFwCfgLib/QemuFwCfgLibMMIO.c | 7 ++++--- > > Maintainers.txt | 6 ++++++ > > 38 files changed, 102 insertions(+), 81 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 > (91%) > > 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 > (100%) > > rename {ArmVirtPkg => OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.c (100%) > > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c => > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMMIO.c (93%) > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81248): https://edk2.groups.io/g/devel/message/81248 Mute This Topic: https://groups.io/mt/85920587/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-