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]> --- Changes in v2: - New patch. --- doc/develop/process.rst | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/doc/develop/process.rst b/doc/develop/process.rst index 6f5bdd3db957..775a855fd7a0 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 spelling corrections, spacing fixes and other + obviously correct trivial changes. They must also in turn amend + the commit message noting their change inside of ``[]`` and add + their own ``Signed-off-by`` 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 -- 2.43.0
