Abner:
I prefer to use git mv command to move those modules from ArmVirtPkg to
OvmfPkg. This way can still keep git history for those modules.

You can create the second patch to update ArmVirtPkg DSC/FDF, or combine
this change into the first patch.

Thanks
Liming

-----邮件原件-----
发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Abner Chang
发送时间: 2021年9月6日 21:05
收件人: Gerd Hoffmann <kraxel@...>; Ard Biesheuvel
<ardb@...>
抄送: Yao, Jiewen <jiewen.yao@...>; devel@edk2.groups.io; gaoliming
<gaoliming@...>; Ard Biesheuvel <ard.biesheuvel@...>;
Kinney, Michael D <michael.d.kinney@...>; Leif Lindholm
<leif@...>; Ni, Ray <ray.ni@...>; Schaefer, Daniel
<daniel.schaefer@...>; Sunil V L <sunilvl@...>; Ard
Biesheuvel <ardb+tianocore@...>
主题: Re: [edk2-devel] [RFC] RISC-V QEMU virtual package

Ok thanks, do we need the two steps to migrate FDT modules under
OvmfPkg/?
1. One patch set to clone those modules under OvmfPkg/
2. One patch set of ArmVirtPkg change to use those modules and deletes the
ones under ArmVirtPkg/.

Or we just do all at once? I prefer to have two steps. How do you think?

Another question, who would like to be the reviewers of OvmfPkg/Fdt/*? Any
volunteers from CC list?

Thanks
Abner

-----Original Message-----
From: Gerd Hoffmann [mailto:kraxel@...]
Sent: Monday, September 6, 2021 8:19 PM
To: Ard Biesheuvel <ardb@...>
Cc: Yao, Jiewen <jiewen.yao@...>; Chang, Abner (HPS SW/FW
Technologist) <abner.chang@...>; devel@edk2.groups.io; gaoliming
<gaoliming@...>; Ard Biesheuvel <ard.biesheuvel@...>;
Kinney, Michael D <michael.d.kinney@...>; Leif Lindholm
<leif@...>; Ni, Ray <ray.ni@...>; Schaefer, Daniel
<daniel.schaefer@...>; Sunil V L <sunilvl@...>; Ard
Biesheuvel <ardb+tianocore@...>
Subject: Re: [edk2-devel] [RFC] RISC-V QEMU virtual package

Hi,

On Mon, Sep 06, 2021 at 02:04:48PM +0200, Ard Biesheuvel wrote:

On Mon, 6 Sept 2021 at 13:44, Yao, Jiewen <jiewen.yao@...>

wrote:

I think it makes sense to put Fdt to OvmfPkg. I suggest an Fdt

folder and

put all things there.

I also think we define Fdt feature in

https://github.com/tianocore/edk2/blob/master/Maintainers.txt to add
reviewer there to help review the code.

Agreed.

Agreeing too.

take care,
Gerd