[edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg
Abner Chang
abner.chang at hpe.com
Wed Sep 29 03:31:53 UTC 2021
BZ #3665 is created for this. I will mention it in the cover letter.
Abner
________________________________
From: Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com>
Sent: Wednesday, September 29, 2021 9:43 AM
To: devel at edk2.groups.io <devel at edk2.groups.io>; Schaefer, Daniel <daniel.schaefer at hpe.com>
Cc: 'Ard Biesheuvel' <ardb+tianocore at kernel.org>; 'Leif Lindholm' <leif at nuviainc.com>; 'Sami Mujawar' <sami.mujawar at arm.com>; 'Jiewen Yao' <jiewen.yao at intel.com>; 'Jordan Justen' <jordan.l.justen at intel.com>; 'Gerd Hoffmann' <kraxel at redhat.com>; 'Sunil V L' <sunilvl at ventanamicro.com>; 'Zhiguang Liu' <zhiguang.liu at intel.com>; 'Michael D Kinney' <michael.d.kinney at 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 at edk2.groups.io <devel at edk2.groups.io> on behalf of gaoliming <gaoliming at byosoft.com.cn>
Sent: Wednesday, September 29, 2021 9:30:44 AM
To: devel at edk2.groups.io <devel at edk2.groups.io>; Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com>; Schaefer, Daniel <daniel.schaefer at hpe.com>
Cc: 'Ard Biesheuvel' <ardb+tianocore at kernel.org>; 'Leif Lindholm' <leif at nuviainc.com>; 'Sami Mujawar' <sami.mujawar at arm.com>; 'Jiewen Yao' <jiewen.yao at intel.com>; 'Jordan Justen' <jordan.l.justen at intel.com>; 'Gerd Hoffmann' <kraxel at redhat.com>; 'Sunil V L' <sunilvl at ventanamicro.com>; 'Zhiguang Liu' <zhiguang.liu at intel.com>; 'Michael D Kinney' <michael.d.kinney at 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 at edk2.groups.io <devel at edk2.groups.io> 代表 Abner Chang
> 发送时间: 2021年9月29日 8:54
> 收件人: Schaefer, Daniel <daniel.schaefer at hpe.com>; devel at edk2.groups.io
> 抄送: Ard Biesheuvel <ardb+tianocore at kernel.org>; Leif Lindholm
> <leif at nuviainc.com>; Sami Mujawar <sami.mujawar at arm.com>; Jiewen Yao
> <jiewen.yao at intel.com>; Jordan Justen <jordan.l.justen at intel.com>; Gerd
> Hoffmann <kraxel at redhat.com>; Sunil V L <sunilvl at ventanamicro.com>;
> Liming Gao <gaoliming at byosoft.com.cn>; Zhiguang Liu
> <zhiguang.liu at intel.com>; Michael D Kinney <michael.d.kinney at 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 at hpe.com>;
> > 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>; Jiewen
> Yao
> > <jiewen.yao at intel.com>; Jordan Justen <jordan.l.justen at intel.com>; Gerd
> > Hoffmann <kraxel at redhat.com>; Sunil V L <sunilvl at ventanamicro.com>;
> > Liming Gao <gaoliming at byosoft.com.cn>; Zhiguang Liu
> > <zhiguang.liu at intel.com>; Michael D Kinney <michael.d.kinney at 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 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 (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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20210929/d88750b0/attachment.htm>
More information about the edk2-devel-archive
mailing list