diff options
| author | Tom Rini <[email protected]> | 2026-01-20 14:11:45 -0600 |
|---|---|---|
| committer | Heinrich Schuchardt <[email protected]> | 2026-01-28 21:12:04 +0100 |
| commit | 7db33677c29ee494739557b5d3e8c1f48b3936e7 (patch) | |
| tree | 9c3f97f02446abfc375f838d312ab31694113a09 /doc/develop | |
| parent | 78e100db5b2354329664fd06fdac870f1ef59c3f (diff) | |
doc: develop: process: Explain when/how Custodians may edit patches
As seen with commit d503633a3676 ("Revert "doc: board: starfive: update
jh7110 common description""), it has not always been clear what is and
isn't allowed by custodians, and what the expectations are. To prevent
further unintentional conflicts, document the limited cases where
custodians are allowed to modify patches directly, and how to do that.
Signed-off-by: Tom Rini <[email protected]>
Reviewed-by: Quentin Schulz <[email protected]>
Diffstat (limited to 'doc/develop')
| -rw-r--r-- | doc/develop/process.rst | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/doc/develop/process.rst b/doc/develop/process.rst index 4159a945707..0651a1c23a4 100644 --- a/doc/develop/process.rst +++ b/doc/develop/process.rst @@ -144,6 +144,12 @@ feedback to the submitter of a patch about what is going on: 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 |