Re: u-boot leaves watchdog enabled by default

2020-09-22 Thread Tom Rini
On Tue, Sep 22, 2020 at 05:41:47PM +0200, Michael Walle wrote: > Hi Tom, > > Am 2020-09-22 16:41, schrieb Tom Rini: > > On Tue, Sep 22, 2020 at 03:18:58PM +0200, Michael Walle wrote: > [..] > > > > But all that said, since we have "wdt stop", perhaps you can find a > > > > place to put that in the

Re: u-boot leaves watchdog enabled by default

2020-09-22 Thread Michael Walle
Hi Tom, Am 2020-09-22 16:41, schrieb Tom Rini: On Tue, Sep 22, 2020 at 03:18:58PM +0200, Michael Walle wrote: [..] > But all that said, since we have "wdt stop", perhaps you can find a > place to put that in the boot script? Or just declare that if we get > far enough to run preboot cmd then

Re: u-boot leaves watchdog enabled by default

2020-09-22 Thread Tom Rini
On Tue, Sep 22, 2020 at 03:18:58PM +0200, Michael Walle wrote: > Hi, > > Am 2020-09-22 14:36, schrieb Tom Rini: > > On Tue, Sep 22, 2020 at 08:59:00AM +0200, Michael Walle wrote: > > > Hi, > > > > > > Am 2020-09-22 03:18, schrieb Tom Rini: > > > > On Mon, Sep 21, 2020 at 10:56:14PM +0200, Michael

Re: u-boot leaves watchdog enabled by default

2020-09-22 Thread Michael Walle
Hi, Am 2020-09-22 14:36, schrieb Tom Rini: On Tue, Sep 22, 2020 at 08:59:00AM +0200, Michael Walle wrote: Hi, Am 2020-09-22 03:18, schrieb Tom Rini: > On Mon, Sep 21, 2020 at 10:56:14PM +0200, Michael Walle wrote: > > Hi, > > [..] > > > > >>> >> called in the bootefi case. So even if I'd

Re: u-boot leaves watchdog enabled by default

2020-09-22 Thread Tom Rini
On Tue, Sep 22, 2020 at 08:59:00AM +0200, Michael Walle wrote: > Hi, > > Am 2020-09-22 03:18, schrieb Tom Rini: > > On Mon, Sep 21, 2020 at 10:56:14PM +0200, Michael Walle wrote: > > > Hi, > > > > [..] > > > > > >>> >> called in the bootefi case. So even if I'd do a > > > > > >>> workaround

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Michael Walle
Hi, Am 2020-09-22 03:18, schrieb Tom Rini: On Mon, Sep 21, 2020 at 10:56:14PM +0200, Michael Walle wrote: Hi, [..] > > >>> >> called in the bootefi case. So even if I'd do a workaround and > > >>> stop it > > >>> >> manually in my board code, I couldn't do that consistently for

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Tom Rini
On Mon, Sep 21, 2020 at 10:56:14PM +0200, Michael Walle wrote: > Hi, > > Am 2020-09-21 20:50, schrieb Tom Rini: > > On Mon, Sep 21, 2020 at 08:29:00PM +0200, Heinrich Schuchardt wrote: > > > On 9/21/20 7:30 PM, Tom Rini wrote: > > > > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: >

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Heinrich Schuchardt
On 9/21/20 10:56 PM, Michael Walle wrote: > Hi, > > Am 2020-09-21 20:50, schrieb Tom Rini: >> On Mon, Sep 21, 2020 at 08:29:00PM +0200, Heinrich Schuchardt wrote: >>> On 9/21/20 7:30 PM, Tom Rini wrote: >>> > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: >>> >> Hi Michael, >>> >> Hi

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Chris Packham
On Tue, Sep 22, 2020 at 8:56 AM Michael Walle wrote: > > Hi, > > Am 2020-09-21 20:50, schrieb Tom Rini: > > On Mon, Sep 21, 2020 at 08:29:00PM +0200, Heinrich Schuchardt wrote: > >> On 9/21/20 7:30 PM, Tom Rini wrote: > >> > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: > >> >> Hi

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Michael Walle
Hi, Am 2020-09-21 20:50, schrieb Tom Rini: On Mon, Sep 21, 2020 at 08:29:00PM +0200, Heinrich Schuchardt wrote: On 9/21/20 7:30 PM, Tom Rini wrote: > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: >> Hi Michael, >> Hi Chris, >> >> On 15.09.20 12:44, Chris Packham wrote: >>> >>> >

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Michael Walle
Hi Stefan, Am 2020-09-21 11:01, schrieb Stefan Roese: Hi Michael, Hi Chris, On 15.09.20 12:44, Chris Packham wrote: On Tue, 15 Sep 2020, 7:54 PM Michael Walle, wrote: Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur: > On Tue, Sep 15, 2020 at 12:56 PM Michael Walle > wrote

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Tom Rini
On Mon, Sep 21, 2020 at 08:29:00PM +0200, Heinrich Schuchardt wrote: > On 9/21/20 7:30 PM, Tom Rini wrote: > > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: > >> Hi Michael, > >> Hi Chris, > >> > >> On 15.09.20 12:44, Chris Packham wrote: > >>> > >>> > >>> On Tue, 15 Sep 2020, 7:54

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Heinrich Schuchardt
On 9/21/20 7:30 PM, Tom Rini wrote: > On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: >> Hi Michael, >> Hi Chris, >> >> On 15.09.20 12:44, Chris Packham wrote: >>> >>> >>> On Tue, 15 Sep 2020, 7:54 PM Michael Walle, wrote: >>> >>> Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur:

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Tom Rini
On Mon, Sep 21, 2020 at 11:01:37AM +0200, Stefan Roese wrote: > Hi Michael, > Hi Chris, > > On 15.09.20 12:44, Chris Packham wrote: > > > > > > On Tue, 15 Sep 2020, 7:54 PM Michael Walle, wrote: > > > > Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur: > > > On Tue, Sep 15, 2020 at 12

Re: u-boot leaves watchdog enabled by default

2020-09-21 Thread Stefan Roese
Hi Michael, Hi Chris, On 15.09.20 12:44, Chris Packham wrote: On Tue, 15 Sep 2020, 7:54 PM Michael Walle, wrote: Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur: > On Tue, Sep 15, 2020 at 12:56 PM Michael Walle > wrote: >> >> Hi Stefan, >> >> it appears th

Re: u-boot leaves watchdog enabled by default

2020-09-15 Thread Chris Packham
On Tue, 15 Sep 2020, 7:54 PM Michael Walle, wrote: > Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur: > > On Tue, Sep 15, 2020 at 12:56 PM Michael Walle > > wrote: > >> > >> Hi Stefan, > >> > >> it appears that since commit 06985289d45 ("watchdog: Implement generic > >> watchdog_reset() version

Re: u-boot leaves watchdog enabled by default

2020-09-15 Thread Michael Walle
Am 2020-09-15 09:44, schrieb Rayagonda Kokatanur: On Tue, Sep 15, 2020 at 12:56 PM Michael Walle wrote: Hi Stefan, it appears that since commit 06985289d45 ("watchdog: Implement generic watchdog_reset() version") - by default - the first watchdog is started unconditionally if CONFIG_WDT is

Re: u-boot leaves watchdog enabled by default

2020-09-15 Thread Rayagonda Kokatanur
On Tue, Sep 15, 2020 at 12:56 PM Michael Walle wrote: > > Hi Stefan, > > it appears that since commit 06985289d45 ("watchdog: Implement generic > watchdog_reset() version") - by default - the first watchdog is started > unconditionally if CONFIG_WDT is set but never stopped before booting > the op

u-boot leaves watchdog enabled by default

2020-09-15 Thread Michael Walle
Hi Stefan, it appears that since commit 06985289d45 ("watchdog: Implement generic watchdog_reset() version") - by default - the first watchdog is started unconditionally if CONFIG_WDT is set but never stopped before booting the operating system. Shouldn't it also be stopped uncondionally? Wh