| Message ID | 20240531065723.1085423-1-claudiu.beznea.uj@bp.renesas.com |
|---|---|
| Headers | show |
| Series | watchdog: rzg2l_wdt: Add support for RZ/G3S | expand |
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(-) >
Hi Claudiu, On Fri, Jul 12, 2024 at 5:39 PM claudiu beznea <claudiu.beznea@tuxon.dev> wrote: > Can you please let me know if there is anything you would like me to > address for this series? Your series was merged in watchdog/master two days ago, and is part of next-20240711 and later. Gr{oetje,eeting}s, Geert
Hi, Geert, On 12.07.2024 20:10, Geert Uytterhoeven wrote: > Hi Claudiu, > > On Fri, Jul 12, 2024 at 5:39 PM claudiu beznea <claudiu.beznea@tuxon.dev> wrote: >> Can you please let me know if there is anything you would like me to >> address for this series? > > Your series was merged in watchdog/master two days ago, and is part > of next-20240711 and later. Didn't noticed, thanks! And sorry for the noise. Claudiu > > Gr{oetje,eeting}s, > > Geert >
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(-)