linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCHv2] watchdog: qcom: Add suspend/resume support
@ 2019-01-17 15:19 Sai Prakash Ranjan
  2019-01-17 15:26 ` Sai Prakash Ranjan
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Sai Prakash Ranjan @ 2019-01-17 15:19 UTC (permalink / raw)
  To: Guenter Roeck, Wim Van Sebroeck, linux-watchdog, Guenter Roeck
  Cc: Rajendra Nayak, Vivek Gautam, Sibi Sankar, Stephen Boyd,
	Doug Anderson, linux-arm-kernel, linux-kernel, linux-arm-msm,
	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 <saiprakash.ranjan@codeaurora.org>

---

v2:
 * Use __maybe_unused instead of #ifdef as suggested by
   Brian and Guenter
---
 drivers/watchdog/qcom-wdt.c | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

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 platform_device *pdev)
 	return 0;
 }
 
+static int __maybe_unused qcom_wdt_suspend(struct device *dev)
+{
+	struct qcom_wdt *wdt = dev_get_drvdata(dev);
+
+	if (watchdog_active(&wdt->wdd))
+		qcom_wdt_stop(&wdt->wdd);
+
+	return 0;
+}
+
+static int __maybe_unused qcom_wdt_resume(struct device *dev)
+{
+	struct qcom_wdt *wdt = dev_get_drvdata(dev);
+
+	if (watchdog_active(&wdt->wdd))
+		qcom_wdt_start(&wdt->wdd);
+
+	return 0;
+}
+
+static SIMPLE_DEV_PM_OPS(qcom_wdt_pm_ops, qcom_wdt_suspend, qcom_wdt_resume);
+
 static const struct of_device_id qcom_wdt_of_table[] = {
 	{ .compatible = "qcom,kpss-timer", .data = reg_offset_data_apcs_tmr },
 	{ .compatible = "qcom,scss-timer", .data = reg_offset_data_apcs_tmr },
@@ -259,6 +281,7 @@ static struct platform_driver qcom_watchdog_driver = {
 	.driver	= {
 		.name		= KBUILD_MODNAME,
 		.of_match_table	= qcom_wdt_of_table,
+		.pm		= &qcom_wdt_pm_ops,
 	},
 };
 module_platform_driver(qcom_watchdog_driver);
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 15:19 [PATCHv2] watchdog: qcom: Add suspend/resume support Sai Prakash Ranjan
@ 2019-01-17 15:26 ` Sai Prakash Ranjan
  2019-01-17 18:16 ` Guenter Roeck
  2019-01-17 18:37 ` Stephen Boyd
  2 siblings, 0 replies; 10+ messages in thread
From: Sai Prakash Ranjan @ 2019-01-17 15:26 UTC (permalink / raw)
  To: Guenter Roeck, Brian Masney, Wim Van Sebroeck, linux-watchdog,
	Guenter Roeck
  Cc: Rajendra Nayak, Stephen Boyd, linux-arm-msm, Doug Anderson,
	linux-kernel, Sibi Sankar, Vivek Gautam, linux-arm-kernel

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 Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
> 
> ---
> 
> v2:
>   * Use __maybe_unused instead of #ifdef as suggested by
>     Brian and Guenter
> ---
>   drivers/watchdog/qcom-wdt.c | 23 +++++++++++++++++++++++
>   1 file changed, 23 insertions(+)
> 
> 

Adding Brian.
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 15:19 [PATCHv2] watchdog: qcom: Add suspend/resume support Sai Prakash Ranjan
  2019-01-17 15:26 ` Sai Prakash Ranjan
@ 2019-01-17 18:16 ` Guenter Roeck
  2019-01-18  2:58   ` Sai Prakash Ranjan
  2019-01-17 18:37 ` Stephen Boyd
  2 siblings, 1 reply; 10+ messages in thread
From: Guenter Roeck @ 2019-01-17 18:16 UTC (permalink / raw)
  To: Sai Prakash Ranjan
  Cc: Wim Van Sebroeck, linux-watchdog, Guenter Roeck, Rajendra Nayak,
	Vivek Gautam, Sibi Sankar, Stephen Boyd, Doug Anderson,
	linux-arm-kernel, linux-kernel, linux-arm-msm

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 time.
> 
> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> 
> ---
> 
> v2:
>  * Use __maybe_unused instead of #ifdef as suggested by
>    Brian and Guenter
> ---
>  drivers/watchdog/qcom-wdt.c | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> 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 platform_device *pdev)
>  	return 0;
>  }
>  
> +static int __maybe_unused qcom_wdt_suspend(struct device *dev)
> +{
> +	struct qcom_wdt *wdt = dev_get_drvdata(dev);
> +
> +	if (watchdog_active(&wdt->wdd))
> +		qcom_wdt_stop(&wdt->wdd);
> +
> +	return 0;
> +}
> +
> +static int __maybe_unused qcom_wdt_resume(struct device *dev)
> +{
> +	struct qcom_wdt *wdt = dev_get_drvdata(dev);
> +
> +	if (watchdog_active(&wdt->wdd))
> +		qcom_wdt_start(&wdt->wdd);
> +
> +	return 0;
> +}
> +
> +static SIMPLE_DEV_PM_OPS(qcom_wdt_pm_ops, qcom_wdt_suspend, qcom_wdt_resume);
> +
>  static const struct of_device_id qcom_wdt_of_table[] = {
>  	{ .compatible = "qcom,kpss-timer", .data = reg_offset_data_apcs_tmr },
>  	{ .compatible = "qcom,scss-timer", .data = reg_offset_data_apcs_tmr },
> @@ -259,6 +281,7 @@ static struct platform_driver qcom_watchdog_driver = {
>  	.driver	= {
>  		.name		= KBUILD_MODNAME,
>  		.of_match_table	= qcom_wdt_of_table,
> +		.pm		= &qcom_wdt_pm_ops,
>  	},
>  };
>  module_platform_driver(qcom_watchdog_driver);
> -- 
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 15:19 [PATCHv2] watchdog: qcom: Add suspend/resume support Sai Prakash Ranjan
  2019-01-17 15:26 ` Sai Prakash Ranjan
  2019-01-17 18:16 ` Guenter Roeck
@ 2019-01-17 18:37 ` Stephen Boyd
  2019-01-17 19:09   ` Guenter Roeck
  2 siblings, 1 reply; 10+ messages in thread
From: Stephen Boyd @ 2019-01-17 18:37 UTC (permalink / raw)
  To: Guenter Roeck, Guenter Roeck, Sai Prakash Ranjan,
	Wim Van Sebroeck, linux-watchdog
  Cc: Rajendra Nayak, Vivek Gautam, Sibi Sankar, Doug Anderson,
	linux-arm-kernel, linux-kernel, linux-arm-msm,
	Sai Prakash Ranjan

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 platform_device *pdev)
>         return 0;
>  }
>  
> +static int __maybe_unused qcom_wdt_suspend(struct device *dev)
> +{
> +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> +
> +       if (watchdog_active(&wdt->wdd))
> +               qcom_wdt_stop(&wdt->wdd);
> +
> +       return 0;
> +}
> +
> +static int __maybe_unused qcom_wdt_resume(struct device *dev)
> +{
> +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> +
> +       if (watchdog_active(&wdt->wdd))
> +               qcom_wdt_start(&wdt->wdd);
> +
> +       return 0;
> +}

This looks fairly generic. For example, the Mediatek driver also stops
and starts (but also pings after starting). Grepping for 'active' in
drivers/watchdog/ finds more examples. Could there be some functions in
watchdog core that do the common things like watchdog_stop() and
watchdog_start() and watchdog_start_and_ping()? Or maybe a bit can be
set during registration so that the 'struct class watchdog_class' can
get PM ops to stop and start on suspend/resume of the watchdog character
device?

Nothing is wrong with the patch, I'm just bemoaning the amount of code
duplication here.


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 18:37 ` Stephen Boyd
@ 2019-01-17 19:09   ` Guenter Roeck
  2019-01-17 21:14     ` Jerry Hoemann
  0 siblings, 1 reply; 10+ messages in thread
From: Guenter Roeck @ 2019-01-17 19:09 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Guenter Roeck, Guenter Roeck, Sai Prakash Ranjan,
	Wim Van Sebroeck, linux-watchdog, Rajendra Nayak, Vivek Gautam,
	Sibi Sankar, Doug Anderson, linux-arm-kernel, linux-kernel,
	linux-arm-msm

On Thu, Jan 17, 2019 at 10:37 AM Stephen Boyd <swboyd@chromium.org> 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
> > +++ b/drivers/watchdog/qcom-wdt.c
> > @@ -245,6 +245,28 @@ static int qcom_wdt_remove(struct platform_device *pdev)
> >         return 0;
> >  }
> >
> > +static int __maybe_unused qcom_wdt_suspend(struct device *dev)
> > +{
> > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > +
> > +       if (watchdog_active(&wdt->wdd))
> > +               qcom_wdt_stop(&wdt->wdd);
> > +
> > +       return 0;
> > +}
> > +
> > +static int __maybe_unused qcom_wdt_resume(struct device *dev)
> > +{
> > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > +
> > +       if (watchdog_active(&wdt->wdd))
> > +               qcom_wdt_start(&wdt->wdd);
> > +
> > +       return 0;
> > +}
>
> This looks fairly generic. For example, the Mediatek driver also stops
> and starts (but also pings after starting). Grepping for 'active' in
> drivers/watchdog/ finds more examples. Could there be some functions in
> watchdog core that do the common things like watchdog_stop() and
> watchdog_start() and watchdog_start_and_ping()? Or maybe a bit can be
> set during registration so that the 'struct class watchdog_class' can
> get PM ops to stop and start on suspend/resume of the watchdog character
> device?
>
> Nothing is wrong with the patch, I'm just bemoaning the amount of code
> duplication here.
>

Patch(es) to add the functionality to the watchdog core are welcome;
it should be possible to move the functionality into the core (maybe
to be enabled with a new watchdog API call). Doing it using the class
device sounds like an excellent idea. This should be straightforward
to implement, though the question of "should we ping on resume or not"
might be an endless source for bike shedding fun.

Guenter

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 19:09   ` Guenter Roeck
@ 2019-01-17 21:14     ` Jerry Hoemann
  2019-01-17 21:16       ` Guenter Roeck
  0 siblings, 1 reply; 10+ messages in thread
From: Jerry Hoemann @ 2019-01-17 21:14 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: Stephen Boyd, Guenter Roeck, Guenter Roeck, Sai Prakash Ranjan,
	Wim Van Sebroeck, linux-watchdog, Rajendra Nayak, Vivek Gautam,
	Sibi Sankar, Doug Anderson, linux-arm-kernel, linux-kernel,
	linux-arm-msm

On Thu, Jan 17, 2019 at 11:09:31AM -0800, Guenter Roeck wrote:
> On Thu, Jan 17, 2019 at 10:37 AM Stephen Boyd <swboyd@chromium.org> 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
> > > +++ b/drivers/watchdog/qcom-wdt.c
> > > @@ -245,6 +245,28 @@ static int qcom_wdt_remove(struct platform_device *pdev)
> > >         return 0;
> > >  }
> > >
> > > +static int __maybe_unused qcom_wdt_suspend(struct device *dev)
> > > +{
> > > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > > +
> > > +       if (watchdog_active(&wdt->wdd))
> > > +               qcom_wdt_stop(&wdt->wdd);
> > > +
> > > +       return 0;
> > > +}
> > > +
> > > +static int __maybe_unused qcom_wdt_resume(struct device *dev)
> > > +{
> > > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > > +
> > > +       if (watchdog_active(&wdt->wdd))
> > > +               qcom_wdt_start(&wdt->wdd);
> > > +
> > > +       return 0;
> > > +}
> >
> > This looks fairly generic. For example, the Mediatek driver also stops
> > and starts (but also pings after starting). Grepping for 'active' in
> > drivers/watchdog/ finds more examples. Could there be some functions in
> > watchdog core that do the common things like watchdog_stop() and
> > watchdog_start() and watchdog_start_and_ping()? Or maybe a bit can be
> > set during registration so that the 'struct class watchdog_class' can
> > get PM ops to stop and start on suspend/resume of the watchdog character
> > device?
> >
> > Nothing is wrong with the patch, I'm just bemoaning the amount of code
> > duplication here.
> >
> 
> Patch(es) to add the functionality to the watchdog core are welcome;
> it should be possible to move the functionality into the core (maybe
> to be enabled with a new watchdog API call). Doing it using the class
> device sounds like an excellent idea. This should be straightforward
> to implement, though the question of "should we ping on resume or not"
> might be an endless source for bike shedding fun.
> 
> Guenter

It could be controlled by a currently unused bit in the
watchdog_info.options.  Then people can agree to disagree.



-- 

-----------------------------------------------------------------------------
Jerry Hoemann                  Software Engineer   Hewlett Packard Enterprise
-----------------------------------------------------------------------------

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 21:14     ` Jerry Hoemann
@ 2019-01-17 21:16       ` Guenter Roeck
  0 siblings, 0 replies; 10+ messages in thread
From: Guenter Roeck @ 2019-01-17 21:16 UTC (permalink / raw)
  To: Jerry.Hoemann
  Cc: Stephen Boyd, Guenter Roeck, Guenter Roeck, Sai Prakash Ranjan,
	Wim Van Sebroeck, linux-watchdog, Rajendra Nayak, Vivek Gautam,
	Sibi Sankar, Doug Anderson, linux-arm-kernel, linux-kernel,
	linux-arm-msm

On Thu, Jan 17, 2019 at 1:14 PM Jerry Hoemann <jerry.hoemann@hpe.com> 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 <swboyd@chromium.org> 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
> > > > +++ b/drivers/watchdog/qcom-wdt.c
> > > > @@ -245,6 +245,28 @@ static int qcom_wdt_remove(struct platform_device *pdev)
> > > >         return 0;
> > > >  }
> > > >
> > > > +static int __maybe_unused qcom_wdt_suspend(struct device *dev)
> > > > +{
> > > > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > > > +
> > > > +       if (watchdog_active(&wdt->wdd))
> > > > +               qcom_wdt_stop(&wdt->wdd);
> > > > +
> > > > +       return 0;
> > > > +}
> > > > +
> > > > +static int __maybe_unused qcom_wdt_resume(struct device *dev)
> > > > +{
> > > > +       struct qcom_wdt *wdt = dev_get_drvdata(dev);
> > > > +
> > > > +       if (watchdog_active(&wdt->wdd))
> > > > +               qcom_wdt_start(&wdt->wdd);
> > > > +
> > > > +       return 0;
> > > > +}
> > >
> > > This looks fairly generic. For example, the Mediatek driver also stops
> > > and starts (but also pings after starting). Grepping for 'active' in
> > > drivers/watchdog/ finds more examples. Could there be some functions in
> > > watchdog core that do the common things like watchdog_stop() and
> > > watchdog_start() and watchdog_start_and_ping()? Or maybe a bit can be
> > > set during registration so that the 'struct class watchdog_class' can
> > > get PM ops to stop and start on suspend/resume of the watchdog character
> > > device?
> > >
> > > Nothing is wrong with the patch, I'm just bemoaning the amount of code
> > > duplication here.
> > >
> >
> > Patch(es) to add the functionality to the watchdog core are welcome;
> > it should be possible to move the functionality into the core (maybe
> > to be enabled with a new watchdog API call). Doing it using the class
> > device sounds like an excellent idea. This should be straightforward
> > to implement, though the question of "should we ping on resume or not"
> > might be an endless source for bike shedding fun.
> >
> > Guenter
>
> It could be controlled by a currently unused bit in the
> watchdog_info.options.  Then people can agree to disagree.
>
I thought about that, but that would result in inconsistent behavior
across drivers. Of course we could be all fancy and make it runtime
configurable via ioctl.

Guenter

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-17 18:16 ` Guenter Roeck
@ 2019-01-18  2:58   ` Sai Prakash Ranjan
  2019-01-18  3:53     ` Guenter Roeck
  0 siblings, 1 reply; 10+ messages in thread
From: Sai Prakash Ranjan @ 2019-01-18  2:58 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: Wim Van Sebroeck, linux-watchdog, Guenter Roeck, Rajendra Nayak,
	Vivek Gautam, Sibi Sankar, Stephen Boyd, Doug Anderson,
	linux-arm-kernel, linux-kernel, linux-arm-msm

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
>> crashes/resets if resume happens after a long time.
>>
>> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
> 
> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> 

Thanks Guenter. Should I send a v3 with your reviewed-by?

- Sai
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-18  2:58   ` Sai Prakash Ranjan
@ 2019-01-18  3:53     ` Guenter Roeck
  2019-01-18  4:25       ` Sai Prakash Ranjan
  0 siblings, 1 reply; 10+ messages in thread
From: Guenter Roeck @ 2019-01-18  3:53 UTC (permalink / raw)
  To: Sai Prakash Ranjan
  Cc: Wim Van Sebroeck, linux-watchdog, Guenter Roeck, Rajendra Nayak,
	Vivek Gautam, Sibi Sankar, Stephen Boyd, Doug Anderson,
	linux-arm-kernel, linux-kernel, linux-arm-msm

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 after suspend would result in unwanted
>>> crashes/resets if resume happens after a long time.
>>>
>>> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
>>
>> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
>>
> 
> Thanks Guenter. Should I send a v3 with your reviewed-by?
> 

I add it to my watchdog-next branch and Wim will pick it up from there.
No need to resend.

Guenter

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCHv2] watchdog: qcom: Add suspend/resume support
  2019-01-18  3:53     ` Guenter Roeck
@ 2019-01-18  4:25       ` Sai Prakash Ranjan
  0 siblings, 0 replies; 10+ messages in thread
From: Sai Prakash Ranjan @ 2019-01-18  4:25 UTC (permalink / raw)
  To: Guenter Roeck
  Cc: Wim Van Sebroeck, linux-watchdog, Guenter Roeck, Rajendra Nayak,
	Vivek Gautam, Sibi Sankar, Stephen Boyd, Doug Anderson,
	linux-arm-kernel, linux-kernel, linux-arm-msm

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 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 <saiprakash.ranjan@codeaurora.org>
>>>
>>> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
>>>
>>
>> Thanks Guenter. Should I send a v3 with your reviewed-by?
>>
> 
> I add it to my watchdog-next branch and Wim will pick it up from there.
> No need to resend.
> 

Ok thanks.

- Sai

-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2019-01-18  4:25 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-17 15:19 [PATCHv2] watchdog: qcom: Add suspend/resume support Sai Prakash Ranjan
2019-01-17 15:26 ` Sai Prakash Ranjan
2019-01-17 18:16 ` Guenter Roeck
2019-01-18  2:58   ` Sai Prakash Ranjan
2019-01-18  3:53     ` Guenter Roeck
2019-01-18  4:25       ` Sai Prakash Ranjan
2019-01-17 18:37 ` Stephen Boyd
2019-01-17 19:09   ` Guenter Roeck
2019-01-17 21:14     ` Jerry Hoemann
2019-01-17 21:16       ` Guenter Roeck

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).