From: Peng Fan <peng.fan@oss.nxp.com>
To: Primoz Fiser <primoz.fiser@norik.com>
Cc: Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>, Frank Li <Frank.Li@nxp.com>,
Sascha Hauer <s.hauer@pengutronix.de>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
Fabio Estevam <festevam@gmail.com>,
Support Opensource <support.opensource@diasemi.com>,
Lee Jones <lee@kernel.org>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Guenter Roeck <linux@roeck-us.net>,
devicetree@vger.kernel.org, imx@lists.linux.dev,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, mfd@lists.linux.dev,
linux-watchdog@vger.kernel.org, upstream@lists.phytec.de,
andrej.picej@norik.com
Subject: Re: [PATCH v2 0/4] da9062/da9061 watchdog timeout mode
Date: Thu, 15 Oct 2026 17:13:43 +0800 [thread overview]
Message-ID: <atCZR27iTkoDuxoG@shlinux89> (raw)
In-Reply-To: <20260924065044.368341-1-primoz.fiser@norik.com>
On Thu, Sep 24, 2026 at 08:50:40AM +0200, Primoz Fiser wrote:
>This is a revival of a series originally posted by Andrej Picej in
>December 2021 [1]. It lets the DA9061/2 PMIC watchdog be configured via
>the "dlg,wdt-sd" DT property, to trigger a full SHUTDOWN instead of the
>default POWERDOWN on watchdog timeout, and enables that on the PHYTEC
>phyCORE-i.MX6 SOM.
>
>From v5 of original series this resend drops patch 4/5 ("dt-bindings:
>watchdog: da9062: add watchdog timeout mode") because it already landed
>upstream as commit 382256b219fb, later converted from .txt to YAML by
>bd888a4377ae. Thus dlg,da9062-watchdog.yaml already documents dlg,wdt-sd
>property.
>
>Patches 1 and 2 (the mfd prerequisites making register CONFIG_I
>writable) needed a rebase: an unrelated commit has since added
>CONFIG_J, the register immediately adjacent to CONFIG_I, to the same
>writeable-range tables. Rather than reapplying the original single-
>register insert next to it, these merge CONFIG_I and CONFIG_J into one
>range, matching the existing CONFIG_G-CONFIG_K style already used in
>this file.
>
>Patch 3 (the watchdog driver support) still applies unmodified.
>
>Patch 4 only needed its target path updated for the arch/arm/boot/dts
>vendor sub-directory move (imx6qdl-phytec-phycore-som.dtsi is now under
>nxp/imx/); its content is unchanged.
>
>Changes in v2:
>
>Patch 1 and 2:
>- capitalize subject description
>
>Patch 3:
>- implement Guenter's remarks (code refactor and update commit message)
>
>Patch 4:
>- capitalize subject description
>
>Link to v1: https://lore.kernel.org/all/20260923111719.2032012-1-primoz.fiser@norik.com/
>
>[1] https://lore.kernel.org/all/20211206064732.280375-1-andrej.picej@norik.com/
>
>Andrej Picej (3):
> mfd: da9061: Make register CONFIG_I writable
> watchdog: da9062: Add support for dlg,wdt-sd property
> ARM: dts: imx6: phycore-som: Set watchdog timeout mode to shutdown
>
>Stefan Christ (1):
> mfd: da9062: Make register CONFIG_I writable
>
LGTM:
Reviewed-by: Peng Fan <peng.fan@nxp.com>
prev parent reply other threads:[~2026-10-15 9:09 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-24 6:50 [PATCH v2 0/4] da9062/da9061 watchdog timeout mode Primoz Fiser
2026-09-24 6:50 ` [PATCH v2 1/4] mfd: da9062: Make register CONFIG_I writable Primoz Fiser
2026-09-24 6:58 ` sashiko-bot
2026-09-24 6:50 ` [PATCH v2 2/4] mfd: da9061: " Primoz Fiser
2026-09-24 6:59 ` sashiko-bot
2026-09-24 6:50 ` [PATCH v2 3/4] watchdog: da9062: Add support for dlg,wdt-sd property Primoz Fiser
2026-09-24 6:56 ` sashiko-bot
2026-09-24 6:50 ` [PATCH v2 4/4] ARM: dts: imx6: phycore-som: Set watchdog timeout mode to shutdown Primoz Fiser
2026-09-24 6:57 ` sashiko-bot
2026-10-15 8:47 ` [PATCH v2 0/4] da9062/da9061 watchdog timeout mode Primoz Fiser
2026-10-15 9:13 ` Peng Fan [this message]
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=atCZR27iTkoDuxoG@shlinux89 \
--to=peng.fan@oss.nxp.com \
--cc=Frank.Li@nxp.com \
--cc=andrej.picej@norik.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=lee@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mfd@lists.linux.dev \
--cc=primoz.fiser@norik.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=support.opensource@diasemi.com \
--cc=upstream@lists.phytec.de \
--cc=wim@linux-watchdog.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.