<div dir="ltr">I deeply sorry Guenter,<div><br></div><div>Each of the bit boot status have the maximum flexibility to represent the entire resets (I will make sure to modify the commit message as in the device tree document)</div><div>So if the user using WD0 to restart the board and WD1 for a board health monitor it can be represented at EXT1 and EXT2.</div><div><br></div><div>If I will use a group for each reset type in a case above the user will need to read the RESSR register (prefer not to do it)  to know which reset occur.</div><div><br></div><div>Sorry again.</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 18:10, 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 8:08 AM, Tomer Maimon wrote:<br>
> Sorry Guenter probebly I didnt explain it well.<br>
> <br>
> <br>
> On Sun, 1 Mar 2020 at 12:48, 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>
>     > During probe NPCM watchdog sets the following bootstatus flags:<br>
>     >       - WDIOF_CARDRESET represent power and core reset.<br>
>     >       - WDIOF_EXTERN1 represent watchdog 0-2 reset.<br>
>     >       - WDIOF_EXTERN2 represent software 1-4 reset.<br>
>     ><br>
>     > Each flag is representing a group of bootstatus.<br>
>     > The user can configure through the device treethe exact reset<br>
>     > to each flag group.<br>
>     ><br>
> <br>
>     Sorry, this doesn't make sense to me. I could understand reporting<br>
>     the above, but it looks to me like devicetree is used to associate<br>
>     a reset bit from the controller with one of the above.<br>
>     Devicetree only seems to be used to associate reset status bits<br>
>     from the controller with WDIOF_CARDRESET, WDIOF_EXTERN1, or<br>
>     WDIOF_EXTERN2. That adds a lot of complexity for little if any<br>
>     gain. <br>
> <br>
>      <br>
> <br>
>     It would make sense to set the bootstatus bits as suggested above,<br>
>     but that doesn't require devicetree properties.<br>
> <br>
>     More comments inline.<br>
> <br>
>     Guenter <br>
> <br>
>  <br>
> <br>
> In the NPCM750 we have the following reset types:<br>
> <br>
>  1. board reset (Power on reset, Core reset)<br>
>  2. WD reset (0-2 WD reset).<br>
>  3. SW reset (1-4 SW reset).<br>
> <br>
> <br>
> Each board can use different reset types, because in the WD status bit there is not enough bits to represent the entire NPCM750 resets.<br>
> <br>
> The NPCM750 reset groups are represent as follow:<br>
> <br>
>  - WDIOF_CARDRESET represent power and core reset.<br>
>  - WDIOF_EXTERN1 represent watchdog 0-2 reset.<br>
>  - WDIOF_EXTERN2 represent software 1-4 reset.<br>
> <br>
Exactly, and I don't see a need to be more specific than that.<br>
This can be implemented without all the DT complexity.<br>
<br>
Guenter<br>
</blockquote></div>