Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree properties for config

From	Guenter Roeck <linux@roeck-us.net>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree properties for config
Date	2017-06-28 20:00 +0200
Message-ID	<tXppY-t5-83@gated-at.bofh.it> (permalink)
References	(5 earlier) <tXmLn-8q5-9@gated-at.bofh.it> <tXmLn-8q5-13@gated-at.bofh.it> <tXnxN-hR-47@gated-at.bofh.it> <tXnHr-AC-11@gated-at.bofh.it> <tXnHs-AC-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Jun 28, 2017 at 11:09:15AM -0500, Christopher Bostic wrote:
> 
> >>
> >>Why would there even be a need configure for no system reset?  The external
> >>mode 'wdt ext: External signal enable after timeout' can be sent off chip
> >>even if system reset is disabled.  The watchdog is doing something in this
> >>case and would require wdt instantiation.
> >>
> >Wouldn't that be "aspeed,external-signal" ?
> Yes that optional parameter would need to be specified in that situation, as
> well as 'aspeed,no-system-reset'.
> 

Ok, makes sense (after looking into the datasheet). The missing detail
is that there is a separate configuration bit for "Reset system after
timeout", which is set in addition to reset_{sys,soc,arm}.

Thanks,
Guenter

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree properties for config Christopher Bostic <cbostic@linux.vnet.ibm.com> - 2017-06-27 23:20 +0200
  Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Guenter Roeck <linux@roeck-us.net> - 2017-06-28 13:40 +0200
    Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Christopher Bostic <cbostic@linux.vnet.ibm.com> - 2017-06-28 16:40 +0200
      Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Guenter Roeck <linux@roeck-us.net> - 2017-06-28 17:00 +0200
        Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Christopher Bostic <cbostic@linux.vnet.ibm.com> - 2017-06-28 17:10 +0200
          Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Guenter Roeck <linux@roeck-us.net> - 2017-06-28 17:10 +0200
            Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Christopher Bostic <cbostic@linux.vnet.ibm.com> - 2017-06-28 18:00 +0200
              Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Guenter Roeck <linux@roeck-us.net> - 2017-06-28 18:10 +0200
                Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Christopher Bostic <cbostic@linux.vnet.ibm.com> - 2017-06-28 18:10 +0200
                Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree  properties for config Guenter Roeck <linux@roeck-us.net> - 2017-06-28 20:00 +0200

csiph-web