[PATCH edk2-platforms 1/3] Maintainers.txt: add Reviewer role to description of section entries

[PATCH edk2-platforms 1/3] Maintainers.txt: add Reviewer role to description of section entries

[Leif Lindholm]

This file was based on an old version of the edk2 Maintainers.txt,
predating both d75b8ac278b and c9e1e28b619. Bring this up to date.

Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Maintainers.txt | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 76feecceb7..85e8dc7fd4 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -12,6 +12,11 @@ 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)
+  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
+     thereof), and/or provides testing or regression testing for the Package
+     (or some modules thereof), in certain platforms and environments.
   W: Web-page with status/info
   T: SCM tree type and location.  Type is one of: git, svn.
   S: Status, one of the following:
-- 
2.11.0

[PATCH edk2-platforms 2/3] Maintainers.txt: update mailing list information

[Leif Lindholm]

The devel mailing list has moved from 01.org to groups.io - update
the mailing list information to reflect this.

Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Maintainers.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Maintainers.txt b/Maintainers.txt
index 85e8dc7fd4..778eec6fed 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -33,7 +33,7 @@ Descriptions of section entries:
 EDK II
 ------
 W: https://github.com/tianocore/tianocore.github.io/wiki/EDK-II
-L: https://lists.01.org/mailman/listinfo/edk2-devel
+L: https://edk2.groups.io/g/devel/
 T: git - https://github.com/tianocore/edk2-platforms.git
 
 Responsible Disclosure, Reporting Security Issues
-- 
2.11.0

[PATCH edk2-platforms 3/3] Maintainers.txt: add reviewer for Marvell platforms

[Leif Lindholm]

Marcin is the author of the majority of the Marvell platform support in
the tree, so clarify that he should be cc:d on patches to Marvell Armada
SoC/Boards and MacchiatoBIN.

Cc: Marcin Wojtas <mw@semihalf.com>
Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 Maintainers.txt | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/Maintainers.txt b/Maintainers.txt
index 778eec6fed..54c98e3b22 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -86,6 +86,12 @@ 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/Marvell
+R: Marcin Wojtas <mw@semihalf.com>
+
+Platform/SolidRun/Armada80x0McBin
+R: Marcin Wojtas <mw@semihalf.com>
+
 Silicon
 M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
 M: Leif Lindholm <leif.lindholm@linaro.org>
@@ -118,3 +124,6 @@ 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>
+
+Silicon/Marvell
+R: Marcin Wojtas <mw@semihalf.com>
-- 
2.11.0

Re: [PATCH edk2-platforms 3/3] Maintainers.txt: add reviewer for Marvell platforms

[Marcin Wojtas]

Hi Leif,

wt., 28 maj 2019 o 12:20 Leif Lindholm <leif.lindholm@linaro.org> napisał(a):
>
> Marcin is the author of the majority of the Marvell platform support in
> the tree, so clarify that he should be cc:d on patches to Marvell Armada
> SoC/Boards and MacchiatoBIN.
>
> Cc: Marcin Wojtas <mw@semihalf.com>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>  Maintainers.txt | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/Maintainers.txt b/Maintainers.txt
> index 778eec6fed..54c98e3b22 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -86,6 +86,12 @@ 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/Marvell
> +R: Marcin Wojtas <mw@semihalf.com>
> +
> +Platform/SolidRun/Armada80x0McBin
> +R: Marcin Wojtas <mw@semihalf.com>
> +
>  Silicon
>  M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>  M: Leif Lindholm <leif.lindholm@linaro.org>
> @@ -118,3 +124,6 @@ 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>
> +
> +Silicon/Marvell
> +R: Marcin Wojtas <mw@semihalf.com>
> --

Thank you for the patch, I will be happy to get it merged.

Best regards,
Marcin

Re: [PATCH edk2-platforms 3/3] Maintainers.txt: add reviewer for Marvell platforms

[Leif Lindholm]

[-- Attachment #1: Type: text/plain, Size: 1567 bytes --]

On Tue, 28 May 2019 at 11:24, Marcin Wojtas <mw@semihalf.com> wrote:
>
> Hi Leif,
>
> wt., 28 maj 2019 o 12:20 Leif Lindholm <leif.lindholm@linaro.org>
napisał(a):
> >
> > Marcin is the author of the majority of the Marvell platform support in
> > the tree, so clarify that he should be cc:d on patches to Marvell Armada
> > SoC/Boards and MacchiatoBIN.
> >
> > Cc: Marcin Wojtas <mw@semihalf.com>
> > Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> > ---
> >  Maintainers.txt | 9 +++++++++
> >  1 file changed, 9 insertions(+)
> >
> > diff --git a/Maintainers.txt b/Maintainers.txt
> > index 778eec6fed..54c98e3b22 100644
> > --- a/Maintainers.txt
> > +++ b/Maintainers.txt
> > @@ -86,6 +86,12 @@ 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/Marvell
> > +R: Marcin Wojtas <mw@semihalf.com>
> > +
> > +Platform/SolidRun/Armada80x0McBin
> > +R: Marcin Wojtas <mw@semihalf.com>
> > +
> >  Silicon
> >  M: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >  M: Leif Lindholm <leif.lindholm@linaro.org>
> > @@ -118,3 +124,6 @@ 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>
> > +
> > +Silicon/Marvell
> > +R: Marcin Wojtas <mw@semihalf.com>
> > --
>
> Thank you for the patch, I will be happy to get it merged.

Thanks! And thanks Ard.

Series pushed as 22d5f49913..c587c76bec

/
    Leif

[-- Attachment #2: Type: text/html, Size: 2698 bytes --]

Re: [PATCH edk2-platforms 1/3] Maintainers.txt: add Reviewer role to description of section entries

[Ard Biesheuvel]

On Tue, 28 May 2019 at 12:20, Leif Lindholm <leif.lindholm@linaro.org> wrote:
>
> This file was based on an old version of the edk2 Maintainers.txt,
> predating both d75b8ac278b and c9e1e28b619. Bring this up to date.
>
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>  Maintainers.txt | 5 +++++
>  1 file changed, 5 insertions(+)
>

(for the series)

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

> diff --git a/Maintainers.txt b/Maintainers.txt
> index 76feecceb7..85e8dc7fd4 100644
> --- a/Maintainers.txt
> +++ b/Maintainers.txt
> @@ -12,6 +12,11 @@ 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)
> +  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
> +     thereof), and/or provides testing or regression testing for the Package
> +     (or some modules thereof), in certain platforms and environments.
>    W: Web-page with status/info
>    T: SCM tree type and location.  Type is one of: git, svn.
>    S: Status, one of the following:
> --
> 2.11.0
>