From: akshay.bhat@timesys.com (Akshay Bhat)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/2] imx6: Implement external watchdog reset
Date: Fri, 30 Oct 2015 13:00:11 -0400 [thread overview]
Message-ID: <5633A21B.2040300@timesys.com> (raw)
In-Reply-To: <1438097230-2653-1-git-send-email-tharvey@gateworks.com>
On 07/28/2015 11:27 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 voltate for
> the CPU when comming out of reset at 800Mhz when it was at 400Mhz prior to
> reset.
>
> This adds a new device-tree property 'ext-reset-output' to fsl-imx-wdt in
> order 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.
>
Hi, was there a reason why this patch did not make it to the upstream
kernel? We would like to use this feature on a Advantech/GE board and I
can rebase the first patch onto the latest linux-next, resubmit if needed.
Thanks, Akshay
> The second patch adds the watchdog configuration and pinmux for Gateworks
> Ventana boards.
>
> I would expect that maintainers of other IMX6 boards that use PMIC's that
> are resettable via WDOG_B to follow-up with similar device-tree patches.
> I've attempted to Cc those individuals here.
>
> [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-March/333689.html
>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: Lucas Stach <l.stach@pengutronix.de>
> Cc: Stefan Roese <sr@denx.de>
> Cc: Iain Paton <ipaton0@gmail.com>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
>
> Tim Harvey (2):
> watchdog: imx2_wdt: add external reset support via 'ext-reset-output'
> dt prop
> ARM: dts: ventana: Add ext-reset support
>
> .../devicetree/bindings/watchdog/fsl-imx-wdt.txt | 2 ++
> arch/arm/boot/dts/imx6qdl-gw51xx.dtsi | 12 ++++++++++++
> arch/arm/boot/dts/imx6qdl-gw52xx.dtsi | 12 ++++++++++++
> arch/arm/boot/dts/imx6qdl-gw53xx.dtsi | 13 +++++++++++++
> arch/arm/boot/dts/imx6qdl-gw54xx.dtsi | 17 +++++++++++++++++
> arch/arm/boot/dts/imx6qdl-gw551x.dtsi | 12 ++++++++++++
> arch/arm/boot/dts/imx6qdl-gw552x.dtsi | 12 ++++++++++++
> drivers/watchdog/imx2_wdt.c | 20 ++++++++++++++++++--
> 8 files changed, 98 insertions(+), 2 deletions(-)
>
next prev parent reply other threads:[~2015-10-30 17:00 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-28 22:54 [PATCH 0/2] imx6: Implement external watchdog reset Tim Harvey
2015-05-28 22:54 ` [PATCH 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset' dt property Tim Harvey
2015-05-29 16:37 ` Lucas Stach
2015-05-28 22:54 ` [PATCH 2/2] ARM: dts: ventana: Add ext-reset support Tim Harvey
2015-05-29 6:17 ` Markus Pargmann
2015-05-29 14:01 ` Tim Harvey
2015-06-04 20:15 ` [PATCH v2 0/2] imx6: Implement external watchdog reset Tim Harvey
2015-06-04 20:15 ` [PATCH v2 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset-output' dt prop Tim Harvey
2015-06-05 16:38 ` Zhi Li
2015-06-25 23:22 ` Tim Harvey
2015-06-22 9:22 ` Lucas Stach
2015-06-28 15:07 ` Shawn Guo
2015-06-29 1:17 ` Shawn Guo
2015-07-02 16:49 ` Tim Harvey
2015-07-03 1:02 ` Shawn Guo
2015-07-28 15:27 ` [PATCH v3 0/2] imx6: Implement external watchdog reset Tim Harvey
2015-07-28 15:27 ` [PATCH v3 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset-output' dt prop Tim Harvey
2015-08-05 13:01 ` Shawn Guo
2015-10-13 14:11 ` Tim Harvey
2015-10-13 14:11 ` Tim Harvey
2015-07-28 15:27 ` [PATCH v3 2/2] ARM: dts: ventana: Add ext-reset support Tim Harvey
2015-10-30 17:00 ` Akshay Bhat [this message]
2015-10-30 17:31 ` [PATCH v3 0/2] imx6: Implement external watchdog reset Tim Harvey
2015-10-30 17:31 ` Tim Harvey
2015-06-04 20:16 ` [PATCH v2 2/2] ARM: dts: ventana: Add ext-reset support Tim Harvey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5633A21B.2040300@timesys.com \
--to=akshay.bhat@timesys.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.