From: "Abner Chang" <abner.chang@hpe.com>
To: "Schaefer, Daniel" <daniel.schaefer@hpe.com>,
"devel@edk2.groups.io" <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
Date: Wed, 29 Sep 2021 00:53:34 +0000 [thread overview]
Message-ID: <CS1PR8401MB1144630E8462E9C551CC1C40FFA99@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <570082d7-7d13-1b27-eea9-639dbf0d76f6@hpe.com>
> -----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%)
> >
next prev parent reply other threads:[~2021-09-29 0:54 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-28 8:30 [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg Abner Chang
2021-09-28 8:30 ` [PATCH V2 1/9] ArmVirtPkg/FdtClintDxe: Move FdtClientDxe to EmbeddedPkg Abner Chang
2021-09-28 23:09 ` Daniel Schaefer
2021-09-28 8:31 ` [PATCH V2 2/9] MdePkg: Add PcdPciIoTranslation PCD Abner Chang
2021-09-28 23:10 ` Daniel Schaefer
2021-09-29 0:51 ` Abner Chang
2021-09-29 1:10 ` 回复: [edk2-devel] " gaoliming
2021-09-28 8:31 ` [PATCH V2 3/9] ArmPkg: Use PcdPciIoTranslation PCD from MdePkg Abner Chang
2021-09-28 23:38 ` Daniel Schaefer
2021-09-28 8:31 ` [PATCH V2 4/9] ArmVirtPkg/FdtPciPcdProducerLib: Relocate PciPcdProducerLib to OvmfPkg Abner Chang
2021-09-28 23:16 ` Daniel Schaefer
[not found] ` <16A91FA9AD0B84BA.7073@groups.io>
2021-09-28 23:41 ` [edk2-devel] " Daniel Schaefer
2021-09-29 0:42 ` Abner Chang
2021-09-28 8:31 ` [PATCH V2 5/9] ArmVirtPkg/HighMemDxe: Relocate HighMemDxe " Abner Chang
2021-09-28 23:34 ` Daniel Schaefer
2021-09-28 8:31 ` [PATCH V2 6/9] ArmVirtPkg/QemuFwCfgLib: Relocate QemuFwCfgLib " Abner Chang
2021-09-28 11:43 ` [edk2-devel] " Gerd Hoffmann
2021-09-29 9:45 ` Sami Mujawar
2021-09-29 10:04 ` Yao, Jiewen
2021-09-29 11:33 ` Sami Mujawar
2021-09-29 12:26 ` [edk2-devel] " Yao, Jiewen
2021-09-29 12:46 ` Abner Chang
2021-09-29 12:57 ` Yao, Jiewen
2021-09-28 8:31 ` [PATCH V2 7/9] MdePkg: Add PcdPciMmio32(64)Translation PCDs Abner Chang
2021-09-28 23:36 ` Daniel Schaefer
2021-09-29 0:54 ` Abner Chang
2021-09-28 8:31 ` [PATCH V2 8/9] ArmVirtPkg/FdtPciHostBridgeLib: Relocate FdtPciHostBridgeLib to OvmfPkg/Fdt Abner Chang
2021-09-28 23:25 ` Daniel Schaefer
2021-09-28 8:31 ` [PATCH V2 9/9] ArmVirtPkg/VirtioFdtDxe: Relocate VirtioFdtDxe " Abner Chang
2021-09-28 23:36 ` Daniel Schaefer
2021-09-28 23:11 ` [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg Daniel Schaefer
2021-09-29 0:53 ` Abner Chang [this message]
2021-09-29 1:30 ` 回复: [edk2-devel] " gaoliming
2021-09-29 1:43 ` Abner Chang
2021-09-29 3:31 ` Abner Chang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CS1PR8401MB1144630E8462E9C551CC1C40FFA99@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.COM \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox