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%)
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.chang@hpe.com> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> Cc: Leif Lindholm <leif@nuviainc.com> Cc: Sami Mujawar <sami.mujawar@arm.com> Cc: Jiewen Yao <jiewen.yao@intel.com> Cc: Jordan Justen <jordan.l.justen@intel.com> Cc: Gerd Hoffmann <kraxel@redhat.com> Cc: Daniel Schaefer <daniel.schaefer@hpe.com> Cc: Sunil V L <sunilvl@ventanamicro.com> Cc: Liming Gao <gaoliming@byosoft.com.cn> Cc: Zhiguang Liu <zhiguang.liu@intel.com> Cc: Michael D Kinney <michael.d.kinney@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%) -- 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81212): https://edk2.groups.io/g/devel/message/81212 Mute This Topic: https://groups.io/mt/85920587/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Is there CI to check that the ArmVirtPkg platforms still builds with this? I assume you haven't checked, 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.chang@hpe.com> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> > Cc: Leif Lindholm <leif@nuviainc.com> > Cc: Sami Mujawar <sami.mujawar@arm.com> > Cc: Jiewen Yao <jiewen.yao@intel.com> > Cc: Jordan Justen <jordan.l.justen@intel.com> > Cc: Gerd Hoffmann <kraxel@redhat.com> > Cc: Daniel Schaefer <daniel.schaefer@hpe.com> > Cc: Sunil V L <sunilvl@ventanamicro.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Cc: Zhiguang Liu <zhiguang.liu@intel.com> > Cc: Michael D Kinney <michael.d.kinney@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 (#81237): https://edk2.groups.io/g/devel/message/81237 Mute This Topic: https://groups.io/mt/85920587/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
> -----Original Message----- > From: Schaefer, Daniel > Sent: Wednesday, September 29, 2021 7:12 AM > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; > devel@edk2.groups.io > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen Yao > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@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.chang@hpe.com> > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> > > Cc: Leif Lindholm <leif@nuviainc.com> > > Cc: Sami Mujawar <sami.mujawar@arm.com> > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > Cc: Jordan Justen <jordan.l.justen@intel.com> > > Cc: Gerd Hoffmann <kraxel@redhat.com> > > Cc: Daniel Schaefer <daniel.schaefer@hpe.com> > > Cc: Sunil V L <sunilvl@ventanamicro.com> > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > Cc: Zhiguang Liu <zhiguang.liu@intel.com> > > Cc: Michael D Kinney <michael.d.kinney@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/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Abner: Is there one BZ for this change? Thanks Liming > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Abner Chang > 发送时间: 2021年9月29日 8:54 > 收件人: Schaefer, Daniel <daniel.schaefer@hpe.com>; devel@edk2.groups.io > 抄送: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen Yao > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@intel.com> > 主题: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to > OvmfPkg > > > > > -----Original Message----- > > From: Schaefer, Daniel > > Sent: Wednesday, September 29, 2021 7:12 AM > > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; > > devel@edk2.groups.io > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen > Yao > > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@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.chang@hpe.com> > > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> > > > Cc: Leif Lindholm <leif@nuviainc.com> > > > Cc: Sami Mujawar <sami.mujawar@arm.com> > > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > > Cc: Jordan Justen <jordan.l.justen@intel.com> > > > Cc: Gerd Hoffmann <kraxel@redhat.com> > > > Cc: Daniel Schaefer <daniel.schaefer@hpe.com> > > > Cc: Sunil V L <sunilvl@ventanamicro.com> > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > Cc: Zhiguang Liu <zhiguang.liu@intel.com> > > > Cc: Michael D Kinney <michael.d.kinney@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 (#81256): https://edk2.groups.io/g/devel/message/81256 Mute This Topic: https://groups.io/mt/85939643/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
Oops..I should create one for this. Thanks for the reminder. Abner Get Outlook for Android<https://aka.ms/AAb9ysg> ________________________________ From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of gaoliming <gaoliming@byosoft.com.cn> Sent: Wednesday, September 29, 2021 9:30:44 AM To: devel@edk2.groups.io <devel@edk2.groups.io>; Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; Schaefer, Daniel <daniel.schaefer@hpe.com> Cc: 'Ard Biesheuvel' <ardb+tianocore@kernel.org>; 'Leif Lindholm' <leif@nuviainc.com>; 'Sami Mujawar' <sami.mujawar@arm.com>; 'Jiewen Yao' <jiewen.yao@intel.com>; 'Jordan Justen' <jordan.l.justen@intel.com>; 'Gerd Hoffmann' <kraxel@redhat.com>; 'Sunil V L' <sunilvl@ventanamicro.com>; 'Zhiguang Liu' <zhiguang.liu@intel.com>; 'Michael D Kinney' <michael.d.kinney@intel.com> Subject: 回复: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg Abner: Is there one BZ for this change? Thanks Liming > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Abner Chang > 发送时间: 2021年9月29日 8:54 > 收件人: Schaefer, Daniel <daniel.schaefer@hpe.com>; devel@edk2.groups.io > 抄送: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen Yao > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@intel.com> > 主题: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to > OvmfPkg > > > > > -----Original Message----- > > From: Schaefer, Daniel > > Sent: Wednesday, September 29, 2021 7:12 AM > > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; > > devel@edk2.groups.io > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen > Yao > > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@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.chang@hpe.com> > > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> > > > Cc: Leif Lindholm <leif@nuviainc.com> > > > Cc: Sami Mujawar <sami.mujawar@arm.com> > > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > > Cc: Jordan Justen <jordan.l.justen@intel.com> > > > Cc: Gerd Hoffmann <kraxel@redhat.com> > > > Cc: Daniel Schaefer <daniel.schaefer@hpe.com> > > > Cc: Sunil V L <sunilvl@ventanamicro.com> > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > Cc: Zhiguang Liu <zhiguang.liu@intel.com> > > > Cc: Michael D Kinney <michael.d.kinney@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 (#81260): https://edk2.groups.io/g/devel/message/81260 Mute This Topic: https://groups.io/mt/85939815/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
BZ #3665 is created for this. I will mention it in the cover letter. Abner ________________________________ From: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com> Sent: Wednesday, September 29, 2021 9:43 AM To: devel@edk2.groups.io <devel@edk2.groups.io>; Schaefer, Daniel <daniel.schaefer@hpe.com> Cc: 'Ard Biesheuvel' <ardb+tianocore@kernel.org>; 'Leif Lindholm' <leif@nuviainc.com>; 'Sami Mujawar' <sami.mujawar@arm.com>; 'Jiewen Yao' <jiewen.yao@intel.com>; 'Jordan Justen' <jordan.l.justen@intel.com>; 'Gerd Hoffmann' <kraxel@redhat.com>; 'Sunil V L' <sunilvl@ventanamicro.com>; 'Zhiguang Liu' <zhiguang.liu@intel.com>; 'Michael D Kinney' <michael.d.kinney@intel.com> Subject: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg Oops..I should create one for this. Thanks for the reminder. Abner Get Outlook for Android<https://aka.ms/AAb9ysg> ________________________________ From: devel@edk2.groups.io <devel@edk2.groups.io> on behalf of gaoliming <gaoliming@byosoft.com.cn> Sent: Wednesday, September 29, 2021 9:30:44 AM To: devel@edk2.groups.io <devel@edk2.groups.io>; Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; Schaefer, Daniel <daniel.schaefer@hpe.com> Cc: 'Ard Biesheuvel' <ardb+tianocore@kernel.org>; 'Leif Lindholm' <leif@nuviainc.com>; 'Sami Mujawar' <sami.mujawar@arm.com>; 'Jiewen Yao' <jiewen.yao@intel.com>; 'Jordan Justen' <jordan.l.justen@intel.com>; 'Gerd Hoffmann' <kraxel@redhat.com>; 'Sunil V L' <sunilvl@ventanamicro.com>; 'Zhiguang Liu' <zhiguang.liu@intel.com>; 'Michael D Kinney' <michael.d.kinney@intel.com> Subject: 回复: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg Abner: Is there one BZ for this change? Thanks Liming > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Abner Chang > 发送时间: 2021年9月29日 8:54 > 收件人: Schaefer, Daniel <daniel.schaefer@hpe.com>; devel@edk2.groups.io > 抄送: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen Yao > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@intel.com> > 主题: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to > OvmfPkg > > > > > -----Original Message----- > > From: Schaefer, Daniel > > Sent: Wednesday, September 29, 2021 7:12 AM > > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; > > devel@edk2.groups.io > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm > > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen > Yao > > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd > > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>; > > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu > > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@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.chang@hpe.com> > > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org> > > > Cc: Leif Lindholm <leif@nuviainc.com> > > > Cc: Sami Mujawar <sami.mujawar@arm.com> > > > Cc: Jiewen Yao <jiewen.yao@intel.com> > > > Cc: Jordan Justen <jordan.l.justen@intel.com> > > > Cc: Gerd Hoffmann <kraxel@redhat.com> > > > Cc: Daniel Schaefer <daniel.schaefer@hpe.com> > > > Cc: Sunil V L <sunilvl@ventanamicro.com> > > > Cc: Liming Gao <gaoliming@byosoft.com.cn> > > > Cc: Zhiguang Liu <zhiguang.liu@intel.com> > > > Cc: Michael D Kinney <michael.d.kinney@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 (#81266): https://edk2.groups.io/g/devel/message/81266 Mute This Topic: https://groups.io/mt/85939815/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.