-----Original Message-----
From: Rebecca Cran <[email protected]>
Sent: Saturday, August 31, 2024 1:58 PM
To: [email protected]; Leif Lindholm <[email protected]>;
Kinney, Michael D <[email protected]>; Michael Kubacki
<[email protected]>
Cc: Rebecca Cran <[email protected]>
Subject: [PATCH edk2-platforms v4 1/1] Move to the GitHub Pull Request
workflow
Migrate data from Maintainers.txt to the GitHub standard CODEOWNERS
files plus REVIEWERS and CONTRIBUTORS.md. The latter file contains
mappings from name to email address and GitHub usernames, which will
help people who want to email maintainers instead of using GitHub.
Add .github/workflows/AssignReviewers.yml which adds reviewers to a
Pull Request based on the content of the REVIEWERS file.
Signed-off-by: Rebecca Cran <[email protected]>
---
.github/workflows/AssignReviewers.yml | 34 ++
CODEOWNERS | 143 +++++++
CONTRIBUTORS.md | 68 +++
Maintainers.txt | 441 --------------------
REVIEWERS | 95 +++++
Readme.md | 30 +-
6 files changed, 365 insertions(+), 446 deletions(-)
diff --git a/.github/workflows/AssignReviewers.yml
b/.github/workflows/AssignReviewers.yml
new file mode 100644
index 000000000000..3fbbc8584000
--- /dev/null
+++ b/.github/workflows/AssignReviewers.yml
@@ -0,0 +1,34 @@
+## @file
+# Assign reviewers from a REVIEWERS file using CODEOWNERS syntax
+#
+# Copyright (c) 2022, Intel Corporation. All rights reserved.<BR>
+# SPDX-License-Identifier: BSD-2-Clause-Patent
+##
+
+name: Assign reviewers from a REVIEWERS file using CODEOWNERS syntax
+
+on:
+ pull_request_target:
+ types: [opened, synchronize, reopened, ready_for_review]
+ branches:
+ - master
+
+jobs:
+ assign_reviewers:
+ if: github.event.pull_request.draft == false
+ runs-on: ubuntu-latest
+ permissions:
+ pull-requests: write
+ steps:
+ - name: Generate Token
+ id: generate-token
+ uses: actions/create-github-app-token@v1
+ with:
+ app-id: ${{ secrets.TIANOCORE_ASSIGN_REVIEWERS_APPLICATION_ID
}}
+ private-key: ${{
secrets.TIANOCORE_ASSIGN_REVIEWERS_APPLICATION_PRIVATE_KEY }}
+ - name: Checkout Pull Request Target
+ uses: actions/checkout@v2
+ - name: Assign Reviewers
+ uses: mdkinney/github-action-assign-reviewers@main
+ with:
+ token: ${{ steps.generate-token.outputs.token }}
diff --git a/CODEOWNERS b/CODEOWNERS
new file mode 100644
index 000000000000..35df26739e22
--- /dev/null
+++ b/CODEOWNERS
@@ -0,0 +1,143 @@
+# This file contains the list of maintainers (i.e. people who own the
+# areas and can commit changes) for various parts of edk2-platforms.
+
+# EDK II Platforms maintainers
+# ============================
+
+* @leiflindholm @mdkinney
+
+/Platform/96Boards/** @ardbiesheuvel @leiflindholm
+
+/Platform/AMD/** @abdattar @changab @pbgrimes
+
+/Platform/AMD/AgesaModulePkg/** @abdattar @changab @pbgrimes
+/Platform/AMD/AgesaPkg/** @abdattar @changab @pbgrimes
+/Platform/AMD/AmdCbsPkg/** @abdattar @changab @pbgrimes
+/Platform/AMD/AmdCpmPkg/** @abdattar @changab @pbgrimes
+
+# AMD Seattle platforms
+/Platform/AMD/OverdriveBoard/** @abdattar @ardbiesheuvel @changab
@leiflindholm @pbgrimes
+/Silicon/AMD/Styx/** @abdattar @ardbiesheuvel @changab
@leiflindholm @pbgrimes
+/Platform/LeMaker/CelloBoard/** @ardbiesheuvel @leiflindholm
+/Platform/SoftIron/ @ardbiesheuvel @leiflindholm
+
+# AMD VanGoghBoard
+/Platform/AMD/VanGoghBoard/** @abdattar @changab @exinghr @pbgrimes
+
+# Ampere Computing
+/Platform/Ampere/** @nhivp
+/Silicon/Ampere/** @nhivp
+
+# ARM
+/Platform/ARM/** @samimujawar
+/Silicon/ARM/** @samimujawar
+
+# BeagleBoard:
+/Platform/BeagleBoard/** @ardbiesheuvel @leiflindholm
+/Silicon/TexasInstruments/** @ardbiesheuvel @leiflindholm
+
+# DisplayLink
+/Drivers/DisplayLink/** @ardbiesheuvel @leiflindholm
+
+/Features/Ext4Pkg/** @heatd
+
+# HiSilicon
+/Platform/Hisilicon/** @leiflindholm
+/Silicon/Hisilicon/** @leiflindholm
+
+/Features/Intel/** @nate-desimone @SaiChaganty
+/Features/Intel/Debugging/** @nate-desimone @SaiChaganty @ydong10
+/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/** @nate-desimone
@SaiChaganty
+/Features/Intel/UserInterface/** @dandanbi @nate-desimone @SaiChaganty
+
+/Features/ManageabilityPkg/** @changab
+
+/Platform/Intel/** @nate-desimone @SaiChaganty
+/Platform/Intel/QuarkPlatformPkg/** @mdkinney @nate-desimone
@SaiChaganty
+/Platform/Intel/Vlv2TbltDevicePkg/** @nate-desimone @SaiChaganty
@yqian4
+/Platform/Intel/BoardModulePkg/** @nate-desimone @SaiChaganty @ydong10
+/Platform/Intel/KabylakeOpenBoardPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/KabylakeOpenBoardPkg/AspireVn7Dash572G/**
@benjamindoron @nate-desimone @SaiChaganty
+/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/** @jackpot51
+/Platform/Intel/MinPlatformPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/PurleyOpenBoardPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/WhiskeylakeOpenBoardPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/CometlakeOpenBoardPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/TigerlakeOpenBoardPkg/** @nate-desimone @SaiChaganty
+/Platform/Intel/AlderlakeOpenBoardPkg/** @nate-desimone @SaiChaganty
+/Platform/Intel/WhitleyOpenBoardPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Platform/Intel/SimicsOpenBoardPkg/** @nate-desimone @SaiChaganty
+/Platform/Intel/Tools/** @BobCF @lgao4 @nate-desimone @SaiChaganty
+
+/Silicon/Intel/** @nate-desimone @SaiChaganty
+/Silicon/Intel/IntelSiliconPkg/** @nate-desimone @niruiyu @SaiChaganty
+/Silicon/Intel/QuarkSocPkg/** @mdkinney @nate-desimone @SaiChaganty
+/Silicon/Intel/Vlv2DeviceRefCodePkg/** @nate-desimone @SaiChaganty
@yqian
+/Silicon/Intel/CoffeelakeSiliconPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Silicon/Intel/KabylakeSiliconPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Silicon/Intel/PurleyRefreshSiliconPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Silicon/Intel/TigerlakeSiliconPkg/** @nate-desimone @SaiChaganty
+/Silicon/Intel/AlderlakeSiliconPkg/** @nate-desimone @SaiChaganty
+/Silicon/Intel/WhitleySiliconPkg/** @ChaselChiu @nate-desimone
@SaiChaganty
+/Silicon/Intel/SimicsX58SktPkg/** @nate-desimone @SaiChaganty
+/Silicon/Intel/SimicsIch10Pkg/** @nate-desimone @SaiChaganty
+/Silicon/Intel/Tools/** @BobCF @lgao4 @nate-desimone @SaiChaganty
+
+# Loongson platforms
+/Platform/Loongson/** @bibo-mao @kilaterlee @lixianglai
+
+# Marvell platforms and silicon
+/Platform/Marvell/** @ndhillonm @wojtas-marcin
+/Platform/SolidRun/** @ndhillonm @wojtas-marcin
+/Silicon/Marvell/** @ndhillonm @wojtas-marcin
+
+# Miscellaneous drivers
+/Silicon/Atmel/** @leiflindholm
+/Silicon/Openmoko/** @leiflindholm
+/Silicon/Synopsys/DesignWare/** @leiflindholm
+
+# NXP platforms and silicon
+/Platform/NXP/** @leiflindholm
+/Silicon/NXP/** @leiflindholm
+
+# OptionRomPkg
+# https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
+/Drivers/OptionRomPkg/** @niruiyu
+
+# Phytium platforms and silicon
+/Platform/Phytium/** @leiflindholm
+/Silicon/Phytium/** @leiflindholm
+
+# QEMU EDK II Minimum Platform Specification implementation
+/Platform/Qemu/QemuOpenBoardPkg/** @heatd
+
+# QEMU sbsa-ref platform
+/Platform/Qemu/SbsaQemu/** @ardbiesheuvel @hrw @leiflindholm
+/Silicon/Qemu/SbsaQemu/** @ardbiesheuvel @hrw @leiflindholm
+
+# Raspberry Pi platforms and silicon
+/Platform/RaspberryPi/** @ardbiesheuvel @leiflindholm
+/Silicon/Broadcom/** @ardbiesheuvel @leiflindholm
+
+# RPMB driver for OP-TEE
+/Drivers/OpTee/OpteeRpmbPkg/** @apalos @samimujawar
+
+/Platform/RISC-V/PlatformPkg/** @vlsunil
+
+/Platform/SiFive/U5SeriesPkg/** @JohnAZoidberg
+
+# Socionext platforms and silicon
+/Platform/Socionext/** @ardbiesheuvel @leiflindholm
+/Silicon/NXP/Library/Pcf8563RealTimeClockLib/** @ardbiesheuvel
@leiflindholm
+/Silicon/Socionext/** @ardbiesheuvel @leiflindholm
+
+/Silicon/RISC-V/ProcessorPkg/** @vlsunil
+
+/Silicon/SiFive/** @JohnAZoidberg
+
+# Sophgo platforms and silicon
+/Platform/Sophgo/** @vlsunil
+/Silicon/Sophgo/SG2042Pkg/** @vlsunil
+
+# StandAloneMMPkg for OP-TEE
+/Platform/StandaloneMm/PlatformStandaloneMmPkg/** @apalos @samimujawar
diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md
new file mode 100644
index 000000000000..eac117c6b7b0
--- /dev/null
+++ b/CONTRIBUTORS.md
@@ -0,0 +1,68 @@
+EDK II Platforms Maintainers and Reviewers
+==========================================
+
+This file is intended to provide an easy way to look up people's names
and email addresses given their GitHub
+usernames from the CODEOWNERS or REVIEWERS files.
+
+Since it's an extra file to remember to update when changing
maintainers or reviewers, it will likely become
+out-of-sync with CODEOWNERS and/or REVIEWERS over time and need an
occasional refresh.
+
+| Name | e-mail address | Github
username |
+|------------------------|----------------------------------|----------
--------------------------------------------|
+| Abdul Lateef Attar | [email protected] |
[@abdattar](https://github.com/abdattar) |
+| Abner Chang | [email protected] |
[@changab](https://github.com/changab) |
+| Ard Biesheuvel | [email protected] |
[@ardbiesheuvel](https://github.com/ardbiesheuvel) |
+| Ashraf Ali S | [email protected] |
[@AshrafAliS](https://github.com/AshrafAliS) |
+| Benjamin Doron | [email protected] |
[@benjamindoron](https://github.com/benjamindoron) |
+| Bibo Mao | [email protected] | [@bibo-
mao](https://github.com/bibo-mao) |
+| Bob Feng | [email protected] |
[@BobCF](https://github.com/BobCF) |
+| caiyuqing379 | [email protected] |
[@caiyuqing379](https://github.com/caiyuqing379) |
+| Chao Li | [email protected] |
[@kilaterlee](https://github.com/kilaterlee) |
+| Chasel Chiu | [email protected] |
[@ChaselChiu](https://github.com/ChaselChiu) |
+| Chuong Tran | [email protected] |
[@chuongtranle](https://github.com/chuongtranle) |
+| dahogn | [email protected] |
[@dahogn](https://github.com/dahogn) |
+| Dandan Bi | [email protected] |
[@dandanbi](https://github.com/dandanbi) |
+| Daniel Schaefer | [email protected] |
[@JohnAZoidberg](https://github.com/JohnAZoidberg) |
+| Deepika Kethi Reddy | [email protected] |
[@dkethire](https://github.com/dkethire) |
+| Duke Zhai | [email protected] |
[@mingxzha](https://github.com/mingxzha) |
+| Eric Dong | [email protected] |
[@ydong10](https://github.com/ydong10) |
+| Eric Xing | [email protected] |
[@exinghr](https://github.com/exinghr) |
+| Graeme Gregory | [email protected] |
|
+| Heng Luo | [email protected] |
|
+| Igniculus Fu | [email protected] |
[@fhh200000](https://github.com/fhh200000) |
+| Ilias Apalodimas | [email protected] |
[@apalos](https://github.com/apalos) |
+| Jeremy Linton | [email protected] |
[@jlinton](https://github.com/jlinton) |
+| Jeremy Soller | [email protected] |
[@jackpot51](https://github.com/jackpot51) |
+| Kathappan Esakkithevar | [email protected] |
[@kesakkit](https://github.com/kesakkit) |
+| Ken Yao | [email protected] |
[@YSHRong](https://github.com/YSHRong) |
+| Leif Lindholm | [email protected] |
[@leiflindholm](https://github.com/leiflindholm) |
+| Liming Gao | [email protected] |
[@lgao4](https://github.com/lgao4) |
+| Ling Jia | [email protected] |
|
+| Marcin Juszkiewicz | [email protected] |
[@hrw](https://github.com/hrw) |
+| Marcin Wojtas | [email protected] | [@wojtas-
marcin](https://github.com/wojtas-marcin) |
+| Marvin Häuser | [email protected] |
[@mhaeuser](https://github.com/mhaeuser) |
+| Meenakshi Aggarwal | [email protected] |
[@Meenakshi88](https://github.com/Meenakshi88) |
+| meng-cz | [email protected] | [@meng-
cz](https://github.com/meng-cz) |
+| Michael D Kinney | [email protected] |
[@mdkinney](https://github.com/mdkinney) |
+| Narinder Dhillon | [email protected] |
[@ndhillonm](https://github.com/ndhillonm) |
+| Nate DeSimone | [email protected] | [@nate-
desimone](https://github.com/nate-desimone) |
+| Nhi Pham | [email protected] |
[@nhivp](https://github.com/nhivp) |
+| Nickle Wang | [email protected] |
[@nicklela](https://github.com/nicklela) |
+| Paul Grimes | [email protected] |
[@pbgrimes](https://github.com/pbgrimes) |
+| Pedro Falcato | [email protected] |
[@heatd](https://github.com/heatd) |
+| Peng Xie | [email protected] |
|
+| Ray Ni | [email protected] |
[@niruiyu](https://github.com/niruiyu) |
+| Rebecca Cran | [email protected] |
[@bcran](https://github.com/bcran) |
+| Rosen Chuang | [email protected] |
[@rbitw](https://github.com/rbitw) |
+| Sai Chaganty | [email protected] |
[@SaiChaganty](https://github.com/SaiChaganty) |
+| Saloni Kasbekar | [email protected] |
[@SaloniKasbekar](https://github.com/SaloniKasbekar) |
+| Sami Mujawar | [email protected] |
[@samimujawar](https://github.com/samimujawar) |
+| Sunil V L | [email protected] |
[@vlsunil](https://github.com/vlsunil) |
+| Theo Jehl | [email protected] |
[@theomacx86](https://github.com/theomacx86) |
+| Thomas Abraham | [email protected] |
|
+| USER0FISH | [email protected] |
[@USER0FISH](https://github.com/USER0FISH) |
+| Wenyi Xie | [email protected] |
|
+| Xianglai li | [email protected] |
[@lixianglai](https://github.com/lixianglai) |
+| Yi Qian | [email protected] |
[@yqian4](https://github.com/yqian4) |
+| Yiqi Shu | [email protected] |
|
+| Yuwei Chen | [email protected] |
[@YuweiChen1110](https://github.com/YuweiChen1110) |
diff --git a/Maintainers.txt b/Maintainers.txt
deleted file mode 100644
index 7b8a9753923b..000000000000
--- a/Maintainers.txt
+++ /dev/null
@@ -1,441 +0,0 @@
-EDK II Platforms Maintainers
-============================
-
-This file provides information about the primary maintainers for
-EDK II Platforms.
-
-In general, you should not privately email the maintainer. You should
-email the edk2-devel list, 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: 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
- 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:
- Supported: Someone is actually paid to look after this.
- Maintained: Someone actually looks after it.
- Odd Fixes: It has a maintainer but they don't have time to do
- much other than throw the odd patch in. See below.
- Orphan: No current maintainer [but maybe you could take the
- role as you write your new code].
- 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 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
-
-All patches CC:d here
-L: [email protected]
-F: *
-F: */
-
-EDK II Platforms maintainers
-----------------------------
-F: *
-M: Leif Lindholm <[email protected]>
-M: Michael D Kinney <[email protected]>
-
-Responsible Disclosure, Reporting Security Issues
--------------------------------------------------
-W: https://github.com/tianocore/tianocore.github.io/wiki/Security
-
-EDK II Platforms Packages:
---------------------------
-
-96Boards
-F: Platform/96Boards/
-M: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-
-AMD
-F: Platform/AMD
-M: Abner Chang <[email protected]>
-M: Abdul Lateef Attar <[email protected]>
-M: Paul Grimes <[email protected]>
-
-F: Platform/AMD/AgesaModulePkg
-F: Platform/AMD/AgesaPkg
-F: Platform/AMD/AmdCbsPkg
-M: Paul Grimes <[email protected]>
-R: Abdul Lateef Attar <[email protected]>
-R: Abner Chang <[email protected]>
-
-F: Platform/AMD/AmdCpmPkg
-M: Abdul Lateef Attar <[email protected]>
-R: Paul Grimes <[email protected]>
-R: Abner Chang <[email protected]>
-
-AMD Seattle
-F: Platform/AMD/OverdriveBoard/
-F: Platform/LeMaker/CelloBoard/
-F: Platform/SoftIron/
-F: Silicon/AMD/Styx/
-M: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-
-AMD VanGoghBoard
-F: Platform/AMD/VanGoghBoard
-F: Silicon/AMD/VanGoghBoard
-M: Eric Xing <[email protected]>
-R: Ken Yao <[email protected]>
-R: Duke Zhai <[email protected]>
-R: Igniculus Fu <[email protected]>
-
-Ampere Computing
-F: Platform/Ampere
-F: Silicon/Ampere
-M: Nhi Pham <[email protected]>
-R: Chuong Tran <[email protected]>
-R: Leif Lindholm <[email protected]>
-R: Rebecca Cran <[email protected]>
-
-ARM
-F: Platform/ARM/
-F: Silicon/ARM/
-R: Thomas Abraham <[email protected]>
-M: Sami Mujawar <[email protected]>
-
-BeagleBoard:
-F: Platform/BeagleBoard/
-F: Silicon/TexasInstruments/
-R: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-
-OptionRomPkg
-F: Drivers/OptionRomPkg/
-W: https://github.com/tianocore/tianocore.github.io/wiki/OptionRomPkg
-M: Ray Ni <[email protected]>
-
-StandAloneMMPkg for OP-TEE
-F: Platform/StandaloneMm/PlatformStandaloneMmPkg/
-M: Sami Mujawar <[email protected]>
-M: Ilias Apalodimas <[email protected]>
-
-DisplayLink
-F: Drivers/DisplayLink/
-M: Leif Lindholm <[email protected]>
-M: Ard Biesheuvel <[email protected]>
-R: Andy Hayes <[email protected]>
-
-HiSilicon
-F: Platform/Hisilicon/
-F: Silicon/Hisilicon/
-M: Leif Lindholm <[email protected]>
-R: Ard Biesheuvel <[email protected]>
-R: Wenyi Xie <[email protected]>
-
-Features/Ext4Pkg
-F: Features/Ext4Pkg/
-M: Pedro Falcato <[email protected]>
-R: Marvin Häuser <[email protected]>
-
-Features/Intel
-F: Features/Intel/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Liming Gao <[email protected]>
-
-Features/Intel/Debugging
-F: Features/Intel/Debugging/
-M: Eric Dong <[email protected]>
-R: Liming Gao <[email protected]>
-
-Features/Intel/OutOfBandManagement/IpmiFeaturePkg
-F: Features/Intel/OutOfBandManagement/IpmiFeaturePkg
-M: Nate DeSimone <[email protected]>
-R: Liming Gao <[email protected]>
-
-Features/Intel/UserInterface
-F: Features/Intel/UserInterface/
-M: Dandan Bi <[email protected]>
-R: Liming Gao <[email protected]>
-
-Features/ManageabilityPkg
-F: Features/ManageabilityPkg/
-M: Abner Chang <[email protected]>
-R: Abdul Lateef Attar <[email protected]>
-R: Nickle Wang <[email protected]>
-
-Platform/Intel
-F: Platform/Intel/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-
-Platform/Intel/QuarkPlatformPkg
-F: Platform/Intel/QuarkPlatformPkg/
-M: Michael D Kinney <[email protected]>
-R: Nate DeSimone <[email protected]>
-
-Platform/Intel/Vlv2TbltDevicePkg
-F: Platform/Intel/Vlv2TbltDevicePkg/
-M: Yi Qian <[email protected]>
-R: Nate DeSimone <[email protected]>
-
-Platform/Intel/BoardModulePkg
-F: Platform/Intel/BoardModulePkg/
-M: Eric Dong <[email protected]>
-R: Liming Gao <[email protected]>
-
-Platform/Intel/KabylakeOpenBoardPkg
-F: Platform/Intel/KabylakeOpenBoardPkg/
-M: Chasel Chiu <[email protected]>
-M: Nate DeSimone <[email protected]>
-
-Platform/Intel/KabylakeOpenBoardPkg/AspireVn7Dash572G
-F: Platform/Intel/KabylakeOpenBoardPkg/AspireVn7Dash572G/
-R: Benjamin Doron <[email protected]>
-
-Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3
-F: Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/
-R: Jeremy Soller <[email protected]>
-
-Platform/Intel/MinPlatformPkg
-F: Platform/Intel/MinPlatformPkg/
-M: Chasel Chiu <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Liming Gao <[email protected]>
-R: Eric Dong <[email protected]>
-
-Platform/Intel/PurleyOpenBoardPkg
-F: Platform/Intel/PurleyOpenBoardPkg/
-M: Nate DeSimone <[email protected]>
-M: Chasel Chiu <[email protected]>
-
-Platform/Intel/WhiskeylakeOpenBoardPkg
-F: Platform/Intel/WhiskeylakeOpenBoardPkg/
-M: Chasel Chiu <[email protected]>
-M: Nate DeSimone <[email protected]>
-
-Platform/Intel/CometlakeOpenBoardPkg
-F: Platform/Intel/CometlakeOpenBoardPkg/
-M: Chasel Chiu <[email protected]>
-M: Nate DeSimone <[email protected]>
-M: Rangasai V Chaganty <[email protected]>
-R: Deepika Kethi Reddy <[email protected]>
-R: Kathappan Esakkithevar <[email protected]>
-
-Platform/Intel/TigerlakeOpenBoardPkg
-F: Platform/Intel/TigerlakeOpenBoardPkg/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Heng Luo <[email protected]>
-
-Platform/Intel/AlderlakeOpenBoardPkg
-F: Platform/Intel/AlderlakeOpenBoardPkg/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Rosen Chuang <[email protected]>
-R: Saloni Kasbekar <[email protected]>
-
-Platform/Intel/WhitleyOpenBoardPkg
-F: Platform/Intel/WhitleyOpenBoardPkg/
-M: Nate DeSimone <[email protected]>
-M: Chasel Chiu <[email protected]>
-
-Platform/Intel/SimicsOpenBoardPkg
-F: Platform/Intel/SimicsOpenBoardPkg/
-M: Nate DeSimone <[email protected]>
-
-Platform/Intel/Tools
-F: Platform/Intel/Tools/
-M: Bob Feng <[email protected]>
-M: Liming Gao <[email protected]>
-R: Yuwei Chen <[email protected]>
-
-Platform/RISC-V/PlatformPkg
-F: Platform/RISC-V/PlatformPkg/
-M: Sunil V L <[email protected]>
-R: Daniel Schaefer <[email protected]>
-
-Platform/SiFive/U5SeriesPkg
-F: Platform/SiFive/U5SeriesPkg/
-M: Daniel Schaefer <[email protected]>
-
-Silicon/Intel
-F: Silicon/Intel/
-M: Rangasai V Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-
-Silicon/Intel/IntelSiliconPkg
-F: Silicon/Intel/IntelSiliconPkg/
-M: Ray Ni <[email protected]>
-M: Rangasai V Chaganty <[email protected]>
-R: Ashraf Ali S <[email protected]>
-
-Silicon/Intel/QuarkSocPkg
-F: Silicon/Intel/QuarkSocPkg/
-M: Michael D Kinney <[email protected]>
-R: Nate DeSimone <[email protected]>
-
-Silicon/Intel/Vlv2DeviceRefCodePkg
-F: Silicon/Intel/Vlv2DeviceRefCodePkg/
-M: Yi Qian <[email protected]>
-R: Nate DeSimone <[email protected]>
-
-Silicon/Intel/CoffeelakeSiliconPkg
-F: Silicon/Intel/CoffeelakeSiliconPkg/
-M: Chasel Chiu <[email protected]>
-M: Sai Chaganty <[email protected]>
-
-Silicon/Intel/KabylakeSiliconPkg
-F: Silicon/Intel/KabylakeSiliconPkg/
-M: Chasel Chiu <[email protected]>
-M: Sai Chaganty <[email protected]>
-
-Silicon/Intel/PurleyRefreshSiliconPkg
-F: Silicon/Intel/PurleyRefreshSiliconPkg/
-M: Chasel Chiu <[email protected]>
-M: Nate DeSimone <[email protected]>
-
-Silicon/Intel/TigerlakeSiliconPkg
-F: Silicon/Intel/TigerlakeSiliconPkg/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Heng Luo <[email protected]>
-
-Platform/Intel/AlderlakeSiliconPkg
-F: Platform/Intel/AlderlakeSiliconPkg/
-M: Sai Chaganty <[email protected]>
-M: Nate DeSimone <[email protected]>
-R: Rosen Chuang <[email protected]>
-R: Saloni Kasbekar <[email protected]>
-
-Silicon/Intel/WhitleySiliconPkg
-F: Silicon/Intel/WhitleySiliconPkg/
-M: Nate DeSimone <[email protected]>
-M: Chasel Chiu <[email protected]>
-
-Silicon/Intel/SimicsX58SktPkg
-F: Silicon/Intel/SimicsX58SktPkg/
-M: Nate DeSimone <[email protected]>
-
-Silicon/Intel/SimicsIch10Pkg
-F: Silicon/Intel/SimicsIch10Pkg/
-M: Nate DeSimone <[email protected]>
-
-Silicon/Intel/Tools
-F: Silicon/Intel/Tools/
-M: Bob Feng <[email protected]>
-M: Liming Gao <[email protected]>
-R: Yuwei Chen <[email protected]>
-
-Loongson platforms
-F: Platform/Loongson/
-M: Bibo Mao <[email protected]>
-M: Xianglai li <[email protected]>
-M: Chao Li <[email protected]>
-
-Marvell platforms and silicon
-F: Platform/Marvell/
-F: Platform/SolidRun/
-F: Silicon/Marvell/
-M: Marcin Wojtas <[email protected]> [wojtas-marcin]
-M: Narinder Dhillon <[email protected]> [ndhillonm]
-R: Leif Lindholm <[email protected]> [leiflindholm]
-
-Miscellaneous drivers
-F: Silicon/Atmel/
-F: Silicon/Openmoko/
-F: Silicon/Synopsys/DesignWare/
-R: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-
-NXP platforms and silicon
-F: Platform/NXP/
-F: Silicon/NXP/
-M: Leif Lindholm <[email protected]>
-R: Meenakshi Aggarwal <[email protected]>
-
-QEMU EDK II Minimum Platform Specification implementation
-F: Platform/Qemu/QemuOpenBoardPkg/
-M: Pedro Falcato <[email protected]>
-R: Theo Jehl <[email protected]>
-
-QEMU sbsa-ref platform
-F: Platform/Qemu/SbsaQemu/
-F: Silicon/Qemu/SbsaQemu/
-M: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-M: Marcin Juszkiewicz <[email protected]> [hrw]
-R: Graeme Gregory <[email protected]>
-
-Raspberry Pi platforms and silicon
-F: Platform/RaspberryPi/
-F: Silicon/Broadcom/
-M: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-R: Jeremy Linton <[email protected]>
-
-RPMB driver for OP-TEE
-F: Drivers/OpTee/OpteeRpmbPkg/
-M: Sami Mujawar <[email protected]>
-M: Ilias Apalodimas <[email protected]>
-
-Socionext platforms and silicon
-F: Platform/Socionext/
-F: Silicon/NXP/Library/Pcf8563RealTimeClockLib/
-F: Silicon/Socionext/
-M: Ard Biesheuvel <[email protected]>
-M: Leif Lindholm <[email protected]>
-
-Silicon/RISC-V/ProcessorPkg
-F: Silicon/RISC-V/ProcessorPkg/
-M: Sunil V L <[email protected]>
-R: Daniel Schaefer <[email protected]>
-
-Silicon/SiFive
-F: Silicon/SiFive/
-M: Daniel Schaefer <[email protected]>
-
-Phytium platforms and silicon
-F: Platform/Phytium/
-F: Silicon/Phytium/
-M: Leif Lindholm <[email protected]>
-R: Peng Xie <[email protected]>
-R: Ling Jia <[email protected]>
-R: Yiqi Shu <[email protected]>
-
-Sophgo platforms and silicon
-F: Platform/Sophgo/
-F: Silicon/Sophgo/SG2042Pkg/
-M: Sunil V L <[email protected]>
-R: dahogn <[email protected]>
-R: meng-cz <[email protected]>
-R: caiyuqing379 <[email protected]>
-R: USER0FISH <[email protected]>
diff --git a/REVIEWERS b/REVIEWERS
new file mode 100644
index 000000000000..5f5a96f4eba1
--- /dev/null
+++ b/REVIEWERS
@@ -0,0 +1,95 @@
+# This file contains the list of reviewers (i.e. people who can
+# review changes, but don't own the areas and can't commit changes)
+# for various parts of edk2-platforms.
+
+# EDK II Platforms reviewers
+# ==========================
+
+# AMD VanGoghBoard
+/Platform/AMD/VanGoghBoard/** @fhh200000 @mingxzha @YSHRong
+
+# Ampere
+/Platform/Ampere/** @bcran @chuongtranle @leiflindholm
+/Silicon/Ampere/** @bcran @chuongtranle @leiflindholm
+
+# ARM
+# Add Thomas Abraham
+# /Platform/ARM/**
+
+# BeagleBoard
+/Platform/BeagleBoard/** @ardbiesheuvel
+/Silicon/TexasInstruments/** @ardbiesheuvel
+
+/Features/Ext4Pkg/** @mhaeuser
+
+# Hisilicon
+# Add Wenyi Xie
+/Platform/Hisilicon/** @ardbiesheuvel
+/Silicon/Hisilicon/** @ardbiesheuvel
+
+/Features/Intel/** @lgao4
+/Features/Intel/Debugging/** @lgao4
+/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/** @lgao4
+/Features/Intel/UserInterface/** @lgao4
+
+/Platform/Intel/QuarkPlatformPkg/** @nate-desimone
+/Platform/Intel/Vlv2TbltDevicePkg/** @nate-desimone
+/Platform/Intel/BoardModulePkg/** @lgao4
+/Platform/Intel/KabylakeOpenBoardPkg/GalagoPro3/** @jackpot5
+/Platform/Intel/MinPlatformPkg/** @lgao4 @ydong10
+/Platform/Intel/WhiskeylakeOpenBoardPkg/** @ChaselChiu
+/Platform/Intel/CometlakeOpenBoardPkg/** @dkethire @kesakkit
+# Add Heng Luo
+# /Platform/Intel/TigerlakeOpenBoardPkg/**
+/Platform/Intel/AlderlakeOpenBoardPkg/** @rbitw @SaloniKasbekar
+/Platform/Intel/Tools/** @YuweiChen1110
+
+/Silicon/Intel/IntelSiliconPkg/** @AshrafAliS
+/Silicon/Intel/QuarkSocPkg/** @nate-desimone
+/Silicon/Intel/Vlv2DeviceRefCodePkg/** @nate-desimone
+
+# Add Heng Luo
+# /Silicon/Intel/TigerlakeSiliconPkg/**
+
+/Silicon/Intel/AlderlakeSiliconPkg/** @rbitw @SaloniKasbekar
+
+/Silicon/Intel/Tools/** @YuweiChen1110
+
+/Features/ManageabilityPkg/** @abdattar @nicklela
+
+/Platform/Marvell/** @leiflindholm
+/Silicon/Marvell/** @leiflindholm
+
+# Miscellaneous Drivers
+/Silicon/Atmel/** @ardbiesheuvel
+/Silicon/Openmoko/** @ardbiesheuvel
+/Silicon/Synopsys/DesignWare/** @ardbiesheuvel
+
+/Platform/NXP/** @Meenakshi88
+/Silicon/NXP/** @Meenakshi88
+
+# QEMU EDK II Minimum Platform Specification implementation
+/Platform/Qemu/QemuOpenBoardPkg/** @theomacx86
+
+# Add Graeme Gregory
+# /Platform/Qemu/SbsaQemu/**
+# /Silicon/Qemu/SbsaQemu/**
+
+# Raspberry Pi platforms and silicon
+/Platform/RaspberryPi/** @jlinton
+/Silicon/Broadcom/** @jlinton
+
+/Platform/RISC-V/PlatformPkg/** @JohnAZoidberg
+/Silicon/RISC-V/ProcessorPkg/** @JohnAZoidberg
+
+# Phytium platforms and silicon
+# Add Peng Xie, Ling Jia and Yiqi Shu
+# /Platform/Phytium/**
+# /Silicon/Phytium/**
+
+/Platform/SolidRun/** @leiflindholm
+
+# Sophgo platforms and silicon
+/Platform/Sophgo/** @caiyiqing379 @dahogn @meng-cz @USER0FISH
+/Silicon/Sophgo/SG2042Pkg/** @caiyiqing379 @dahogn @meng-cz @USER0FISH
+
diff --git a/Readme.md b/Readme.md
index 8de9eaa7cfbb..792e27e51843 100644
--- a/Readme.md
+++ b/Readme.md
@@ -32,10 +32,30 @@ Many platforms require additional image processing
beyond the EDK2 build.
Any such steps should be documented (as a Readme.md), and any necessary
helper
scripts be contained, under said platform directory.
-Any contributions to this branch should be submitted via email to the
-edk2-devel mailing list with a subject prefix of `[platforms]`. See
-[Laszlo's excellent
guide](https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-
unkempt-git-guide-for-edk2-contributors-and-maintainers) for details
-on how to do this successfully.
+Any contributions to this repo should be submitted via GitHub Pull
Request.
+
+For details of who owns code in certain parts of the repo, see the
CODEOWNERS and
+REVIEWERS files. Look in CONTRIBUTORS.md to find out people's names and
their
+email addresses.
+
+In general, you should not privately email the maintainer. You should
+email the edk2-devel list, and Cc the area maintainers and
+reviewers.
+
+If the maintainer wants to hand over the role to other people,
+they should create a PR on GitHub to update CODEOWNERS,
+REVIEWERS and CONTRIBUTORS.md with new maintainer, and the new
maintainer
+should review the PR and approve it.
+
+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
+
+Responsible Disclosure, Reporting Security Issues
+-------------------------------------------------
+W: https://github.com/tianocore/tianocore.github.io/wiki/Security
# How to build (Linux Environment)
@@ -294,7 +314,7 @@ For more information, see the
# Maintainers
-See [Maintainers.txt](Maintainers.txt).
+See [CONTRIBUTORS.md](CONTRIBUTORS.md), [CODEOWNERS](CODEOWNERS) and
[REVIEWERS](REVIEWERS).
# Submodules
--
2.46.0
