List of upstream coreboot maintainers and how to submit coreboot changes This represents the list of maintainers that work on the upstream coreboot code base (on coreboot.org). Maintainers are assigned to topics and when applicable to subtrees of the source tree. You'll find some subtrees that don't have a maintainer. If you are looking for reviewers for such a sub- tree, it's often a good choice to look at the git history to see who worked on it last. Please try to follow the guidelines below. This will make things easier on the maintainers. Not all of these guidelines matter for every trivial patch so apply some common sense. 1. Always _test_ your changes, however small, on at least 1 or 2 people, preferably many more. 2. Try to release a few ALPHA test versions to gerrit. Announce them onto the coreboot mailing list and IRC channel and await results. This is especially important on coreboot core changes, but also for device drivers, because often that's the only way you will find things like the fact revision 3 chipset needs a magic fix you didn't know about, or some clown changed the chips on a board and not its name. (Don't laugh!) 3. Make sure your changes compile correctly in multiple configurations. In particular check that changes work for all boards in the tree (use abuild!) 4. When you are happy with a change make it generally available for testing in gerrit and await feedback. 5. Make your patch available through coreboot's gerrit code review system, and add the relevant maintainer from this list as a code reviewer. Be prepared to get your changes sent back with seemingly silly requests about formatting and variable names. These aren't as silly as they seem. One job the maintainers do is to keep things looking the same. Sometimes this means that the clever hack in your mainboard or chipset to get around a problem actually needs to become a generalized coreboot feature ready for next time. PLEASE check your patch with the automated style checker (util/lint/checkpatch.pl) to catch trival style violations. See https://www.coreboot.org/Coding_Style for guidance here. PLEASE add the maintainers that are generated by util/scripts/get_maintainer.pl as reviewers. The results returned by the script will be best if you have git installed and are making your changes in a branch derived from coreboot.org's latest git tree. PLEASE try to include any credit lines you want added with the patch. It avoids people being missed off by mistake and makes it easier to know who wants adding and who doesn't. PLEASE document known bugs. If it doesn't work for everything or does something very odd once a month document it. PLEASE remember that submissions must be made under the terms of the OSDL certificate of contribution and should include a Signed-off-by: line. The current version of this "Developer's Certificate of Origin" (DCO) is listed at https://www.coreboot.org/Development_Guidelines#Sign-off_Procedure. 6. Make sure you have the right to send any changes you make. If you do changes at work you may find your employer owns the patch not you. 7. Happy hacking. Descriptions of section entries: M: Maintainer: FullName Must be registered to Gerrit (https://review.coreboot.org/). Should have experience with upstream coreboot development. R: Designated reviewer: FullName These reviewers should be CCed on patches. L: Mailing list that is relevant to this area W: Web-page with status/info Q: Patchwork web based patch tracking system site T: SCM tree type and location. Type is one of: git, hg, quilt, stgit, topgit S: Status, one of the following: Supported: Someone is continuously paid to look after this and a reaction to review requests can be expected within a few days, a month at most. Maintained: Someone actually looks after it and a reaction to review requests can usually be expected within a few weeks. 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: drivers/net/ all files in and below drivers/net F: drivers/net/* all files in drivers/net, but not below F: */net/* all files in "any top level directory"/net One pattern per line. Multiple F: lines acceptable. N: Files and directories with regex patterns. N: [^a-z]tegra all files whose path contains the word tegra One pattern per line. Multiple N: lines acceptable. scripts/get_maintainer.pl has different behavior for files that match F: pattern and matches of N: patterns. By default, get_maintainer will not look at git log history when an F: pattern match occurs. When an N: match occurs, git log history is used to also notify the people that have git commit signatures. X: Files and directories that are NOT maintained, same rules as F: Files exclusions are tested before file matches. Can be useful for excluding a specific subdirectory, for instance: F: net/ X: net/ipv6/ matches all files in and below net excluding net/ipv6/ K: Keyword perl extended regex pattern to match content in a patch or file. For instance: K: of_get_profile matches patches or files that contain "of_get_profile" K: \b(printk|pr_(info|err))\b matches patches or files that contain one or more of the words printk, pr_info or pr_err One regex pattern per line. Multiple K: lines acceptable. Note: For the hard of thinking, this list is meant to remain in alphabetical order. If you could add yourselves to it in alphabetical order that would be so much easier [Ed] Maintainers List (try to look for most precise areas first) ----------------------------------- RISC-V ARCHITECTURE M: Ronald Minnich R: Philipp Hug S: Maintained F: src/arch/riscv/ F: src/soc/sifive/ F: src/soc/ucb/ F: src/mainboard/emulation/*-riscv/ F: src/mainboard/sifive/ F: util/riscv/ PPC64 ARCHITECTURE M: Ronald Minnich M: Timothy Pearson S: Maintained F: src/arch/ppc64/ F: src/cpu/qemu-power8/ F: src/mainboard/emulation/qemu-power8/ LENOVO EC M: Alexander Couzens S: Maintained F: src/ec/lenovo/ LENOVO MAINBOARDS M: Alexander Couzens M: Patrick Rudolph S: Maintained F: src/mainboard/lenovo/ LENOVO G505S MAINBOARD M: Mike Banon S: Maintained F: src/mainboard/lenovo/g505s/ APPLE MAINBOARDS M: Evgeny Zinoviev S: Maintained F: src/mainboard/apple/ GETAC P470 MAINBOARD M: Patrick Georgi S: Maintained F: src/mainboard/getac/p470 INTEL PINEVIEW CHIPSET M: Damien Zammit S: Odd Fixes F: src/northbridge/intel/pineview/ INTEL D510MO MAINBOARD M: Damien Zammit S: Odd Fixes F: src/mainboard/intel/d510mo INTEL X4X CHIPSET M: Damien Zammit S: Odd Fixes F: src/northbridge/intel/x4x/ GIGABYTE GA-G41M-ES2L MAINBOARD M: Damien Zammit S: Odd Fixes F: src/mainboard/gigabyte/ga-g41m-es2l GIGABYTE GA-H61M-S2PV MAINBOARD M: Angel Pons S: Maintained F: src/mainboard/gigabyte/ga-h61m-s2pv GOOGLE PANTHER MAINBOARD M: Stefan Reinauer S: Supported F: src/mainboard/google/panther/ GOOGLE MAINBOARDS (Intel-based, legacy/inactive) M: Matt DeVillier S: Maintained F: src/mainboard/google/auron/ F: src/mainboard/google/beltino/ F: src/mainboard/google/butterfly/ F: src/mainboard/google/cyan/ F: src/mainboard/google/glados/ F: src/mainboard/google/jecht/ F: src/mainboard/google/link/ F: src/mainboard/google/parrot/ F: src/mainboard/google/slippy/ F: src/mainboard/google/stout/ OPENCELLULAR MAINBOARDS M: Philipp Deppenwiese M: Patrick Rudolph S: Supported F: src/mainboard/opencellular/elgon/ PURISM MAINBOARDS M: Matt DeVillier S: Supported F: src/mainboard/purism SAMSUNG CHROMEOS MAINBOARDS M: Matt DeVillier S: Maintained F: src/mainboard/samsung/lumpy/ F: src/mainboard/samsung/stumpy/ INTEL FSP DENVERTON-NS SOC & HARCUVAR CRB M: Vanessa Eusebio M: David Guckian S: Odd Fixes F: src/mainboard/intel/harcuvar/ F: src/soc/intel/denverton_ns/ F: src/vendorcode/intel/fsp/fsp2_0/denverton_ns/ INTEL FSP 1.1 M: Lee Leahy M: Huang Jin M: York Yang S: Supported F: src/drivers/intel/fsp1_1/ INTEL FSP 2.0 M: Andrey Petrov S: Maintained F: src/drivers/intel/fsp2_0/ INTEL STRAGO MAINBOARD M: Hannah Williams S: Supported F: /src/mainboard/intel/strago/ INTEL BRASWELL SOC M: Piotr Król M: Michał Żygowski M: Frans Hendriks S: Maintained F: /src/soc/intel/braswell F: /src/vendorcode/intel/fsp/fsp1_1/braswell INTEL APOLLOLAKE_SOC M: Andrey Petrov S: Maintained F: src/soc/intel/apollolake/ ASROCK G41C-GS MAINBOARD & VARIANTS M: Angel Pons S: Maintained F: src/mainboard/asrock/g41c-gs/ ASROCK H81M-HDS MAINBOARD M: Tristan Corrick S: Maintained F: src/mainboard/asrock/h81m-hds/ ASUS AM1I-A MAINBOARD M: Mike Banon S: Maintained F: src/mainboard/asus/am1i-a/ ASUS MAXIMUS IV GENE-Z MAINBOARD M: Tristan Corrick S: Maintained F: src/mainboard/asus/maximus_iv_gene-z/ ASUS P5QC PRO MAINBOARD & VARIANTS M: Angel Pons S: Maintained F: src/mainboard/asus/p5qc/ ASUS P8H61-M LX MAINBOARD M: Tristan Corrick S: Maintained F: src/mainboard/asus/p8h61-m_lx/ ASUS P8H61-M PRO MAINBOARD M: Angel Pons S: Maintained F: src/mainboard/asus/p8h61-m_pro/ ASUS P8Z77-M PRO MAINBOARD M: Vlado Cibic S: Maintained F: src/mainboard/asus/p8z77-m_pro/ PC ENGINES ALL MAINBOARDS M: Piotr Król M: Michał Żygowski S: Supported F: src/mainboard/pcengines/ PROTECTLI ALL MAINBOARDS M: Piotr Król M: Michał Żygowski S: Maintained F: src/mainboard/protectli/ SIEMENS MC_xxxx MAINBOARDS M: Werner Zeh S: Maintained F: src/mainboard/siemens/mc_apl1/ SUPERMICRO X10SLM+-F MAINBOARD M: Tristan Corrick S: Maintained F: src/mainboard/supermicro/x10slm-f/ SUPERMICRO X11-LGA1151-SERIES M: Michael Niewöhner S: Maintained F: src/mainboard/supermicro/x11-lga1151-series FACEBOOK FBG1701 MAINBOARD M: Frans Hendriks M: Wim Vervoorn S: Maintained F: src/mainboard/facebook/fbg1701/ FACEBOOK MONOLITH MAINBOARD M: Frans Hendriks M: Wim Vervoorn S: Maintained F: src/mainboard/facebook/monolith/ PORTWELL PQ-M107 MAINBOARD M: Frans Hendriks M: Wim Vervoorn S: Maintained F: src/mainboard/portwell/m107/ ASPEED AST2050 DRIVER & COMMON CODE M: Timothy Pearson S: Supported F: src/drivers/aspeed/common/ F: src/drivers/aspeed/ast2050/ ATI MACH64 Driver S: Orphan F: src/drivers/ati/mach64/ ABUILD M: Patrick Georgi M: Martin Roth S: Supported F: util/abuild/ ACPI F: src/acpi/ F: src/arch/x86/acpi/ F: util/acpi/ LZ4 COMPRESSION M: Julius Werner S: Supported F: src/commonlib/lz4* F: payloads/libpayload/liblz4/ F: util/cbfstool/lz4/ ARM ARCHITECTURE M: Julius Werner S: Supported F: src/arch/arm/ F: src/arch/arm64/ F: src/soc/mediatek/ F: src/soc/nvidia/ F: src/soc/rockchip/ F: util/nvidia/ F: util/rockchip/ ORPHANED ARM SOCS S: Orphaned F: src/cpu/allwinner/ F: src/cpu/armltd/ F: src/cpu/ti/ F: src/soc/marvell/ F: src/soc/qualcomm/ F: src/soc/samsung/ F: util/arm_boot_tools/ F: util/exynos/ F: util/ipqheader/ X86 ARCHITECTURE F: src/arch/x86/ F: src/cpu/x86/ F: src/drivers/pc80/ F: src/include/pc80/ F: src/include/cpu/x86/ INTEL SUPPORT M: Patrick Rudolph S: Maintained F: src/vendorcode/intel/ F: src/cpu/intel/ F: src/northbridge/intel/ F: src/southbridge/intel/ F: src/soc/intel/ F: src/drivers/intel/ F: src/include/cpu/intel/ AMD SUPPORT F: src/vendorcode/amd/ F: src/cpu/amd/ F: src/northbridge/amd/ F: src/southbridge/amd/ F: src/include/cpu/amd/ LINT SCRIPTS M: Patrick Georgi M: Martin Roth S: Supported F: util/lint/ INTELTOOL M: Stefan Reinauer F: util/inteltool/ INTELMETOOL M: Philipp Deppenwiese F: util/intelmetool/ ME_CLEANER M: Nicola Corna W: https://github.com/corna/me_cleaner S: Maintained F: util/me_cleaner/ IFDTOOL M: Stefan Reinauer F: util/ifdtool/ BUILD SYSTEM M: Patrick Georgi M: Martin Roth S: Supported F: Makefile F: *.inc F: src/include/kconfig.h F: util/kconfig/ F: util/sconfig/ F: util/xcompile/ F: util/genbuild_h/ BOARD STATUS F: util/board_status/ BINARY OBJECTS F: 3rdparty/blobs/ VERIFIED BOOT F: 3rdparty/vboot/ F: src/vendorcode/google/chromeos/ F: src/include/tpm.h F: src/include/tpm_lite/ RESOURCE ALLOCATOR F: src/device/* F: src/include/device/ F: src/include/cpu/cpu.h OPTION ROM EXECUTION & X86EMU F: src/device/oprom/ CBFS F: src/include/cbfs.h F: src/commonlib/bsd/include/commonlib/bsd/cbfs_serialized.h F: util/cbfstool/ CBMEM F: src/include/cbmem.h F: src/include/cbmem_id.h F: util/cbmem/ CONSOLE F: src/console/ F: src/include/console/ F: src/drivers/uart/ NVRAM F: util/nvramtool/ F: util/optionlist/ F: payloads/nvramcui/ LIBPAYLOAD F: payloads/libpayload/ BAYOU PAYLOAD F: payloads/bayou/ COREINFO PAYLOAD F: payloads/coreinfo/ EXTERNAL PAYLOADS INTEGRATION M: Stefan Reinauer M: Martin Roth F: payloads/external LINUXBOOT PAYLOAD INTEGRATION M: Philipp Deppenwiese M: Marcello Sylvester Bauer S: Supported F: payloads/external/LinuxBoot VERIFIED BOOT 2 M: Aaron Durbin F: src/security/vboot/ TPM SUPPORT M: Philipp Deppenwiese S: Supported F: src/drivers/*/tpm/ F: src/security/vboot/vboot_crtm.* F: src/security/tpm DOCKER M: Martin Roth S: Supported F: util/docker/ TOOLCHAIN F: util/crossgcc/ GIT F: .git* F: /util/gitconfig SUPERIOS & SUPERIOTOOL M: Felix Held S: Maintained F: src/superio/ F: util/superiotool/ MEMLAYOUT M: Julius Werner S: Supported F: */memlayout.h F: *.ld ELTAN VENDORCODE M: Frans Hendriks M: Wim Vervoorn S: Maintained F: src/vendorcode/eltan MISSING: TIMERS / DELAYS MISSING: TIMESTAMPS MISSING: FMAP MISSING: GPIO MISSING: SMP MISSING: DMP / QEMU-X86 MISSING: ELOG MISSING: SPI # *** Infrastructure Owners*** # This is intended to let people know who they should contact for issues with various infrastructure pieces. # Hardware # Owners: Stefan, Patrick # Backups: # Web Server # Owners: Stefan, Patrick # Backups: # Website # Owners: Martin, Philipp # Backups: Patrick, Stefan # Documentation Website # Owners: Patrick, Philipp # Backups: CODE OF CONDUCT M: Stefan Reinauer M: Patrick Georgi M: Ronald Minnich M: Martin Roth S: Maintained F: Documentation/community/code_of_conduct.md # Wiki # Owners: Stefan, Patrick # Backups: # Gerrit # Owners: Stefan, Patrick # Backups: Martin # Jenkins # Owners: Patrick, Martin # Backups: # Bug Tracker # Owners: Lynxis, # Backups: Martin, # Mailing List # Owners: Stefan, Patrick # Backups: Martin, # Software Freedom Conservancy # Main contact: Martin # “Official” contact: Stefan