Re: [PATCH v3 5/6] Watchdog: introduce ARM SBSA watchdog driver

[Guenter Roeck]

On 05/29/2015 02:11 AM, Fu Wei wrote:
[ ... ]

> > > +
> > > +       status = sbsa_gwdt_cf_read(SBSA_GWDT_WCS, wdd);
> > > +       if (status & SBSA_GWDT_WCS_WS1) {
> > > +               dev_warn(dev, "System reset by WDT(WCS: %x, WCV: %llx)\n",
> > > +                        status, sbsa_gwdt_get_wcv(wdd));
> >
> >
> > Does this message (specifically the WCS / WCV values) have any
> > useful meaning for the user ?
>
> I think so, according to SBSA spec:
> If WS0 is asserted and a timeout refresh occurs then the following must occur:
>      If the system is compliant to SBSA level 2 or higher the compare
> value must retain its current value. This means that the compare value
> records the time that WS1 is asserted.
>
> So, I think WCV can log the time when system reset by WDT, that may
> help user figure out the problem. but WCS can be delete I think.

How would that help ? It is just a number with no reference to anything.

Thanks,
Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/