[PATCH v1 4/4] watchdog: npcm: sets card ext1 and ext2 bootstatus during probe

[Tomer Maimon]

I deeply sorry Guenter,

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)
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.

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.

Sorry again.

Tomer

On Sun, 1 Mar 2020 at 18:10, Guenter Roeck <linux at roeck-us.net> wrote:

> On 3/1/20 8:08 AM, Tomer Maimon wrote:
> > Sorry Guenter probebly I didnt explain it well.
> >
> >
> > On Sun, 1 Mar 2020 at 12:48, Guenter Roeck <linux at roeck-us.net <mailto:
> linux at roeck-us.net>> wrote:
> >
> >     On 3/1/20 1:40 AM, Tomer Maimon wrote:
> >     > During probe NPCM watchdog sets the following bootstatus flags:
> >     >       - WDIOF_CARDRESET represent power and core reset.
> >     >       - WDIOF_EXTERN1 represent watchdog 0-2 reset.
> >     >       - WDIOF_EXTERN2 represent software 1-4 reset.
> >     >
> >     > Each flag is representing a group of bootstatus.
> >     > The user can configure through the device treethe exact reset
> >     > to each flag group.
> >     >
> >
> >     Sorry, this doesn't make sense to me. I could understand reporting
> >     the above, but it looks to me like devicetree is used to associate
> >     a reset bit from the controller with one of the above.
> >     Devicetree only seems to be used to associate reset status bits
> >     from the controller with WDIOF_CARDRESET, WDIOF_EXTERN1, or
> >     WDIOF_EXTERN2. That adds a lot of complexity for little if any
> >     gain.
> >
> >
> >
> >     It would make sense to set the bootstatus bits as suggested above,
> >     but that doesn't require devicetree properties.
> >
> >     More comments inline.
> >
> >     Guenter
> >
> >
> >
> > In the NPCM750 we have the following reset types:
> >
> >  1. board reset (Power on reset, Core reset)
> >  2. WD reset (0-2 WD reset).
> >  3. SW reset (1-4 SW reset).
> >
> >
> > Each board can use different reset types, because in the WD status bit
> there is not enough bits to represent the entire NPCM750 resets.
> >
> > The NPCM750 reset groups are represent as follow:
> >
> >  - WDIOF_CARDRESET represent power and core reset.
> >  - WDIOF_EXTERN1 represent watchdog 0-2 reset.
> >  - WDIOF_EXTERN2 represent software 1-4 reset.
> >
> Exactly, and I don't see a need to be more specific than that.
> This can be implemented without all the DT complexity.
>
> Guenter
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200301/32dddf49/attachment-0001.htm>