[edk2-devel] [PATCH 1/2] Maintainers.txt: Add reviewers for the OvmfPkg TDX-related files

[Laszlo Ersek]

On 03/10/21 03:55, Min Xu wrote:
> Register reviewers for the TDX-related files in OvmfPkg.
> 
> Cc: Andrew Fish <afish at apple.com>
> Cc: Laszlo Ersek <lersek at redhat.com>
> Cc: Leif Lindholm <leif at nuviainc.com>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Jiewen Yao <jiewen.yao at intel.com>
> Signed-off-by: Min Xu <min.m.xu at 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.lendacky at amd.com>
>  R: Brijesh Singh <brijesh.singh at amd.com>
>  
> +OvmfPkg: TDX-related modules
> +R: Jiewen Yao <jiewen.yao at intel.com>
> +R: Min Xu <min.m.xu at 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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-