Krzysztof Kozlowski <[email protected]> writes: > The paragraph mentions only removal of Tested-by and Reviewed-by tags as > action needing mentioning in patch changelog, so some developers treat > it too literally. Acks, as a weaker form of review/approval, should > rarely be removed, but if that happens it should be explained as well. > > Signed-off-by: Krzysztof Kozlowski <[email protected]> > --- > Documentation/process/submitting-patches.rst | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Documentation/process/submitting-patches.rst > b/Documentation/process/submitting-patches.rst > index 910e8fc9e3c8..9a509f1a6873 100644 > --- a/Documentation/process/submitting-patches.rst > +++ b/Documentation/process/submitting-patches.rst > @@ -592,8 +592,9 @@ Both Tested-by and Reviewed-by tags, once received on > mailing list from tester > or reviewer, should be added by author to the applicable patches when sending > next versions. However if the patch has changed substantially in following > version, these tags might not be applicable anymore and thus should be > removed. > -Usually removal of someone's Tested-by or Reviewed-by tags should be > mentioned > -in the patch changelog (after the '---' separator). > +Usually removal of someone's Acked-by, Tested-by or Reviewed-by tags should > be > +mentioned in the patch changelog with an explanation (after the '---' > +separator).
Applied, thanks. jon
