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-11 Thread Daniel Kurtz
On Tue, Aug 11, 2015 at 12:42 PM, Guenter Roeck li...@roeck-us.net 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 matthias@gmail.com wrote: On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From:

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

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

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 matthias@gmail.com wrote: On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From: Greta Zhang greta.zh...@mediatek.com Without .shutdown(), watchdog might reset the

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 matthias@gmail.com wrote: On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From: Greta Zhang greta.zh...@mediatek.com Without .shutdown(), watchdog might reset the system during power off. For example, if

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

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 greta.zh...@mediatek.com 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 Matthias Brugger
On Thursday, July 23, 2015 01:49:11 PM Eddie Huang wrote: From: Greta Zhang greta.zh...@mediatek.com 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

[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:

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

2015-07-22 Thread Eddie Huang
From: Greta Zhang greta.zh...@mediatek.com 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