Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Marco Felsch
Hi Ahmad, On 19-11-04 10:57, Ahmad Fatoum wrote: > Hello Marco, > > On 11/4/19 10:44 AM, Marco Felsch wrote: > >> This means that your boot time would increase by 200 ms. If this matter to > >> you, > >> you might want to change this, so watchdog_set_timeout is called only once. > > > >

Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Ahmad Fatoum
Hello Marco, On 11/4/19 10:44 AM, Marco Felsch wrote: >> This means that your boot time would increase by 200 ms. If this matter to >> you, >> you might want to change this, so watchdog_set_timeout is called only once. > > Increasing the delay isn't a big deal. But after we discussed it again I

Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Marco Felsch
Hi Ahmad, On 19-11-04 09:51, Ahmad Fatoum wrote: > Hello Marco, > > On 11/4/19 9:34 AM, Marco Felsch wrote: > > On 19-11-04 09:27, Sascha Hauer wrote: > >> On Wed, Oct 30, 2019 at 06:06:53PM +0100, Marco Felsch wrote: > >>> The watchdog resets the system if the watchdog gets pinged to fast. >

Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Ahmad Fatoum
Hello Marco, On 11/4/19 9:34 AM, Marco Felsch wrote: > On 19-11-04 09:27, Sascha Hauer wrote: >> On Wed, Oct 30, 2019 at 06:06:53PM +0100, Marco Felsch wrote: >>> The watchdog resets the system if the watchdog gets pinged to fast. >>> Between each watchdog ping must be a pause of at least 200ms.

Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Marco Felsch
On 19-11-04 09:27, Sascha Hauer wrote: > On Wed, Oct 30, 2019 at 06:06:53PM +0100, Marco Felsch wrote: > > The watchdog resets the system if the watchdog gets pinged to fast. > > Between each watchdog ping must be a pause of at least 200ms. > > > > Signed-off-by: Marco Felsch > > --- > >

Re: [PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-11-04 Thread Sascha Hauer
On Wed, Oct 30, 2019 at 06:06:53PM +0100, Marco Felsch wrote: > The watchdog resets the system if the watchdog gets pinged to fast. > Between each watchdog ping must be a pause of at least 200ms. > > Signed-off-by: Marco Felsch > --- > drivers/mfd/da9063.c | 11 +++ > 1 file changed, 11

[PATCH 2/2] mfd: da9063: fix watchdog ping execution

2019-10-30 Thread Marco Felsch
The watchdog resets the system if the watchdog gets pinged to fast. Between each watchdog ping must be a pause of at least 200ms. Signed-off-by: Marco Felsch --- drivers/mfd/da9063.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/mfd/da9063.c b/drivers/mfd/da9063.c