[PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv

E Shattow posted 2 patches 1 week, 1 day ago
[PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by E Shattow 1 week, 1 day ago
From: Icenowy Zheng <uwu@icenowy.me>

Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based board.

Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
Signed-off-by: E Shattow <e@freeshell.de>
---
 Documentation/devicetree/bindings/riscv/starfive.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml
index 04510341a71e..3fdeab9d8f08 100644
--- a/Documentation/devicetree/bindings/riscv/starfive.yaml
+++ b/Documentation/devicetree/bindings/riscv/starfive.yaml
@@ -33,6 +33,7 @@ properties:
               - pine64,star64
               - starfive,visionfive-2-v1.2a
               - starfive,visionfive-2-v1.3b
+              - xunlong,orangepi-rv
           - const: starfive,jh7110
 
 additionalProperties: true
-- 
2.50.0
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Krzysztof Kozlowski 1 week ago
On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
> From: Icenowy Zheng <uwu@icenowy.me>
> 
> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based board.
> 
> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
> Signed-off-by: E Shattow <e@freeshell.de>

<form letter>
This is a friendly reminder during the review process.

It looks like you received a tag and forgot to add it.

If you do not know the process, here is a short explanation:
Please add Acked-by/Reviewed-by/Tested-by tags when posting new
versions of patchset, under or above your Signed-off-by tag, unless
patch changed significantly (e.g. new properties added to the DT
bindings). Tag is "received", when provided in a message replied to you
on the mailing list. Tools like b4 can help here. However, there's no
need to repost patches *only* to add the tags. The upstream maintainer
will do that for tags received on the version they apply.

Please read:
https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577

If a tag was not added on purpose, please state why and what changed.
</form letter>
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Conor Dooley 1 week ago
On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski wrote:
> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
> > From: Icenowy Zheng <uwu@icenowy.me>
> > 
> > Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based board.
> > 
> > Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
> > Signed-off-by: E Shattow <e@freeshell.de>
> 
> <form letter>
> This is a friendly reminder during the review process.
> 
> It looks like you received a tag and forgot to add it.

It's from me, don't resubmit just to add it since it'll be me applying
anyway.

> 
> If you do not know the process, here is a short explanation:
> Please add Acked-by/Reviewed-by/Tested-by tags when posting new
> versions of patchset, under or above your Signed-off-by tag, unless
> patch changed significantly (e.g. new properties added to the DT
> bindings). Tag is "received", when provided in a message replied to you
> on the mailing list. Tools like b4 can help here. However, there's no
> need to repost patches *only* to add the tags. The upstream maintainer
> will do that for tags received on the version they apply.
> 
> Please read:
> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
> 
> If a tag was not added on purpose, please state why and what changed.
> </form letter>
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by E Shattow 1 week ago
On 11/24/25 05:22, Conor Dooley wrote:
> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski wrote:
>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>
>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based board.
>>>
>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>> Signed-off-by: E Shattow <e@freeshell.de>
>>
>> <form letter>
>> This is a friendly reminder during the review process.
>>
>> It looks like you received a tag and forgot to add it.
> 
> It's from me, don't resubmit just to add it since it'll be me applying
> anyway.
> 

Hi Conor,

Okay. Yes I'd dropped the tag since the commit message is appreciably
different, and you would be handling it again anyways. Thanks! And thank
you Krzysztof for the reminder -E

>>
>> If you do not know the process, here is a short explanation:
>> Please add Acked-by/Reviewed-by/Tested-by tags when posting new
>> versions of patchset, under or above your Signed-off-by tag, unless
>> patch changed significantly (e.g. new properties added to the DT
>> bindings). Tag is "received", when provided in a message replied to you
>> on the mailing list. Tools like b4 can help here. However, there's no
>> need to repost patches *only* to add the tags. The upstream maintainer
>> will do that for tags received on the version they apply.
>>
>> Please read:
>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
>>
>> If a tag was not added on purpose, please state why and what changed.
>> </form letter>
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Krzysztof Kozlowski 6 days, 17 hours ago
On 24/11/2025 22:59, E Shattow wrote:
> 
> On 11/24/25 05:22, Conor Dooley wrote:
>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski wrote:
>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>>
>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based board.
>>>>
>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>>> Signed-off-by: E Shattow <e@freeshell.de>
>>>
>>> <form letter>
>>> This is a friendly reminder during the review process.
>>>
>>> It looks like you received a tag and forgot to add it.
>>
>> It's from me, don't resubmit just to add it since it'll be me applying
>> anyway.
>>
> 
> Hi Conor,
> 
> Okay. Yes I'd dropped the tag since the commit message is appreciably
> different, and you would be handling it again anyways. Thanks! And thank
> you Krzysztof for the reminder -E


And where did you explain that you dropped the tag because of that?
Please read the form letter carefully.

Best regards,
Krzysztof
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Icenowy Zheng 6 days, 17 hours ago
在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
> On 24/11/2025 22:59, E Shattow wrote:
> > 
> > On 11/24/25 05:22, Conor Dooley wrote:
> > > On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
> > > wrote:
> > > > On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
> > > > > From: Icenowy Zheng <uwu@icenowy.me>
> > > > > 
> > > > > Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
> > > > > board.
> > > > > 
> > > > > Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
> > > > > Signed-off-by: E Shattow <e@freeshell.de>
> > > > 
> > > > <form letter>
> > > > This is a friendly reminder during the review process.
> > > > 
> > > > It looks like you received a tag and forgot to add it.
> > > 
> > > It's from me, don't resubmit just to add it since it'll be me
> > > applying
> > > anyway.
> > > 
> > 
> > Hi Conor,
> > 
> > Okay. Yes I'd dropped the tag since the commit message is
> > appreciably
> > different, and you would be handling it again anyways. Thanks! And
> > thank
> > you Krzysztof for the reminder -E
> 
> 
> And where did you explain that you dropped the tag because of that?
> Please read the form letter carefully.

Well I think there's no clear definition of "the patch has changed
substantially" here.

E may think for this such-short patch, the commit message weighs a lot
and the change to it is significant to the patch (e.g. making the patch
not clear enough).

> 
> Best regards,
> Krzysztof
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Krzysztof Kozlowski 6 days, 17 hours ago
On 25/11/2025 08:33, Icenowy Zheng wrote:
> 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
>> On 24/11/2025 22:59, E Shattow wrote:
>>>
>>> On 11/24/25 05:22, Conor Dooley wrote:
>>>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
>>>> wrote:
>>>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>>>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>>>>
>>>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
>>>>>> board.
>>>>>>
>>>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>>>>> Signed-off-by: E Shattow <e@freeshell.de>
>>>>>
>>>>> <form letter>
>>>>> This is a friendly reminder during the review process.
>>>>>
>>>>> It looks like you received a tag and forgot to add it.
>>>>
>>>> It's from me, don't resubmit just to add it since it'll be me
>>>> applying
>>>> anyway.
>>>>
>>>
>>> Hi Conor,
>>>
>>> Okay. Yes I'd dropped the tag since the commit message is
>>> appreciably
>>> different, and you would be handling it again anyways. Thanks! And
>>> thank
>>> you Krzysztof for the reminder -E
>>
>>
>> And where did you explain that you dropped the tag because of that?
>> Please read the form letter carefully.
> 
> Well I think there's no clear definition of "the patch has changed
> substantially" here.
> 
> E may think for this such-short patch, the commit message weighs a lot
> and the change to it is significant to the patch (e.g. making the patch
> not clear enough).

You still did not bother to read what we expect. I do not discuss if
this changed significantly or not, although it is obvious that it did
not change and tag should have been retained.

Look again:
<QUOTE>
Please read:
https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577

If a tag was not added on purpose, please state why and what changed.
</QUOTE>

Where did you explain that?


Best regards,
Krzysztof
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Icenowy Zheng 6 days, 17 hours ago
在 2025-11-25星期二的 08:48 +0100,Krzysztof Kozlowski写道:
> On 25/11/2025 08:33, Icenowy Zheng wrote:
> > 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
> > > On 24/11/2025 22:59, E Shattow wrote:
> > > > 
> > > > On 11/24/25 05:22, Conor Dooley wrote:
> > > > > On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
> > > > > wrote:
> > > > > > On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
> > > > > > > From: Icenowy Zheng <uwu@icenowy.me>
> > > > > > > 
> > > > > > > Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
> > > > > > > board.
> > > > > > > 
> > > > > > > Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
> > > > > > > Signed-off-by: E Shattow <e@freeshell.de>
> > > > > > 
> > > > > > <form letter>
> > > > > > This is a friendly reminder during the review process.
> > > > > > 
> > > > > > It looks like you received a tag and forgot to add it.
> > > > > 
> > > > > It's from me, don't resubmit just to add it since it'll be me
> > > > > applying
> > > > > anyway.
> > > > > 
> > > > 
> > > > Hi Conor,
> > > > 
> > > > Okay. Yes I'd dropped the tag since the commit message is
> > > > appreciably
> > > > different, and you would be handling it again anyways. Thanks!
> > > > And
> > > > thank
> > > > you Krzysztof for the reminder -E
> > > 
> > > 
> > > And where did you explain that you dropped the tag because of
> > > that?
> > > Please read the form letter carefully.
> > 
> > Well I think there's no clear definition of "the patch has changed
> > substantially" here.
> > 
> > E may think for this such-short patch, the commit message weighs a
> > lot
> > and the change to it is significant to the patch (e.g. making the
> > patch
> > not clear enough).
> 
> You still did not bother to read what we expect. I do not discuss if
> this changed significantly or not, although it is obvious that it did
> not change and tag should have been retained.
> 
> Look again:
> <QUOTE>
> Please read:
> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
> 
> If a tag was not added on purpose, please state why and what changed.
> </QUOTE>
> 
> Where did you explain that?

Sure, I agree that explaination of this should be delivered.

> 
> 
> Best regards,
> Krzysztof
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by E Shattow 6 days, 11 hours ago

On 11/25/25 00:01, Icenowy Zheng wrote:
> 在 2025-11-25星期二的 08:48 +0100,Krzysztof Kozlowski写道:
>> On 25/11/2025 08:33, Icenowy Zheng wrote:
>>> 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
>>>> On 24/11/2025 22:59, E Shattow wrote:
>>>>>
>>>>> On 11/24/25 05:22, Conor Dooley wrote:
>>>>>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
>>>>>> wrote:
>>>>>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>>>>>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>>
>>>>>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
>>>>>>>> board.
>>>>>>>>
>>>>>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>> Signed-off-by: E Shattow <e@freeshell.de>
>>>>>>>
>>>>>>> <form letter>
>>>>>>> This is a friendly reminder during the review process.
>>>>>>>
>>>>>>> It looks like you received a tag and forgot to add it.
>>>>>>
>>>>>> It's from me, don't resubmit just to add it since it'll be me
>>>>>> applying
>>>>>> anyway.
>>>>>>
>>>>>
>>>>> Hi Conor,
>>>>>
>>>>> Okay. Yes I'd dropped the tag since the commit message is
>>>>> appreciably
>>>>> different, and you would be handling it again anyways. Thanks!
>>>>> And
>>>>> thank
>>>>> you Krzysztof for the reminder -E
>>>>
>>>>
>>>> And where did you explain that you dropped the tag because of
>>>> that?
>>>> Please read the form letter carefully.
>>>
>>> Well I think there's no clear definition of "the patch has changed
>>> substantially" here.
>>>
>>> E may think for this such-short patch, the commit message weighs a
>>> lot
>>> and the change to it is significant to the patch (e.g. making the
>>> patch
>>> not clear enough).
>>
>> You still did not bother to read what we expect. I do not discuss if
>> this changed significantly or not, although it is obvious that it did
>> not change and tag should have been retained.
>>
>> Look again:
>> <QUOTE>
>> Please read:
>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
>>
>> If a tag was not added on purpose, please state why and what changed.
>> </QUOTE>
>>
>> Where did you explain that?
> 
> Sure, I agree that explaination of this should be delivered.
> 
>>
>>
>> Best regards,
>> Krzysztof
> 

Krzysztof,

The submitting-patches document does not support your assertion in this
situation. I evidently have a different understanding of than you, as
when I read it before sending the series I did arrive at the action of:

1. Acked-by is less formal and so it is a matter of opinion and
judgement whether to retain it or not. I choose not to here due to
changes visible in the commit log (the commit messages and titles). I
would let Conor decide if they will acknowledge the revised series
instead of making that decision for them.

2. I am not required to mention the drop of any Acked-by tag in the
cover letter for this situation. Conor is handling this series so I do
need to participate on the mailing list when there's a question or
comment about picking up the lack of Acked-by.

I am in the understanding at the exchange between Conor and I that this
concern of yours will be resolved without my input depending on the
review from Emil. Let us know what/if more specific and actionable needs
done and whom by you expect it to be done.

Thank you,

-E
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Krzysztof Kozlowski 5 days, 16 hours ago
On 25/11/2025 14:07, E Shattow wrote:
> 
> 
> On 11/25/25 00:01, Icenowy Zheng wrote:
>> 在 2025-11-25星期二的 08:48 +0100,Krzysztof Kozlowski写道:
>>> On 25/11/2025 08:33, Icenowy Zheng wrote:
>>>> 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
>>>>> On 24/11/2025 22:59, E Shattow wrote:
>>>>>>
>>>>>> On 11/24/25 05:22, Conor Dooley wrote:
>>>>>>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
>>>>>>> wrote:
>>>>>>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>>>>>>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>>>
>>>>>>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
>>>>>>>>> board.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>>> Signed-off-by: E Shattow <e@freeshell.de>
>>>>>>>>
>>>>>>>> <form letter>
>>>>>>>> This is a friendly reminder during the review process.
>>>>>>>>
>>>>>>>> It looks like you received a tag and forgot to add it.
>>>>>>>
>>>>>>> It's from me, don't resubmit just to add it since it'll be me
>>>>>>> applying
>>>>>>> anyway.
>>>>>>>
>>>>>>
>>>>>> Hi Conor,
>>>>>>
>>>>>> Okay. Yes I'd dropped the tag since the commit message is
>>>>>> appreciably
>>>>>> different, and you would be handling it again anyways. Thanks!
>>>>>> And
>>>>>> thank
>>>>>> you Krzysztof for the reminder -E
>>>>>
>>>>>
>>>>> And where did you explain that you dropped the tag because of
>>>>> that?
>>>>> Please read the form letter carefully.
>>>>
>>>> Well I think there's no clear definition of "the patch has changed
>>>> substantially" here.
>>>>
>>>> E may think for this such-short patch, the commit message weighs a
>>>> lot
>>>> and the change to it is significant to the patch (e.g. making the
>>>> patch
>>>> not clear enough).
>>>
>>> You still did not bother to read what we expect. I do not discuss if
>>> this changed significantly or not, although it is obvious that it did
>>> not change and tag should have been retained.
>>>
>>> Look again:
>>> <QUOTE>
>>> Please read:
>>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
>>>
>>> If a tag was not added on purpose, please state why and what changed.
>>> </QUOTE>
>>>
>>> Where did you explain that?
>>
>> Sure, I agree that explaination of this should be delivered.
>>
>>>
>>>
>>> Best regards,
>>> Krzysztof
>>
> 
> Krzysztof,
> 
> The submitting-patches document does not support your assertion in this

I sent patch to fix that because that was too literal. Ack has the same
rules as Rb.

> situation. I evidently have a different understanding of than you, as
> when I read it before sending the series I did arrive at the action of:
> 
> 1. Acked-by is less formal and so it is a matter of opinion and
> judgement whether to retain it or not. I choose not to here due to
> changes visible in the commit log (the commit messages and titles). I
> would let Conor decide if they will acknowledge the revised series
> instead of making that decision for them.

I value my time and you dropping the tag causes the patch appearing for
our review AGAIN. Doing review again without any explanation from
submitter feels like someone is not respecting our time, therefore,
since you keep disagreeing on that matter, I will switch to patches from
other authors.


Best regards,
Krzysztof
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by Conor Dooley 6 days, 5 hours ago
On Tue, Nov 25, 2025 at 05:07:00AM -0800, E Shattow wrote:
> 
> 
> On 11/25/25 00:01, Icenowy Zheng wrote:
> > 在 2025-11-25星期二的 08:48 +0100,Krzysztof Kozlowski写道:
> >> On 25/11/2025 08:33, Icenowy Zheng wrote:
> >>> 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
> >>>> On 24/11/2025 22:59, E Shattow wrote:
> >>>>>
> >>>>> On 11/24/25 05:22, Conor Dooley wrote:
> >>>>>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
> >>>>>> wrote:
> >>>>>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
> >>>>>>>> From: Icenowy Zheng <uwu@icenowy.me>
> >>>>>>>>
> >>>>>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
> >>>>>>>> board.
> >>>>>>>>
> >>>>>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
> >>>>>>>> Signed-off-by: E Shattow <e@freeshell.de>
> >>>>>>>
> >>>>>>> <form letter>
> >>>>>>> This is a friendly reminder during the review process.
> >>>>>>>
> >>>>>>> It looks like you received a tag and forgot to add it.
> >>>>>>
> >>>>>> It's from me, don't resubmit just to add it since it'll be me
> >>>>>> applying
> >>>>>> anyway.
> >>>>>>
> >>>>>
> >>>>> Hi Conor,
> >>>>>
> >>>>> Okay. Yes I'd dropped the tag since the commit message is
> >>>>> appreciably
> >>>>> different, and you would be handling it again anyways. Thanks!
> >>>>> And
> >>>>> thank
> >>>>> you Krzysztof for the reminder -E
> >>>>
> >>>>
> >>>> And where did you explain that you dropped the tag because of
> >>>> that?
> >>>> Please read the form letter carefully.
> >>>
> >>> Well I think there's no clear definition of "the patch has changed
> >>> substantially" here.
> >>>
> >>> E may think for this such-short patch, the commit message weighs a
> >>> lot
> >>> and the change to it is significant to the patch (e.g. making the
> >>> patch
> >>> not clear enough).
> >>
> >> You still did not bother to read what we expect. I do not discuss if
> >> this changed significantly or not, although it is obvious that it did
> >> not change and tag should have been retained.
> >>
> >> Look again:
> >> <QUOTE>
> >> Please read:
> >> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
> >>
> >> If a tag was not added on purpose, please state why and what changed.
> >> </QUOTE>
> >>
> >> Where did you explain that?
> > 
> > Sure, I agree that explaination of this should be delivered.
> > 
> >>
> >>
> >> Best regards,
> >> Krzysztof
> > 
> 
> Krzysztof,
> 
> The submitting-patches document does not support your assertion in this
> situation. I evidently have a different understanding of than you, as
> when I read it before sending the series I did arrive at the action of:
> 
> 1. Acked-by is less formal and so it is a matter of opinion and
> judgement whether to retain it or not. I choose not to here due to
> changes visible in the commit log (the commit messages and titles). I
> would let Conor decide if they will acknowledge the revised series
> instead of making that decision for them.

If anything, acked-by being "less formal" would mean that it's more
resistant to chances in the patch.

> 2. I am not required to mention the drop of any Acked-by tag in the
> cover letter for this situation. Conor is handling this series so I do
> need to participate on the mailing list when there's a question or
> comment about picking up the lack of Acked-by.

If I were you, I would pretty much always mention the reason for
dropping tags unless the changelog or cover letter contains some pretty
drastic changes between versions thereby making it really obvious why
the tags are gone.
Re: [PATCH v3 1/2] dt-bindings: riscv: starfive: add xunlong,orangepi-rv
Posted by E Shattow 5 days, 4 hours ago
On 11/25/25 11:14, Conor Dooley wrote:
> On Tue, Nov 25, 2025 at 05:07:00AM -0800, E Shattow wrote:
>>
>>
>> On 11/25/25 00:01, Icenowy Zheng wrote:
>>> 在 2025-11-25星期二的 08:48 +0100,Krzysztof Kozlowski写道:
>>>> On 25/11/2025 08:33, Icenowy Zheng wrote:
>>>>> 在 2025-11-25星期二的 08:28 +0100,Krzysztof Kozlowski写道:
>>>>>> On 24/11/2025 22:59, E Shattow wrote:
>>>>>>>
>>>>>>> On 11/24/25 05:22, Conor Dooley wrote:
>>>>>>>> On Mon, Nov 24, 2025 at 08:28:10AM +0100, Krzysztof Kozlowski
>>>>>>>> wrote:
>>>>>>>>> On Sun, Nov 23, 2025 at 02:50:44PM -0800, E Shattow wrote:
>>>>>>>>>> From: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>>>>
>>>>>>>>>> Add "xunlong,orangepi-rv" as a StarFive JH7110 SoC-based
>>>>>>>>>> board.
>>>>>>>>>>
>>>>>>>>>> Signed-off-by: Icenowy Zheng <uwu@icenowy.me>
>>>>>>>>>> Signed-off-by: E Shattow <e@freeshell.de>
>>>>>>>>>
>>>>>>>>> <form letter>
>>>>>>>>> This is a friendly reminder during the review process.
>>>>>>>>>
>>>>>>>>> It looks like you received a tag and forgot to add it.
>>>>>>>>
>>>>>>>> It's from me, don't resubmit just to add it since it'll be me
>>>>>>>> applying
>>>>>>>> anyway.
>>>>>>>>
>>>>>>>
>>>>>>> Hi Conor,
>>>>>>>
>>>>>>> Okay. Yes I'd dropped the tag since the commit message is
>>>>>>> appreciably
>>>>>>> different, and you would be handling it again anyways. Thanks!
>>>>>>> And
>>>>>>> thank
>>>>>>> you Krzysztof for the reminder -E
>>>>>>
>>>>>>
>>>>>> And where did you explain that you dropped the tag because of
>>>>>> that?
>>>>>> Please read the form letter carefully.
>>>>>
>>>>> Well I think there's no clear definition of "the patch has changed
>>>>> substantially" here.
>>>>>
>>>>> E may think for this such-short patch, the commit message weighs a
>>>>> lot
>>>>> and the change to it is significant to the patch (e.g. making the
>>>>> patch
>>>>> not clear enough).
>>>>
>>>> You still did not bother to read what we expect. I do not discuss if
>>>> this changed significantly or not, although it is obvious that it did
>>>> not change and tag should have been retained.
>>>>
>>>> Look again:
>>>> <QUOTE>
>>>> Please read:
>>>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
>>>>
>>>> If a tag was not added on purpose, please state why and what changed.
>>>> </QUOTE>
>>>>
>>>> Where did you explain that?
>>>
>>> Sure, I agree that explaination of this should be delivered.
>>>
>>>>
>>>>
>>>> Best regards,
>>>> Krzysztof
>>>
>>
>> Krzysztof,
>>
>> The submitting-patches document does not support your assertion in this
>> situation. I evidently have a different understanding of than you, as
>> when I read it before sending the series I did arrive at the action of:
>>
>> 1. Acked-by is less formal and so it is a matter of opinion and
>> judgement whether to retain it or not. I choose not to here due to
>> changes visible in the commit log (the commit messages and titles). I
>> would let Conor decide if they will acknowledge the revised series
>> instead of making that decision for them.
> 
> If anything, acked-by being "less formal" would mean that it's more
> resistant to chances in the patch.
> 

That is an interesting "read between the lines" opposite meaning of
formality than what I understand it to be. I'm making my internal mental
note, weird, but okay.

>> 2. I am not required to mention the drop of any Acked-by tag in the
>> cover letter for this situation. Conor is handling this series so I do
>> need to participate on the mailing list when there's a question or
>> comment about picking up the lack of Acked-by.
> 
> If I were you, I would pretty much always mention the reason for
> dropping tags unless the changelog or cover letter contains some pretty
> drastic changes between versions thereby making it really obvious why
> the tags are gone.

Sounds good to me and I will comply. I did infer this from the initial
form-letter reply by Krzysztof that something had run afoul of norms. No
problem for me to explain my understanding so it can be worked out what
needs to happen, and so thank you for the clarification. This was not
the time-saving noise reduction I intended for.

-E