<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, 16 Feb 2026 at 16:48, 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 16/02/2026 15:37, Tomer Maimon wrote:<br>
> Hi Krzysztof,<br>
> <br>
> Thanks for your comments.<br>
> <br>
> On Mon, 16 Feb 2026 at 13:16, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>> wrote:<br>
> <br>
>> On 16/02/2026 12:10, Tomer Maimon wrote:<br>
>>> Hi Krzysztof,<br>
>>><br>
>>> Thanks for your review.<br>
>>><br>
>>> On Tue, 10 Feb 2026 at 18:11, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>><br>
>> wrote:<br>
>>><br>
>>>> On 10/02/2026 14:38, Tomer Maimon wrote:<br>
>>>>> Add reset status detection for NPCM7XX and NPCM8XX platforms via syscon<br>
>>>>> integration. Document syscon property and three configurable reset type<br>
>>>>> properties (nuvoton,card-reset-type, nuvoton,ext1-reset-type,<br>
>>>>> nuvoton,ext2-reset-type)that map reset signal detection to specific<br>
>>>>> reset bit positions.<br>
>>>>><br>
>>>>> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
>>>>> ---<br>
>>>>>  .../watchdog/nuvoton,npcm750-wdt.yaml         | 51 ++++++++++++++++++-<br>
>>>>>  1 file changed, 49 insertions(+), 2 deletions(-)<br>
>>>>><br>
>>>>> diff --git<br>
>>>> a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<br>
>>>> b/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<br>
>>>>> index 7aa30f5b5c49..054cc0115af2 100644<br>
>>>>> ---<br>
>> a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<br>
>>>>> +++<br>
>> b/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<br>
>>>>> @@ -12,7 +12,7 @@ maintainers:<br>
>>>>>  description:<br>
>>>>>    Nuvoton NPCM timer module provides five 24-bit timer counters, and a<br>
>>>> watchdog.<br>
>>>>>    The watchdog supports a pre-timeout interrupt that fires 10ms before<br>
>>>> the<br>
>>>>> -  expiry.<br>
>>>>> +  expiry and reset status detection via syscon integration.<br>
>>>>><br>
>>>>>  allOf:<br>
>>>>>    - $ref: watchdog.yaml#<br>
>>>>> @@ -40,12 +40,55 @@ properties:<br>
>>>>>    clock-frequency:<br>
>>>>>      description: Frequency in Hz of the clock that drives the NPCM<br>
>>>> timer.<br>
>>>>><br>
>>>>> +  syscon:<br>
>>>><br>
>>>> First iteration. See "How to Get Your DT Schema Bindings Accepted 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). Commit<br>
>> msg MUST be explicit about it and provide the reasons. If wrong (e.g.<br>
>> discouraged) ABI was already used then it depends how and when it got<br>
>> into the kernel, e.g. if someone bypassed DT completely just to get it<br>
>> inside.<br>
>><br>
> The syscon property is already used in the upstream NPCM8xx DTSI watchdog<br>
> node, so I will document it as existing ABI and mark it deprecated. I 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></blockquote><div><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">Understood. The <code style="">syscon</code> phandle is used by the watchdog driver to read and clear the GCR reset‑status registers and then report the reset cause through the watchdog <code style="">bootstatus</code> bits. </font></div><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">Therefore, this property should appear in the binding only because the watchdog driver actually uses it — which I am implementing in this patch set. </font></div><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">I will document it accordingly, and also add the new <code style="">nuvoton,sysgcr</code> phandle as the preferred name.</font></div></div><div style="font-family:"Segoe UI";font-size:14px;line-height:20px"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> add a new vendor‑specific property (nuvoton,sysgcr) as the preferred one,<br>
> and explain this clearly in the commit message.<br>
> <br>
>><br>
>>> 1. Modify the syson to nuvoton,sys-gcr like in the dtsi?<br>
>>> 2. Still use the syscon property in the dtsi file, therefore stick with<br>
>> the<br>
>>> syscon add.<br>
<br>
<br>
Best regards,<br>
Krzysztof<br></blockquote><div><br></div><div>Best regards,</div><div><br></div><div>Tomer </div></div></div>