Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Sai Prakash Ranjan
On 1/18/2019 9:23 AM, Guenter Roeck wrote: On 1/17/19 6:58 PM, Sai Prakash Ranjan wrote: On 1/17/2019 11:46 PM, Guenter Roeck wrote: On Thu, Jan 17, 2019 at 08:49:42PM +0530, Sai Prakash Ranjan wrote: This adds the support for qcom watchdog suspend and resume when entering and exiting deep

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Guenter Roeck
On 1/17/19 6:58 PM, Sai Prakash Ranjan wrote: On 1/17/2019 11:46 PM, Guenter Roeck wrote: On Thu, Jan 17, 2019 at 08:49:42PM +0530, Sai Prakash Ranjan wrote: This adds the support for qcom watchdog suspend and resume when entering and exiting deep sleep states. Otherwise having watchdog active

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Sai Prakash Ranjan
On 1/17/2019 11:46 PM, Guenter Roeck wrote: On Thu, Jan 17, 2019 at 08:49:42PM +0530, Sai Prakash Ranjan wrote: This adds the support for qcom watchdog suspend and resume when entering and exiting deep sleep states. Otherwise having watchdog active after suspend would result in unwanted

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Guenter Roeck
On Thu, Jan 17, 2019 at 1:14 PM Jerry Hoemann wrote: > > On Thu, Jan 17, 2019 at 11:09:31AM -0800, Guenter Roeck wrote: > > On Thu, Jan 17, 2019 at 10:37 AM Stephen Boyd wrote: > > > > > > Quoting Sai Prakash Ranjan (2019-01-17 07:19:42) > > > > diff --git a/drivers/watchdog/qcom-wdt.c

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Jerry Hoemann
On Thu, Jan 17, 2019 at 11:09:31AM -0800, Guenter Roeck wrote: > On Thu, Jan 17, 2019 at 10:37 AM Stephen Boyd wrote: > > > > Quoting Sai Prakash Ranjan (2019-01-17 07:19:42) > > > diff --git a/drivers/watchdog/qcom-wdt.c b/drivers/watchdog/qcom-wdt.c > > > index 780971318810..5dfd604477a4 100644

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Guenter Roeck
On Thu, Jan 17, 2019 at 10:37 AM Stephen Boyd wrote: > > Quoting Sai Prakash Ranjan (2019-01-17 07:19:42) > > diff --git a/drivers/watchdog/qcom-wdt.c b/drivers/watchdog/qcom-wdt.c > > index 780971318810..5dfd604477a4 100644 > > --- a/drivers/watchdog/qcom-wdt.c > > +++

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Stephen Boyd
Quoting Sai Prakash Ranjan (2019-01-17 07:19:42) > diff --git a/drivers/watchdog/qcom-wdt.c b/drivers/watchdog/qcom-wdt.c > index 780971318810..5dfd604477a4 100644 > --- a/drivers/watchdog/qcom-wdt.c > +++ b/drivers/watchdog/qcom-wdt.c > @@ -245,6 +245,28 @@ static int qcom_wdt_remove(struct

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Guenter Roeck
On Thu, Jan 17, 2019 at 08:49:42PM +0530, Sai Prakash Ranjan wrote: > This adds the support for qcom watchdog suspend and resume > when entering and exiting deep sleep states. Otherwise > having watchdog active after suspend would result in unwanted > crashes/resets if resume happens after a long

Re: [PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Sai Prakash Ranjan
On 1/17/2019 8:49 PM, Sai Prakash Ranjan wrote: This adds the support for qcom watchdog suspend and resume when entering and exiting deep sleep states. Otherwise having watchdog active after suspend would result in unwanted crashes/resets if resume happens after a long time. Signed-off-by: Sai

[PATCHv2] watchdog: qcom: Add suspend/resume support

2019-01-17 Thread Sai Prakash Ranjan
This adds the support for qcom watchdog suspend and resume when entering and exiting deep sleep states. Otherwise having watchdog active after suspend would result in unwanted crashes/resets if resume happens after a long time. Signed-off-by: Sai Prakash Ranjan --- v2: * Use __maybe_unused