From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH 1/5] doc: develop: process: Move Custodians section
Date: Tue, 20 Jan 2026 16:31:42 -0600 [thread overview]
Message-ID: <20260120223405.469050-2-trini@konsulko.com> (raw)
In-Reply-To: <20260120223405.469050-1-trini@konsulko.com>
Move the "Custodians" section to be after the "Review Process, Git Tags"
section, in preparation for more re-organization.
Signed-off-by: Tom Rini <trini@konsulko.com>
---
doc/develop/process.rst | 82 ++++++++++++++++++++---------------------
1 file changed, 41 insertions(+), 41 deletions(-)
diff --git a/doc/develop/process.rst b/doc/develop/process.rst
index 0651a1c23a48..3bda5e6b51dd 100644
--- a/doc/develop/process.rst
+++ b/doc/develop/process.rst
@@ -120,47 +120,6 @@ these can be "cherry-picked" and are subject to the normal merge rules. For
example, a bug fix can come in later in the window but a full re-sync only
happens within the merge window itself.
-.. _custodians:
-
-Custodians
-----------
-
-The Custodians take responsibility for some area of the U-Boot code. The
-in-tree ``MAINTAINERS`` files list who is responsible for which areas.
-
-It is their responsibility to pick up patches from the mailing list
-that fall into their responsibility, and to process these.
-
-A very important responsibility of each custodian is to provide
-feedback to the submitter of a patch about what is going on:
-
- * If the patch was accepted, or if it was rejected (with exact list
- of reasons), if it needs to be reworked (with respective review
- comments). Even a "I have no time now, will look into it later"
- message is better than nothing. Also, if there are remarks to a
- patch, these should leave no doubt if they were just comments and
- the patch will be accepted anyway, or if the patch should be
- reworked/resubmitted, or if it was rejected. However, if a submitter
- feels it has been too long since posting their patch and not
- received any feedback, it is OK to follow-up and ask.
-
- * A custodian may make changes suggested by :doc:`checkpatch.pl
- <checkpatch>`. They must also in turn amend the commit message noting
- their change, for example ``[trini: Fix typos]``, and add their own
- :ref:`Signed-off-by <dco>` tag. All other changes must be handled by
- another iteration of the patch, or follow-up patch.
-
- * If the patch itself can still be applied to the tree. The custodian
- is expected to put in a "best effort" if a patch does not apply
- cleanly, but can be made to apply still. It is up to the custodian
- to decide how recent of a commit the patch must be against. It is
- acceptable to request patches against the last officially released
- version of U-Boot or newer. Of course a custodian can also accept
- patches against older code. It can be difficult to find the correct
- balance between putting too much work on the custodian or too much
- work on an individual submitting a patch when something does not
- apply cleanly.
-
Review Process, Git Tags
------------------------
@@ -232,6 +191,47 @@ document.
For example, when your change affects a specific board or driver, then makes
a lot of sense to put the respective maintainer of this code on Cc:
+.. _custodians:
+
+Custodians
+----------
+
+The Custodians take responsibility for some area of the U-Boot code. The
+in-tree ``MAINTAINERS`` files list who is responsible for which areas.
+
+It is their responsibility to pick up patches from the mailing list
+that fall into their responsibility, and to process these.
+
+A very important responsibility of each custodian is to provide
+feedback to the submitter of a patch about what is going on:
+
+ * If the patch was accepted, or if it was rejected (with exact list
+ of reasons), if it needs to be reworked (with respective review
+ comments). Even a "I have no time now, will look into it later"
+ message is better than nothing. Also, if there are remarks to a
+ patch, these should leave no doubt if they were just comments and
+ the patch will be accepted anyway, or if the patch should be
+ reworked/resubmitted, or if it was rejected. However, if a submitter
+ feels it has been too long since posting their patch and not
+ received any feedback, it is OK to follow-up and ask.
+
+ * A custodian may make changes suggested by :doc:`checkpatch.pl
+ <checkpatch>`. They must also in turn amend the commit message noting
+ their change, for example ``[trini: Fix typos]``, and add their own
+ :ref:`Signed-off-by <dco>` tag. All other changes must be handled by
+ another iteration of the patch, or follow-up patch.
+
+ * If the patch itself can still be applied to the tree. The custodian
+ is expected to put in a "best effort" if a patch does not apply
+ cleanly, but can be made to apply still. It is up to the custodian
+ to decide how recent of a commit the patch must be against. It is
+ acceptable to request patches against the last officially released
+ version of U-Boot or newer. Of course a custodian can also accept
+ patches against older code. It can be difficult to find the correct
+ balance between putting too much work on the custodian or too much
+ work on an individual submitting a patch when something does not
+ apply cleanly.
+
Work flow of a Custodian
------------------------
--
2.43.0
next prev parent reply other threads:[~2026-01-20 22:34 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-20 22:31 [PATCH 0/5] doc: develop: Document b4 for custodians Tom Rini
2026-01-20 22:31 ` Tom Rini [this message]
2026-01-21 10:55 ` [PATCH 1/5] doc: develop: process: Move Custodians section Quentin Schulz
2026-01-20 22:31 ` [PATCH 2/5] doc: develop: process: Make "Work flow of a Custodian" a subsection Tom Rini
2026-01-21 9:44 ` Michael Nazzareno Trimarchi
2026-01-21 14:51 ` Tom Rini
2026-01-21 10:57 ` Quentin Schulz
2026-01-20 22:31 ` [PATCH 3/5] doc: develop: codingstyle: Update b4 external link Tom Rini
2026-01-21 10:58 ` Quentin Schulz
2026-01-20 22:31 ` [PATCH 4/5] doc: develop: sending_patches: Update link to patchwork Tom Rini
2026-01-21 10:59 ` Quentin Schulz
2026-01-20 22:31 ` [PATCH 5/5] doc: develop: process: Document using b4 and patchwork for custodians Tom Rini
2026-01-21 11:16 ` Quentin Schulz
2026-01-21 15:06 ` Tom Rini
2026-01-26 13:17 ` Mattijs Korpershoek
2026-01-26 13:18 ` Mattijs Korpershoek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260120223405.469050-2-trini@konsulko.com \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox