<div dir="ltr">Thank you for the clarification. <br><div><br></div><div>I will remove the priority patch, and send a new patch set after receiving comments for the boot status patch.</div><div><br></div><div>Tomer</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 1 Mar 2020 at 17:46, 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 7:36 AM, Tomer Maimon wrote:<br>
> <br>
> <br>
> On Sun, 1 Mar 2020 at 12:06, Guenter Roeck <<a href="mailto:linux@roeck-us.net" target="_blank">linux@roeck-us.net</a> <mailto:<a href="mailto:linux@roeck-us.net" target="_blank">linux@roeck-us.net</a>>> wrote:<br>
> <br>
>     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>
> <br>
> <br>
> In the NPCM750 there is two initiated restarts:<br>
> <br>
>   * Software reset<br>
>   * WD reset<br>
> <br>
> the Software restart found at NPCM reset driver<br>
> <a href="https://github.com/torvalds/linux/blob/master/drivers/reset/reset-npcm.c" rel="noreferrer" target="_blank">https://github.com/torvalds/linux/blob/master/drivers/reset/reset-npcm.c</a><br>
> <br>
> 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<br>
> <br>
<br>
This is not the intended use case for restart priority. It is not<br>
intended to be user configurable. The idea is that the more thorough<br>
restart gets higher priority. This is implied by the restart method,<br>
not by user preferences.<br>
<br>
Also, the idea behind supporting multiple means to reset the system<br>
is to be able to support multiple means to restart, some of which<br>
may not always be available. In that situation, the priority means,<br>
and is supposed to mean, "pick the best restart method available".<br>
Again, that is determined by system design, and not supposed to<br>
be configurable by the user.<br>
<br>
On top of that, a watchdog driver based reset is almost always<br>
a reset of last resort, to be chosen only if nothing else is available<br>
in a given system. The existence of the reset driver confirms that<br>
this is not different for this driver/chip.<br>
<br>
Guenter<br>
</blockquote></div>