On Tue, May 23, 2023 at 02:59:50PM -0700, Tuan Phan wrote: > Thanks Andrei, > Sunil, any comments or it is good to go. As this patchset spans across > MdePkg, UefiCpuPkg and OvmfPkg, do I need to separate it so each package > maintainer can merge independently? > Hi Tuan,
I don't see an issue when a patch set spans across multiple modules as long as corresponding maintainers review/ack. However, it is better if the change in single patch is restricted to one module/package. In your series, PATCH 3/6 is spread across 3 different modules. Could you separate the change for each package? Also, while all other patches affects RISC-V only, PATCH 5/6 in your series is a generic change which affects other architectures. Ard is the maintainer for this and he should to agree for the change. Did I miss a response to his question for this patch? Thanks! Sunil -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#105214): https://edk2.groups.io/g/devel/message/105214 Mute This Topic: https://groups.io/mt/98269071/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
