Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-05 Thread Sergei Shtylyov
On 03/05/2018 05:08 PM, Fabrizio Castro wrote: >>> On R-Car Gen2 and RZ/G1 the watchdog IP clock needs to be always ON, >>> on R-Car Gen3 we power the IP down during suspend. >>> >>> This commit adds suspend/resume support, so that the watchdog counting >>> "pauses" during suspend on all of the

RE: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-05 Thread Fabrizio Castro
Hello Sergei, thank you for your feedback! > Subject: Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support > > Hello! > > On 3/1/2018 9:17 PM, Fabrizio Castro wrote: > > > On R-Car Gen2 and RZ/G1 the watchdog IP clock needs to be always ON, > > on R

RE: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-02 Thread Fabrizio Castro
Hi Guenter, thank you for your feedback! > Subject: Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support > > On 03/02/2018 02:45 AM, Fabrizio Castro wrote: > > Dear All, > > > > perhaps someone from this email thread could explain to me what's the actua

Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-02 Thread Guenter Roeck
, meaning I rather give the watchdog a bit of additional slack during resume. Having said that, as mentioned before, I am willing to accept the patch as is, in the assumption that the authors know what they are doing. Guenter Thanks, Fab Subject: Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add

RE: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-02 Thread Fabrizio Castro
doing so from user space, which I don't think is what we want here, but I may be wrong. Could someone please shed some light here? Thanks, Fab > Subject: Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support > > Hi Fabrizio, > > On Thu, Mar 1, 2018 at 7:17 PM,

Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-02 Thread Sergei Shtylyov
Hello! On 3/1/2018 9:17 PM, Fabrizio Castro wrote: On R-Car Gen2 and RZ/G1 the watchdog IP clock needs to be always ON, on R-Car Gen3 we power the IP down during suspend. This commit adds suspend/resume support, so that the watchdog counting "pauses" during suspend on all of the SoCs

Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-01 Thread Guenter Roeck
On Thu, Mar 01, 2018 at 06:17:21PM +, Fabrizio Castro wrote: > On R-Car Gen2 and RZ/G1 the watchdog IP clock needs to be always ON, > on R-Car Gen3 we power the IP down during suspend. > > This commit adds suspend/resume support, so that the watchdog counting > "pauses" during suspend on all

Re: [PATCH v7 1/3] watchdog: renesas_wdt: Add suspend/resume support

2018-03-01 Thread Wolfram Sang
On Thu, Mar 01, 2018 at 06:17:21PM +, Fabrizio Castro wrote: > On R-Car Gen2 and RZ/G1 the watchdog IP clock needs to be always ON, > on R-Car Gen3 we power the IP down during suspend. > > This commit adds suspend/resume support, so that the watchdog counting > "pauses" during suspend on all