From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web10.3259.1632879050214808855 for ; Tue, 28 Sep 2021 18:30:51 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Wed, 29 Sep 2021 09:30:43 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , , "'Schaefer, Daniel'" Cc: "'Ard Biesheuvel'" , "'Leif Lindholm'" , "'Sami Mujawar'" , "'Jiewen Yao'" , "'Jordan Justen'" , "'Gerd Hoffmann'" , "'Sunil V L'" , "'Zhiguang Liu'" , "'Michael D Kinney'" References: <20210928083107.7418-1-abner.chang@hpe.com> <570082d7-7d13-1b27-eea9-639dbf0d76f6@hpe.com> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIFYyIDAvOV0gTWlncmF0ZSBBcm1WaXJ0UGtnIG1vZHVsZXMgdG8gT3ZtZlBrZw==?= Date: Wed, 29 Sep 2021 09:30:44 +0800 Message-ID: <000201d7b4d1$9fc6cf10$df546d30$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQHhj5a6n0zTGjPeRFpaUzM0aDy1yQJF4fnPArULCBKrfqyUAA== Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Abner: Is there one BZ for this change? Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io = =E4=BB=A3=E8=A1=A8 Abner Chang > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2021=E5=B9=B49=E6=9C=8829=E6=97=A5 = 8:54 > =E6=94=B6=E4=BB=B6=E4=BA=BA: Schaefer, Daniel ; = devel@edk2.groups.io > =E6=8A=84=E9=80=81: Ard Biesheuvel ; Leif Lind= holm > ; Sami Mujawar ; Jiewen Yao > ; Jordan Justen ; Gerd > Hoffmann ; Sunil V L ; > Liming Gao ; Zhiguang Liu > ; Michael D Kinney > =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH V2 0/9] Migrate ArmVirtPkg mo= dules to > OvmfPkg >=20 >=20 >=20 > > -----Original Message----- > > From: Schaefer, Daniel > > Sent: Wednesday, September 29, 2021 7:12 AM > > To: Chang, Abner (HPS SW/FW Technologist) ; > > devel@edk2.groups.io > > Cc: Ard Biesheuvel ; Leif Lindholm > > ; Sami Mujawar ; Jiewen > Yao > > ; Jordan Justen ; Gerd > > Hoffmann ; Sunil V L ; > > Liming Gao ; Zhiguang Liu > > ; Michael D Kinney > > Subject: Re: [PATCH V2 0/9] Migrate ArmVirtPkg modules to OvmfPkg > > > > Is there CI to check that the ArmVirtPkg platforms still builds with th= is? > > I assume you haven't checked, Abner? > Yes, this patch set passed the CI before I sending it out :). > Abner >=20 > > > > 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 > > > Cc: Ard Biesheuvel > > > Cc: Leif Lindholm > > > Cc: Sami Mujawar > > > Cc: Jiewen Yao > > > Cc: Jordan Justen > > > Cc: Gerd Hoffmann > > > Cc: Daniel Schaefer > > > Cc: Sunil V L > > > Cc: Liming Gao > > > Cc: Zhiguang Liu > > > Cc: Michael D Kinney > > > > > > 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 =3D> > > EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.inf (92%) > > > rename {ArmVirtPkg/Library =3D> > > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.inf (77%) > > > rename {ArmVirtPkg/Library =3D> > > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.inf (87%) > > > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.inf > > (91%) > > > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.inf > (92%) > > > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.inf =3D> > > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMMIO.inf (86%) > > > rename {ArmVirtPkg =3D> EmbeddedPkg}/Include/Protocol/FdtClient.h > > (100%) > > > rename {ArmVirtPkg =3D> > > EmbeddedPkg/Drivers}/FdtClientDxe/FdtClientDxe.c (100%) > > > rename {ArmVirtPkg/Library =3D> > > OvmfPkg/Fdt}/FdtPciHostBridgeLib/FdtPciHostBridgeLib.c (100%) > > > rename {ArmVirtPkg/Library =3D> > > OvmfPkg/Fdt}/FdtPciPcdProducerLib/FdtPciPcdProducerLib.c (100%) > > > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/HighMemDxe/HighMemDxe.c > > (100%) > > > rename {ArmVirtPkg =3D> OvmfPkg/Fdt}/VirtioFdtDxe/VirtioFdtDxe.c > (100%) > > > rename ArmVirtPkg/Library/QemuFwCfgLib/QemuFwCfgLib.c =3D> > > OvmfPkg/Library/QemuFwCfgLib/QemuFwCfgLibMMIO.c (93%) > > > >=20 >=20 >=20 >=20