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

[Krzysztof Kozlowski <krzk@kernel.org>]

On 14/11/2025 09:51, Cristian Ciocaltea wrote:
> On 11/14/25 9:17 AM, Dragan Simic wrote:
>> Hello Krzysztof,
>>
>> 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.
> 
> Hmm, actually this tag seems to be documented, or at least given as an example:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/maintainer-tip.rst?h=v6.18-rc5#n309

Yes, for the tip folks (so three people). If you send to the TIP, their
maintainer profile applies. There are also other specific rules in TIP
which are in contrary to common (as most used) process, e.g. completely
reversed tags (see also Konstantin's explanation about the order which
he implemented in b4).

If you want to use it outside of tip, first this should be added to
common docs and checkpatch. Just like b4 should be changed if you want
to use their order of tags.

Best regards,
Krzysztof