Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-07-01 Thread Tero Kristo
On 01/07/2020 16:34, Guenter Roeck wrote: On 6/30/20 10:50 PM, Tero Kristo wrote: [ ... ] Hardware supports changing the timeout value, however it only updates this during the next window (preload values are picked once user pings the watchdog.) The current driver doesn't support or use that

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-07-01 Thread Guenter Roeck
On 6/30/20 10:50 PM, Tero Kristo wrote: [ ... ] > Hardware supports changing the timeout value, however it only updates this > during the next window (preload values are picked once user pings the > watchdog.) > The current driver doesn't support or use that, though, since the start function is

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-30 Thread Tero Kristo
On 30/06/2020 23:23, Guenter Roeck wrote: On Thu, Jun 25, 2020 at 08:04:50PM +0300, Tero Kristo wrote: On 25/06/2020 16:35, Guenter Roeck wrote: On 6/25/20 1:32 AM, Tero Kristo wrote: On 24/06/2020 18:24, Jan Kiszka wrote: On 24.06.20 13:45, Tero Kristo wrote: If the RTI watchdog has been st

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-30 Thread Guenter Roeck
On Thu, Jun 25, 2020 at 08:04:50PM +0300, Tero Kristo wrote: > On 25/06/2020 16:35, Guenter Roeck wrote: > > On 6/25/20 1:32 AM, Tero Kristo wrote: > > > On 24/06/2020 18:24, Jan Kiszka wrote: > > > > On 24.06.20 13:45, Tero Kristo wrote: > > > > > If the RTI watchdog has been started by someone (l

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-25 Thread Tero Kristo
On 25/06/2020 16:35, Guenter Roeck wrote: On 6/25/20 1:32 AM, Tero Kristo wrote: On 24/06/2020 18:24, Jan Kiszka wrote: On 24.06.20 13:45, Tero Kristo wrote: If the RTI watchdog has been started by someone (like bootloader) when the driver probes, we must adjust the initial ping timeout to mat

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-25 Thread Guenter Roeck
On 6/25/20 1:32 AM, Tero Kristo wrote: > On 24/06/2020 18:24, Jan Kiszka wrote: >> On 24.06.20 13:45, Tero Kristo wrote: >>> If the RTI watchdog has been started by someone (like bootloader) when >>> the driver probes, we must adjust the initial ping timeout to match the >>> currently running watch

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-25 Thread Jan Kiszka
On 25.06.20 10:32, Tero Kristo wrote: On 24/06/2020 18:24, Jan Kiszka wrote: On 24.06.20 13:45, Tero Kristo wrote: If the RTI watchdog has been started by someone (like bootloader) when the driver probes, we must adjust the initial ping timeout to match the currently running watchdog window to

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-25 Thread Tero Kristo
On 24/06/2020 18:24, Jan Kiszka wrote: On 24.06.20 13:45, Tero Kristo wrote: If the RTI watchdog has been started by someone (like bootloader) when the driver probes, we must adjust the initial ping timeout to match the currently running watchdog window to avoid generating watchdog reset. Signe

Re: [PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-24 Thread Jan Kiszka
On 24.06.20 13:45, Tero Kristo wrote: If the RTI watchdog has been started by someone (like bootloader) when the driver probes, we must adjust the initial ping timeout to match the currently running watchdog window to avoid generating watchdog reset. Signed-off-by: Tero Kristo --- drivers/wat

[PATCH 2/2] watchdog: rti: tweak min_hw_heartbeat_ms to match initial allowed window

2020-06-24 Thread Tero Kristo
If the RTI watchdog has been started by someone (like bootloader) when the driver probes, we must adjust the initial ping timeout to match the currently running watchdog window to avoid generating watchdog reset. Signed-off-by: Tero Kristo --- drivers/watchdog/rti_wdt.c | 25