Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Lothar Waßmann
Hi, On Fri, 4 Mar 2016 16:26:59 +0100 Romain Izard wrote: > Hi Lothar, > > 2016-03-04 15:59 GMT+01:00 Lothar Waßmann : > >> > I also check the WDT_MR register before and after enabling > >> > watchdog, the WDV and WDD fields are correct. > >> > > >> > Can you check it again? thank

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Romain Izard
Hi Lothar, 2016-03-04 15:59 GMT+01:00 Lothar Waßmann : >> > I also check the WDT_MR register before and after enabling >> > watchdog, the WDV and WDD fields are correct. >> > >> > Can you check it again? thank you. >> >>> >> >>> >> >>> Working case: >> >>> MR on kernel startup: 0

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Lothar Waßmann
Hi, On Fri, 4 Mar 2016 05:56:20 -0800 Guenter Roeck wrote: > On 03/04/2016 05:26 AM, Romain Izard wrote: > > 2016-03-04 14:09 GMT+01:00 Guenter Roeck : > >> On 03/04/2016 01:06 AM, Romain Izard wrote: > >>> 2016-03-04 6:23 GMT+01:00 Guenter Roeck : > On 03/03/2016 05:35 PM, Yang, Wenyou wrote

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Guenter Roeck
On 03/04/2016 05:26 AM, Romain Izard wrote: 2016-03-04 14:09 GMT+01:00 Guenter Roeck : On 03/04/2016 01:06 AM, Romain Izard wrote: 2016-03-04 6:23 GMT+01:00 Guenter Roeck : On 03/03/2016 05:35 PM, Yang, Wenyou wrote: On 2016/3/3 18:29, Romain Izard wrote: If the internal counter is not refr

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Romain Izard
2016-03-04 14:09 GMT+01:00 Guenter Roeck : > On 03/04/2016 01:06 AM, Romain Izard wrote: >> 2016-03-04 6:23 GMT+01:00 Guenter Roeck : >>> On 03/03/2016 05:35 PM, Yang, Wenyou wrote: On 2016/3/3 18:29, Romain Izard wrote: > > If the internal counter is not refreshed when the watchdog is

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Guenter Roeck
On 03/04/2016 01:06 AM, Romain Izard wrote: Hi Wenyou, Guenter, 2016-03-04 6:23 GMT+01:00 Guenter Roeck : On 03/03/2016 05:35 PM, Yang, Wenyou wrote: On 2016/3/3 18:29, Romain Izard wrote: If the internal counter is not refreshed when the watchdog is started for the first time, the watchdog

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-04 Thread Romain Izard
Hi Wenyou, Guenter, 2016-03-04 6:23 GMT+01:00 Guenter Roeck : > On 03/03/2016 05:35 PM, Yang, Wenyou wrote: >> On 2016/3/3 18:29, Romain Izard wrote: >>> >>> If the internal counter is not refreshed when the watchdog is >>> started for the first time, the watchdog will trigger very rapidly. >>> Fo

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Guenter Roeck
On 03/03/2016 05:35 PM, Yang, Wenyou wrote: Hi Romain, On 2016/3/3 18:29, Romain Izard wrote: If the internal counter is not refreshed when the watchdog is started for the first time, the watchdog will trigger very rapidly. For example, opening /dev/watchdog without writing in it will immediate

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Yang, Wenyou
Hi Romain, On 2016/3/3 18:29, Romain Izard wrote: If the internal counter is not refreshed when the watchdog is started for the first time, the watchdog will trigger very rapidly. For example, opening /dev/watchdog without writing in it will immediately trigger a reboot, instead of waiting for t

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Guenter Roeck
On Thu, Mar 03, 2016 at 01:53:47PM +0100, Romain Izard wrote: > Hi Guenter, > > 2016-03-03 13:10 GMT+01:00 Guenter Roeck : > > On 03/03/2016 02:29 AM, Romain Izard wrote: > >> > >> If the internal counter is not refreshed when the watchdog is started > >> for the first time, the watchdog will trig

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Romain Izard
Hi Guenter, 2016-03-03 13:10 GMT+01:00 Guenter Roeck : > On 03/03/2016 02:29 AM, Romain Izard wrote: >> >> If the internal counter is not refreshed when the watchdog is started >> for the first time, the watchdog will trigger very rapidly. For >> example, opening /dev/watchdog without writing in i

Re: [PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Guenter Roeck
On 03/03/2016 02:29 AM, Romain Izard wrote: If the internal counter is not refreshed when the watchdog is started for the first time, the watchdog will trigger very rapidly. For example, opening /dev/watchdog without writing in it will immediately trigger a reboot, instead of waiting for the dela

[PATCH v1] watchdog: sama5d4_wdt: Reset delay on start

2016-03-03 Thread Romain Izard
If the internal counter is not refreshed when the watchdog is started for the first time, the watchdog will trigger very rapidly. For example, opening /dev/watchdog without writing in it will immediately trigger a reboot, instead of waiting for the delay to expire. To avoid this problem, reload th