Re: [PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-08-11 Thread Daniel Kurtz
On Tue, Aug 11, 2015 at 12:42 PM, Guenter Roeck wrote: > On 08/10/2015 09:28 PM, Daniel Kurtz wrote: >> >> Hi Guenter, Matthias, >> >> On Fri, Jul 24, 2015 at 1:37 AM, Matthias Brugger >> wrote: >>> >>> On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From: Greta Zhang >>

Re: [PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-08-10 Thread Guenter Roeck
On 08/10/2015 09:28 PM, Daniel Kurtz wrote: Hi Guenter, Matthias, On Fri, Jul 24, 2015 at 1:37 AM, Matthias Brugger wrote: On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From: Greta Zhang Without .shutdown(), watchdog might reset the system during power off. For example, if watch

Re: [PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-08-10 Thread Daniel Kurtz
Hi Guenter, Matthias, On Fri, Jul 24, 2015 at 1:37 AM, Matthias Brugger wrote: > On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: >> From: Greta Zhang >> >> Without .shutdown(), watchdog might reset the system during power off. >> For example, if watchdog's timeout is set to 30s, then i

Re: [PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-07-23 Thread Matthias Brugger
On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: > From: Greta Zhang > > Without .shutdown(), watchdog might reset the system during power off. > For example, if watchdog's timeout is set to 30s, then it is reset to > zero by mtk_wdt_ping(). During power off, no app will ping watchdog, >

Re: [PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-07-23 Thread Guenter Roeck
On 07/22/2015 10:49 PM, Eddie Huang wrote: From: Greta Zhang Without .shutdown(), watchdog might reset the system during power off. For example, if watchdog's timeout is set to 30s, then it is reset to zero by mtk_wdt_ping(). During power off, no app will ping watchdog, but watchdog is still ru

[PATCH v2 2/2] watchdog: add wdt shutdown callback to disable wdt if enabled

2015-07-22 Thread Eddie Huang
From: Greta Zhang Without .shutdown(), watchdog might reset the system during power off. For example, if watchdog's timeout is set to 30s, then it is reset to zero by mtk_wdt_ping(). During power off, no app will ping watchdog, but watchdog is still running and may trigger reset. Signed-off-by: