Re: [PATCH v6 0/3] Add Radxa CM5 and IO Board

[Krzysztof Kozlowski <krzk@kernel.org>]

On 14/11/2025 08:26, Dragan Simic wrote:
> On Friday, November 14, 2025 08:22 CET, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>> On 14/11/2025 08:17, Dragan Simic wrote:
>>> On Friday, November 14, 2025 08:10 CET, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>> On 14/11/2025 06:03, FUKAUMI Naoki wrote:
>>>>> On 11/12/25 09:46, Dragan Simic wrote:
>>>>>> On Wednesday, November 12, 2025 00:26 CET, FUKAUMI Naoki <naoki@radxa.com> wrote:
>>>>>>> On 11/11/25 23:33, Dragan Simic wrote:
>>>>>>>> On Tuesday, November 11, 2025 12:52 CET, FUKAUMI Naoki <naoki@radxa.com> wrote:
>>>>>>>>> On 11/6/25 02:48, Joseph Kogut wrote:
>>>>>>>>>> On Wed, Nov 5, 2025 at 4:15 AM FUKAUMI Naoki <naoki@radxa.com> wrote:
>>>>>>>>>>> I'd like to clarify the situation regarding the v6 patch series I submitted.
>>>>>>>>>>>
>>>>>>>>>>> The original device tree work for the Radxa CM5 and IO Board was
>>>>>>>>>>> authored by Joseph Kogut. I took over the responsibility of getting it
>>>>>>>>>>> upstreamed with his agreement.
>>>>>>>>>>
>>>>>>>>>> I'll confirm this. I've been in communication with Naoki. They made a
>>>>>>>>>> large number of revisions to my original patch series, which I think
>>>>>>>>>> have technical merit. I suggested they submit the patches themselves,
>>>>>>>>>> and gave them explicit permission to add my Signed-off-by and CC me.
>>>>>>>>>>
>>>>>>>>>> I assume this was the correct way for them to continue the work I
>>>>>>>>>> started, but if not, please let us know the best way to proceed.
>>>>>>>>>
>>>>>>>>> Can anyone help us?
>>>>>>>>
>>>>>>>> I'm not exactly sure how to resolve the current situation, but for
>>>>>>>> Signed-off-by tags to be present, in this case you'd need to have
>>>>>>>> Co-developed-by tags as well, because the final patch versions,
>>>>>>>> which would be submitted by Naoki, would differ significantly from
>>>>>>>> the versions that Joseph actively worked on, if I understood
>>>>>>>> everything correctly.  Though, for Joseph's Signed-off-by tags to
>>>>>>>> be included there, he would also need to participate actively in
>>>>>>>> the development of the final patch versions.
>>>>>>>
>>>>>>> https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by
>>>>>>>
>>>>>>> If
>>>>>>> ----
>>>>>>> From: Joseph Kogut <joseph.kogut@gmail.com>
>>>>>>>
>>>>>>> <changelog>
>>>>>>>
>>>>>>> Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com>
>>>>>>> Co-developed-by: FUKAUMI Naoki <naoki@radxa.com>
>>>>>>> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
>>>>>>> ----
>>>>>>> then I can submit my patch series?
>>>>>>
>>>>>> Actually, the Co-developed-by tags would be pointing to Joseph
>>>>>> in that case, but as I described it above, this approach basically
>>>>>> cannot be used, because Joseph's original work differs a lot from
>>>>>> what you'd actually submit to the mailing list(s).
>>>>>>
>>>>>>> Or,
>>>>>>>
>>>>>>>> Another option, technically a bit simpler, would be to include just
>>>>>>>> Originally-by tags for Joseph, which would indicate that Joseph gave
>>>>>>>> up on the development of the patches and handed them over to Naoki
>>>>>>>> for future development and submission to the mailing lists. Though,
>>>>>>>> that would require Joseph to publicly state exactly that.
>>>>>>>
>>>>>>> I cannot find any documentation about "Originally-by".
>>>>>>> Is this correct?
>>>>>>> ----
>>>>>>> <changelog>
>>>>>>>
>>>>>>> Originally-by: Joseph Kogut <joseph.kogut@gmail.com>
>>>>
>>>> There is no such tag. Don't invent tags.
>>>
>>> True, it doesn't exist officially, but it's been used fairly often.
>>>
>>>>>>> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
>>>>>>> ----
>>>>>>
>>>>>> That approach is the only I see applicable in this case.  However,
>>>>>> let's hear opinions from other people as well.
>>>>>
>>>>> I see. I want to do this approach.
>>>>>
>>>>> Joseph, could you give me a statement to do this?
>>>>
>>>> Use standard authorship and standard tags, some of which are explained
>>>> in Submitting patches.
>>>
>>> Frankly, your suggestion is useless, because it doesn't explain what 
>>> to do in this particular case.
>>
>> Maybe because you did not read the doc...
> 
> I think you already know the answer: I read it multiple times and 
> used it as a reference more than once.

Then if you read it and saw my objection to inventing tags, you could
guess that only first solution for patches with changes coming from
multiple authors is allowed. Also you would find from that doc, that
patches which were not changed - like in this patchset - must be
attributed to single author followed by SoB. So both cases nicely
explained. Pretty simple once you remove the invented tag option,
because it is really unnecessary.


Best regards,
Krzysztof