On 03/10/21 03:55, Min Xu wrote: > Register reviewers for the TDX-related files in OvmfPkg. > > Cc: Andrew Fish <af...@apple.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Cc: Leif Lindholm <l...@nuviainc.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Jiewen Yao <jiewen....@intel.com> > Signed-off-by: Min Xu <min.m...@intel.com> > --- > Maintainers.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Maintainers.txt b/Maintainers.txt > index e38497123857..7d9fe89d6d28 100644 > --- a/Maintainers.txt > +++ b/Maintainers.txt > @@ -465,6 +465,10 @@ F: OvmfPkg/PlatformPei/AmdSev.c > R: Tom Lendacky <thomas.lenda...@amd.com> > R: Brijesh Singh <brijesh.si...@amd.com> > > +OvmfPkg: TDX-related modules > +R: Jiewen Yao <jiewen....@intel.com> > +R: Min Xu <min.m...@intel.com> > + > OvmfPkg: TCG- and TPM2-related modules > F: OvmfPkg/Include/IndustryStandard/QemuTpm.h > F: OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c >
I'm OK with the patch content, however the placement of the new section is not right. We strive to keep the section headers alphabetically ordered. Here's an excerpt (pre-patch): OvmfPkg: SEV-related modules OvmfPkg: TCG- and TPM2-related modules OvmfPkg: Xen-related modules I think that the new section belongs between TCG and Xen. Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72626): https://edk2.groups.io/g/devel/message/72626 Mute This Topic: https://groups.io/mt/81219132/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
