Anastasia Klimchuk has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/76771?usp=email )
Change subject: doc: Add link to flashrom patches to dev guide
......................................................................
doc: Add link to flashrom patches to dev guide
People might be interested in what's currently ongoing in
development, and even more practically, have a look at examples
before creating their own patch. In fact, the latter is a good
idea to do. Giving the link in the intro of dev guide will
encourage that.
Viewing the patches does not require an account, so can be done
at the very beginning.
Change-Id: Iecd31a5e9a3683480b33fb9ba331545a0cf669b4
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
---
M doc/dev_guide/development_guide.rst
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/71/76771/1
diff --git a/doc/dev_guide/development_guide.rst b/doc/dev_guide/development_guide.rst
index 950fa91..35cd927 100644
--- a/doc/dev_guide/development_guide.rst
+++ b/doc/dev_guide/development_guide.rst
@@ -13,6 +13,7 @@
adjusted and you avoid frustration if there is some disagreement about the
design.
+You can look at the latest flashrom development efforts `here <https://review.coreboot.org/q/project:flashrom>`_.
Set up the git repository and dev environment
=============================================
--
To view, visit https://review.coreboot.org/c/flashrom/+/76771?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iecd31a5e9a3683480b33fb9ba331545a0cf669b4
Gerrit-Change-Number: 76771
Gerrit-PatchSet: 1
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-MessageType: newchange
Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/76675?usp=email )
Change subject: doc: Fix Contact link in README
......................................................................
doc: Fix Contact link in README
The link should be to another doc, that will work with any website
configuration. Old link only worked for wiki.
Change-Id: I6e2ec4f73494bebd5bd47bbeaa2051439d1d3bbd
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/76675
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Peter Marheine <pmarheine(a)chromium.org>
---
M README.rst
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
build bot (Jenkins): Verified
Peter Marheine: Looks good to me, approved
diff --git a/README.rst b/README.rst
index 428ae57..0a0ea08 100644
--- a/README.rst
+++ b/README.rst
@@ -55,6 +55,4 @@
https://www.flashrom.org/
-Available contact methods are
-
- https://www.flashrom.org/Contact
+For available contact methods see :doc:`contact`
--
To view, visit https://review.coreboot.org/c/flashrom/+/76675?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I6e2ec4f73494bebd5bd47bbeaa2051439d1d3bbd
Gerrit-Change-Number: 76675
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/76663?usp=email )
Change subject: doc: Fix the example Signed-off-by line to be displayed properly
......................................................................
doc: Fix the example Signed-off-by line to be displayed properly
It was not displayed, so needed to be fixed. Secondly, it is just
one line and can be an inline code markup instead of a code-block
(the latter was not displayed for some reasons).
Change-Id: I0f26d6fb03d1491daefa4a362068f674230e94a6
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/76663
Reviewed-by: Peter Marheine <pmarheine(a)chromium.org>
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
---
M doc/dev_guide/development_guide.rst
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
build bot (Jenkins): Verified
Peter Marheine: Looks good to me, approved
diff --git a/doc/dev_guide/development_guide.rst b/doc/dev_guide/development_guide.rst
index b4887ef..950fa91 100644
--- a/doc/dev_guide/development_guide.rst
+++ b/doc/dev_guide/development_guide.rst
@@ -87,9 +87,7 @@
<http://web.archive.org/web/20070306195036/http://osdlab.org/newsroom/press_…>`_
do. Add a note such as
-.. code-block::
-
- Signed-off-by: Random J Developer <random(a)developer.example.org>
+:code:`Signed-off-by: Random J Developer <random(a)developer.example.org>`
to your email/patch if you agree with the Developer's Certificate of Origin 1.1
printed below. Read `this post on the LKML
--
To view, visit https://review.coreboot.org/c/flashrom/+/76663?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I0f26d6fb03d1491daefa4a362068f674230e94a6
Gerrit-Change-Number: 76663
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Anastasia Klimchuk has submitted this change. ( https://review.coreboot.org/c/flashrom/+/75744?usp=email )
Change subject: flash: Update the comment for flashchip eraseblocks
......................................................................
flash: Update the comment for flashchip eraseblocks
Change-Id: Ica7790667ac4c1baf961cb7a330e08178e2c0c28
Signed-off-by: Anastasia Klimchuk <aklm(a)chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/75744
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Aarya <aarya.chaumal(a)gmail.com>
---
M include/flash.h
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Aarya: Looks good to me, approved
build bot (Jenkins): Verified
diff --git a/include/flash.h b/include/flash.h
index 0eace15..4c08549 100644
--- a/include/flash.h
+++ b/include/flash.h
@@ -449,9 +449,7 @@
/*
* Erase blocks and associated erase function. Any chip erase function
* is stored as chip-sized virtual block together with said function.
- * The first one that fits will be chosen. There is currently no way to
- * influence that behaviour. For testing just comment out the other
- * elements or set the function pointer to NULL.
+ * The logic for how to optimally select erase functions is in erasure_layout.c
*/
struct block_eraser {
struct eraseblock {
--
To view, visit https://review.coreboot.org/c/flashrom/+/75744?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ica7790667ac4c1baf961cb7a330e08178e2c0c28
Gerrit-Change-Number: 75744
Gerrit-PatchSet: 2
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Aarya <aarya.chaumal(a)gmail.com>
Gerrit-Reviewer: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Edward O'Callaghan <quasisec(a)chromium.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: merged
Attention is currently required from: Alexander Goncharov, Anastasia Klimchuk, Angel Pons, Martin L Roth, Patrick Georgi, Peter Marheine, Stefan Reinauer, Thomas Heijligen.
Carl-Daniel Hailfinger has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/76455?usp=email )
Change subject: doc: Add code of conduct
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS2:
> It seems the main misunderstanding comes from "Acknowledge that rules already exist".
> Maybe some rules exist, but code of conduct does not exist. The project needs one.
Right. If the name is important, we can just rename the existing rules to "code of conduct". Then we have one. And the wikipedia definition of "code of conduct" also matches the old rules. (No, I didn't edit wikipedia to agree with me.)
> I see two options:
>
> 1) This patch adds code of conduct copied from coreboot as a starting point. We start discussions/reviews of a Friendliness page with the goal to rename and promote it to code of conduct. Once this is done, the new page replaces this initial version.
>
> 2) This patch adds code of conduct copied from coreboot as a starting point. We start discussions/reviews of a Friendliness page as an independent page, and then once it is done we have two pages: code of conduct and friendliness.
>
> There are few reasons why Friendliness cannot be migrated to the new format straight away, see below. Otherwise I would migrate it already.
>
> 1) Some parts are out-dated (we don't do code reviews on mailing list anymore)
> 2) Some parts unfinished (todo in the middle)
> 3) Some parts belong to dev guidelines
> 4) Some info better be on the Contacts page
> 5) Missing: clear definition of unacceptable behavior
> 6) Missing: clear statement that this rules are enforced (and how), clear explanation of consequences of 5)
What about option three?
3) Acknowledge that the friendliness document is a code of conduct?
Even if you disagree with its scope, enforcement, and level of detail, it is a code of conduct.
> I am more than happy to also add these as comments to the friendliness patch, once it is out for review.
>
> The very important property of code of conduct is that one can't violate it. That was missing from Friendliness: it was violated multiple times over the recent years and nothing happened.
How do you know that nothing happened? Speaking as past maintainer, I know that something happened because I had to handle such stuff in the past. However, I do believe in not shaming people in public, so the enforcement actions were not noticed publically.
> As a side note: I am really interested to hear the story about "conflicts bigger than the one which caused explosions two months ago"! Seriously, tell me about it, I haven't heard about it!
The involved parties know, and I don't want to shame people in public regardless of how long ago this happened.
--
To view, visit https://review.coreboot.org/c/flashrom/+/76455?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic4c646bc47d5db4d064a411d60a44f16559dfcee
Gerrit-Change-Number: 76455
Gerrit-PatchSet: 3
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin L Roth <gaumless(a)gmail.com>
Gerrit-CC: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Comment-Date: Wed, 26 Jul 2023 01:37:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org>
Comment-In-Reply-To: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-MessageType: comment
Attention is currently required from: Alexander Goncharov, Anastasia Klimchuk, Angel Pons, Martin L Roth, Patrick Georgi, Peter Marheine, Stefan Reinauer, Thomas Heijligen.
Carl-Daniel Hailfinger has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/76455?usp=email )
Change subject: doc: Add code of conduct
......................................................................
Patch Set 3: Code-Review-2
(1 comment)
Patchset:
PS2:
> also added CB:75554 , it is already approved.
It is approved in coreboot code review, but not merged in coreboot (unresolved comments). This would make our code of conduct more strict than the code of conduct of coreboot.
It would even make our code of conduct one of the strictest codes of conduct out there. Quick googling didn't find any similar clauses on other FOSS projects.
To quote that clause: "Threatening or starting legal action against the project, sibling projects hosted on coreboot.org infrastructure, project or infrastructure
maintainers leads to an immediate ban from coreboot.org and related
systems."
--
To view, visit https://review.coreboot.org/c/flashrom/+/76455?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic4c646bc47d5db4d064a411d60a44f16559dfcee
Gerrit-Change-Number: 76455
Gerrit-PatchSet: 3
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin L Roth <gaumless(a)gmail.com>
Gerrit-CC: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Comment-Date: Wed, 26 Jul 2023 01:26:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Patrick Georgi <patrick(a)coreboot.org>
Comment-In-Reply-To: Anastasia Klimchuk <aklm(a)chromium.org>
Comment-In-Reply-To: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Alexander Goncharov, Angel Pons, Carl-Daniel Hailfinger, Martin L Roth, Patrick Georgi, Peter Marheine, Stefan Reinauer, Thomas Heijligen.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/76455?usp=email )
Change subject: doc: Add code of conduct
......................................................................
Patch Set 3:
(2 comments)
Patchset:
PS2:
> Thanks for bringing that up! […]
Since two people already asked to copy the text instead of linking it, I did that in the new patchset. I copied the existing content of https://doc.coreboot.org/community/code_of_conduct.html and also added CB:75554 , it is already approved.
Totally valid point about the arbitration team, sorry that I missed that!
Yes agree, that's a bootstrapping measure until we figure out our own.
PS2:
It seems the main misunderstanding comes from "Acknowledge that rules already exist".
Maybe some rules exist, but code of conduct does not exist. The project needs one.
As I said in commit message,
> For now link to coreboot's one, in the absence of our own.
It is always possible to create our own later, if desired,
but meanwhile we can share code of conduct with coreboot. We
do have the same servers and infrastructure anyway.
As Carl-Daniel explained here and also on the mailing list, adopting flashrom tailored code of conduct can take some time. And this is fine. Meanwhile, we can use coreboot's one.
I see two options:
1) This patch adds code of conduct copied from coreboot as a starting point. We start discussions/reviews of a Friendliness page with the goal to rename and promote it to code of conduct. Once this is done, the new page replaces this initial version.
2) This patch adds code of conduct copied from coreboot as a starting point. We start discussions/reviews of a Friendliness page as an independent page, and then once it is done we have two pages: code of conduct and friendliness.
There are few reasons why Friendliness cannot be migrated to the new format straight away, see below. Otherwise I would migrate it already.
1) Some parts are out-dated (we don't do code reviews on mailing list anymore)
2) Some parts unfinished (todo in the middle)
3) Some parts belong to dev guidelines
4) Some info better be on the Contacts page
5) Missing: clear definition of unacceptable behavior
6) Missing: clear statement that this rules are enforced (and how), clear explanation of consequences of 5)
I am more than happy to also add these as comments to the friendliness patch, once it is out for review.
The very important property of code of conduct is that one can't violate it. That was missing from Friendliness: it was violated multiple times over the recent years and nothing happened.
As a side note: I am really interested to hear the story about "conflicts bigger than the one which caused explosions two months ago"! Seriously, tell me about it, I haven't heard about it!
--
To view, visit https://review.coreboot.org/c/flashrom/+/76455?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic4c646bc47d5db4d064a411d60a44f16559dfcee
Gerrit-Change-Number: 76455
Gerrit-PatchSet: 3
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin L Roth <gaumless(a)gmail.com>
Gerrit-CC: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-Attention: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Comment-Date: Mon, 24 Jul 2023 13:07:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Patrick Georgi <patrick(a)coreboot.org>
Comment-In-Reply-To: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Comment-In-Reply-To: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Comment-In-Reply-To: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Alexander Goncharov, Anastasia Klimchuk, Angel Pons, Martin L Roth, Stefan Reinauer, Thomas Heijligen.
Hello Alexander Goncharov, Angel Pons, Carl-Daniel Hailfinger, Peter Marheine, Stefan Reinauer, Thomas Heijligen, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/76455?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: doc: Add code of conduct
......................................................................
doc: Add code of conduct
For now link to coreboot's one, in the absence of our own.
It is always possible to create our own later, if desired,
but meanwhile we can share code of conduct with coreboot. We
do have the same servers and infrastructure anyway.
Change-Id: Ic4c646bc47d5db4d064a411d60a44f16559dfcee
Signed-off-by: Anastasia Klimchuk <aklm(a)flashrom.org>
---
A doc/about_flashrom/code_of_conduct.rst
M doc/about_flashrom/index.rst
M doc/about_flashrom/team.rst
3 files changed, 144 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/55/76455/3
--
To view, visit https://review.coreboot.org/c/flashrom/+/76455?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic4c646bc47d5db4d064a411d60a44f16559dfcee
Gerrit-Change-Number: 76455
Gerrit-PatchSet: 3
Gerrit-Owner: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Reviewer: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net>
Gerrit-Reviewer: Peter Marheine <pmarheine(a)chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: Thomas Heijligen <src(a)posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin L Roth <gaumless(a)gmail.com>
Gerrit-CC: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Thomas Heijligen <src(a)posteo.de>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Anastasia Klimchuk <aklm(a)chromium.org>
Gerrit-Attention: Alexander Goncharov <chat(a)joursoir.net>
Gerrit-MessageType: newpatchset