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

[PATCH edk2-platforms 0/6] 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.

Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
Cc: Marcin Wojtas <mw@semihalf.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>

Leif Lindholm (6):
  Maintainers.txt: Update documentation for new format
  Maintainers.txt: drop ',' from maintainer names
  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 | 175 ++++++++++++++++++++++++++++++++++++++----------
 1 file changed, 140 insertions(+), 35 deletions(-)

--
2.20.1

[PATCH edk2-platforms 1/6] 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>
---
 Maintainers.txt | 42 +++++++++++++++++++++++++++++++++---------
 1 file changed, 33 insertions(+), 9 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 1260c8d2465b..f824e2b05de1 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.
+EDK Platforms II.
 
 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

Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format

[Philippe Mathieu-Daudé]

On 10/9/19 2:00 PM, Leif Lindholm wrote:
> 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>
> ---
>   Maintainers.txt | 42 +++++++++++++++++++++++++++++++++---------
>   1 file changed, 33 insertions(+), 9 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 1260c8d2465b..f824e2b05de1 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.
> +EDK Platforms II.

This one looks odd :)

>   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
> 

Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format

[Leif Lindholm]

On Wed, Oct 09, 2019 at 02:51:38PM +0200, Philippe Mathieu-Daudé wrote:
> On 10/9/19 2:00 PM, Leif Lindholm wrote:
> > 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>
> > ---
> >   Maintainers.txt | 42 +++++++++++++++++++++++++++++++++---------
> >   1 file changed, 33 insertions(+), 9 deletions(-)
> > 
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index 1260c8d2465b..f824e2b05de1 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.
> > +EDK Platforms II.
> 
> This one looks odd :)

I might have been getting a bit cross-eyed at this point.

Thanks!

/
    Leif

> >   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
> > 
> 
> 
> 
> 

Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format

[Philippe Mathieu-Daudé]

On 10/9/19 2:54 PM, Leif Lindholm wrote:
> On Wed, Oct 09, 2019 at 02:51:38PM +0200, Philippe Mathieu-Daudé wrote:
>> On 10/9/19 2:00 PM, Leif Lindholm wrote:
>>> 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>
>>> ---
>>>    Maintainers.txt | 42 +++++++++++++++++++++++++++++++++---------
>>>    1 file changed, 33 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/Maintainers.txt b/Maintainers.txt
>>> index 1260c8d2465b..f824e2b05de1 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.
>>> +EDK Platforms II.
>>
>> This one looks odd :)
> 
> I might have been getting a bit cross-eyed at this point.

8)

No need to respin the series IMO, so once fixed:
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

> 
> Thanks!
> 
> /
>      Leif
> 
>>>    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
>>>
>>
>>
>>
>>
> 
> 
> 

[PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names

[Leif Lindholm]

The ',' character, when used in the name part of email addresses,
must be escaped. Not doing so leads to all kinds of fun with SMTP.
So rework all M: entries containing a , to not do so.

Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Maintainers.txt | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index f824e2b05de1..1d27dde741f0 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -172,19 +172,19 @@ M: Michael Kubacki <michael.a.kubacki@intel.com>
 M: Sai Chaganty <rangasai.v.chaganty@intel.com>
 
 Silicon/Intel/LewisburgPkg
-M: Piwko, Maciej <maciej.piwko@intel.com>
-M: Bu, Daocheng <daocheng.bu@intel.com>
-M: Oram, Isaac W <isaac.w.oram@intel.com>
+M: Maciej Piwko<maciej.piwko@intel.com>
+M: Daocheng Bu <daocheng.bu@intel.com>
+M: Isaac W Oram <isaac.w.oram@intel.com>
 
 Silicon/Intel/PurleyRcPkg
-M: Gillispie, Thad <thad.gillispie@intel.com>
-M: Bu, Daocheng <daocheng.bu@intel.com>
-M: Oram, Isaac W <isaac.w.oram@intel.com>
+M: Thad Gillispie <thad.gillispie@intel.com>
+M: Daocheng Bu <daocheng.bu@intel.com>
+M: Isaac W Oram <isaac.w.oram@intel.com>
 
 Silicon/Intel/PurleySktPkg
-M: Gillispie, Thad <thad.gillispie@intel.com>
-M: Bu, Daocheng <daocheng.bu@intel.com>
-M: Oram, Isaac W <isaac.w.oram@intel.com>
+M: Thad Gillispie <thad.gillispie@intel.com>
+M: Daocheng Bu <daocheng.bu@intel.com>
+M: Isaac W Oram <isaac.w.oram@intel.com>
 
 Silicon/Intel/SimicsX58SktPkg
 M: Wei David Y <david.y.wei@intel.com>
-- 
2.20.1

Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names

[Philippe Mathieu-Daudé]

On 10/9/19 2:00 PM, Leif Lindholm wrote:
> The ',' character, when used in the name part of email addresses,
> must be escaped. Not doing so leads to all kinds of fun with SMTP.
> So rework all M: entries containing a , to not do so.

Oh, good tip!

> Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>   Maintainers.txt | 18 +++++++++---------
>   1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index f824e2b05de1..1d27dde741f0 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -172,19 +172,19 @@ M: Michael Kubacki <michael.a.kubacki@intel.com>
>   M: Sai Chaganty <rangasai.v.chaganty@intel.com>
>   
>   Silicon/Intel/LewisburgPkg
> -M: Piwko, Maciej <maciej.piwko@intel.com>
> -M: Bu, Daocheng <daocheng.bu@intel.com>
> -M: Oram, Isaac W <isaac.w.oram@intel.com>
> +M: Maciej Piwko<maciej.piwko@intel.com>

Name [space] <email> ?

> +M: Daocheng Bu <daocheng.bu@intel.com>
> +M: Isaac W Oram <isaac.w.oram@intel.com>
>   
>   Silicon/Intel/PurleyRcPkg
> -M: Gillispie, Thad <thad.gillispie@intel.com>
> -M: Bu, Daocheng <daocheng.bu@intel.com>
> -M: Oram, Isaac W <isaac.w.oram@intel.com>
> +M: Thad Gillispie <thad.gillispie@intel.com>
> +M: Daocheng Bu <daocheng.bu@intel.com>
> +M: Isaac W Oram <isaac.w.oram@intel.com>
>   
>   Silicon/Intel/PurleySktPkg
> -M: Gillispie, Thad <thad.gillispie@intel.com>
> -M: Bu, Daocheng <daocheng.bu@intel.com>
> -M: Oram, Isaac W <isaac.w.oram@intel.com>
> +M: Thad Gillispie <thad.gillispie@intel.com>
> +M: Daocheng Bu <daocheng.bu@intel.com>
> +M: Isaac W Oram <isaac.w.oram@intel.com>
>   
>   Silicon/Intel/SimicsX58SktPkg
>   M: Wei David Y <david.y.wei@intel.com>
> 

Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names

[Leif Lindholm]

On Wed, Oct 09, 2019 at 02:53:34PM +0200, Philippe Mathieu-Daudé wrote:
> On 10/9/19 2:00 PM, Leif Lindholm wrote:
> > The ',' character, when used in the name part of email addresses,
> > must be escaped. Not doing so leads to all kinds of fun with SMTP.
> > So rework all M: entries containing a , to not do so.
> 
> Oh, good tip!
> 
> > Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> > ---
> >   Maintainers.txt | 18 +++++++++---------
> >   1 file changed, 9 insertions(+), 9 deletions(-)
> > 
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index f824e2b05de1..1d27dde741f0 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -172,19 +172,19 @@ M: Michael Kubacki <michael.a.kubacki@intel.com>
> >   M: Sai Chaganty <rangasai.v.chaganty@intel.com>
> >   Silicon/Intel/LewisburgPkg
> > -M: Piwko, Maciej <maciej.piwko@intel.com>
> > -M: Bu, Daocheng <daocheng.bu@intel.com>
> > -M: Oram, Isaac W <isaac.w.oram@intel.com>
> > +M: Maciej Piwko<maciej.piwko@intel.com>
> 
> Name [space] <email> ?

Huh, yeah, as previously mentioned...
OK to fold in?

> > +M: Daocheng Bu <daocheng.bu@intel.com>
> > +M: Isaac W Oram <isaac.w.oram@intel.com>
> >   Silicon/Intel/PurleyRcPkg
> > -M: Gillispie, Thad <thad.gillispie@intel.com>
> > -M: Bu, Daocheng <daocheng.bu@intel.com>
> > -M: Oram, Isaac W <isaac.w.oram@intel.com>
> > +M: Thad Gillispie <thad.gillispie@intel.com>
> > +M: Daocheng Bu <daocheng.bu@intel.com>
> > +M: Isaac W Oram <isaac.w.oram@intel.com>
> >   Silicon/Intel/PurleySktPkg
> > -M: Gillispie, Thad <thad.gillispie@intel.com>
> > -M: Bu, Daocheng <daocheng.bu@intel.com>
> > -M: Oram, Isaac W <isaac.w.oram@intel.com>
> > +M: Thad Gillispie <thad.gillispie@intel.com>
> > +M: Daocheng Bu <daocheng.bu@intel.com>
> > +M: Isaac W Oram <isaac.w.oram@intel.com>
> >   Silicon/Intel/SimicsX58SktPkg
> >   M: Wei David Y <david.y.wei@intel.com>
> > 
> 

Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names

[Philippe Mathieu-Daudé]

On 10/9/19 3:16 PM, Leif Lindholm wrote:
> On Wed, Oct 09, 2019 at 02:53:34PM +0200, Philippe Mathieu-Daudé wrote:
>> On 10/9/19 2:00 PM, Leif Lindholm wrote:
>>> The ',' character, when used in the name part of email addresses,
>>> must be escaped. Not doing so leads to all kinds of fun with SMTP.
>>> So rework all M: entries containing a , to not do so.
>>
>> Oh, good tip!
>>
>>> Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
>>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>>> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
>>> ---
>>>    Maintainers.txt | 18 +++++++++---------
>>>    1 file changed, 9 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/Maintainers.txt b/Maintainers.txt
>>> index f824e2b05de1..1d27dde741f0 100644
>>> --- a/Maintainers.txt
>>> +++ b/Maintainers.txt
>>> @@ -172,19 +172,19 @@ M: Michael Kubacki <michael.a.kubacki@intel.com>
>>>    M: Sai Chaganty <rangasai.v.chaganty@intel.com>
>>>    Silicon/Intel/LewisburgPkg
>>> -M: Piwko, Maciej <maciej.piwko@intel.com>
>>> -M: Bu, Daocheng <daocheng.bu@intel.com>
>>> -M: Oram, Isaac W <isaac.w.oram@intel.com>
>>> +M: Maciej Piwko<maciej.piwko@intel.com>
>>
>> Name [space] <email> ?
> 
> Huh, yeah, as previously mentioned...
> OK to fold in?

Sure:
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

>>> +M: Daocheng Bu <daocheng.bu@intel.com>
>>> +M: Isaac W Oram <isaac.w.oram@intel.com>
>>>    Silicon/Intel/PurleyRcPkg
>>> -M: Gillispie, Thad <thad.gillispie@intel.com>
>>> -M: Bu, Daocheng <daocheng.bu@intel.com>
>>> -M: Oram, Isaac W <isaac.w.oram@intel.com>
>>> +M: Thad Gillispie <thad.gillispie@intel.com>
>>> +M: Daocheng Bu <daocheng.bu@intel.com>
>>> +M: Isaac W Oram <isaac.w.oram@intel.com>
>>>    Silicon/Intel/PurleySktPkg
>>> -M: Gillispie, Thad <thad.gillispie@intel.com>
>>> -M: Bu, Daocheng <daocheng.bu@intel.com>
>>> -M: Oram, Isaac W <isaac.w.oram@intel.com>
>>> +M: Thad Gillispie <thad.gillispie@intel.com>
>>> +M: Daocheng Bu <daocheng.bu@intel.com>
>>> +M: Isaac W Oram <isaac.w.oram@intel.com>
>>>    Silicon/Intel/SimicsX58SktPkg
>>>    M: Wei David Y <david.y.wei@intel.com>
>>>
>>

[PATCH edk2-platforms 3/6] 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>
---
 Maintainers.txt | 56 ++++++++++++++++++++++++++++++++++++++-----------
 1 file changed, 44 insertions(+), 12 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 1d27dde741f0..43803592d369 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 Bieshuevel <ard.bieshuevel@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,136 +79,156 @@ 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/PurleyOpenBoardPkg
+F: Platform/Intel/PurleyOpenBoardPkg/
 M: Shifei A Lu <shifei.a.lu@intel.com>
 M: Xiaohu Zhou <bowen.zhou@intel.com>
 M: Isaac W Oram <isaac.w.oram@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/LewisburgPkg
+F: Silicon/Intel/LewisburgPkg/
 M: Maciej Piwko<maciej.piwko@intel.com>
 M: Daocheng Bu <daocheng.bu@intel.com>
 M: Isaac W Oram <isaac.w.oram@intel.com>
 
 Silicon/Intel/PurleyRcPkg
+F: Silicon/Intel/PurleyRcPkg/
 M: Thad Gillispie <thad.gillispie@intel.com>
 M: Daocheng Bu <daocheng.bu@intel.com>
 M: Isaac W Oram <isaac.w.oram@intel.com>
 
 Silicon/Intel/PurleySktPkg
+F: Silicon/Intel/PurleySktPkg/
 M: Thad Gillispie <thad.gillispie@intel.com>
 M: Daocheng Bu <daocheng.bu@intel.com>
 M: Isaac W Oram <isaac.w.oram@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

Re: [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format

[Ard Biesheuvel]

On Wed, 9 Oct 2019 at 14:00, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>
> 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>
> ---
>  Maintainers.txt | 56 ++++++++++++++++++++++++++++++++++++++-----------
>  1 file changed, 44 insertions(+), 12 deletions(-)
>
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 1d27dde741f0..43803592d369 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 Bieshuevel <ard.bieshuevel@linaro.org>

Typo here ^^

Not a big deal since you are removing this line again in the last
patch, but I thought I'd mention it nonetheless


> +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,136 +79,156 @@ 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/PurleyOpenBoardPkg
> +F: Platform/Intel/PurleyOpenBoardPkg/
>  M: Shifei A Lu <shifei.a.lu@intel.com>
>  M: Xiaohu Zhou <bowen.zhou@intel.com>
>  M: Isaac W Oram <isaac.w.oram@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/LewisburgPkg
> +F: Silicon/Intel/LewisburgPkg/
>  M: Maciej Piwko<maciej.piwko@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@intel.com>
>
>  Silicon/Intel/PurleyRcPkg
> +F: Silicon/Intel/PurleyRcPkg/
>  M: Thad Gillispie <thad.gillispie@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@intel.com>
>
>  Silicon/Intel/PurleySktPkg
> +F: Silicon/Intel/PurleySktPkg/
>  M: Thad Gillispie <thad.gillispie@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@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
>

Re: [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format

[Leif Lindholm]

On Wed, Oct 09, 2019 at 02:31:21PM +0200, Ard Biesheuvel wrote:
> On Wed, 9 Oct 2019 at 14:00, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> >
> > 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>
> > ---
> >  Maintainers.txt | 56 ++++++++++++++++++++++++++++++++++++++-----------
> >  1 file changed, 44 insertions(+), 12 deletions(-)
> >
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index 1d27dde741f0..43803592d369 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 Bieshuevel <ard.bieshuevel@linaro.org>
> 
> Typo here ^^
> 
> Not a big deal since you are removing this line again in the last
> patch, but I thought I'd mention it nonetheless

Yeah, I'll fix that before pushing, or a v2.

Thanks!

/
    Leif

> > +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,136 +79,156 @@ 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/PurleyOpenBoardPkg
> > +F: Platform/Intel/PurleyOpenBoardPkg/
> >  M: Shifei A Lu <shifei.a.lu@intel.com>
> >  M: Xiaohu Zhou <bowen.zhou@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@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/LewisburgPkg
> > +F: Silicon/Intel/LewisburgPkg/
> >  M: Maciej Piwko<maciej.piwko@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@intel.com>
> >
> >  Silicon/Intel/PurleyRcPkg
> > +F: Silicon/Intel/PurleyRcPkg/
> >  M: Thad Gillispie <thad.gillispie@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@intel.com>
> >
> >  Silicon/Intel/PurleySktPkg
> > +F: Silicon/Intel/PurleySktPkg/
> >  M: Thad Gillispie <thad.gillispie@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@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
> >

Re: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format

[Liming Gao]

Leif:

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif Lindholm
> Sent: Wednesday, October 9, 2019 8:00 PM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
> 
> 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>
> ---
>  Maintainers.txt | 56 ++++++++++++++++++++++++++++++++++++++-----------
>  1 file changed, 44 insertions(+), 12 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 1d27dde741f0..43803592d369 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 Bieshuevel <ard.bieshuevel@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,136 +79,156 @@ 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/PurleyOpenBoardPkg
> +F: Platform/Intel/PurleyOpenBoardPkg/
>  M: Shifei A Lu <shifei.a.lu@intel.com>
>  M: Xiaohu Zhou <bowen.zhou@intel.com>
>  M: Isaac W Oram <isaac.w.oram@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/
This is a typo. It should be F: Platform/Marvell/. Right?

Thanks
Liming
>  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/LewisburgPkg
> +F: Silicon/Intel/LewisburgPkg/
>  M: Maciej Piwko<maciej.piwko@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@intel.com>
> 
>  Silicon/Intel/PurleyRcPkg
> +F: Silicon/Intel/PurleyRcPkg/
>  M: Thad Gillispie <thad.gillispie@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@intel.com>
> 
>  Silicon/Intel/PurleySktPkg
> +F: Silicon/Intel/PurleySktPkg/
>  M: Thad Gillispie <thad.gillispie@intel.com>
>  M: Daocheng Bu <daocheng.bu@intel.com>
>  M: Isaac W Oram <isaac.w.oram@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
> 
> 
> 

Re: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format

[Leif Lindholm]

On Wed, Oct 09, 2019 at 01:48:57PM +0000, Gao, Liming wrote:
> Leif:
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Leif Lindholm
> > Sent: Wednesday, October 9, 2019 8:00 PM
> > To: devel@edk2.groups.io
> > Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>; Kinney, Michael D <michael.d.kinney@intel.com>
> > Subject: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
> > 
> > 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>
> > ---
> >  Maintainers.txt | 56 ++++++++++++++++++++++++++++++++++++++-----------
> >  1 file changed, 44 insertions(+), 12 deletions(-)
> > 
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index 1d27dde741f0..43803592d369 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 Bieshuevel <ard.bieshuevel@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,136 +79,156 @@ 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/PurleyOpenBoardPkg
> > +F: Platform/Intel/PurleyOpenBoardPkg/
> >  M: Shifei A Lu <shifei.a.lu@intel.com>
> >  M: Xiaohu Zhou <bowen.zhou@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@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/
> This is a typo. It should be F: Platform/Marvell/. Right?

Yes. Thanks!

/
    Leif

> Thanks
> Liming
> >  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/LewisburgPkg
> > +F: Silicon/Intel/LewisburgPkg/
> >  M: Maciej Piwko<maciej.piwko@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@intel.com>
> > 
> >  Silicon/Intel/PurleyRcPkg
> > +F: Silicon/Intel/PurleyRcPkg/
> >  M: Thad Gillispie <thad.gillispie@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@intel.com>
> > 
> >  Silicon/Intel/PurleySktPkg
> > +F: Silicon/Intel/PurleySktPkg/
> >  M: Thad Gillispie <thad.gillispie@intel.com>
> >  M: Daocheng Bu <daocheng.bu@intel.com>
> >  M: Isaac W Oram <isaac.w.oram@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 4/6] 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>
---
 Maintainers.txt | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 43803592d369..56b12dd79c3f 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -160,15 +160,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>
@@ -229,6 +220,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

Re: [edk2-devel] [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections

[Philippe Mathieu-Daudé]

On 10/9/19 2:00 PM, Leif Lindholm wrote:
> Cc: Marcin Wojtas <mw@semihalf.com>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>   Maintainers.txt | 14 ++++----------
>   1 file changed, 4 insertions(+), 10 deletions(-)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 43803592d369..56b12dd79c3f 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -160,15 +160,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>
> @@ -229,6 +220,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>
> 

Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

Re: [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections

[Marcin Wojtas]

Hi Leif,

śr., 9 paź 2019 o 14:00 Leif Lindholm <leif.lindholm@linaro.org> napisał(a):
>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>  Maintainers.txt | 14 ++++----------
>  1 file changed, 4 insertions(+), 10 deletions(-)
>
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 43803592d369..56b12dd79c3f 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -160,15 +160,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>
> @@ -229,6 +220,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
>

Ok for me, so:
Reviewed-by: Marcin Wojtas <mw@semihalf.com>

Thanks,
Marcin

[PATCH edk2-platforms 5/6] 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 Bieshuevel <ard.bieshuevel@linaro.org>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Maintainers.txt | 56 +++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 56 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 56b12dd79c3f..ebb878e22ac4 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 Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+ARM
+F: Platform/ARM/
+R: Ard Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+BeagleBoard:
+F: Platform/BeagleBoard/
+F: Silicon/TexasInstruments/
+R: Ard Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+Comcast
+F: Platform/Comcast/
+M: Ard Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
 OptionRomPkg
 F: Drivers/OptionRomPkg/
 W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
@@ -90,6 +119,12 @@ M: Leif Lindholm <leif.lindholm@linaro.org>
 M: Ard Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
+
 Platform/Intel/QuarkPlatformPkg
 F: Platform/Intel/QuarkPlatformPkg/
 M: Michael D Kinney <michael.d.kinney@intel.com>
@@ -226,3 +261,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 Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
+
+Raspberry Pi platforms and silicon
+F: Platform/RaspberryPi/
+F: Silicon/Broadcom/
+M: Ard Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
+M: Leif Lindholm <leif.lindholm@linaro.org>
-- 
2.20.1

Re: [edk2-devel] [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules

[Philippe Mathieu-Daudé]

On 10/9/19 2:00 PM, Leif Lindholm wrote:
> 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 Bieshuevel <ard.bieshuevel@linaro.org>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>   Maintainers.txt | 56 +++++++++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 56 insertions(+)
> 
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 56b12dd79c3f..ebb878e22ac4 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 Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> +
> +ARM
> +F: Platform/ARM/
> +R: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> +
> +BeagleBoard:
> +F: Platform/BeagleBoard/
> +F: Silicon/TexasInstruments/
> +R: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> +
> +Comcast
> +F: Platform/Comcast/
> +M: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> +
>   OptionRomPkg
>   F: Drivers/OptionRomPkg/
>   W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
> @@ -90,6 +119,12 @@ M: Leif Lindholm <leif.lindholm@linaro.org>
>   M: Ard Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
> +
>   Platform/Intel/QuarkPlatformPkg
>   F: Platform/Intel/QuarkPlatformPkg/
>   M: Michael D Kinney <michael.d.kinney@intel.com>
> @@ -226,3 +261,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 Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> +
> +Raspberry Pi platforms and silicon
> +F: Platform/RaspberryPi/
> +F: Silicon/Broadcom/
> +M: Ard Bieshuevel <ard.bieshuevel@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 Bieshuevel <ard.bieshuevel@linaro.org>
> +M: Leif Lindholm <leif.lindholm@linaro.org>
> 

FWIW:
Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>

[PATCH edk2-platforms 6/6] 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>
---
 Maintainers.txt | 1 -
 1 file changed, 1 deletion(-)

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

Re: [PATCH edk2-platforms 0/6] Update Maintainers.txt

[Ard Biesheuvel]

On Wed, 9 Oct 2019 at 14:00, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>
> 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.
>
> Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>

Typo here ^^^

... but I'm sure the Google mail robot has informed you by now? :-)

In any case, for the series (modulo the typoes)

Acked-by: Ard Bieshulevl <ard.lehvieshbe@linaro.org>


> Cc: Marcin Wojtas <mw@semihalf.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>
> Leif Lindholm (6):
>   Maintainers.txt: Update documentation for new format
>   Maintainers.txt: drop ',' from maintainer names
>   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 | 175 ++++++++++++++++++++++++++++++++++++++----------
>  1 file changed, 140 insertions(+), 35 deletions(-)
>
> --
> 2.20.1
>

Re: [PATCH edk2-platforms 0/6] Update Maintainers.txt

[Leif Lindholm]

On Wed, Oct 09, 2019 at 02:32:37PM +0200, Ard Biesheuvel wrote:
> On Wed, 9 Oct 2019 at 14:00, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> >
> > 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.
> >
> > Cc: Ard Bieshuevel <ard.bieshuevel@linaro.org>
> 
> Typo here ^^^
> 
> ... but I'm sure the Google mail robot has informed you by now? :-)

Many times...
Funny thing is, I didn't type out a single address - I copy-pasted it
from the GetMaintainers.py output (from another window where I was
going through the whitespace fixups, using the script to get at it).

Turns out your name/address was always wrong in
https://github.com/tianocore/edk2-platforms/blob/master/Maintainers.txt#L52
:)

Gosh hecking darnit!

> In any case, for the series (modulo the typoes)
> 
> Acked-by: Ard Bieshulevl <ard.lehvieshbe@linaro.org>

Don't tempt me.

(Thanks!)

> 
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> >
> > Leif Lindholm (6):
> >   Maintainers.txt: Update documentation for new format
> >   Maintainers.txt: drop ',' from maintainer names
> >   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 | 175 ++++++++++++++++++++++++++++++++++++++----------
> >  1 file changed, 140 insertions(+), 35 deletions(-)
> >
> > --
> > 2.20.1
> >