[PATCH edk2-platforms v2 0/5] Update Maintainers.txt

[PATCH edk2-platforms v2 0/5] Update Maintainers.txt

[Leif Lindholm]

This set does a few things:
- Imports the documentation header from edk2/Maintainers.txt
- Reformats some maintainer entries to get rid of ',' characters
  in the names.
- Adds F: entries for all sections, and a default fallback section
  for those where there is no match.
- Merges the Marvell sections (designed to work with this format)
  into a single area of responsibility.
- Adds sections for those that now ended up having none other than
  the fallback one (mostly ARM platforms).
- Remove Ard from the default section.

I am intentionally not Cc:ing all affected maintainers, since this
is a reformatting exercise. All roles actually affected are Cc:d.

Changes since v1:
- 2/6 ("Maintainers.txt: drop ',' from maintainer names") dropped,
  since commit 3adcb58818ca ("Maintainers.txt: Update Intel package
  maintainers") dropped the affected sections.
- Fixed the spelling of Ard's surname :|
- Fixed other reported typos.
- Added R-b/A-b.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>

Leif Lindholm (5):
  Maintainers.txt: Update documentation for new format
  Maintainers.txt: revamp file to machine-readable format
  Maintainers.txt: merge Marvell platforms and silicon sections
  Maintainers.txt: create explicit sections for remaining modules
  Maintainers.txt: drop Ard from default section

 Maintainers.txt | 153 ++++++++++++++++++++++++++++++++++++++++--------
 1 file changed, 127 insertions(+), 26 deletions(-)

-- 
2.20.1

[PATCH edk2-platforms v2 1/5] Maintainers.txt: Update documentation for new format

[Leif Lindholm]

Synchronize header with edk2/Maintainers.txt and change references
from "EDK II to EDK II Platforms".

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
 Maintainers.txt | 40 ++++++++++++++++++++++++++++++++--------
 1 file changed, 32 insertions(+), 8 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 9599024e20a6..033b4c88a680 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -1,17 +1,24 @@
-EDK II Maintainers
-==================
+EDK II Platforms Maintainers
+============================
 
 This file provides information about the primary maintainers for
 EDK II Platforms.
 
 In general, you should not privately email the maintainer. You should
-email the edk2-devel list, but you can also Cc the maintainer.
+email the edk2-devel list, and Cc the package maintainers and
+reviewers.
+
+If the package maintainer wants to hand over the role to other people,
+the package maintainer should send the patch to update Maintainers.txt
+with new maintainer, and the new maintainer should follow up with
+an Acked-by or a Reviewed-by.
 
 Descriptions of section entries:
 
   L: Mailing list that is relevant to this area (default is edk2-devel)
      Patches and questions should be sent to the email list.
-  M: Cc address for patches and questions (ie, the package maintainer)
+  M: Package Maintainer: Cc address for patches and questions. Responsible
+     for reviewing and pushing package changes to source control.
   R: Package Reviewer: Cc address for patches and questions. Reviewers help
      maintainers review code, but don't have push access. A designated Package
      Reviewer is reasonably familiar with the Package (or some modules
@@ -29,9 +36,26 @@ Descriptions of section entries:
      Obsolete:   Old code. Something tagged obsolete generally means
                  it has been replaced by a better system and you
                  should be using that.
+  F: Files and directories with wildcard patterns.
+     A trailing slash includes all files and subdirectory files.
+     F:   MdeModulePkg/   all files in and below MdeModulePkg
+     F:   MdeModulePkg/*  all files in MdeModulePkg, but not below
+     F:   */Pci/*         all files in a directory called Pci, at any depth in
+                          the hierarchy, but not below
+     One pattern per line.  Multiple F: lines per section acceptable.
+  X: Files and directories that are NOT maintained, same rules as F:
+     Files exclusions are tested after file matches.
+     Can be useful for excluding a specific subdirectory, for instance:
+     F:   NetworkPkg/
+     X:   NetworkPkg/Ip6Dxe/
+     matches all files in and below NetworkPkg excluding NetworkPkg/Ip6Dxe/
+  Filenames not caught by any F: rule get matched as being located in the top-
+  level directory. (Internally, the script looks for a match called '<default>',
+  so please don't add a file called that in the top-level directory.)
 
-EDK II
-------
+
+EDK II Platforms
+----------------
 W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II
 L: https://edk2.groups.io/g/devel/
 T: git - https://github.com/tianocore/edk2-platforms.git
@@ -40,8 +64,8 @@ Responsible Disclosure, Reporting Security Issues
 -------------------------------------------------
 W: https://github.com/tianocore/tianocore.github.io/wiki/Security
 
-EDK II Packages:
-----------------
+EDK II Platforms Packages:
+--------------------------
 
 Drivers/OptionRomPkg
 W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
-- 
2.20.1

[PATCH edk2-platforms v2 2/5] Maintainers.txt: revamp file to machine-readable format

[Leif Lindholm]

Convert existing package maintainer/reviewer descriptions
with adding the new path definitions.

Delete the "Platform" and "Silicon" section maintainership
descriptions. The EDK II Platforms maintainers section will
now match that.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Maintainers.txt | 52 +++++++++++++++++++++++++++++++++++++------------
 1 file changed, 40 insertions(+), 12 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 033b4c88a680..17613936c37e 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -60,6 +60,18 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II
 L: https://edk2.groups.io/g/devel/
 T: git - https://github.com/tianocore/edk2-platforms.git
 
+All patches CC:d here
+L: devel@edk2.groups.io
+F: *
+F: */
+
+EDK II Platforms maintainers
+----------------------------
+F: *
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+M: Michael D Kinney <michael.d.kinney@intel.com>
+
 Responsible Disclosure, Reporting Security Issues
 -------------------------------------------------
 W: https://github.com/tianocore/tianocore.github.io/wiki/Security
@@ -67,116 +79,132 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/Security
 EDK II Platforms Packages:
 --------------------------
 
-Drivers/OptionRomPkg
+OptionRomPkg
+F: Drivers/OptionRomPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
 M: Ray Ni <ray.ni@intel.com>
 
-Drivers/DisplayLink
+DisplayLink
+F: Drivers/DisplayLink/
 M: Leif Lindholm <leif.lindholm@linaro.org>
 M: Ard Bieshuevel <ard.bieshuevel@linaro.org>
 R: Andy Hayes <andy.hayes@displaylink.com>
 
-Platform
-M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
-M: Leif Lindholm <leif.lindholm@linaro.org>
-M: Michael D Kinney <michael.d.kinney@intel.com>
-
 Platform/Intel/QuarkPlatformPkg
+F: Platform/Intel/QuarkPlatformPkg/
 M: Michael D Kinney <michael.d.kinney@intel.com>
 M: Kelly Steele <kelly.steele@intel.com>
 
 Platform/Intel/Vlv2TbltDevicePkg
+F: Platform/Intel/Vlv2TbltDevicePkg/
 M: Zailiang Sun <zailiang.sun@intel.com>
 M: Yi Qian <yi.qian@intel.com>
 
 Platform/Intel/AdvancedFeaturePkg
+F: Platform/Intel/AdvancedFeaturePkg/
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Sai Chaganty <rangasai.v.chaganty@intel.com>
 R: Liming Gao <liming.gao@intel.com>
 
 Platform/Intel/BoardModulePkg
+F: Platform/Intel/BoardModulePkg/
 M: Eric Dong <eric.dong@intel.com>
 R: Liming Gao <liming.gao@intel.com>
 
 Platform/Intel/DebugFeaturePkg
+F: Platform/Intel/DebugFeaturePkg/
 M: Eric Dong <eric.dong@intel.com>
 R: Liming Gao <liming.gao@intel.com>
 
 Platform/Intel/UserInterfaceFeaturePkg
+F: Platform/Intel/UserInterfaceFeaturePkg/
 M: Dandan Bi <dandan.bi@intel.com>
 R: Liming Gao <liming.gao@intel.com>
 
 Platform/Intel/KabylakeOpenBoardPkg
+F: Platform/Intel/KabylakeOpenBoardPkg/
 M: Chasel Chiu <chasel.chiu@intel.com>
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Nate DeSimone <nathaniel.l.desimone@intel.com>
 
 Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3
+F: Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/
 R: Jeremy Soller <jeremy@system76.com>
 
 Platform/Intel/MinPlatformPkg
+F: Platform/Intel/MinPlatformPkg/
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Chasel Chiu <chasel.chiu@intel.com>
 M: Nate DeSimone <nathaniel.l.desimone@intel.com>
 R: Liming Gao <liming.gao@intel.com>
 
 Platform/Intel/WhiskeylakeOpenBoardPkg
+F: Platform/Intel/WhiskeylakeOpenBoardPkg/
 M: Chasel Chiu <chasel.chiu@intel.com>
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Nate DeSimone <nathaniel.l.desimone@intel.com>
 
 Platform/Intel/SimicsOpenBoardPkg
+F: Platform/Intel/SimicsOpenBoardPkg/
 M: Wei David Y <david.y.wei@intel.com>
 M: Agyeman Prince <prince.agyeman@intel.com>
 
 Platform/Intel/Tools
+F: Platform/Intel/Tools/
 M: Bob Feng <bob.c.feng@intel.com>
 M: Liming Gao <liming.gao@intel.com>
 
 Platform/Marvell
+F; Platform/Marvell/
 R: Marcin Wojtas <mw@semihalf.com>
+M: Leif Lindholm <leif.lindholm@linaro.org>
 
 Platform/SolidRun/Armada80x0McBin
+F: Platform/SolidRun/Armada80x0McBin/
 R: Marcin Wojtas <mw@semihalf.com>
 
-Silicon
-M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
-M: Leif Lindholm <leif.lindholm@linaro.org>
-M: Michael D Kinney <michael.d.kinney@intel.com>
-
 Silicon/Intel/IntelSiliconPkg
+F: Silicon/Intel/IntelSiliconPkg/
 M: Ray Ni <ray.ni@intel.com>
 M: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
 
 Silicon/Intel/QuarkSocPkg
+F: Silicon/Intel/QuarkSocPkg/
 M: Michael D Kinney <michael.d.kinney@intel.com>
 M: Kelly Steele <kelly.steele@intel.com>
 
 Silicon/Intel/Vlv2DeviceRefCodePkg
+F: Silicon/Intel/Vlv2DeviceRefCodePkg/
 M: Zailiang Sun <zailiang.sun@intel.com>
 M: Yi Qian <yi.qian@intel.com>
 
 Silicon/Intel/CoffeelakeSiliconPkg
+F: Silicon/Intel/CoffeelakeSiliconPkg/
 M: Chasel Chiu <chasel.chiu@intel.com>
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Sai Chaganty <rangasai.v.chaganty@intel.com>
 
 Silicon/Intel/KabylakeSiliconPkg
+F: Silicon/Intel/KabylakeSiliconPkg/
 M: Chasel Chiu <chasel.chiu@intel.com>
 M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Sai Chaganty <rangasai.v.chaganty@intel.com>
 
 Silicon/Intel/SimicsX58SktPkg
+F: Silicon/Intel/SimicsX58SktPkg/
 M: Wei David Y <david.y.wei@intel.com>
 M: Agyeman Prince <prince.agyeman@intel.com>
 
 Silicon/Intel/SimicsIch10Pkg
+F: Silicon/Intel/SimicsIch10Pkg/
 M: Wei David Y <david.y.wei@intel.com>
 M: Agyeman Prince <prince.agyeman@intel.com>
 
 Silicon/Intel/Tools
+F: Silicon/Intel/Tools/
 M: Bob Feng <bob.c.feng@intel.com>
 M: Liming Gao <liming.gao@intel.com>
 
 Silicon/Marvell
+F: Silicon/Marvell/
 R: Marcin Wojtas <mw@semihalf.com>
-- 
2.20.1

[PATCH edk2-platforms v2 3/5] Maintainers.txt: merge Marvell platforms and silicon sections

[Leif Lindholm]

Cc: Marcin Wojtas <mw@semihalf.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Marcin Wojtas <mw@semihalf.com>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
 Maintainers.txt | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 17613936c37e..80e41a98ce11 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -154,15 +154,6 @@ F: Platform/Intel/Tools/
 M: Bob Feng <bob.c.feng@intel.com>
 M: Liming Gao <liming.gao@intel.com>
 
-Platform/Marvell
-F; Platform/Marvell/
-R: Marcin Wojtas <mw@semihalf.com>
-M: Leif Lindholm <leif.lindholm@linaro.org>
-
-Platform/SolidRun/Armada80x0McBin
-F: Platform/SolidRun/Armada80x0McBin/
-R: Marcin Wojtas <mw@semihalf.com>
-
 Silicon/Intel/IntelSiliconPkg
 F: Silicon/Intel/IntelSiliconPkg/
 M: Ray Ni <ray.ni@intel.com>
@@ -205,6 +196,9 @@ F: Silicon/Intel/Tools/
 M: Bob Feng <bob.c.feng@intel.com>
 M: Liming Gao <liming.gao@intel.com>
 
-Silicon/Marvell
+Marvell platforms and silicon
+F: Platform/Marvell/
+F: Platform/SolidRun/Armada80x0McBin/
 F: Silicon/Marvell/
 R: Marcin Wojtas <mw@semihalf.com>
+M: Leif Lindholm <leif.lindholm@linaro.org>
-- 
2.20.1

[PATCH edk2-platforms v2 4/5] Maintainers.txt: create explicit sections for remaining modules

[Leif Lindholm]

Many of the original (ARM) platforms, and drivers brought in for them,
had no explicit maintainership defined. Add entries for these, roughly
corresponding to how they have already been handled in the past.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
---
 Maintainers.txt | 58 ++++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 57 insertions(+), 1 deletion(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 80e41a98ce11..86ec3c1b172d 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -79,6 +79,35 @@ W: https://github.com/tianocore/tianocore.github.io/wiki/Security
 EDK II Platforms Packages:
 --------------------------
 
+96Boards
+F: Platform/96Boards/
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+AMD Seattle
+F: Platform/AMD/OverdriveBoard/
+F: Platform/LeMaker/CelloBoard/
+F: Platform/SoftIron/
+F: Silicon/AMD/Styx/
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+ARM
+F: Platform/ARM/
+R: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+BeagleBoard:
+F: Platform/BeagleBoard/
+F: Silicon/TexasInstruments/
+R: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+Comcast
+F: Platform/Comcast/
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
 OptionRomPkg
 F: Drivers/OptionRomPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
@@ -87,9 +116,15 @@ M: Ray Ni <ray.ni@intel.com>
 DisplayLink
 F: Drivers/DisplayLink/
 M: Leif Lindholm <leif.lindholm@linaro.org>
-M: Ard Bieshuevel <ard.bieshuevel@linaro.org>
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
 R: Andy Hayes <andy.hayes@displaylink.com>
 
+HiSilicon
+F: Platform/Hisilicon/
+F: Silicon/Hisilicon/
+M: Leif Lindholm <leif.lindholm@linaro.org>
+R: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+
 Platform/Intel/QuarkPlatformPkg
 F: Platform/Intel/QuarkPlatformPkg/
 M: Michael D Kinney <michael.d.kinney@intel.com>
@@ -202,3 +237,24 @@ F: Platform/SolidRun/Armada80x0McBin/
 F: Silicon/Marvell/
 R: Marcin Wojtas <mw@semihalf.com>
 M: Leif Lindholm <leif.lindholm@linaro.org>
+
+Miscellaneous drivers
+F: Silicon/Atmel/
+F: Silicon/NXP/
+F: Silicon/Openmoko/
+F: Silicon/Synopsys/DesignWare/
+R: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+Raspberry Pi platforms and silicon
+F: Platform/RaspberryPi/
+F: Silicon/Broadcom/
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+R: Pete Batard <pete@akeo.ie>
+
+Socionext platforms and silicon
+F: Platform/Socionext/
+F: Silicon/Socionext/
+M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
-- 
2.20.1

[PATCH edk2-platforms v2 5/5] Maintainers.txt: drop Ard from default section

[Leif Lindholm]

Ard will still be looking after specific platforms and drivers, but he
was mainly added to the "default" maintainers group to offload me until
edk2-platforms got off the ground, so drop him from that target now we
add the GetMaintainers.py.

Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
Acked-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Maintainers.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 86ec3c1b172d..cb7ae0a95d44 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -68,7 +68,6 @@ F: */
 EDK II Platforms maintainers
 ----------------------------
 F: *
-M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
 M: Leif Lindholm <leif.lindholm@linaro.org>
 M: Michael D Kinney <michael.d.kinney@intel.com>
 
-- 
2.20.1

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

[Michael D Kinney]

Series 

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On
> Behalf Of Leif Lindholm
> Sent: Wednesday, November 13, 2019 10:00 AM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Marcin
> Wojtas <mw@semihalf.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>
> Subject: [edk2-devel] [PATCH edk2-platforms v2 0/5]
> Update Maintainers.txt
> 
> This set does a few things:
> - Imports the documentation header from
> edk2/Maintainers.txt
> - Reformats some maintainer entries to get rid of ','
> characters
>   in the names.
> - Adds F: entries for all sections, and a default
> fallback section
>   for those where there is no match.
> - Merges the Marvell sections (designed to work with
> this format)
>   into a single area of responsibility.
> - Adds sections for those that now ended up having none
> other than
>   the fallback one (mostly ARM platforms).
> - Remove Ard from the default section.
> 
> I am intentionally not Cc:ing all affected maintainers,
> since this is a reformatting exercise. All roles
> actually affected are Cc:d.
> 
> Changes since v1:
> - 2/6 ("Maintainers.txt: drop ',' from maintainer
> names") dropped,
>   since commit 3adcb58818ca ("Maintainers.txt: Update
> Intel package
>   maintainers") dropped the affected sections.
> - Fixed the spelling of Ard's surname :|
> - Fixed other reported typos.
> - Added R-b/A-b.
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> 
> Leif Lindholm (5):
>   Maintainers.txt: Update documentation for new format
>   Maintainers.txt: revamp file to machine-readable
> format
>   Maintainers.txt: merge Marvell platforms and silicon
> sections
>   Maintainers.txt: create explicit sections for
> remaining modules
>   Maintainers.txt: drop Ard from default section
> 
>  Maintainers.txt | 153
> ++++++++++++++++++++++++++++++++++++++++--------
>  1 file changed, 127 insertions(+), 26 deletions(-)
> 
> --
> 2.20.1
> 
> 
> 

Re: [edk2-devel] [PATCH edk2-platforms v2 0/5] Update Maintainers.txt

[Leif Lindholm]

Thanks Mike (and other reviewers).

Pushed as cee28ab1c8da..f7271a06170b.

/
    Leif

On Wed, Nov 13, 2019 at 06:42:16PM +0000, Kinney, Michael D wrote:
> Series 
> 
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> Mike
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On
> > Behalf Of Leif Lindholm
> > Sent: Wednesday, November 13, 2019 10:00 AM
> > To: devel@edk2.groups.io
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Marcin
> > Wojtas <mw@semihalf.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Wang, Jian J
> > <jian.j.wang@intel.com>
> > Subject: [edk2-devel] [PATCH edk2-platforms v2 0/5]
> > Update Maintainers.txt
> > 
> > This set does a few things:
> > - Imports the documentation header from
> > edk2/Maintainers.txt
> > - Reformats some maintainer entries to get rid of ','
> > characters
> >   in the names.
> > - Adds F: entries for all sections, and a default
> > fallback section
> >   for those where there is no match.
> > - Merges the Marvell sections (designed to work with
> > this format)
> >   into a single area of responsibility.
> > - Adds sections for those that now ended up having none
> > other than
> >   the fallback one (mostly ARM platforms).
> > - Remove Ard from the default section.
> > 
> > I am intentionally not Cc:ing all affected maintainers,
> > since this is a reformatting exercise. All roles
> > actually affected are Cc:d.
> > 
> > Changes since v1:
> > - 2/6 ("Maintainers.txt: drop ',' from maintainer
> > names") dropped,
> >   since commit 3adcb58818ca ("Maintainers.txt: Update
> > Intel package
> >   maintainers") dropped the affected sections.
> > - Fixed the spelling of Ard's surname :|
> > - Fixed other reported typos.
> > - Added R-b/A-b.
> > 
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > 
> > Leif Lindholm (5):
> >   Maintainers.txt: Update documentation for new format
> >   Maintainers.txt: revamp file to machine-readable
> > format
> >   Maintainers.txt: merge Marvell platforms and silicon
> > sections
> >   Maintainers.txt: create explicit sections for
> > remaining modules
> >   Maintainers.txt: drop Ard from default section
> > 
> >  Maintainers.txt | 153
> > ++++++++++++++++++++++++++++++++++++++++--------
> >  1 file changed, 127 insertions(+), 26 deletions(-)
> > 
> > --
> > 2.20.1
> > 
> > 
> > 
>