[PATCH v2 0/4] da9062/da9061 watchdog timeout mode

[PATCH v2 0/4] da9062/da9061 watchdog timeout mode

[Primoz Fiser]

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

 .../nxp/imx/imx6qdl-phytec-phycore-som.dtsi    |  1 +
 drivers/mfd/da9062-core.c                      |  4 ++--
 drivers/watchdog/da9062_wdt.c                  | 18 ++++++++++++++++++
 3 files changed, 21 insertions(+), 2 deletions(-)

-- 
2.34.1

[PATCH v2 1/4] mfd: da9062: Make register CONFIG_I writable

[Primoz Fiser]

From: Stefan Christ <s.christ@phytec.de>

Make the config register CONFIG_I writable to change the watchdog mode.

Signed-off-by: Stefan Christ <s.christ@phytec.de>
Signed-off-by: Andrej Picej <andrej.picej@norik.com>
Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
Acked-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>
---
Changes in v2:
- capitalize subject description

 drivers/mfd/da9062-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
index ad198183aff7..32dd24483ee0 100644
--- a/drivers/mfd/da9062-core.c
+++ b/drivers/mfd/da9062-core.c
@@ -531,7 +531,7 @@ static const struct regmap_range da9062_aa_writeable_ranges[] = {
 	regmap_reg_range(DA9062AA_VBUCK3_B, DA9062AA_VBUCK3_B),
 	regmap_reg_range(DA9062AA_VLDO1_B, DA9062AA_VLDO4_B),
 	regmap_reg_range(DA9062AA_BBAT_CONT, DA9062AA_BBAT_CONT),
-	regmap_reg_range(DA9062AA_CONFIG_J, DA9062AA_CONFIG_J),
+	regmap_reg_range(DA9062AA_CONFIG_I, DA9062AA_CONFIG_J),
 	regmap_reg_range(DA9062AA_GP_ID_0, DA9062AA_GP_ID_19),
 };
 
-- 
2.34.1

Re: [PATCH v2 1/4] mfd: da9062: Make register CONFIG_I writable

[sashiko-bot]

> From: Stefan Christ <s.christ@phytec.de>
> 
> Make the config register CONFIG_I writable to change the watchdog mode.
> 
> Signed-off-by: Stefan Christ <s.christ@phytec.de>
> Signed-off-by: Andrej Picej <andrej.picej@norik.com>
> Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
> Acked-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260924065044.368341-1-primoz.fiser@norik.com?part=1

[PATCH v2 2/4] mfd: da9061: Make register CONFIG_I writable

[Primoz Fiser]

From: Andrej Picej <andrej.picej@norik.com>

Make the config register CONFIG_I writable to change the watchdog mode.

Signed-off-by: Andrej Picej <andrej.picej@norik.com>
Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>
---
Changes in v2:
- capitalize subject description

 drivers/mfd/da9062-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
index 32dd24483ee0..c987e533824d 100644
--- a/drivers/mfd/da9062-core.c
+++ b/drivers/mfd/da9062-core.c
@@ -427,7 +427,7 @@ static const struct regmap_range da9061_aa_writeable_ranges[] = {
 	regmap_reg_range(DA9062AA_VBUCK1_B, DA9062AA_VBUCK4_B),
 	regmap_reg_range(DA9062AA_VBUCK3_B, DA9062AA_VBUCK3_B),
 	regmap_reg_range(DA9062AA_VLDO1_B, DA9062AA_VLDO4_B),
-	regmap_reg_range(DA9062AA_CONFIG_J, DA9062AA_CONFIG_J),
+	regmap_reg_range(DA9062AA_CONFIG_I, DA9062AA_CONFIG_J),
 	regmap_reg_range(DA9062AA_GP_ID_0, DA9062AA_GP_ID_19),
 };
 
-- 
2.34.1

Re: [PATCH v2 2/4] mfd: da9061: Make register CONFIG_I writable

[sashiko-bot]

> From: Andrej Picej <andrej.picej@norik.com>
> 
> Make the config register CONFIG_I writable to change the watchdog mode.
> 
> Signed-off-by: Andrej Picej <andrej.picej@norik.com>
> Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
> Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
> Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260924065044.368341-1-primoz.fiser@norik.com?part=2

[PATCH v2 3/4] watchdog: da9062: Add support for dlg,wdt-sd property

[Primoz Fiser]

From: Andrej Picej <andrej.picej@norik.com>

Implement a method to change watchdog timeout configuration based on the
"dlg,wdt-sd" devicetree property. Setting the WATCHDOG_SD bit enables
SHUTDOWN mode, and clearing it enables POWERDOWN mode on watchdog
timeout.

If no devicetree property is specified, the WATCHDOG_SD bit stays in the
default configuration, not breaking behaviour of devices which might
depend on default fuse configuration.

Signed-off-by: Andrej Picej <andrej.picej@norik.com>
Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
Acked-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>
---
Changes in v2:
- implement Guenter's remarks (code refactor and update commit message)

 drivers/watchdog/da9062_wdt.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/drivers/watchdog/da9062_wdt.c b/drivers/watchdog/da9062_wdt.c
index 3d7bf2f351db..0c007847b7c4 100644
--- a/drivers/watchdog/da9062_wdt.c
+++ b/drivers/watchdog/da9062_wdt.c
@@ -206,6 +206,8 @@ static int da9062_wdt_probe(struct platform_device *pdev)
 	unsigned int timeout;
 	struct da9062 *chip;
 	struct da9062_watchdog *wdt;
+	int ret;
+	u32 val;
 
 	chip = dev_get_drvdata(dev->parent);
 	if (!chip)
@@ -245,6 +247,22 @@ static int da9062_wdt_probe(struct platform_device *pdev)
 		set_bit(WDOG_HW_RUNNING, &wdt->wdtdev.status);
 	}
 
+	/*
+	 * Configure what happens on watchdog timeout. Can be specified with
+	 * the "dlg,wdt-sd" devicetree property (0 -> POWERDOWN, non-zero ->
+	 * SHUTDOWN). If the property is not set, leave the default in place.
+	 */
+	ret = device_property_read_u32(dev, "dlg,wdt-sd", &val);
+	if (!ret) {
+		ret = regmap_update_bits(wdt->hw->regmap,
+					 DA9062AA_CONFIG_I,
+					 DA9062AA_WATCHDOG_SD_MASK,
+					 val ? DA9062AA_WATCHDOG_SD_MASK : 0);
+		if (ret)
+			return dev_err_probe(dev, ret,
+					     "failed to set wdt reset mode\n");
+	}
+
 	return devm_watchdog_register_device(dev, &wdt->wdtdev);
 }
 
-- 
2.34.1

Re: [PATCH v2 3/4] watchdog: da9062: Add support for dlg,wdt-sd property

[sashiko-bot]

> From: Andrej Picej <andrej.picej@norik.com>
> 
> Implement a method to change watchdog timeout configuration based on the
> "dlg,wdt-sd" devicetree property. Setting the WATCHDOG_SD bit enables
> SHUTDOWN mode, and clearing it enables POWERDOWN mode on watchdog
> timeout.
> 
> If no devicetree property is specified, the WATCHDOG_SD bit stays in the
> default configuration, not breaking behaviour of devices which might
> depend on default fuse configuration.
> 
> Signed-off-by: Andrej Picej <andrej.picej@norik.com>
> Tested-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
> Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
> Acked-by: Guenter Roeck <linux@roeck-us.net>
> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260924065044.368341-1-primoz.fiser@norik.com?part=3

[PATCH v2 4/4] ARM: dts: imx6: phycore-som: Set watchdog timeout mode to shutdown

[Primoz Fiser]

From: Andrej Picej <andrej.picej@norik.com>

Enable system restart when the watchdog timeout occurs.

Signed-off-by: Andrej Picej <andrej.picej@norik.com>
Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>
---
Changes in v2:
- capitalize subject description

 arch/arm/boot/dts/nxp/imx/imx6qdl-phytec-phycore-som.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-phytec-phycore-som.dtsi b/arch/arm/boot/dts/nxp/imx/imx6qdl-phytec-phycore-som.dtsi
index 85e278eb2016..ddbfecd591f5 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-phytec-phycore-som.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-phytec-phycore-som.dtsi
@@ -115,6 +115,7 @@ da9062_onkey: onkey {
 		watchdog {
 			compatible = "dlg,da9062-watchdog";
 			dlg,use-sw-pm;
+			dlg,wdt-sd = <1>;
 		};
 
 		thermal {
-- 
2.34.1

Re: [PATCH v2 4/4] ARM: dts: imx6: phycore-som: Set watchdog timeout mode to shutdown

[sashiko-bot]

> From: Andrej Picej <andrej.picej@norik.com>
> 
> Enable system restart when the watchdog timeout occurs.
> 
> Signed-off-by: Andrej Picej <andrej.picej@norik.com>
> Reviewed-by: Adam Thomson <Adam.Thomson.Opensource@diasemi.com>
> Signed-off-by: Primoz Fiser <primoz.fiser@norik.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260924065044.368341-1-primoz.fiser@norik.com?part=4

Re: [PATCH v2 0/4] da9062/da9061 watchdog timeout mode

[Primoz Fiser]

Hi,

gentle ping, can we get this applied soon?

Or is there still something to fix?

BR,
Primoz


On 9/24/26 08:50, 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
> 
>  .../nxp/imx/imx6qdl-phytec-phycore-som.dtsi    |  1 +
>  drivers/mfd/da9062-core.c                      |  4 ++--
>  drivers/watchdog/da9062_wdt.c                  | 18 ++++++++++++++++++
>  3 files changed, 21 insertions(+), 2 deletions(-)
> 

-- 
Primoz Fiser
phone: +386-41-390-545
email: primoz.fiser@norik.com
--
Norik systems d.o.o.
Your embedded software partner
Slovenia, EU
phone: +386-41-540-545
email: info@norik.com

Re: [PATCH v2 0/4] da9062/da9061 watchdog timeout mode

[Peng Fan]

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>