* [PATCH edk2-platforms 0/6] Update Maintainers.txt
@ 2019-10-09 12:00 Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format Leif Lindholm
` (6 more replies)
0 siblings, 7 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel, Marcin Wojtas
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
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:51 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names Leif Lindholm
` (5 subsequent siblings)
6 siblings, 1 reply; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:53 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format Leif Lindholm
` (4 subsequent siblings)
6 siblings, 1 reply; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:31 ` Ard Biesheuvel
2019-10-09 13:48 ` [edk2-devel] " Liming Gao
2019-10-09 12:00 ` [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections Leif Lindholm
` (3 subsequent siblings)
6 siblings, 2 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
` (2 preceding siblings ...)
2019-10-09 12:00 ` [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:56 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 13:21 ` Marcin Wojtas
2019-10-09 12:00 ` [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules Leif Lindholm
` (2 subsequent siblings)
6 siblings, 2 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney, Marcin Wojtas
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
` (3 preceding siblings ...)
2019-10-09 12:00 ` [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:57 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 6/6] Maintainers.txt: drop Ard from default section Leif Lindholm
2019-10-09 12:32 ` [PATCH edk2-platforms 0/6] Update Maintainers.txt Ard Biesheuvel
6 siblings, 1 reply; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH edk2-platforms 6/6] Maintainers.txt: drop Ard from default section
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
` (4 preceding siblings ...)
2019-10-09 12:00 ` [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules Leif Lindholm
@ 2019-10-09 12:00 ` Leif Lindholm
2019-10-09 12:32 ` [PATCH edk2-platforms 0/6] Update Maintainers.txt Ard Biesheuvel
6 siblings, 0 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:00 UTC (permalink / raw)
To: devel; +Cc: Ard Biesheuvel, Michael D Kinney
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
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
2019-10-09 12:00 ` [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format Leif Lindholm
@ 2019-10-09 12:31 ` Ard Biesheuvel
2019-10-09 12:47 ` Leif Lindholm
2019-10-09 13:48 ` [edk2-devel] " Liming Gao
1 sibling, 1 reply; 22+ messages in thread
From: Ard Biesheuvel @ 2019-10-09 12:31 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel-groups-io, Michael D Kinney
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
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH edk2-platforms 0/6] Update Maintainers.txt
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
` (5 preceding siblings ...)
2019-10-09 12:00 ` [PATCH edk2-platforms 6/6] Maintainers.txt: drop Ard from default section Leif Lindholm
@ 2019-10-09 12:32 ` Ard Biesheuvel
2019-10-09 12:46 ` Leif Lindholm
6 siblings, 1 reply; 22+ messages in thread
From: Ard Biesheuvel @ 2019-10-09 12:32 UTC (permalink / raw)
To: Leif Lindholm
Cc: edk2-devel-groups-io, Michael D Kinney, Ard Bieshuevel,
Marcin Wojtas
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
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH edk2-platforms 0/6] Update Maintainers.txt
2019-10-09 12:32 ` [PATCH edk2-platforms 0/6] Update Maintainers.txt Ard Biesheuvel
@ 2019-10-09 12:46 ` Leif Lindholm
0 siblings, 0 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:46 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel-groups-io, Michael D Kinney
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
> >
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
2019-10-09 12:31 ` Ard Biesheuvel
@ 2019-10-09 12:47 ` Leif Lindholm
0 siblings, 0 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:47 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel-groups-io, Michael D Kinney
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
> >
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format
2019-10-09 12:00 ` [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format Leif Lindholm
@ 2019-10-09 12:51 ` Philippe Mathieu-Daudé
2019-10-09 12:54 ` Leif Lindholm
0 siblings, 1 reply; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 12:51 UTC (permalink / raw)
To: devel, leif.lindholm; +Cc: Ard Biesheuvel, Michael D Kinney
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
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names
2019-10-09 12:00 ` [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names Leif Lindholm
@ 2019-10-09 12:53 ` Philippe Mathieu-Daudé
2019-10-09 13:16 ` Leif Lindholm
0 siblings, 1 reply; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 12:53 UTC (permalink / raw)
To: devel, leif.lindholm; +Cc: Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format
2019-10-09 12:51 ` [edk2-devel] " Philippe Mathieu-Daudé
@ 2019-10-09 12:54 ` Leif Lindholm
2019-10-09 12:59 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 12:54 UTC (permalink / raw)
To: devel, philmd; +Cc: Ard Biesheuvel, Michael D Kinney
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
> >
>
>
>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections
2019-10-09 12:00 ` [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections Leif Lindholm
@ 2019-10-09 12:56 ` Philippe Mathieu-Daudé
2019-10-09 13:21 ` Marcin Wojtas
1 sibling, 0 replies; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 12:56 UTC (permalink / raw)
To: devel, leif.lindholm; +Cc: Ard Biesheuvel, Michael D Kinney, Marcin Wojtas
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>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules
2019-10-09 12:00 ` [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules Leif Lindholm
@ 2019-10-09 12:57 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 12:57 UTC (permalink / raw)
To: devel, leif.lindholm; +Cc: Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format
2019-10-09 12:54 ` Leif Lindholm
@ 2019-10-09 12:59 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 12:59 UTC (permalink / raw)
To: devel, leif.lindholm; +Cc: Ard Biesheuvel, Michael D Kinney
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
>>>
>>
>>
>>
>>
>
>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names
2019-10-09 12:53 ` [edk2-devel] " Philippe Mathieu-Daudé
@ 2019-10-09 13:16 ` Leif Lindholm
2019-10-09 13:36 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 13:16 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: devel, Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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>
> >
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections
2019-10-09 12:00 ` [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections Leif Lindholm
2019-10-09 12:56 ` [edk2-devel] " Philippe Mathieu-Daudé
@ 2019-10-09 13:21 ` Marcin Wojtas
1 sibling, 0 replies; 22+ messages in thread
From: Marcin Wojtas @ 2019-10-09 13:21 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel-groups-io, Ard Biesheuvel, Michael D Kinney
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
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names
2019-10-09 13:16 ` Leif Lindholm
@ 2019-10-09 13:36 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 22+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-10-09 13:36 UTC (permalink / raw)
To: Leif Lindholm; +Cc: devel, Ard Biesheuvel, Michael D Kinney, Ard Bieshuevel
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>
>>>
>>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
2019-10-09 12:00 ` [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format Leif Lindholm
2019-10-09 12:31 ` Ard Biesheuvel
@ 2019-10-09 13:48 ` Liming Gao
2019-10-09 14:27 ` Leif Lindholm
1 sibling, 1 reply; 22+ messages in thread
From: Liming Gao @ 2019-10-09 13:48 UTC (permalink / raw)
To: devel@edk2.groups.io, leif.lindholm@linaro.org
Cc: Ard Biesheuvel, Kinney, Michael D
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
>
>
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [edk2-devel] [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format
2019-10-09 13:48 ` [edk2-devel] " Liming Gao
@ 2019-10-09 14:27 ` Leif Lindholm
0 siblings, 0 replies; 22+ messages in thread
From: Leif Lindholm @ 2019-10-09 14:27 UTC (permalink / raw)
To: Gao, Liming; +Cc: devel@edk2.groups.io, Ard Biesheuvel, Kinney, Michael D
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
> >
> >
> >
>
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2019-10-09 14:27 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-10-09 12:00 [PATCH edk2-platforms 0/6] Update Maintainers.txt Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 1/6] Maintainers.txt: Update documentation for new format Leif Lindholm
2019-10-09 12:51 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 12:54 ` Leif Lindholm
2019-10-09 12:59 ` Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 2/6] Maintainers.txt: drop ',' from maintainer names Leif Lindholm
2019-10-09 12:53 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 13:16 ` Leif Lindholm
2019-10-09 13:36 ` Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 3/6] Maintainers.txt: revamp file to machine-readable format Leif Lindholm
2019-10-09 12:31 ` Ard Biesheuvel
2019-10-09 12:47 ` Leif Lindholm
2019-10-09 13:48 ` [edk2-devel] " Liming Gao
2019-10-09 14:27 ` Leif Lindholm
2019-10-09 12:00 ` [PATCH edk2-platforms 4/6] Maintainers.txt: merge Marvell platforms and silicon sections Leif Lindholm
2019-10-09 12:56 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 13:21 ` Marcin Wojtas
2019-10-09 12:00 ` [PATCH edk2-platforms 5/6] Maintainers.txt: create explicit sections for remaining modules Leif Lindholm
2019-10-09 12:57 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-09 12:00 ` [PATCH edk2-platforms 6/6] Maintainers.txt: drop Ard from default section Leif Lindholm
2019-10-09 12:32 ` [PATCH edk2-platforms 0/6] Update Maintainers.txt Ard Biesheuvel
2019-10-09 12:46 ` Leif Lindholm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox