<div dir="ltr"><div dir="ltr">Hi Krzysztof,<div><br></div><div>Thanks for your review.</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, 10 Feb 2026 at 18:11, 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 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 a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml b/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<br>
> index 7aa30f5b5c49..054cc0115af2 100644<br>
> --- a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm750-wdt.yaml<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 watchdog.<br>
>    The watchdog supports a pre-timeout interrupt that fires 10ms before 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 timer.<br>
>  <br>
> +  syscon:<br>
<br>
First iteration. See "How to Get Your DT Schema Bindings Accepted in<br>
Less Than 10 Iterations"<br></blockquote><div>Thanks, it was very helpful.</div><div>the syscon property is already found in the WD node in nuvoton-common-npcm8xx.dtsi file, what should I do:</div><div>1. Modify the syson to nuvoton,sys-gcr like in the dtsi?</div><div>2. Still use the syscon property in the dtsi file, therefore stick with the syscon add.</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
or just read the docs please.<br>
<br>
> +    description: phandle to the Global Control Register (GCR) syscon node.<br>
> +    $ref: /schemas/types.yaml#/definitions/phandle<br>
> +<br>
> +  nuvoton,card-reset-type:<br>
> +    description: Reset type for external card reset signal detection.<br>
> +    enum:<br>
> +      - porst<br>
> +      - corst<br>
> +      - wd0<br>
> +      - wd1<br>
> +      - wd2<br>
> +      - sw1<br>
> +      - sw2<br>
> +      - sw3<br>
> +      - sw4<br>
<br>
You want it to be static configuration, so cannot be changed runtime? Why?<br></blockquote><div><div style="font-family:"Segoe UI";font-size:14px;line-height:20px">Yes, it is only an indication of the most recent reset in the BMC. In addition:</div></div><div><span style="font-family:"Segoe UI";font-size:14px">1. The driver reads the reset register during the probe. After this read, the reset register should be cleared so it’s ready for the next system reset.</span></div><div><font face="Segoe UI"><span style="font-size:14px">2. </span></font><span style="font-family:"Segoe UI";font-size:14px">I’m not aware of any service function that allows changing the reset indication at runtime.</span><font face="Segoe UI"><span style="font-size:14px"><br></span></font><font face="Segoe UI"><span style="font-size:14px">.</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>
> +  nuvoton,ext1-reset-type:<br>
> +    description: Reset type for external reset signal 1 detection.<br>
> +    enum:<br>
> +      - porst<br>
> +      - corst<br>
> +      - wd0<br>
> +      - wd1<br>
> +      - wd2<br>
> +      - sw1<br>
> +      - sw2<br>
> +      - sw3<br>
> +      - sw4<br>
> +<br>
> +  nuvoton,ext2-reset-type:<br>
> +    description: Reset type for external reset signal 2 detection.<br>
> +    enum:<br>
> +      - porst<br>
> +      - corst<br>
> +      - wd0<br>
> +      - wd1<br>
> +      - wd2<br>
> +      - sw1<br>
> +      - sw2<br>
> +      - sw3<br>
> +      - sw4<br>
> +<br>
>  required:<br>
>    - compatible<br>
>    - reg<br>
>    - interrupts<br>
>  <br>
> -unevaluatedProperties: false<br>
> +additionalProperties: false<br>
<br>
Why? Nothing explains this in the commit msg.<br></blockquote><div>Mistake, will back to unevaluatedProperties: false</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>
>  <br>
>  examples:<br>
>    - |<br>
> @@ -57,4 +100,8 @@ examples:<br>
>          interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;<br>
>          reg = <0xf000801c 0x4>;<br>
>          clocks = <&clk NPCM7XX_CLK_TIMER>;<br>
> +        syscon = <&gcr>;<br>
> +        nuvoton,card-reset-type = "porst";<br>
> +        nuvoton,ext1-reset-type = "wd0";<br>
> +        nuvoton,ext2-reset-type = "wd2";<br>
>      };<br>
<br>
<br>
Best regards,<br>
Krzysztof<br></blockquote><div><br></div><div>Best regards,</div><div><br></div><div>Tomer</div></div></div>