Hi, watchdog maintainers, Can you please let me know if there is anything you would like me to address for this series? Thank you, Claudiu Beznea On 31.05.2024 09:57, Claudiu wrote: > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > > Hi, > > Series adds watchdog support for Renesas RZ/G3S (R9A08G045) SoC. > > Patches do the following: > - patch 1/9 makes the driver depend on ARCH_RZG2L || ARCH_R9A09G011 > - patch 2/9 makes the driver depend on PM > - patches 3-7/9 adds fixes and cleanups for the watchdog driver > - patch 8/9 adds suspend to RAM to the watchdog driver (to be used by > RZ/G3S) > - patch 9/9 documents the RZ/G3S support > > Thank you, > Claudiu Beznea > > Changes in v9: > - dropped patch "watchdog: rzg2l_wdt: Power on the PM domain in > rzg2l_wdt_restart()" until further clarifications; this will not > impact any behavior as the RZ/G3S power domain support is not > instantiated > - on patch "watchdog: rzg2l_wdt: Remove reset de-assert from probe" > call pm_runtime_put() in case the reset deassert fails > > Changes in v8: > - added patch 9 > - collected tags > > Changes in v7: > - updated the dependency on patch 2/9 > > Changes in v6: > - update patch 2/9 description > - fixed the dependency on COMPILE_TEST previously introduced in patch > 2/9 > > Changes in v5: > - updated description of patch 2/9 > - simplify the code in patch 2/9 by using on a new line: > depends on PM || COMPILE_TEST > > Changes in v4: > - added patch "watchdog: rzg2l_wdt: Restrict the driver to ARCH_RZG2L and > ARCH_R9A09G011" > - collected tags > > Changes in v3: > - make driver depend on PM not select it > - drop patches already accepted (patches 1, 10, 11 from v2) > - re-arranged the tags in patch 8/8 as they were messed by b4 am/shazam > > Changes in v2: > - added patch "watchdog: rzg2l_wdt: Select PM" > - propagate the return status of rzg2l_wdt_start() to it's callers > in patch "watchdog: rzg2l_wdt: Use pm_runtime_resume_and_get()" > - propagate the return status of rzg2l_wdt_stop() to it's callers > in patch "watchdog: rzg2l_wdt: Check return status of pm_runtime_put()" > - removed pm_ptr() from patch "watchdog: rzg2l_wdt: Add suspend/resume support" > - s/G2UL/G2L in patch "dt-bindings: watchdog: renesas,wdt: Document RZ/G3S support" > - collected tags > > Claudiu Beznea (9): > watchdog: rzg2l_wdt: Restrict the driver to ARCH_RZG2L and > ARCH_R9A09G011 > watchdog: rzg2l_wdt: Make the driver depend on PM > watchdog: rzg2l_wdt: Use pm_runtime_resume_and_get() > watchdog: rzg2l_wdt: Check return status of pm_runtime_put() > watchdog: rzg2l_wdt: Remove reset de-assert from probe > watchdog: rzg2l_wdt: Remove comparison with zero > watchdog: rzg2l_wdt: Rely on the reset driver for doing proper reset > watchdog: rzg2l_wdt: Add suspend/resume support > dt-bindings: watchdog: renesas,wdt: Document RZ/G3S support > > .../bindings/watchdog/renesas,wdt.yaml | 1 + > drivers/watchdog/Kconfig | 3 +- > drivers/watchdog/rzg2l_wdt.c | 113 ++++++++++-------- > 3 files changed, 66 insertions(+), 51 deletions(-) >