On Tue, Feb 03, 2026 at 10:29:48AM +0530, Amitabh Srivastava wrote:
> docs: process: maintainer-pgp-guide.rst: updated the http link for
I missed this the last time, but you don't need to duplicate the "docs:
process:..." line here.
> 'add the kernel.org UID to your key'. Added SPDX-License-Identifier
> to fix checkpatch.pl warning. Tested kernel using qemu-system-aarch64
You don't need to mention how you tested it (great that you did, but it
doesn't need to be in the commit).
> with busybox inside a multipass ubuntu VM on macos.
I suggest that your submission should look like this:
Subject: docs: process: maintainer-pgp-guide: update kernel.org docs link
Update http link to the documentation about how to add a kernel.org UID to
the maintainer's key. Add missing SPDX-License-Identifier to fix a
checkpatch warning.
Signed-off-by: ...
This has the following improvements:
- removes duplicate info that is already in the subject
- uses imperative mood that is required for commits (update vs updated)
see:
https://docs.kernel.org/process/submitting-patches.html#describe-your-changes
> Signed-off-by: Amitabh Srivastava <[email protected]>
> Reviewed-by: Konstantin Ryabitsev <[email protected]>
You shouldn't do this -- I didn't actually reply with a "Reviewed-by"
trailer. These trailers are given by the reviewer to indicate "I have reviewed
this and didn't find any problems with the patch. You should only add trailers
that you have actually received from reviewers.
Please send a v3 with these fixes and that should earn you a "Reviewed-by."
Best regards,
--
KR
