Messages in this thread | | | Date | Tue, 09 Jun 2015 11:29:47 -0500 | From | Timur Tabi <> | Subject | Re: [PATCH v4 5/7] Watchdog: introduce ARM SBSA watchdog driver |
| |
On 06/09/2015 11:22 AM, Guenter Roeck wrote: > > > but I see your point. Essentially, the specification is broken > for all practical purposes, since, as you point out, enabling > the watchdog overwrites and explicitly sets WCV. Effectively > this means that just using WCV to program the timeout period > is not really possible. > > I am not really sure how to address this. We can either only use WOR, > and forget about pretimeout, or we can enforce a minimum pretimeout. > In the latter case, we'll have to write WCV after writing WOR.
In talking with our hardware engineers, using WCV to program the timeout period is not a valid operation. This is why I keep arguing against the pre-timeout feature, and I don't agree that servers should always use pre-timeout.
-- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
| |