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

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

quoted

quoted

+
+       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