Messages in this thread | | | Date | Thu, 11 Oct 2018 14:43:33 +0200 | From | Niklas Cassel <> | Subject | Re: [PATCH v8 2/2] cpufreq: qcom-hw: Add support for QCOM cpufreq HW driver |
| |
On Thu, Oct 11, 2018 at 04:45:25PM +0530, Taniya Das wrote:
> > WARNING: Duplicate signature > > #370: > > Signed-off-by: Taniya Das <tdas@codeaurora.org> > > > > If you are more than one author, you can use the Co-Developed-by tag. > > > > Hmm, I do see multiple SOBs being used and merged. >
From Documentation/process/submitting-patches.rst:
""" A Co-Developed-by: states that the patch was also created by another developer along with the original author. This is useful at times when multiple people work on a single patch. Note, this person also needs to have a Signed-off-by: line in the patch as well. """
So it is not wrong to have two Signed-off-bys, just that having the Co-Developed-by tag clearly shows that there was more than one author.
Without the Co-Developed-by tag, the second Signed-off-by tag is ambiguous, since the seconds Signed-off-by tag might just have meant that that person was in the delivery chain of the patch.
I think that the Co-Developed-by tag is optional, so if the co-developer doesn't care about taking credit, it can probably be skipped.
Kind regards, Niklas
| |