<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=gb2312">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);">
BZ #3665 is created for this. I will mention it in the cover letter.</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);">
<br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);">
Abner</div>
<div id="appendonsend"></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size: 11pt;"><b>From:</b> Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com><br>
<b>Sent:</b> Wednesday, September 29, 2021 9:43 AM<br>
<b>To:</b> devel@edk2.groups.io <devel@edk2.groups.io>; Schaefer, Daniel <daniel.schaefer@hpe.com><br>
<b>Cc:</b> '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><br>
<b>Subject:</b> Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg</font>
<div> </div>
</div>
<div>
<div dir="auto" style="color: rgb(33, 33, 33); background-color: rgb(255, 255, 255);">
Oops..I should create one for this.</div>
<div dir="auto" style="color: rgb(33, 33, 33); background-color: rgb(255, 255, 255);">
Thanks for the reminder.</div>
<div dir="auto" style="color: rgb(33, 33, 33); background-color: rgb(255, 255, 255);">
<br>
</div>
<div dir="auto" style="color: rgb(33, 33, 33); background-color: rgb(255, 255, 255);">
Abner</div>
<div id="x_ms-outlook-mobile-signature">
<div><br>
</div>
Get <a href="https://aka.ms/AAb9ysg">Outlook for Android</a></div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size: 11pt;"><b>From:</b> devel@edk2.groups.io <devel@edk2.groups.io> on behalf of gaoliming <gaoliming@byosoft.com.cn><br>
<b>Sent:</b> Wednesday, September 29, 2021 9:30:44 AM<br>
<b>To:</b> devel@edk2.groups.io <devel@edk2.groups.io>; Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; Schaefer, Daniel <daniel.schaefer@hpe.com><br>
<b>Cc:</b> '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><br>
<b>Subject:</b> 回复: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg</font>
<div> </div>
</div>
<div class="x_BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="x_PlainText">Abner:<br>
Is there one BZ for this change?<br>
<br>
Thanks<br>
Liming<br>
> -----邮件原件-----<br>
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Abner Chang<br>
> 发送时间: 2021年9月29日 8:54<br>
> 收件人: Schaefer, Daniel <daniel.schaefer@hpe.com>; devel@edk2.groups.io<br>
> 抄送: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm<br>
> <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen Yao<br>
> <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd<br>
> Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>;<br>
> Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu<br>
> <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@intel.com><br>
> 主题: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg modules to<br>
> OvmfPkg<br>
> <br>
> <br>
> <br>
> > -----Original Message-----<br>
> > From: Schaefer, Daniel<br>
> > Sent: Wednesday, September 29, 2021 7:12 AM<br>
> > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>;<br>
> > devel@edk2.groups.io<br>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Leif Lindholm<br>
> > <leif@nuviainc.com>; Sami Mujawar <sami.mujawar@arm.com>; Jiewen<br>
> Yao<br>
> > <jiewen.yao@intel.com>; Jordan Justen <jordan.l.justen@intel.com>; Gerd<br>
> > Hoffmann <kraxel@redhat.com>; Sunil V L <sunilvl@ventanamicro.com>;<br>
> > Liming Gao <gaoliming@byosoft.com.cn>; Zhiguang Liu<br>
> > <zhiguang.liu@intel.com>; Michael D Kinney <michael.d.kinney@intel.com><br>
> > Subject: Re: [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg<br>
> ><br>
> > Is there CI to check that the ArmVirtPkg platforms still builds with this?<br>
> > I assume you haven't checked, Abner?<br>
> Yes, this patch set passed the CI before I sending it out :).<br>
> Abner<br>
> <br>
> ><br>
> > On 9/28/21 16:30, Abner Chang wrote:<br>
> > > In V2: Remove HPE license on the files that just moved around or<br>
> > > the changes in the file are just code removal.<br>
> > ><br>
> > > This pacthes set is to migrate some modules from ArmVirtPkg<br>
> > > to under OvmfPkg for the upcoming RiscVVirtPkg that can leverage<br>
> > > those modules without the dependency with Arm*Pkg.<br>
> > ><br>
> > > The modules moved from ArmVirtPkg to OvmfPkg are,<br>
> > > - FdtClientDxe<br>
> > > - PciPcdProducerLib<br>
> > > - HighMemDxe<br>
> > > - QemuFwCfgLib<br>
> > > - FdtPciHostBridgeLib<br>
> > > - VirtioFdtDxe<br>
> > ><br>
> > > Below PCDs are moved to under MdePkg and leverage by RiscVVirtPkg.<br>
> > > This change also remove the dependency on ArmPkg of OvmfPkg.<br>
> > > - PcdPciIoTranslation<br>
> > > - PcdPciIoTranslation<br>
> > > - PcdPciMmio32(64)Translation<br>
> > ><br>
> > > Signed-off-by: Abner Chang <abner.chang@hpe.com><br>
> > > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org><br>
> > > Cc: Leif Lindholm <leif@nuviainc.com><br>
> > > Cc: Sami Mujawar <sami.mujawar@arm.com><br>
> > > Cc: Jiewen Yao <jiewen.yao@intel.com><br>
> > > Cc: Jordan Justen <jordan.l.justen@intel.com><br>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com><br>
> > > Cc: Daniel Schaefer <daniel.schaefer@hpe.com><br>
> > > Cc: Sunil V L <sunilvl@ventanamicro.com><br>
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn><br>
> > > Cc: Zhiguang Liu <zhiguang.liu@intel.com><br>
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com><br>
> > ><br>
> > > Abner Chang (9):<br>
> > > ArmVirtPkg/FdtClintDxe: Move FdtClientDxe to EmbeddedPkg<br>
> > > MdePkg: Add PcdPciIoTranslation PCD<br>
> > > ArmPkg: Use PcdPciIoTranslation PCD from MdePkg<br>
> > > ArmVirtPkg/FdtPciPcdProducerLib: Relocate PciPcdProducerLib to<br>
> > OvmfPkg<br>
> > > ArmVirtPkg/HighMemDxe: Relocate HighMemDxe to OvmfPkg<br>
> > > ArmVirtPkg/QemuFwCfgLib: Relocate QemuFwCfgLib to OvmfPkg<br>
> > > MdePkg: Add PcdPciMmio32(64)Translation PCDs<br>
> > > ArmVirtPkg/FdtPciHostBridgeLib: Relocate FdtPciHostBridgeLib to<br>
> > > OvmfPkg/Fdt<br>
> > > ArmVirtPkg/VirtioFdtDxe: Relocate VirtioFdtDxe to OvmfPkg/Fdt<br>
> > ><br>
> > > ArmPkg/ArmPkg.dec | 15<br>
> ++++++--------<br>
> > > ArmVirtPkg/ArmVirtPkg.dec | 3 ---<br>
> > > EmbeddedPkg/EmbeddedPkg.dec | 1 +<br>
> > > MdePkg/MdePkg.dec | 12<br>
> +++++++++++<br>
> > > ArmVirtPkg/ArmVirtCloudHv.dsc | 18<br>
> ++++++++---------<br>
> > > ArmVirtPkg/ArmVirtKvmTool.dsc | 18<br>
> ++++++++---------<br>
> > > ArmVirtPkg/ArmVirtQemu.dsc | 20<br>
> +++++++++----------<br>
> > > ArmVirtPkg/ArmVirtQemuKernel.dsc | 20<br>
> +++++++++----------<br>
> > > ArmVirtPkg/ArmVirtXen.dsc | 2 +-<br>
> > > EmbeddedPkg/EmbeddedPkg.dsc | 1 +<br>
> > > ArmVirtPkg/ArmVirtCloudHv.fdf | 6 +++---<br>
> > > ArmVirtPkg/ArmVirtKvmTool.fdf | 6 +++---<br>
> > > ArmVirtPkg/ArmVirtXen.fdf | 2 +-<br>
> > > ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc | 6 +++---<br>
> > > .../ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf | 2 +-<br>
> > > .../ArmVirtGicArchLib/ArmVirtGicArchLib.inf | 1 +<br>
> > > .../ArmVirtPL031FdtClientLib.inf | 1 +<br>
> > > .../ArmVirtPsciResetSystemLib.inf | 1 +<br>
> > > .../ArmVirtTimerFdtClientLib.inf | 1 +<br>
> > > .../KvmtoolRtcFdtClientLib.inf | 1 +<br>
> > > .../NorFlashKvmtoolLib/NorFlashKvmtoolLib.inf | 1 +<br>
> > > .../NorFlashQemuLib/NorFlashQemuLib.inf | 1 +<br>
> > > .../XenAcpiPlatformDxe/XenAcpiPlatformDxe.inf | 1 +<br>
> > > ArmVirtPkg/XenioFdtDxe/XenioFdtDxe.inf | 1 +<br>
> > > .../Drivers}/FdtClientDxe/FdtClientDxe.inf | 1 -<br>
> > > .../FdtPciHostBridgeLib.inf | 11 +++++-----<br>
> > > .../FdtPciPcdProducerLib.inf | 5 ++---<br>
> > > .../Fdt}/HighMemDxe/HighMemDxe.inf | 4 ++--<br>
> > > .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf | 2 +-<br>
> > > .../Library/QemuFwCfgLib/QemuFwCfgLibMMIO.inf | 6 +++---<br>
> > > .../Include/Protocol/FdtClient.h | 0<br>
> > > .../Drivers}/FdtClientDxe/FdtClientDxe.c | 0<br>
> > > .../FdtPciHostBridgeLib/FdtPciHostBridgeLib.c | 0<br>
> > > .../FdtPciPcdProducerLib.c | 0<br>
> > > .../Fdt}/HighMemDxe/HighMemDxe.c | 0<br>
> > > .../Fdt}/VirtioFdtDxe/VirtioFdtDxe.c | 0<br>
> > > .../Library/QemuFwCfgLib/QemuFwCfgLibMMIO.c | 7 ++++---<br>
> > > Maintainers.txt | 6 ++++++<br>
> > > 38 files changed, 102 insertions(+), 81 deletions(-)<br>
> > > rename {ArmVirtPkg =><br>
> > EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.inf (92%)<br>
> > > rename {ArmVirtPkg/Library =><br>
> > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.inf (77%)<br>
> > > rename {ArmVirtPkg/Library =><br>
> > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf (87%)<br>
> > > rename {ArmVirtPkg => OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.inf<br>
> > (91%)<br>
> > > rename {ArmVirtPkg => OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf<br>
> (92%)<br>
> > > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.inf =><br>
> > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMMIO.inf (86%)<br>
> > > rename {ArmVirtPkg => EmbeddedPkg}/Include/Protocol/FdtClient.h<br>
> > (100%)<br>
> > > rename {ArmVirtPkg =><br>
> > EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.c (100%)<br>
> > > rename {ArmVirtPkg/Library =><br>
> > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.c (100%)<br>
> > > rename {ArmVirtPkg/Library =><br>
> > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.c (100%)<br>
> > > rename {ArmVirtPkg => OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.c<br>
> > (100%)<br>
> > > rename {ArmVirtPkg => OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.c<br>
> (100%)<br>
> > > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c =><br>
> > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMMIO.c (93%)<br>
> > ><br>
> <br>
> <br>
> <br>
> <br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
</div>
</span></font></div>
</div>
</body>
</html>
<div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links:<p> You receive all messages sent to this group. <p> <a target="_blank" href="https://edk2.groups.io/g/devel/message/81266">View/Reply Online (#81266)</a> | | <a target="_blank" href="https://groups.io/mt/85939815/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com]<br> <div width="1" style="color:white;clear:both">_._,_._,_</div>