<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 1 Mar 2020 at 12:06, Guenter Roeck <<a href="mailto:linux@roeck-us.net">linux@roeck-us.net</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 3/1/20 1:40 AM, Tomer Maimon wrote:<br>
> Add device tree restart priority documentation.<br>
> <br>
<br>
I think this warrants an explanation _why_ this is needed.<br>
What is the use case ? Not just theory, please.<br></blockquote><div><br></div><div>In the NPCM750 there is two <font face="arial, sans-serif"><span style="color:rgb(51,51,51);font-size:14px">initiated</span> </font>restarts:</div><div><ul><li>Software reset</li><li>WD reset</li></ul><div>the Software restart found at NPCM reset driver</div></div><div><a href="https://github.com/torvalds/linux/blob/master/drivers/reset/reset-npcm.c">https://github.com/torvalds/linux/blob/master/drivers/reset/reset-npcm.c</a><br></div><div><br></div><div>In NPCM WD driver the restart is configure as well, I will like to add the priority so the user will have maximum flexibility if he using both restarts</div><div><br></div><div>Thanks,</div><div><br></div><div>Tomer</div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Guenter<br>
<br>
> Signed-off-by: Tomer Maimon <<a href="mailto:tmaimon77@gmail.com" target="_blank">tmaimon77@gmail.com</a>><br>
> ---<br>
>  Documentation/devicetree/bindings/watchdog/nuvoton,npcm-wdt.txt | 2 ++<br>
>  1 file changed, 2 insertions(+)<br>
> <br>
> diff --git a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm-wdt.txt b/Documentation/devicetree/bindings/watchdog/nuvoton,npcm-wdt.txt<br>
> index 6d593003c933..0a0f86a25eb0 100644<br>
> --- a/Documentation/devicetree/bindings/watchdog/nuvoton,npcm-wdt.txt<br>
> +++ b/Documentation/devicetree/bindings/watchdog/nuvoton,npcm-wdt.txt<br>
> @@ -17,6 +17,7 @@ Required clocking property, have to be one of:<br>
>  <br>
>  Optional properties:<br>
>  - timeout-sec : Contains the watchdog timeout in seconds<br>
> +- nuvoton,restart-priority : Contains the card restart priority.<br>
>  <br>
>  Example:<br>
>  <br>
> @@ -25,4 +26,5 @@ timer@f000801c {<br>
>      interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;<br>
>      reg = <0xf000801c 0x4>;<br>
>      clocks = <&clk NPCM7XX_CLK_TIMER>;<br>
> +     nuvoton,restart-priority = <155>;<br>
>  };<br>
> <br>
<br>
</blockquote></div></div>