<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, 17 Feb 2026 at 09:12, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org">krzk@kernel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 17/02/2026 07:53, Tomer Maimon wrote:<br>
> On Mon, 16 Feb 2026 at 20:03, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>> wrote:<br>
> <br>
>> On Mon, Feb 16, 2026 at 04:59:18PM +0200, Tomer Maimon wrote:<br>
>>> On Mon, 16 Feb 2026 at 16:48, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>><br>
>> wrote:<br>
>>>>>>>>> +  syscon:<br>
>>>>>>>><br>
>>>>>>>> First iteration. See "How to Get Your DT Schema Bindings Accepted<br>
>> in<br>
>>>>>>>> Less Than 10 Iterations"<br>
>>>>>>>><br>
>>>>>>> Thanks, it was very helpful.<br>
>>>>>>> the syscon property is already found in the WD node<br>
>>>>>>> in nuvoton-common-npcm8xx.dtsi file, what should I do:<br>
>>>>>><br>
>>>>>> How is that file related to this binding?<br>
>>>>>><br>
>>>>>> Either you document existing ABI or you add new (for new device).<br>
>> Commit<br>
>>>>>> msg MUST be explicit about it and provide the reasons. If wrong<br>
>> (e.g.<br>
>>>>>> discouraged) ABI was already used then it depends how and when it<br>
>> got<br>
>>>>>> into the kernel, e.g. if someone bypassed DT completely just to get<br>
>> it<br>
>>>>>> inside.<br>
>>>>>><br>
>>>>> The syscon property is already used in the upstream NPCM8xx DTSI<br>
>> watchdog<br>
>>>>> node, so I will document it as existing ABI and mark it deprecated. I<br>
>>>> will<br>
>>>><br>
>>>> And how it is used? I cannot find its usage, so I do not agree on<br>
>>>> documenting it. Property should be removed or at least provide the<br>
>>>> justification/impact of removal, if you need it to stay.<br>
>>>><br>
>>>> Understood. The syscon phandle is used by the watchdog driver to read<br>
>> and<br>
>><br>
>> You messed up quotes.<br>
>><br>
>> Can you point me to the line? I REALLY want to be sure that we are not<br>
>> wasting each other time, e.g. me looking at wrong code or you telling me<br>
>> some bollocks from downstream.<br>
>><br>
> The syscon property is not used in the current upstream npcm_wdt driver.<br>
> It is used *in this patch set*, which introduces the function<br>
<br>
Why would that matter for the ABI?<br>
<br>
So no, your buggy DTS sneaked into the kernel before submitting bindings<br>
is not acceptable thus ABI which was NEVER reviewed must be removed.</blockquote><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">It will be removed from the DTSI.<br></font></div><div><font face="arial, sans-serif"><span style="font-size:14px">I’ll send a new patch set that adds proper reset‑indication support using a new </span><code style="font-size:14px">nuvoton,sysgcr</code><span style="font-size:14px"> phandle.</span> </font></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
Best regards,<br>
Krzysztof<br></blockquote><div><br></div><div>Best regards,</div><div><br></div><div>Tomer </div></div></div>