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@bp.renesas.com>
>
> 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(-)
>