From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Subject: Re: [PATCH v5 1/2] watchdog: imx2_wdt: add external reset support via dt prop To: Guenter Roeck , Tim Harvey , linux-watchdog@vger.kernel.org, Shawn Guo References: <1459523804-11390-1-git-send-email-tharvey@gateworks.com> <1459523804-11390-2-git-send-email-tharvey@gateworks.com> <570704E1.6010700@roeck-us.net> Cc: linux-arm-kernel@lists.infradead.org, Fabio Estevam , Lucas Stach , Wim Van Sebroeck , Stefan Roese , Iain Paton , Sascha Hauer From: Akshay Bhat Message-ID: <571505B6.7030808@timesys.com> Date: Mon, 18 Apr 2016 12:05:10 -0400 MIME-Version: 1.0 In-Reply-To: <570704E1.6010700@roeck-us.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit List-ID: On 04/07/2016 09:09 PM, Guenter Roeck wrote: > On 04/01/2016 08:16 AM, Tim Harvey wrote: >> The IMX6 watchdog supports assertion of a signal (WDOG_B) which >> can be pinmux'd to an external pin. This is typically used for boards >> that >> have PMIC's in control of the IMX6 power rails. In fact, failure to use >> such an external reset on boards with external PMIC's can result in >> various >> hangs due to the IMX6 not being fully reset [1] as well as the board >> failing >> to reset because its PMIC has not been reset to provide adequate >> voltage for >> the CPU when coming out of reset at 800Mhz. >> >> This uses a new device-tree property 'fsl,ext-reset-output' to >> indicate the >> board has such a reset and to cause the watchdog to be configured to >> assert >> WDOG_B instead of an internal reset both on a watchdog timeout and in >> system_restart. >> >> [1] >> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-March/333689.html >> >> >> Cc: Fabio Estevam >> Cc: Lucas Stach >> Cc: Stefan Roese >> Cc: Iain Paton >> Cc: Sascha Hauer >> Cc: Akshay Bhat >> Cc: Guenter Roeck >> Signed-off-by: Tim Harvey >> Reviewed-by: Lucas Stach >> Acked-by: Shawn Guo > > Reviewed-by: Guenter Roeck > Hi Wim, When will this patch be applied to the watchdog maintainers git tree? I have another board that can use this feature and I am guessing I should wait before for the above patch to be applied before submitting mine. Thanks, Akshay From mboxrd@z Thu Jan 1 00:00:00 1970 From: akshay.bhat@timesys.com (Akshay Bhat) Date: Mon, 18 Apr 2016 12:05:10 -0400 Subject: [PATCH v5 1/2] watchdog: imx2_wdt: add external reset support via dt prop In-Reply-To: <570704E1.6010700@roeck-us.net> References: <1459523804-11390-1-git-send-email-tharvey@gateworks.com> <1459523804-11390-2-git-send-email-tharvey@gateworks.com> <570704E1.6010700@roeck-us.net> Message-ID: <571505B6.7030808@timesys.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 04/07/2016 09:09 PM, Guenter Roeck wrote: > On 04/01/2016 08:16 AM, Tim Harvey wrote: >> The IMX6 watchdog supports assertion of a signal (WDOG_B) which >> can be pinmux'd to an external pin. This is typically used for boards >> that >> have PMIC's in control of the IMX6 power rails. In fact, failure to use >> such an external reset on boards with external PMIC's can result in >> various >> hangs due to the IMX6 not being fully reset [1] as well as the board >> failing >> to reset because its PMIC has not been reset to provide adequate >> voltage for >> the CPU when coming out of reset at 800Mhz. >> >> This uses a new device-tree property 'fsl,ext-reset-output' to >> indicate the >> board has such a reset and to cause the watchdog to be configured to >> assert >> WDOG_B instead of an internal reset both on a watchdog timeout and in >> system_restart. >> >> [1] >> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-March/333689.html >> >> >> Cc: Fabio Estevam >> Cc: Lucas Stach >> Cc: Stefan Roese >> Cc: Iain Paton >> Cc: Sascha Hauer >> Cc: Akshay Bhat >> Cc: Guenter Roeck >> Signed-off-by: Tim Harvey >> Reviewed-by: Lucas Stach >> Acked-by: Shawn Guo > > Reviewed-by: Guenter Roeck > Hi Wim, When will this patch be applied to the watchdog maintainers git tree? I have another board that can use this feature and I am guessing I should wait before for the above patch to be applied before submitting mine. Thanks, Akshay