Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

Hi, > > > > > > > > > During update the not serviced watchdog reset the board and you may > > > end up with brick. > > > > > > > Depends on your setup and needs. If you have board with only one > > watchdog when you boot from qspi but > > reading variables from qspi failed (for whatever

Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

Hi Michal, > Hi, > > 2018-02-28 10:42 GMT+01:00 Lukasz Majewski : > > > On Wed, 28 Feb 2018 10:29:24 +0100 > > Michal Simek wrote: > > > > > HI, > > > > > > 2018-02-28 9:55 GMT+01:00 Lukasz Majewski : > > > > > > > On Mon, 26 Feb 2018

Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

Hi, 2018-02-28 10:42 GMT+01:00 Lukasz Majewski : > On Wed, 28 Feb 2018 10:29:24 +0100 > Michal Simek wrote: > > > HI, > > > > 2018-02-28 9:55 GMT+01:00 Lukasz Majewski : > > > > > On Mon, 26 Feb 2018 10:09:55 +0100 > > > Michal Simek

Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

On Wed, 28 Feb 2018 10:29:24 +0100 Michal Simek wrote: > HI, > > 2018-02-28 9:55 GMT+01:00 Lukasz Majewski : > > > On Mon, 26 Feb 2018 10:09:55 +0100 > > Michal Simek wrote: > > > > > Watchdog is only enabled in full u-boot. Adoption

Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

HI, 2018-02-28 9:55 GMT+01:00 Lukasz Majewski : > On Mon, 26 Feb 2018 10:09:55 +0100 > Michal Simek wrote: > > > Watchdog is only enabled in full u-boot. Adoption for SPL should be > > also done because that's the right place where watchdog should be > >

Re: [U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

On Mon, 26 Feb 2018 10:09:55 +0100 Michal Simek wrote: > Watchdog is only enabled in full u-boot. Adoption for SPL should be > also done because that's the right place where watchdog should be > enabled. > > Signed-off-by: Michal Simek > --- >

[U-Boot] [PATCH v2 2/3] arm: zynq: Wire watchdog internals

Watchdog is only enabled in full u-boot. Adoption for SPL should be also done because that's the right place where watchdog should be enabled. Signed-off-by: Michal Simek --- Changes in v2: - Make watchdog_reset depends on CONFIG_WATCHDOG not WDT This will handle use