Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-04-07 Thread Guenter Roeck
On 03/23/2016 03:05 PM, Matthew McClintock wrote: For certain parts and some versions of TZ, TZ will reset the chip when a BARK is triggered even though it was not configured here. So by default let's configure this BARK time as well. Signed-off-by: Matthew McClintock

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-04-07 Thread Guenter Roeck
On 03/23/2016 03:05 PM, Matthew McClintock wrote: For certain parts and some versions of TZ, TZ will reset the chip when a BARK is triggered even though it was not configured here. So by default let's configure this BARK time as well. Signed-off-by: Matthew McClintock Reviewed-by: Guenter

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Matthew McClintock
On Mar 24, 2016, at 11:17 AM, Guenter Roeck wrote: > >>> Why isn't TZ configuring the bark time to what it wants? I'm lost >>> why we have to do this for them. >> >> So it was done like this to ensure we had a valid upgrade. The bootloader is >> using the watchdog to ensure

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Matthew McClintock
On Mar 24, 2016, at 11:17 AM, Guenter Roeck wrote: > >>> Why isn't TZ configuring the bark time to what it wants? I'm lost >>> why we have to do this for them. >> >> So it was done like this to ensure we had a valid upgrade. The bootloader is >> using the watchdog to ensure the system is

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Guenter Roeck
On Thu, Mar 24, 2016 at 10:46:42AM -0500, Matthew McClintock wrote: > On Mar 23, 2016, at 5:42 PM, Stephen Boyd wrote: > > > > On 03/23, Matthew McClintock wrote: > >> For certain parts and some versions of TZ, TZ will reset the chip > >> when a BARK is triggered even

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Guenter Roeck
On Thu, Mar 24, 2016 at 10:46:42AM -0500, Matthew McClintock wrote: > On Mar 23, 2016, at 5:42 PM, Stephen Boyd wrote: > > > > On 03/23, Matthew McClintock wrote: > >> For certain parts and some versions of TZ, TZ will reset the chip > >> when a BARK is triggered even though it was not

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Matthew McClintock
On Mar 23, 2016, at 5:42 PM, Stephen Boyd wrote: > > On 03/23, Matthew McClintock wrote: >> For certain parts and some versions of TZ, TZ will reset the chip >> when a BARK is triggered even though it was not configured here. So >> by default let's configure this BARK time

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-24 Thread Matthew McClintock
On Mar 23, 2016, at 5:42 PM, Stephen Boyd wrote: > > On 03/23, Matthew McClintock wrote: >> For certain parts and some versions of TZ, TZ will reset the chip >> when a BARK is triggered even though it was not configured here. So >> by default let's configure this BARK time as well. >> > > Why

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-23 Thread Stephen Boyd
On 03/23, Matthew McClintock wrote: > For certain parts and some versions of TZ, TZ will reset the chip > when a BARK is triggered even though it was not configured here. So > by default let's configure this BARK time as well. > Why isn't TZ configuring the bark time to what it wants? I'm lost

Re: [PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-23 Thread Stephen Boyd
On 03/23, Matthew McClintock wrote: > For certain parts and some versions of TZ, TZ will reset the chip > when a BARK is triggered even though it was not configured here. So > by default let's configure this BARK time as well. > Why isn't TZ configuring the bark time to what it wants? I'm lost

[PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-23 Thread Matthew McClintock
For certain parts and some versions of TZ, TZ will reset the chip when a BARK is triggered even though it was not configured here. So by default let's configure this BARK time as well. Signed-off-by: Matthew McClintock --- drivers/watchdog/qcom-wdt.c | 5 + 1 file

[PATCH 08/17] watchdog: qcom: configure BARK time in addition to BITE time

2016-03-23 Thread Matthew McClintock
For certain parts and some versions of TZ, TZ will reset the chip when a BARK is triggered even though it was not configured here. So by default let's configure this BARK time as well. Signed-off-by: Matthew McClintock --- drivers/watchdog/qcom-wdt.c | 5 + 1 file changed, 5 insertions(+)