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@xxxxxxxxx/                                         
>                                                                                 
> [1] https://lore.kernel.org/all/20211206064732.280375-1-andrej.picej@xxxxxxxxx/
> 
> 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@xxxxxxxxx
--
Norik systems d.o.o.
Your embedded software partner
Slovenia, EU
phone: +386-41-540-545
email: info@xxxxxxxxx