On 5/28/24 13:13, Rasmus Villemoes wrote:
watchdog_reset is no more. That is, it is no longer called from
anywhere, so any leftover definitions are just dead weight, and
references in comments need to be updated.

I don't have access to any m68k, sh or powerpc boards. Only lightly
compile-tested.

Apart from the two sh4 patches, these are independent of each other
and can be applied individually.

Cc: Angelo Dureghello <ang...@kernel-space.org>
Cc: Christophe Leroy <christophe.le...@csgroup.eu>
Cc: Huan Wang <alison.w...@nxp.com>
Cc: Nobuhiro Iwamatsu <iwama...@nigauri.org>

Thanks for continuing the cleanup here:

Reviewed-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

Rasmus Villemoes (7):
   m68k: remove dead code
   wdt-uclass: watchdog_reset cleanup
   serial: ns16550: fix comment to mention schedule instead of
     watchdog_reset
   sh4: move reset_cpu() from watchdog.c to cpu.c
   sh4: remove watchdog.c file
   powerpc: mpc83xx: remove unused watchdog_reset() function
   powerpc: mpc85xx: remove dead watchdog-related code

  arch/m68k/cpu/mcf52x2/cpu.c    | 108 ---------------------------------
  arch/powerpc/cpu/mpc83xx/cpu.c |  15 -----
  arch/powerpc/cpu/mpc85xx/cpu.c |  31 ----------
  arch/sh/cpu/sh4/Makefile       |   2 +-
  arch/sh/cpu/sh4/cpu.c          |  10 +++
  arch/sh/cpu/sh4/watchdog.c     |  61 -------------------
  drivers/serial/ns16550.c       |   8 +--
  drivers/watchdog/wdt-uclass.c  |  19 +-----
  include/watchdog.h             |   3 -
  9 files changed, 17 insertions(+), 240 deletions(-)
  delete mode 100644 arch/sh/cpu/sh4/watchdog.c


Viele Grüße,
Stefan Roese

--
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to