public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH v2 0/1] Maintainers.txt: Review ownership for MdeModulePkg
@ 2019-07-18  1:19 Wu, Hao A
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
  2019-07-18 12:37 ` [PATCH v2 0/1] Maintainers.txt: Review " Laszlo Ersek
  0 siblings, 2 replies; 8+ messages in thread
From: Wu, Hao A @ 2019-07-18  1:19 UTC (permalink / raw)
  To: devel
  Cc: Hao A Wu, Andrew Fish, Laszlo Ersek, Leif Lindholm,
	Michael D Kinney, Dandan Bi, Eric Dong, Liming Gao, Ray Ni,
	Star Zeng, Zhichao Gao

This patch is also available at:
https://github.com/hwu25/edk2/tree/mdemodulepkg_reviewers_v2

Also, please note that on the above branch, the proposed patch
(MdeModulePkg reviewers) is inserted between:

* [edk2-devel] [PATCH 4/4] Maintainers.txt: split out section "OvmfPkg: CSM modules"
(from Laszlo)
https://edk2.groups.io/g/devel/message/43732

and

* [edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script
(from Leif)
https://edk2.groups.io/g/devel/message/43666


V2 changes:
A. Sort "F:" patterns in alphabetical order for each section;
B. Serveral module renaming:
   Console and Graphic -> Console and Graphics
   S3                  -> ACPI S3
   SMM                 -> Management Mode (MM, SMM)
   Variable            -> UEFI Variable


Cc: Andrew Fish <afish@apple.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>

Hao A Wu (1):
  Maintainers.txt: Fine-grained review ownership for MdeModulePkg

 Maintainers.txt | 151 +++++++++++++++++++-
 1 file changed, 149 insertions(+), 2 deletions(-)

-- 
2.12.0.windows.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 [PATCH v2 0/1] Maintainers.txt: Review ownership for MdeModulePkg Wu, Hao A
@ 2019-07-18  1:19 ` Wu, Hao A
  2019-07-18  1:37   ` [edk2-devel] " Liming Gao
                     ` (4 more replies)
  2019-07-18 12:37 ` [PATCH v2 0/1] Maintainers.txt: Review " Laszlo Ersek
  1 sibling, 5 replies; 8+ messages in thread
From: Wu, Hao A @ 2019-07-18  1:19 UTC (permalink / raw)
  To: devel
  Cc: Hao A Wu, Andrew Fish, Laszlo Ersek, Leif Lindholm,
	Michael D Kinney, Dandan Bi, Eric Dong, Liming Gao, Star Zeng,
	Zhichao Gao

This commit add the reviewers information for modules within MdeModulePkg.

For now the modules list includes:
ACPI
ACPI S3
BDS
Console and Graphics
Core services (PEI, DXE and Runtime)
Device and Peripheral
Firmware Update
HII and UI
Management Mode (MM, SMM)
Reset
SMBIOS
UEFI Variable

Please note that, for MdeModulePkg components not included in the above
list, the reviewers will fall back to the package maintainers.

Cc: Andrew Fish <afish@apple.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Zhichao Gao <zhichao.gao@intel.com>
Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
Reviewed-by: Ray Ni <ray.ni@intel.com>
---
 Maintainers.txt | 151 +++++++++++++++++++-
 1 file changed, 149 insertions(+), 2 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 18fd2ef43c..0298010809 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
 
 MdeModulePkg
 F: MdeModulePkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
 M: Jian J Wang <jian.j.wang@intel.com>
 M: Hao A Wu <hao.a.wu@intel.com>
+
+MdeModulePkg: ACPI modules
+F: MdeModulePkg/Include/*Acpi*.h
+F: MdeModulePkg/Universal/Acpi/
+R: Dandan Bi <dandan.bi@intel.com>
+R: Liming Gao <liming.gao@intel.com>
+
+MdeModulePkg: ACPI S3 modules
+F: MdeModulePkg/*LockBox*/
+F: MdeModulePkg/Include/*BootScript*.h
+F: MdeModulePkg/Include/*LockBox*.h
+F: MdeModulePkg/Include/*S3*.h
+F: MdeModulePkg/Library/*S3*/
+R: Hao A Wu <hao.a.wu@intel.com>
+R: Eric Dong <eric.dong@intel.com>
+
+MdeModulePkg: BDS modules
+F: MdeModulePkg/*BootManager*/
+F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
+F: MdeModulePkg/Universal/BdsDxe/
+F: MdeModulePkg/Universal/DevicePathDxe/
+F: MdeModulePkg/Universal/DriverHealthManagerDxe/
+F: MdeModulePkg/Universal/LoadFileOnFv2/
+F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
+R: Zhichao Gao <zhichao.gao@intel.com>
+R: Ray Ni <ray.ni@intel.com>
+
+MdeModulePkg: Console and Graphics modules
+F: MdeModulePkg/*Logo*/
+F: MdeModulePkg/Include/*Logo*.h
+F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
+F: MdeModulePkg/Include/Guid/Console*.h
+F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
+F: MdeModulePkg/Include/Guid/TtyTerm.h
+F: MdeModulePkg/Include/Library/BmpSupportLib.h
+F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
+F: MdeModulePkg/Library/BaseBmpSupportLib/
+F: MdeModulePkg/Library/FrameBufferBltLib/
+F: MdeModulePkg/Universal/Console/
+R: Zhichao Gao <zhichao.gao@intel.com>
+R: Ray Ni <ray.ni@intel.com>
+
+MdeModulePkg: Core services (PEI, DXE and Runtime) modules
+F: MdeModulePkg/*Mem*/
+F: MdeModulePkg/*SectionExtract*/
+F: MdeModulePkg/*StatusCode*/
+F: MdeModulePkg/Application/DumpDynPcd/
+F: MdeModulePkg/Core/Dxe/
+F: MdeModulePkg/Core/DxeIplPeim/
+F: MdeModulePkg/Core/Pei/
+F: MdeModulePkg/Core/RuntimeDxe/
+F: MdeModulePkg/Include/*Mem*.h
+F: MdeModulePkg/Include/*Pcd*.h
+F: MdeModulePkg/Include/*Perf*.h
+F: MdeModulePkg/Include/*StatusCode*.h
+F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
+F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
+F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
+F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
+F: MdeModulePkg/Include/Guid/LzmaDecompress.h
+F: MdeModulePkg/Include/Library/SecurityManagementLib.h
+F: MdeModulePkg/Library/*Decompress*/
+F: MdeModulePkg/Library/*Perf*/
+F: MdeModulePkg/Library/DxeSecurityManagementLib/
+F: MdeModulePkg/Universal/PCD/
+F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
+F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
+R: Dandan Bi <dandan.bi@intel.com>
+R: Liming Gao <liming.gao@intel.com>
+
+MdeModulePkg: Device and Peripheral modules
+F: MdeModulePkg/*PciHostBridge*/
+F: MdeModulePkg/*Serial*/
+F: MdeModulePkg/Bus/
+F: MdeModulePkg/Include/*Ata*.h
+F: MdeModulePkg/Include/*IoMmu*.h
+F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
+F: MdeModulePkg/Include/*NvmExpress*.h
+F: MdeModulePkg/Include/*SdMmc*.h
+F: MdeModulePkg/Include/*SerialPort*.h
+F: MdeModulePkg/Include/*Ufs*.h
+F: MdeModulePkg/Include/*Usb*.h
+F: MdeModulePkg/Include/Guid/RecoveryDevice.h
+F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
+F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
+F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
+F: MdeModulePkg/Include/Protocol/Ps2Policy.h
+F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
+F: MdeModulePkg/Universal/Disk/
+F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
+R: Hao A Wu <hao.a.wu@intel.com>
 R: Ray Ni <ray.ni@intel.com>
-  (especially for Bus, Universal/Console, Universal/Disk,
-   Universal/BdsDxe and related libraries, header files)
+
+MdeModulePkg: Firmware Update modules
+F: MdeModulePkg/*Capsule*/
+F: MdeModulePkg/Include/*Capsule*.h
+F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
+F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
+F: MdeModulePkg/Include/Protocol/EsrtManagement.h
+F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
+F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
+F: MdeModulePkg/Library/FmpAuthenticationLibNull/
+F: MdeModulePkg/Universal/Esrt*/
+R: Hao A Wu <hao.a.wu@intel.com>
+R: Liming Gao <liming.gao@intel.com>
+
+MdeModulePkg: HII and UI modules
+F: MdeModulePkg/*FileExplorer*/
+F: MdeModulePkg/*Hii*/
+F: MdeModulePkg/*Ui*/
+F: MdeModulePkg/Application/BootManagerMenuApp/
+F: MdeModulePkg/Include/*FileExplorer*.h
+F: MdeModulePkg/Include/*FormBrowser*.h
+F: MdeModulePkg/Include/*Hii*.h
+F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
+F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
+F: MdeModulePkg/Library/CustomizedDisplayLib/
+F: MdeModulePkg/Universal/DisplayEngineDxe/
+F: MdeModulePkg/Universal/DriverSampleDxe/
+F: MdeModulePkg/Universal/SetupBrowserDxe/
+R: Dandan Bi <dandan.bi@intel.com>
+R: Eric Dong <eric.dong@intel.com>
+
+MdeModulePkg: Management Mode (MM, SMM) modules
+F: MdeModulePkg/*Smi*/
+F: MdeModulePkg/*Smm*/
+F: MdeModulePkg/Include/*Smi*.h
+F: MdeModulePkg/Include/*Smm*.h
+R: Eric Dong <eric.dong@intel.com>
+R: Ray Ni <ray.ni@intel.com>
+
+MdeModulePkg: Reset modules
+F: MdeModulePkg/*Reset*/
+F: MdeModulePkg/Include/*Reset*.h
+R: Zhichao Gao <zhichao.gao@intel.com>
+R: Ray Ni <ray.ni@intel.com>
+
+MdeModulePkg: SMBIOS modules
+F: MdeModulePkg/Universal/Smbios*/
+R: Dandan Bi <dandan.bi@intel.com>
 R: Star Zeng <star.zeng@intel.com>
 
+MdeModulePkg: UEFI Variable modules
+F: MdeModulePkg/*Var*/
+F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
+F: MdeModulePkg/Include/*/*Var*.h
+F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
+F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
+F: MdeModulePkg/Universal/FaultTolerantWrite*/
+R: Hao A Wu <hao.a.wu@intel.com>
+R: Liming Gao <liming.gao@intel.com>
+
 MdePkg
 F: MdePkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
 M: Michael D Kinney <michael.d.kinney@intel.com>
 M: Liming Gao <liming.gao@intel.com>
 
-- 
2.12.0.windows.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [edk2-devel] [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
@ 2019-07-18  1:37   ` Liming Gao
  2019-07-18  1:53   ` Dong, Eric
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Liming Gao @ 2019-07-18  1:37 UTC (permalink / raw)
  To: devel@edk2.groups.io, Wu, Hao A
  Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D,
	Bi, Dandan, Dong, Eric, Zeng, Star, Gao, Zhichao

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Wu, Hao A
> Sent: Thursday, July 18, 2019 9:19 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Andrew Fish <afish@apple.com>; Laszlo Ersek <lersek@redhat.com>; Leif Lindholm
> <leif.lindholm@linaro.org>; Kinney, Michael D <michael.d.kinney@intel.com>; Bi, Dandan <dandan.bi@intel.com>; Dong, Eric
> <eric.dong@intel.com>; Gao, Liming <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>
> Subject: [edk2-devel] [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
> 
> This commit add the reviewers information for modules within MdeModulePkg.
> 
> For now the modules list includes:
> ACPI
> ACPI S3
> BDS
> Console and Graphics
> Core services (PEI, DXE and Runtime)
> Device and Peripheral
> Firmware Update
> HII and UI
> Management Mode (MM, SMM)
> Reset
> SMBIOS
> UEFI Variable
> 
> Please note that, for MdeModulePkg components not included in the above
> list, the reviewers will fall back to the package maintainers.
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> ---
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 18fd2ef43c..0298010809 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
> 
>  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
>  M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao A Wu <hao.a.wu@intel.com>
> +
> +MdeModulePkg: ACPI modules
> +F: MdeModulePkg/Include/*Acpi*.h
> +F: MdeModulePkg/Universal/Acpi/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: ACPI S3 modules
> +F: MdeModulePkg/*LockBox*/
> +F: MdeModulePkg/Include/*BootScript*.h
> +F: MdeModulePkg/Include/*LockBox*.h
> +F: MdeModulePkg/Include/*S3*.h
> +F: MdeModulePkg/Library/*S3*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: BDS modules
> +F: MdeModulePkg/*BootManager*/
> +F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> +F: MdeModulePkg/Universal/BdsDxe/
> +F: MdeModulePkg/Universal/DevicePathDxe/
> +F: MdeModulePkg/Universal/DriverHealthManagerDxe/
> +F: MdeModulePkg/Universal/LoadFileOnFv2/
> +F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Console and Graphics modules
> +F: MdeModulePkg/*Logo*/
> +F: MdeModulePkg/Include/*Logo*.h
> +F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
> +F: MdeModulePkg/Include/Guid/Console*.h
> +F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
> +F: MdeModulePkg/Include/Guid/TtyTerm.h
> +F: MdeModulePkg/Include/Library/BmpSupportLib.h
> +F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
> +F: MdeModulePkg/Library/BaseBmpSupportLib/
> +F: MdeModulePkg/Library/FrameBufferBltLib/
> +F: MdeModulePkg/Universal/Console/
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> +F: MdeModulePkg/*Mem*/
> +F: MdeModulePkg/*SectionExtract*/
> +F: MdeModulePkg/*StatusCode*/
> +F: MdeModulePkg/Application/DumpDynPcd/
> +F: MdeModulePkg/Core/Dxe/
> +F: MdeModulePkg/Core/DxeIplPeim/
> +F: MdeModulePkg/Core/Pei/
> +F: MdeModulePkg/Core/RuntimeDxe/
> +F: MdeModulePkg/Include/*Mem*.h
> +F: MdeModulePkg/Include/*Pcd*.h
> +F: MdeModulePkg/Include/*Perf*.h
> +F: MdeModulePkg/Include/*StatusCode*.h
> +F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
> +F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
> +F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
> +F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
> +F: MdeModulePkg/Include/Guid/LzmaDecompress.h
> +F: MdeModulePkg/Include/Library/SecurityManagementLib.h
> +F: MdeModulePkg/Library/*Decompress*/
> +F: MdeModulePkg/Library/*Perf*/
> +F: MdeModulePkg/Library/DxeSecurityManagementLib/
> +F: MdeModulePkg/Universal/PCD/
> +F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
> +F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: Device and Peripheral modules
> +F: MdeModulePkg/*PciHostBridge*/
> +F: MdeModulePkg/*Serial*/
> +F: MdeModulePkg/Bus/
> +F: MdeModulePkg/Include/*Ata*.h
> +F: MdeModulePkg/Include/*IoMmu*.h
> +F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
> +F: MdeModulePkg/Include/*NvmExpress*.h
> +F: MdeModulePkg/Include/*SdMmc*.h
> +F: MdeModulePkg/Include/*SerialPort*.h
> +F: MdeModulePkg/Include/*Ufs*.h
> +F: MdeModulePkg/Include/*Usb*.h
> +F: MdeModulePkg/Include/Guid/RecoveryDevice.h
> +F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
> +F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
> +F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
> +F: MdeModulePkg/Include/Protocol/Ps2Policy.h
> +F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
> +F: MdeModulePkg/Universal/Disk/
> +F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> +R: Hao A Wu <hao.a.wu@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
> -  (especially for Bus, Universal/Console, Universal/Disk,
> -   Universal/BdsDxe and related libraries, header files)
> +
> +MdeModulePkg: Firmware Update modules
> +F: MdeModulePkg/*Capsule*/
> +F: MdeModulePkg/Include/*Capsule*.h
> +F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
> +F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
> +F: MdeModulePkg/Include/Protocol/EsrtManagement.h
> +F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
> +F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
> +F: MdeModulePkg/Library/FmpAuthenticationLibNull/
> +F: MdeModulePkg/Universal/Esrt*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: HII and UI modules
> +F: MdeModulePkg/*FileExplorer*/
> +F: MdeModulePkg/*Hii*/
> +F: MdeModulePkg/*Ui*/
> +F: MdeModulePkg/Application/BootManagerMenuApp/
> +F: MdeModulePkg/Include/*FileExplorer*.h
> +F: MdeModulePkg/Include/*FormBrowser*.h
> +F: MdeModulePkg/Include/*Hii*.h
> +F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
> +F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
> +F: MdeModulePkg/Library/CustomizedDisplayLib/
> +F: MdeModulePkg/Universal/DisplayEngineDxe/
> +F: MdeModulePkg/Universal/DriverSampleDxe/
> +F: MdeModulePkg/Universal/SetupBrowserDxe/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: Management Mode (MM, SMM) modules
> +F: MdeModulePkg/*Smi*/
> +F: MdeModulePkg/*Smm*/
> +F: MdeModulePkg/Include/*Smi*.h
> +F: MdeModulePkg/Include/*Smm*.h
> +R: Eric Dong <eric.dong@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Reset modules
> +F: MdeModulePkg/*Reset*/
> +F: MdeModulePkg/Include/*Reset*.h
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: SMBIOS modules
> +F: MdeModulePkg/Universal/Smbios*/
> +R: Dandan Bi <dandan.bi@intel.com>
>  R: Star Zeng <star.zeng@intel.com>
> 
> +MdeModulePkg: UEFI Variable modules
> +F: MdeModulePkg/*Var*/
> +F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
> +F: MdeModulePkg/Include/*/*Var*.h
> +F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
> +F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
> +F: MdeModulePkg/Universal/FaultTolerantWrite*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
>  MdePkg
>  F: MdePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
> 
> --
> 2.12.0.windows.1
> 
> 
> 


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
  2019-07-18  1:37   ` [edk2-devel] " Liming Gao
@ 2019-07-18  1:53   ` Dong, Eric
  2019-07-18  5:25   ` Gao, Zhichao
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Dong, Eric @ 2019-07-18  1:53 UTC (permalink / raw)
  To: Wu, Hao A, devel@edk2.groups.io
  Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D,
	Bi, Dandan, Gao, Liming, Zeng, Star, Gao, Zhichao

Reviewed-by: Eric Dong <eric.dong@intel.com>

> -----Original Message-----
> From: Wu, Hao A
> Sent: Thursday, July 18, 2019 9:19 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Andrew Fish <afish@apple.com>;
> Laszlo Ersek <lersek@redhat.com>; Leif Lindholm <leif.lindholm@linaro.org>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Bi, Dandan
> <dandan.bi@intel.com>; Dong, Eric <eric.dong@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>
> Subject: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for
> MdeModulePkg
> 
> This commit add the reviewers information for modules within
> MdeModulePkg.
> 
> For now the modules list includes:
> ACPI
> ACPI S3
> BDS
> Console and Graphics
> Core services (PEI, DXE and Runtime)
> Device and Peripheral
> Firmware Update
> HII and UI
> Management Mode (MM, SMM)
> Reset
> SMBIOS
> UEFI Variable
> 
> Please note that, for MdeModulePkg components not included in the above
> list, the reviewers will fall back to the package maintainers.
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> ---
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt index 18fd2ef43c..0298010809
> 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
> 
>  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
>  M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao A Wu <hao.a.wu@intel.com>
> +
> +MdeModulePkg: ACPI modules
> +F: MdeModulePkg/Include/*Acpi*.h
> +F: MdeModulePkg/Universal/Acpi/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: ACPI S3 modules
> +F: MdeModulePkg/*LockBox*/
> +F: MdeModulePkg/Include/*BootScript*.h
> +F: MdeModulePkg/Include/*LockBox*.h
> +F: MdeModulePkg/Include/*S3*.h
> +F: MdeModulePkg/Library/*S3*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: BDS modules
> +F: MdeModulePkg/*BootManager*/
> +F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> +F: MdeModulePkg/Universal/BdsDxe/
> +F: MdeModulePkg/Universal/DevicePathDxe/
> +F: MdeModulePkg/Universal/DriverHealthManagerDxe/
> +F: MdeModulePkg/Universal/LoadFileOnFv2/
> +F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Console and Graphics modules
> +F: MdeModulePkg/*Logo*/
> +F: MdeModulePkg/Include/*Logo*.h
> +F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
> +F: MdeModulePkg/Include/Guid/Console*.h
> +F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
> +F: MdeModulePkg/Include/Guid/TtyTerm.h
> +F: MdeModulePkg/Include/Library/BmpSupportLib.h
> +F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
> +F: MdeModulePkg/Library/BaseBmpSupportLib/
> +F: MdeModulePkg/Library/FrameBufferBltLib/
> +F: MdeModulePkg/Universal/Console/
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> +F: MdeModulePkg/*Mem*/
> +F: MdeModulePkg/*SectionExtract*/
> +F: MdeModulePkg/*StatusCode*/
> +F: MdeModulePkg/Application/DumpDynPcd/
> +F: MdeModulePkg/Core/Dxe/
> +F: MdeModulePkg/Core/DxeIplPeim/
> +F: MdeModulePkg/Core/Pei/
> +F: MdeModulePkg/Core/RuntimeDxe/
> +F: MdeModulePkg/Include/*Mem*.h
> +F: MdeModulePkg/Include/*Pcd*.h
> +F: MdeModulePkg/Include/*Perf*.h
> +F: MdeModulePkg/Include/*StatusCode*.h
> +F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
> +F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
> +F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
> +F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
> +F: MdeModulePkg/Include/Guid/LzmaDecompress.h
> +F: MdeModulePkg/Include/Library/SecurityManagementLib.h
> +F: MdeModulePkg/Library/*Decompress*/
> +F: MdeModulePkg/Library/*Perf*/
> +F: MdeModulePkg/Library/DxeSecurityManagementLib/
> +F: MdeModulePkg/Universal/PCD/
> +F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
> +F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: Device and Peripheral modules
> +F: MdeModulePkg/*PciHostBridge*/
> +F: MdeModulePkg/*Serial*/
> +F: MdeModulePkg/Bus/
> +F: MdeModulePkg/Include/*Ata*.h
> +F: MdeModulePkg/Include/*IoMmu*.h
> +F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
> +F: MdeModulePkg/Include/*NvmExpress*.h
> +F: MdeModulePkg/Include/*SdMmc*.h
> +F: MdeModulePkg/Include/*SerialPort*.h
> +F: MdeModulePkg/Include/*Ufs*.h
> +F: MdeModulePkg/Include/*Usb*.h
> +F: MdeModulePkg/Include/Guid/RecoveryDevice.h
> +F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
> +F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
> +F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
> +F: MdeModulePkg/Include/Protocol/Ps2Policy.h
> +F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
> +F: MdeModulePkg/Universal/Disk/
> +F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> +R: Hao A Wu <hao.a.wu@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
> -  (especially for Bus, Universal/Console, Universal/Disk,
> -   Universal/BdsDxe and related libraries, header files)
> +
> +MdeModulePkg: Firmware Update modules
> +F: MdeModulePkg/*Capsule*/
> +F: MdeModulePkg/Include/*Capsule*.h
> +F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
> +F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
> +F: MdeModulePkg/Include/Protocol/EsrtManagement.h
> +F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
> +F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
> +F: MdeModulePkg/Library/FmpAuthenticationLibNull/
> +F: MdeModulePkg/Universal/Esrt*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: HII and UI modules
> +F: MdeModulePkg/*FileExplorer*/
> +F: MdeModulePkg/*Hii*/
> +F: MdeModulePkg/*Ui*/
> +F: MdeModulePkg/Application/BootManagerMenuApp/
> +F: MdeModulePkg/Include/*FileExplorer*.h
> +F: MdeModulePkg/Include/*FormBrowser*.h
> +F: MdeModulePkg/Include/*Hii*.h
> +F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
> +F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
> +F: MdeModulePkg/Library/CustomizedDisplayLib/
> +F: MdeModulePkg/Universal/DisplayEngineDxe/
> +F: MdeModulePkg/Universal/DriverSampleDxe/
> +F: MdeModulePkg/Universal/SetupBrowserDxe/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: Management Mode (MM, SMM) modules
> +F: MdeModulePkg/*Smi*/
> +F: MdeModulePkg/*Smm*/
> +F: MdeModulePkg/Include/*Smi*.h
> +F: MdeModulePkg/Include/*Smm*.h
> +R: Eric Dong <eric.dong@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Reset modules
> +F: MdeModulePkg/*Reset*/
> +F: MdeModulePkg/Include/*Reset*.h
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: SMBIOS modules
> +F: MdeModulePkg/Universal/Smbios*/
> +R: Dandan Bi <dandan.bi@intel.com>
>  R: Star Zeng <star.zeng@intel.com>
> 
> +MdeModulePkg: UEFI Variable modules
> +F: MdeModulePkg/*Var*/
> +F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
> +F: MdeModulePkg/Include/*/*Var*.h
> +F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
> +F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
> +F: MdeModulePkg/Universal/FaultTolerantWrite*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
>  MdePkg
>  F: MdePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
> 
> --
> 2.12.0.windows.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
  2019-07-18  1:37   ` [edk2-devel] " Liming Gao
  2019-07-18  1:53   ` Dong, Eric
@ 2019-07-18  5:25   ` Gao, Zhichao
  2019-07-18  5:26   ` Zeng, Star
  2019-07-18  5:39   ` Dandan Bi
  4 siblings, 0 replies; 8+ messages in thread
From: Gao, Zhichao @ 2019-07-18  5:25 UTC (permalink / raw)
  To: Wu, Hao A, devel@edk2.groups.io
  Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D,
	Bi, Dandan, Dong, Eric, Gao, Liming, Zeng, Star

Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>

> -----Original Message-----
> From: Wu, Hao A
> Sent: Thursday, July 18, 2019 9:19 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Andrew Fish <afish@apple.com>;
> Laszlo Ersek <lersek@redhat.com>; Leif Lindholm <leif.lindholm@linaro.org>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Bi, Dandan
> <dandan.bi@intel.com>; Dong, Eric <eric.dong@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>
> Subject: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for
> MdeModulePkg
> 
> This commit add the reviewers information for modules within
> MdeModulePkg.
> 
> For now the modules list includes:
> ACPI
> ACPI S3
> BDS
> Console and Graphics
> Core services (PEI, DXE and Runtime)
> Device and Peripheral
> Firmware Update
> HII and UI
> Management Mode (MM, SMM)
> Reset
> SMBIOS
> UEFI Variable
> 
> Please note that, for MdeModulePkg components not included in the above
> list, the reviewers will fall back to the package maintainers.
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> ---
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt index 18fd2ef43c..0298010809
> 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
> 
>  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
>  M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao A Wu <hao.a.wu@intel.com>
> +
> +MdeModulePkg: ACPI modules
> +F: MdeModulePkg/Include/*Acpi*.h
> +F: MdeModulePkg/Universal/Acpi/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: ACPI S3 modules
> +F: MdeModulePkg/*LockBox*/
> +F: MdeModulePkg/Include/*BootScript*.h
> +F: MdeModulePkg/Include/*LockBox*.h
> +F: MdeModulePkg/Include/*S3*.h
> +F: MdeModulePkg/Library/*S3*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: BDS modules
> +F: MdeModulePkg/*BootManager*/
> +F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> +F: MdeModulePkg/Universal/BdsDxe/
> +F: MdeModulePkg/Universal/DevicePathDxe/
> +F: MdeModulePkg/Universal/DriverHealthManagerDxe/
> +F: MdeModulePkg/Universal/LoadFileOnFv2/
> +F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Console and Graphics modules
> +F: MdeModulePkg/*Logo*/
> +F: MdeModulePkg/Include/*Logo*.h
> +F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
> +F: MdeModulePkg/Include/Guid/Console*.h
> +F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
> +F: MdeModulePkg/Include/Guid/TtyTerm.h
> +F: MdeModulePkg/Include/Library/BmpSupportLib.h
> +F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
> +F: MdeModulePkg/Library/BaseBmpSupportLib/
> +F: MdeModulePkg/Library/FrameBufferBltLib/
> +F: MdeModulePkg/Universal/Console/
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> +F: MdeModulePkg/*Mem*/
> +F: MdeModulePkg/*SectionExtract*/
> +F: MdeModulePkg/*StatusCode*/
> +F: MdeModulePkg/Application/DumpDynPcd/
> +F: MdeModulePkg/Core/Dxe/
> +F: MdeModulePkg/Core/DxeIplPeim/
> +F: MdeModulePkg/Core/Pei/
> +F: MdeModulePkg/Core/RuntimeDxe/
> +F: MdeModulePkg/Include/*Mem*.h
> +F: MdeModulePkg/Include/*Pcd*.h
> +F: MdeModulePkg/Include/*Perf*.h
> +F: MdeModulePkg/Include/*StatusCode*.h
> +F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
> +F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
> +F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
> +F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
> +F: MdeModulePkg/Include/Guid/LzmaDecompress.h
> +F: MdeModulePkg/Include/Library/SecurityManagementLib.h
> +F: MdeModulePkg/Library/*Decompress*/
> +F: MdeModulePkg/Library/*Perf*/
> +F: MdeModulePkg/Library/DxeSecurityManagementLib/
> +F: MdeModulePkg/Universal/PCD/
> +F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
> +F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: Device and Peripheral modules
> +F: MdeModulePkg/*PciHostBridge*/
> +F: MdeModulePkg/*Serial*/
> +F: MdeModulePkg/Bus/
> +F: MdeModulePkg/Include/*Ata*.h
> +F: MdeModulePkg/Include/*IoMmu*.h
> +F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
> +F: MdeModulePkg/Include/*NvmExpress*.h
> +F: MdeModulePkg/Include/*SdMmc*.h
> +F: MdeModulePkg/Include/*SerialPort*.h
> +F: MdeModulePkg/Include/*Ufs*.h
> +F: MdeModulePkg/Include/*Usb*.h
> +F: MdeModulePkg/Include/Guid/RecoveryDevice.h
> +F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
> +F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
> +F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
> +F: MdeModulePkg/Include/Protocol/Ps2Policy.h
> +F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
> +F: MdeModulePkg/Universal/Disk/
> +F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> +R: Hao A Wu <hao.a.wu@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
> -  (especially for Bus, Universal/Console, Universal/Disk,
> -   Universal/BdsDxe and related libraries, header files)
> +
> +MdeModulePkg: Firmware Update modules
> +F: MdeModulePkg/*Capsule*/
> +F: MdeModulePkg/Include/*Capsule*.h
> +F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
> +F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
> +F: MdeModulePkg/Include/Protocol/EsrtManagement.h
> +F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
> +F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
> +F: MdeModulePkg/Library/FmpAuthenticationLibNull/
> +F: MdeModulePkg/Universal/Esrt*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: HII and UI modules
> +F: MdeModulePkg/*FileExplorer*/
> +F: MdeModulePkg/*Hii*/
> +F: MdeModulePkg/*Ui*/
> +F: MdeModulePkg/Application/BootManagerMenuApp/
> +F: MdeModulePkg/Include/*FileExplorer*.h
> +F: MdeModulePkg/Include/*FormBrowser*.h
> +F: MdeModulePkg/Include/*Hii*.h
> +F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
> +F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
> +F: MdeModulePkg/Library/CustomizedDisplayLib/
> +F: MdeModulePkg/Universal/DisplayEngineDxe/
> +F: MdeModulePkg/Universal/DriverSampleDxe/
> +F: MdeModulePkg/Universal/SetupBrowserDxe/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: Management Mode (MM, SMM) modules
> +F: MdeModulePkg/*Smi*/
> +F: MdeModulePkg/*Smm*/
> +F: MdeModulePkg/Include/*Smi*.h
> +F: MdeModulePkg/Include/*Smm*.h
> +R: Eric Dong <eric.dong@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Reset modules
> +F: MdeModulePkg/*Reset*/
> +F: MdeModulePkg/Include/*Reset*.h
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: SMBIOS modules
> +F: MdeModulePkg/Universal/Smbios*/
> +R: Dandan Bi <dandan.bi@intel.com>
>  R: Star Zeng <star.zeng@intel.com>
> 
> +MdeModulePkg: UEFI Variable modules
> +F: MdeModulePkg/*Var*/
> +F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
> +F: MdeModulePkg/Include/*/*Var*.h
> +F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
> +F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
> +F: MdeModulePkg/Universal/FaultTolerantWrite*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
>  MdePkg
>  F: MdePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
> 
> --
> 2.12.0.windows.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
                     ` (2 preceding siblings ...)
  2019-07-18  5:25   ` Gao, Zhichao
@ 2019-07-18  5:26   ` Zeng, Star
  2019-07-18  5:39   ` Dandan Bi
  4 siblings, 0 replies; 8+ messages in thread
From: Zeng, Star @ 2019-07-18  5:26 UTC (permalink / raw)
  To: Wu, Hao A, devel@edk2.groups.io
  Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D,
	Bi, Dandan, Dong, Eric, Gao, Liming, Gao, Zhichao, Zeng, Star

Reviewed-by: Star Zeng <star.zeng@intel.com> for my part.

> -----Original Message-----
> From: Wu, Hao A
> Sent: Thursday, July 18, 2019 9:19 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Andrew Fish <afish@apple.com>;
> Laszlo Ersek <lersek@redhat.com>; Leif Lindholm <leif.lindholm@linaro.org>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Bi, Dandan
> <dandan.bi@intel.com>; Dong, Eric <eric.dong@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>
> Subject: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for
> MdeModulePkg
> 
> This commit add the reviewers information for modules within
> MdeModulePkg.
> 
> For now the modules list includes:
> ACPI
> ACPI S3
> BDS
> Console and Graphics
> Core services (PEI, DXE and Runtime)
> Device and Peripheral
> Firmware Update
> HII and UI
> Management Mode (MM, SMM)
> Reset
> SMBIOS
> UEFI Variable
> 
> Please note that, for MdeModulePkg components not included in the above
> list, the reviewers will fall back to the package maintainers.
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> ---
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt index 18fd2ef43c..0298010809
> 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
> 
>  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
>  M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao A Wu <hao.a.wu@intel.com>
> +
> +MdeModulePkg: ACPI modules
> +F: MdeModulePkg/Include/*Acpi*.h
> +F: MdeModulePkg/Universal/Acpi/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: ACPI S3 modules
> +F: MdeModulePkg/*LockBox*/
> +F: MdeModulePkg/Include/*BootScript*.h
> +F: MdeModulePkg/Include/*LockBox*.h
> +F: MdeModulePkg/Include/*S3*.h
> +F: MdeModulePkg/Library/*S3*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: BDS modules
> +F: MdeModulePkg/*BootManager*/
> +F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> +F: MdeModulePkg/Universal/BdsDxe/
> +F: MdeModulePkg/Universal/DevicePathDxe/
> +F: MdeModulePkg/Universal/DriverHealthManagerDxe/
> +F: MdeModulePkg/Universal/LoadFileOnFv2/
> +F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Console and Graphics modules
> +F: MdeModulePkg/*Logo*/
> +F: MdeModulePkg/Include/*Logo*.h
> +F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
> +F: MdeModulePkg/Include/Guid/Console*.h
> +F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
> +F: MdeModulePkg/Include/Guid/TtyTerm.h
> +F: MdeModulePkg/Include/Library/BmpSupportLib.h
> +F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
> +F: MdeModulePkg/Library/BaseBmpSupportLib/
> +F: MdeModulePkg/Library/FrameBufferBltLib/
> +F: MdeModulePkg/Universal/Console/
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> +F: MdeModulePkg/*Mem*/
> +F: MdeModulePkg/*SectionExtract*/
> +F: MdeModulePkg/*StatusCode*/
> +F: MdeModulePkg/Application/DumpDynPcd/
> +F: MdeModulePkg/Core/Dxe/
> +F: MdeModulePkg/Core/DxeIplPeim/
> +F: MdeModulePkg/Core/Pei/
> +F: MdeModulePkg/Core/RuntimeDxe/
> +F: MdeModulePkg/Include/*Mem*.h
> +F: MdeModulePkg/Include/*Pcd*.h
> +F: MdeModulePkg/Include/*Perf*.h
> +F: MdeModulePkg/Include/*StatusCode*.h
> +F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
> +F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
> +F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
> +F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
> +F: MdeModulePkg/Include/Guid/LzmaDecompress.h
> +F: MdeModulePkg/Include/Library/SecurityManagementLib.h
> +F: MdeModulePkg/Library/*Decompress*/
> +F: MdeModulePkg/Library/*Perf*/
> +F: MdeModulePkg/Library/DxeSecurityManagementLib/
> +F: MdeModulePkg/Universal/PCD/
> +F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
> +F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: Device and Peripheral modules
> +F: MdeModulePkg/*PciHostBridge*/
> +F: MdeModulePkg/*Serial*/
> +F: MdeModulePkg/Bus/
> +F: MdeModulePkg/Include/*Ata*.h
> +F: MdeModulePkg/Include/*IoMmu*.h
> +F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
> +F: MdeModulePkg/Include/*NvmExpress*.h
> +F: MdeModulePkg/Include/*SdMmc*.h
> +F: MdeModulePkg/Include/*SerialPort*.h
> +F: MdeModulePkg/Include/*Ufs*.h
> +F: MdeModulePkg/Include/*Usb*.h
> +F: MdeModulePkg/Include/Guid/RecoveryDevice.h
> +F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
> +F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
> +F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
> +F: MdeModulePkg/Include/Protocol/Ps2Policy.h
> +F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
> +F: MdeModulePkg/Universal/Disk/
> +F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> +R: Hao A Wu <hao.a.wu@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
> -  (especially for Bus, Universal/Console, Universal/Disk,
> -   Universal/BdsDxe and related libraries, header files)
> +
> +MdeModulePkg: Firmware Update modules
> +F: MdeModulePkg/*Capsule*/
> +F: MdeModulePkg/Include/*Capsule*.h
> +F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
> +F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
> +F: MdeModulePkg/Include/Protocol/EsrtManagement.h
> +F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
> +F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
> +F: MdeModulePkg/Library/FmpAuthenticationLibNull/
> +F: MdeModulePkg/Universal/Esrt*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: HII and UI modules
> +F: MdeModulePkg/*FileExplorer*/
> +F: MdeModulePkg/*Hii*/
> +F: MdeModulePkg/*Ui*/
> +F: MdeModulePkg/Application/BootManagerMenuApp/
> +F: MdeModulePkg/Include/*FileExplorer*.h
> +F: MdeModulePkg/Include/*FormBrowser*.h
> +F: MdeModulePkg/Include/*Hii*.h
> +F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
> +F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
> +F: MdeModulePkg/Library/CustomizedDisplayLib/
> +F: MdeModulePkg/Universal/DisplayEngineDxe/
> +F: MdeModulePkg/Universal/DriverSampleDxe/
> +F: MdeModulePkg/Universal/SetupBrowserDxe/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: Management Mode (MM, SMM) modules
> +F: MdeModulePkg/*Smi*/
> +F: MdeModulePkg/*Smm*/
> +F: MdeModulePkg/Include/*Smi*.h
> +F: MdeModulePkg/Include/*Smm*.h
> +R: Eric Dong <eric.dong@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Reset modules
> +F: MdeModulePkg/*Reset*/
> +F: MdeModulePkg/Include/*Reset*.h
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: SMBIOS modules
> +F: MdeModulePkg/Universal/Smbios*/
> +R: Dandan Bi <dandan.bi@intel.com>
>  R: Star Zeng <star.zeng@intel.com>
> 
> +MdeModulePkg: UEFI Variable modules
> +F: MdeModulePkg/*Var*/
> +F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
> +F: MdeModulePkg/Include/*/*Var*.h
> +F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
> +F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
> +F: MdeModulePkg/Universal/FaultTolerantWrite*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
>  MdePkg
>  F: MdePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
> 
> --
> 2.12.0.windows.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for MdeModulePkg
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
                     ` (3 preceding siblings ...)
  2019-07-18  5:26   ` Zeng, Star
@ 2019-07-18  5:39   ` Dandan Bi
  4 siblings, 0 replies; 8+ messages in thread
From: Dandan Bi @ 2019-07-18  5:39 UTC (permalink / raw)
  To: Wu, Hao A, devel@edk2.groups.io
  Cc: Andrew Fish, Laszlo Ersek, Leif Lindholm, Kinney, Michael D,
	Dong, Eric, Gao, Liming, Zeng, Star, Gao, Zhichao

Reviewed-by: Dandan Bi <dandan.bi@intel.com>

Thanks,
Dandan
> -----Original Message-----
> From: Wu, Hao A
> Sent: Thursday, July 18, 2019 9:19 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a.wu@intel.com>; Andrew Fish <afish@apple.com>;
> Laszlo Ersek <lersek@redhat.com>; Leif Lindholm <leif.lindholm@linaro.org>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Bi, Dandan
> <dandan.bi@intel.com>; Dong, Eric <eric.dong@intel.com>; Gao, Liming
> <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>
> Subject: [PATCH v2 1/1] Maintainers.txt: Fine-grained review ownership for
> MdeModulePkg
> 
> This commit add the reviewers information for modules within
> MdeModulePkg.
> 
> For now the modules list includes:
> ACPI
> ACPI S3
> BDS
> Console and Graphics
> Core services (PEI, DXE and Runtime)
> Device and Peripheral
> Firmware Update
> HII and UI
> Management Mode (MM, SMM)
> Reset
> SMBIOS
> UEFI Variable
> 
> Please note that, for MdeModulePkg components not included in the above
> list, the reviewers will fall back to the package maintainers.
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Signed-off-by: Hao A Wu <hao.a.wu@intel.com>
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> ---
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt index 18fd2ef43c..0298010809
> 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -183,17 +183,164 @@ R: Star Zeng <star.zeng@intel.com>
> 
>  MdeModulePkg
>  F: MdeModulePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdeModulePkg
>  M: Jian J Wang <jian.j.wang@intel.com>
>  M: Hao A Wu <hao.a.wu@intel.com>
> +
> +MdeModulePkg: ACPI modules
> +F: MdeModulePkg/Include/*Acpi*.h
> +F: MdeModulePkg/Universal/Acpi/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: ACPI S3 modules
> +F: MdeModulePkg/*LockBox*/
> +F: MdeModulePkg/Include/*BootScript*.h
> +F: MdeModulePkg/Include/*LockBox*.h
> +F: MdeModulePkg/Include/*S3*.h
> +F: MdeModulePkg/Library/*S3*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: BDS modules
> +F: MdeModulePkg/*BootManager*/
> +F: MdeModulePkg/Include/Library/UefiBootManagerLib.h
> +F: MdeModulePkg/Universal/BdsDxe/
> +F: MdeModulePkg/Universal/DevicePathDxe/
> +F: MdeModulePkg/Universal/DriverHealthManagerDxe/
> +F: MdeModulePkg/Universal/LoadFileOnFv2/
> +F: MdeModulePkg/Universal/SecurityStubDxe/Defer3rdPartyImageLoad.*
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Console and Graphics modules
> +F: MdeModulePkg/*Logo*/
> +F: MdeModulePkg/Include/*Logo*.h
> +F: MdeModulePkg/Include/Guid/ConnectConInEvent.h
> +F: MdeModulePkg/Include/Guid/Console*.h
> +F: MdeModulePkg/Include/Guid/StandardErrorDevice.h
> +F: MdeModulePkg/Include/Guid/TtyTerm.h
> +F: MdeModulePkg/Include/Library/BmpSupportLib.h
> +F: MdeModulePkg/Include/Library/FrameBufferBltLib.h
> +F: MdeModulePkg/Library/BaseBmpSupportLib/
> +F: MdeModulePkg/Library/FrameBufferBltLib/
> +F: MdeModulePkg/Universal/Console/
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Core services (PEI, DXE and Runtime) modules
> +F: MdeModulePkg/*Mem*/
> +F: MdeModulePkg/*SectionExtract*/
> +F: MdeModulePkg/*StatusCode*/
> +F: MdeModulePkg/Application/DumpDynPcd/
> +F: MdeModulePkg/Core/Dxe/
> +F: MdeModulePkg/Core/DxeIplPeim/
> +F: MdeModulePkg/Core/Pei/
> +F: MdeModulePkg/Core/RuntimeDxe/
> +F: MdeModulePkg/Include/*Mem*.h
> +F: MdeModulePkg/Include/*Pcd*.h
> +F: MdeModulePkg/Include/*Perf*.h
> +F: MdeModulePkg/Include/*StatusCode*.h
> +F: MdeModulePkg/Include/Guid/Crc32GuidedSectionExtraction.h
> +F: MdeModulePkg/Include/Guid/EventExitBootServiceFailed.h
> +F: MdeModulePkg/Include/Guid/IdleLoopEvent.h
> +F: MdeModulePkg/Include/Guid/LoadModuleAtFixedAddress.h
> +F: MdeModulePkg/Include/Guid/LzmaDecompress.h
> +F: MdeModulePkg/Include/Library/SecurityManagementLib.h
> +F: MdeModulePkg/Library/*Decompress*/
> +F: MdeModulePkg/Library/*Perf*/
> +F: MdeModulePkg/Library/DxeSecurityManagementLib/
> +F: MdeModulePkg/Universal/PCD/
> +F: MdeModulePkg/Universal/PlatformDriOverrideDxe/
> +F: MdeModulePkg/Universal/SecurityStubDxe/SecurityStub.c
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: Device and Peripheral modules
> +F: MdeModulePkg/*PciHostBridge*/
> +F: MdeModulePkg/*Serial*/
> +F: MdeModulePkg/Bus/
> +F: MdeModulePkg/Include/*Ata*.h
> +F: MdeModulePkg/Include/*IoMmu*.h
> +F: MdeModulePkg/Include/*NonDiscoverableDevice*.h
> +F: MdeModulePkg/Include/*NvmExpress*.h
> +F: MdeModulePkg/Include/*SdMmc*.h
> +F: MdeModulePkg/Include/*SerialPort*.h
> +F: MdeModulePkg/Include/*Ufs*.h
> +F: MdeModulePkg/Include/*Usb*.h
> +F: MdeModulePkg/Include/Guid/RecoveryDevice.h
> +F: MdeModulePkg/Include/Guid/S3StorageDeviceInitList.h
> +F: MdeModulePkg/Include/Library/PciHostBridgeLib.h
> +F: MdeModulePkg/Include/Ppi/StorageSecurityCommand.h
> +F: MdeModulePkg/Include/Protocol/Ps2Policy.h
> +F: MdeModulePkg/Library/NonDiscoverableDeviceRegistrationLib/
> +F: MdeModulePkg/Universal/Disk/
> +F: MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/
> +R: Hao A Wu <hao.a.wu@intel.com>
>  R: Ray Ni <ray.ni@intel.com>
> -  (especially for Bus, Universal/Console, Universal/Disk,
> -   Universal/BdsDxe and related libraries, header files)
> +
> +MdeModulePkg: Firmware Update modules
> +F: MdeModulePkg/*Capsule*/
> +F: MdeModulePkg/Include/*Capsule*.h
> +F: MdeModulePkg/Include/Library/DisplayUpdateProgressLib.h
> +F: MdeModulePkg/Include/Library/FmpAuthenticationLib.h
> +F: MdeModulePkg/Include/Protocol/EsrtManagement.h
> +F: MdeModulePkg/Include/Protocol/FirmwareManagementProgress.h
> +F: MdeModulePkg/Library/DisplayUpdateProgressLib*/
> +F: MdeModulePkg/Library/FmpAuthenticationLibNull/
> +F: MdeModulePkg/Universal/Esrt*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
> +MdeModulePkg: HII and UI modules
> +F: MdeModulePkg/*FileExplorer*/
> +F: MdeModulePkg/*Hii*/
> +F: MdeModulePkg/*Ui*/
> +F: MdeModulePkg/Application/BootManagerMenuApp/
> +F: MdeModulePkg/Include/*FileExplorer*.h
> +F: MdeModulePkg/Include/*FormBrowser*.h
> +F: MdeModulePkg/Include/*Hii*.h
> +F: MdeModulePkg/Include/Library/CustomizedDisplayLib.h
> +F: MdeModulePkg/Include/Protocol/DisplayProtocol.h
> +F: MdeModulePkg/Library/CustomizedDisplayLib/
> +F: MdeModulePkg/Universal/DisplayEngineDxe/
> +F: MdeModulePkg/Universal/DriverSampleDxe/
> +F: MdeModulePkg/Universal/SetupBrowserDxe/
> +R: Dandan Bi <dandan.bi@intel.com>
> +R: Eric Dong <eric.dong@intel.com>
> +
> +MdeModulePkg: Management Mode (MM, SMM) modules
> +F: MdeModulePkg/*Smi*/
> +F: MdeModulePkg/*Smm*/
> +F: MdeModulePkg/Include/*Smi*.h
> +F: MdeModulePkg/Include/*Smm*.h
> +R: Eric Dong <eric.dong@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: Reset modules
> +F: MdeModulePkg/*Reset*/
> +F: MdeModulePkg/Include/*Reset*.h
> +R: Zhichao Gao <zhichao.gao@intel.com>
> +R: Ray Ni <ray.ni@intel.com>
> +
> +MdeModulePkg: SMBIOS modules
> +F: MdeModulePkg/Universal/Smbios*/
> +R: Dandan Bi <dandan.bi@intel.com>
>  R: Star Zeng <star.zeng@intel.com>
> 
> +MdeModulePkg: UEFI Variable modules
> +F: MdeModulePkg/*Var*/
> +F: MdeModulePkg/Include/*/*FaultTolerantWrite*.h
> +F: MdeModulePkg/Include/*/*Var*.h
> +F: MdeModulePkg/Include/Guid/SystemNvDataGuid.h
> +F: MdeModulePkg/Include/Protocol/SwapAddressRange.h
> +F: MdeModulePkg/Universal/FaultTolerantWrite*/
> +R: Hao A Wu <hao.a.wu@intel.com>
> +R: Liming Gao <liming.gao@intel.com>
> +
>  MdePkg
>  F: MdePkg/
>  W: https://github.com/tianocore/tianocore.github.io/wiki/MdePkg
>  M: Michael D Kinney <michael.d.kinney@intel.com>
>  M: Liming Gao <liming.gao@intel.com>
> 
> --
> 2.12.0.windows.1


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH v2 0/1] Maintainers.txt: Review ownership for MdeModulePkg
  2019-07-18  1:19 [PATCH v2 0/1] Maintainers.txt: Review ownership for MdeModulePkg Wu, Hao A
  2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
@ 2019-07-18 12:37 ` Laszlo Ersek
  1 sibling, 0 replies; 8+ messages in thread
From: Laszlo Ersek @ 2019-07-18 12:37 UTC (permalink / raw)
  To: Hao A Wu, devel
  Cc: Andrew Fish, Leif Lindholm, Michael D Kinney, Dandan Bi,
	Eric Dong, Liming Gao, Ray Ni, Star Zeng, Zhichao Gao

On 07/18/19 03:19, Hao A Wu wrote:
> This patch is also available at:
> https://github.com/hwu25/edk2/tree/mdemodulepkg_reviewers_v2
> 
> Also, please note that on the above branch, the proposed patch
> (MdeModulePkg reviewers) is inserted between:
> 
> * [edk2-devel] [PATCH 4/4] Maintainers.txt: split out section "OvmfPkg: CSM modules"
> (from Laszlo)
> https://edk2.groups.io/g/devel/message/43732
> 
> and
> 
> * [edk2-devel] [PATCH 3/3] BaseTools: add GetMaintainer.py script
> (from Leif)
> https://edk2.groups.io/g/devel/message/43666
> 
> 
> V2 changes:
> A. Sort "F:" patterns in alphabetical order for each section;
> B. Serveral module renaming:
>    Console and Graphic -> Console and Graphics
>    S3                  -> ACPI S3
>    SMM                 -> Management Mode (MM, SMM)
>    Variable            -> UEFI Variable
> 
> 
> Cc: Andrew Fish <afish@apple.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Dandan Bi <dandan.bi@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> 
> Hao A Wu (1):
>   Maintainers.txt: Fine-grained review ownership for MdeModulePkg
> 
>  Maintainers.txt | 151 +++++++++++++++++++-
>  1 file changed, 149 insertions(+), 2 deletions(-)
> 

Acked-by: Laszlo Ersek <lersek@redhat.com>

Thanks
Laszlo

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2019-07-18 12:37 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-18  1:19 [PATCH v2 0/1] Maintainers.txt: Review ownership for MdeModulePkg Wu, Hao A
2019-07-18  1:19 ` [PATCH v2 1/1] Maintainers.txt: Fine-grained review " Wu, Hao A
2019-07-18  1:37   ` [edk2-devel] " Liming Gao
2019-07-18  1:53   ` Dong, Eric
2019-07-18  5:25   ` Gao, Zhichao
2019-07-18  5:26   ` Zeng, Star
2019-07-18  5:39   ` Dandan Bi
2019-07-18 12:37 ` [PATCH v2 0/1] Maintainers.txt: Review " Laszlo Ersek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox