[PATCH] docs: arm: tcm: Fix typo in description of TCM and MMU usage

Jonathan Corbet corbet at lwn.net
Thu Jun 9 10:27:51 PDT 2022


Simon Horman <simon.horman at corigine.com> writes:

> On Thu, Jun 09, 2022 at 10:01:01AM -0600, Jonathan Corbet wrote:
>> Simon Horman <simon.horman at corigine.com> writes:
>> 
>> > Correct a typo in the description of interaction between
>> > the TCM and MMU.
>> >
>> > Found by inspection.
>> >
>> > Signed-off-by: Louis Peens <louis.peens at corigine.com>
>> > Signed-off-by: Simon Horman <simon.horman at corigine.com>
>> 
>> Here too I have to ask: what does this signoff chain mean?  If two
>> developers were needed for a single-character typo fix, then a
>> Co-developed-by line is appropriate.  If you are forwarding a patch from
>> Louis, then a From: line to get the authorship right is indicated ... ?
>
> The patch was created by me.
>
> Due to internal processes a release by a colleague is provided, that is the
> purpose of Louis's sign off (overkill in this case, I agree).
>
> If Louis's sign-off is not necessary from your perspective then I'm happy
> for you to drop it or for my to repost the patch without it.

So are you perhaps using it like a Reviewed-by?  If so, that's the tag
that should be used.  But Signed-off-by has a well-defined meaning
described by the DCO, and "release by a colleague" isn't in there, so
I think that tag is not appropriate in this patch.

Thanks,

jon



More information about the linux-arm-kernel mailing list