* [PATCH v3 1/5] drivers: watchdog: add mcf watchdog support
2023-07-24 6:49 [PATCH v3 0/5] m68k: add ColdFire watchdog driver Angelo Dureghello
@ 2023-07-24 6:49 ` Angelo Dureghello
2023-07-24 7:40 ` Stefan Roese
2023-07-24 6:49 ` [PATCH v3 2/5] m68k: move watchdog functions in mcf_wdt driver Angelo Dureghello
` (3 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Angelo Dureghello @ 2023-07-24 6:49 UTC (permalink / raw)
To: trini, sr; +Cc: u-boot, Angelo Dureghello
This watchdog driver applies to the following
mcf families:
- mcf52x2 (5271 5275 5282)
- mcf532x (5329 5373)
- mcf523x (5235)
Cpu's not listed for each family does not have WDT module.
Note, after some attempts testing by qemu on 5208 i
finally abandoned, watchdog seems not implemented properly.
The driver has been tested in a real M5282EVM.
Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
---
Changes for v2:
- remove unnecessary hardcoded timeouts
- remove unnecessary hw_watchdog_xxx stuff
- rewrite wdog module reg calculation
- using IS_ENABLED() where possible
Changes for v3:
- remove hardcoded 4s test
---
drivers/watchdog/Kconfig | 7 ++
drivers/watchdog/Makefile | 1 +
drivers/watchdog/mcf_wdt.c | 135 +++++++++++++++++++++++++++++++++++++
3 files changed, 143 insertions(+)
create mode 100644 drivers/watchdog/mcf_wdt.c
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index 646663528a..07fc4940e9 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -178,6 +178,13 @@ config WDT_MAX6370
help
Select this to enable max6370 watchdog timer.
+config WDT_MCF
+ bool "ColdFire family watchdog timer support"
+ depends on WDT
+ help
+ Select this to enable ColdFire watchdog timer,
+ which supports mcf52x2 mcf532x mcf523x families.
+
config WDT_MESON_GXBB
bool "Amlogic watchdog timer support"
depends on WDT
diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
index fd5d9c7376..eef786f5e7 100644
--- a/drivers/watchdog/Makefile
+++ b/drivers/watchdog/Makefile
@@ -31,6 +31,7 @@ obj-$(CONFIG_WDT_CDNS) += cdns_wdt.o
obj-$(CONFIG_WDT_FTWDT010) += ftwdt010_wdt.o
obj-$(CONFIG_WDT_GPIO) += gpio_wdt.o
obj-$(CONFIG_WDT_MAX6370) += max6370_wdt.o
+obj-$(CONFIG_WDT_MCF) += mcf_wdt.o
obj-$(CONFIG_WDT_MESON_GXBB) += meson_gxbb_wdt.o
obj-$(CONFIG_WDT_MPC8xxx) += mpc8xxx_wdt.o
obj-$(CONFIG_WDT_MT7620) += mt7620_wdt.o
diff --git a/drivers/watchdog/mcf_wdt.c b/drivers/watchdog/mcf_wdt.c
new file mode 100644
index 0000000000..b36488bb5b
--- /dev/null
+++ b/drivers/watchdog/mcf_wdt.c
@@ -0,0 +1,135 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * mcf_wdt.c - driver for ColdFire on-chip watchdog
+ *
+ * Author: Angelo Dureghello <angelo@kernel-space.org>
+ *
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <hang.h>
+#include <asm/io.h>
+#include <wdt.h>
+#include <linux/bitops.h>
+
+#define DIVIDER_5XXX 4096
+#define DIVIDER_5282 8192
+
+#define WCR_EN BIT(0)
+#define WCR_HALTED BIT(1)
+#define WCR_DOZE BIT(2)
+#define WCR_WAIT BIT(3)
+
+struct watchdog_regs {
+ u16 wcr; /* Control */
+ u16 wmr; /* Service */
+ u16 wcntr; /* Counter */
+ u16 wsr; /* Reset Status */
+};
+
+static void mcf_watchdog_reset(struct watchdog_regs *wdog)
+{
+ if (!IS_ENABLED(CONFIG_WATCHDOG_RESET_DISABLE)) {
+ writew(0x5555, &wdog->wsr);
+ writew(0xaaaa, &wdog->wsr);
+ }
+}
+
+static void mcf_watchdog_init(struct watchdog_regs *wdog, u32 fixed_divider,
+ u64 timeout_msecs)
+{
+ u32 wdog_module, cycles_per_sec;
+
+ cycles_per_sec = CFG_SYS_CLK / fixed_divider;
+
+ wdog_module = cycles_per_sec * ((u32)timeout_msecs / 1000);
+ wdog_module += (cycles_per_sec / 1000) * ((u32)timeout_msecs % 1000);
+
+ /* Limit check, max 16 bits */
+ if (wdog_module > 0xffff)
+ wdog_module = 0xffff;
+
+ /* Set timeout and enable watchdog */
+ writew((u16)wdog_module, &wdog->wmr);
+ writew(WCR_EN, &wdog->wcr);
+
+ mcf_watchdog_reset(wdog);
+}
+
+struct mcf_wdt_priv {
+ void __iomem *base;
+ u32 fixed_divider;
+};
+
+static int mcf_wdt_expire_now(struct udevice *dev, ulong flags)
+{
+ hang();
+
+ return 0;
+}
+
+static int mcf_wdt_reset(struct udevice *dev)
+{
+ struct mcf_wdt_priv *priv = dev_get_priv(dev);
+
+ mcf_watchdog_reset(priv->base);
+
+ return 0;
+}
+
+static int mcf_wdt_start(struct udevice *dev, u64 timeout, ulong flags)
+{
+ struct mcf_wdt_priv *priv = dev_get_priv(dev);
+
+ /* Timeout from fdt (timeout) comes in milliseconds */
+ mcf_watchdog_init(priv->base, priv->fixed_divider, timeout);
+
+ return 0;
+}
+
+static int mcf_wdt_stop(struct udevice *dev)
+{
+ struct mcf_wdt_priv *priv = dev_get_priv(dev);
+ struct watchdog_regs *wdog = (struct watchdog_regs *)priv->base;
+
+ setbits_be16(&wdog->wcr, WCR_HALTED);
+
+ return 0;
+}
+
+static int mcf_wdt_probe(struct udevice *dev)
+{
+ struct mcf_wdt_priv *priv = dev_get_priv(dev);
+
+ priv->base = dev_read_addr_ptr(dev);
+ if (!priv->base)
+ return -ENOENT;
+
+ priv->fixed_divider = (u32)dev_get_driver_data(dev);
+
+ return 0;
+}
+
+static const struct wdt_ops mcf_wdt_ops = {
+ .start = mcf_wdt_start,
+ .stop = mcf_wdt_stop,
+ .reset = mcf_wdt_reset,
+ .expire_now = mcf_wdt_expire_now,
+};
+
+static const struct udevice_id mcf_wdt_ids[] = {
+ { .compatible = "fsl,mcf5208-wdt", .data = DIVIDER_5XXX },
+ { .compatible = "fsl,mcf5282-wdt", .data = DIVIDER_5282 },
+ {}
+};
+
+U_BOOT_DRIVER(mcf_wdt) = {
+ .name = "mcf_wdt",
+ .id = UCLASS_WDT,
+ .of_match = mcf_wdt_ids,
+ .probe = mcf_wdt_probe,
+ .ops = &mcf_wdt_ops,
+ .priv_auto = sizeof(struct mcf_wdt_priv),
+ .flags = DM_FLAG_PRE_RELOC,
+};
--
2.41.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v3 1/5] drivers: watchdog: add mcf watchdog support
2023-07-24 6:49 ` [PATCH v3 1/5] drivers: watchdog: add mcf watchdog support Angelo Dureghello
@ 2023-07-24 7:40 ` Stefan Roese
0 siblings, 0 replies; 11+ messages in thread
From: Stefan Roese @ 2023-07-24 7:40 UTC (permalink / raw)
To: Angelo Dureghello, trini; +Cc: u-boot
On 7/24/23 08:49, Angelo Dureghello wrote:
> This watchdog driver applies to the following
> mcf families:
>
> - mcf52x2 (5271 5275 5282)
> - mcf532x (5329 5373)
> - mcf523x (5235)
>
> Cpu's not listed for each family does not have WDT module.
>
> Note, after some attempts testing by qemu on 5208 i
> finally abandoned, watchdog seems not implemented properly.
>
> The driver has been tested in a real M5282EVM.
>
> Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> Changes for v2:
> - remove unnecessary hardcoded timeouts
> - remove unnecessary hw_watchdog_xxx stuff
> - rewrite wdog module reg calculation
> - using IS_ENABLED() where possible
> Changes for v3:
> - remove hardcoded 4s test
> ---
> drivers/watchdog/Kconfig | 7 ++
> drivers/watchdog/Makefile | 1 +
> drivers/watchdog/mcf_wdt.c | 135 +++++++++++++++++++++++++++++++++++++
> 3 files changed, 143 insertions(+)
> create mode 100644 drivers/watchdog/mcf_wdt.c
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index 646663528a..07fc4940e9 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -178,6 +178,13 @@ config WDT_MAX6370
> help
> Select this to enable max6370 watchdog timer.
>
> +config WDT_MCF
> + bool "ColdFire family watchdog timer support"
> + depends on WDT
> + help
> + Select this to enable ColdFire watchdog timer,
> + which supports mcf52x2 mcf532x mcf523x families.
> +
> config WDT_MESON_GXBB
> bool "Amlogic watchdog timer support"
> depends on WDT
> diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
> index fd5d9c7376..eef786f5e7 100644
> --- a/drivers/watchdog/Makefile
> +++ b/drivers/watchdog/Makefile
> @@ -31,6 +31,7 @@ obj-$(CONFIG_WDT_CDNS) += cdns_wdt.o
> obj-$(CONFIG_WDT_FTWDT010) += ftwdt010_wdt.o
> obj-$(CONFIG_WDT_GPIO) += gpio_wdt.o
> obj-$(CONFIG_WDT_MAX6370) += max6370_wdt.o
> +obj-$(CONFIG_WDT_MCF) += mcf_wdt.o
> obj-$(CONFIG_WDT_MESON_GXBB) += meson_gxbb_wdt.o
> obj-$(CONFIG_WDT_MPC8xxx) += mpc8xxx_wdt.o
> obj-$(CONFIG_WDT_MT7620) += mt7620_wdt.o
> diff --git a/drivers/watchdog/mcf_wdt.c b/drivers/watchdog/mcf_wdt.c
> new file mode 100644
> index 0000000000..b36488bb5b
> --- /dev/null
> +++ b/drivers/watchdog/mcf_wdt.c
> @@ -0,0 +1,135 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * mcf_wdt.c - driver for ColdFire on-chip watchdog
> + *
> + * Author: Angelo Dureghello <angelo@kernel-space.org>
> + *
> + */
> +
> +#include <common.h>
> +#include <dm.h>
> +#include <hang.h>
> +#include <asm/io.h>
> +#include <wdt.h>
> +#include <linux/bitops.h>
> +
> +#define DIVIDER_5XXX 4096
> +#define DIVIDER_5282 8192
> +
> +#define WCR_EN BIT(0)
> +#define WCR_HALTED BIT(1)
> +#define WCR_DOZE BIT(2)
> +#define WCR_WAIT BIT(3)
> +
> +struct watchdog_regs {
> + u16 wcr; /* Control */
> + u16 wmr; /* Service */
> + u16 wcntr; /* Counter */
> + u16 wsr; /* Reset Status */
> +};
> +
> +static void mcf_watchdog_reset(struct watchdog_regs *wdog)
> +{
> + if (!IS_ENABLED(CONFIG_WATCHDOG_RESET_DISABLE)) {
> + writew(0x5555, &wdog->wsr);
> + writew(0xaaaa, &wdog->wsr);
> + }
> +}
> +
> +static void mcf_watchdog_init(struct watchdog_regs *wdog, u32 fixed_divider,
> + u64 timeout_msecs)
> +{
> + u32 wdog_module, cycles_per_sec;
> +
> + cycles_per_sec = CFG_SYS_CLK / fixed_divider;
> +
> + wdog_module = cycles_per_sec * ((u32)timeout_msecs / 1000);
> + wdog_module += (cycles_per_sec / 1000) * ((u32)timeout_msecs % 1000);
> +
> + /* Limit check, max 16 bits */
> + if (wdog_module > 0xffff)
> + wdog_module = 0xffff;
> +
> + /* Set timeout and enable watchdog */
> + writew((u16)wdog_module, &wdog->wmr);
> + writew(WCR_EN, &wdog->wcr);
> +
> + mcf_watchdog_reset(wdog);
> +}
> +
> +struct mcf_wdt_priv {
> + void __iomem *base;
> + u32 fixed_divider;
> +};
> +
> +static int mcf_wdt_expire_now(struct udevice *dev, ulong flags)
> +{
> + hang();
> +
> + return 0;
> +}
> +
> +static int mcf_wdt_reset(struct udevice *dev)
> +{
> + struct mcf_wdt_priv *priv = dev_get_priv(dev);
> +
> + mcf_watchdog_reset(priv->base);
> +
> + return 0;
> +}
> +
> +static int mcf_wdt_start(struct udevice *dev, u64 timeout, ulong flags)
> +{
> + struct mcf_wdt_priv *priv = dev_get_priv(dev);
> +
> + /* Timeout from fdt (timeout) comes in milliseconds */
> + mcf_watchdog_init(priv->base, priv->fixed_divider, timeout);
> +
> + return 0;
> +}
> +
> +static int mcf_wdt_stop(struct udevice *dev)
> +{
> + struct mcf_wdt_priv *priv = dev_get_priv(dev);
> + struct watchdog_regs *wdog = (struct watchdog_regs *)priv->base;
> +
> + setbits_be16(&wdog->wcr, WCR_HALTED);
> +
> + return 0;
> +}
> +
> +static int mcf_wdt_probe(struct udevice *dev)
> +{
> + struct mcf_wdt_priv *priv = dev_get_priv(dev);
> +
> + priv->base = dev_read_addr_ptr(dev);
> + if (!priv->base)
> + return -ENOENT;
> +
> + priv->fixed_divider = (u32)dev_get_driver_data(dev);
> +
> + return 0;
> +}
> +
> +static const struct wdt_ops mcf_wdt_ops = {
> + .start = mcf_wdt_start,
> + .stop = mcf_wdt_stop,
> + .reset = mcf_wdt_reset,
> + .expire_now = mcf_wdt_expire_now,
> +};
> +
> +static const struct udevice_id mcf_wdt_ids[] = {
> + { .compatible = "fsl,mcf5208-wdt", .data = DIVIDER_5XXX },
> + { .compatible = "fsl,mcf5282-wdt", .data = DIVIDER_5282 },
> + {}
> +};
> +
> +U_BOOT_DRIVER(mcf_wdt) = {
> + .name = "mcf_wdt",
> + .id = UCLASS_WDT,
> + .of_match = mcf_wdt_ids,
> + .probe = mcf_wdt_probe,
> + .ops = &mcf_wdt_ops,
> + .priv_auto = sizeof(struct mcf_wdt_priv),
> + .flags = DM_FLAG_PRE_RELOC,
> +};
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: sr@denx.de
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 2/5] m68k: move watchdog functions in mcf_wdt driver
2023-07-24 6:49 [PATCH v3 0/5] m68k: add ColdFire watchdog driver Angelo Dureghello
2023-07-24 6:49 ` [PATCH v3 1/5] drivers: watchdog: add mcf watchdog support Angelo Dureghello
@ 2023-07-24 6:49 ` Angelo Dureghello
2023-07-24 7:41 ` Stefan Roese
2023-07-24 6:49 ` [PATCH v3 3/5] m68k: dts: add watchdog node Angelo Dureghello
` (2 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Angelo Dureghello @ 2023-07-24 6:49 UTC (permalink / raw)
To: trini, sr; +Cc: u-boot, Angelo Dureghello
Move watchdog functions inside a separate watchdog driver.
Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
---
Changes for v2:
- none
Changes for v3:
- none
---
arch/m68k/cpu/mcf523x/cpu.c | 42 ---------------------------------
arch/m68k/cpu/mcf52x2/cpu.c | 47 +------------------------------------
arch/m68k/cpu/mcf532x/cpu.c | 44 ----------------------------------
3 files changed, 1 insertion(+), 132 deletions(-)
diff --git a/arch/m68k/cpu/mcf523x/cpu.c b/arch/m68k/cpu/mcf523x/cpu.c
index ba2c228911..bef67767b4 100644
--- a/arch/m68k/cpu/mcf523x/cpu.c
+++ b/arch/m68k/cpu/mcf523x/cpu.c
@@ -12,7 +12,6 @@
#include <init.h>
#include <net.h>
#include <vsprintf.h>
-#include <watchdog.h>
#include <command.h>
#include <netdev.h>
#include <asm/global_data.h>
@@ -62,47 +61,6 @@ int print_cpuinfo(void)
};
#endif /* CONFIG_DISPLAY_CPUINFO */
-#if defined(CONFIG_WATCHDOG)
-/* Called by macro WATCHDOG_RESET */
-void watchdog_reset(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
-
- /* Count register */
- out_be16(&wdp->sr, 0x5555);
- asm("nop");
- out_be16(&wdp->sr, 0xaaaa);
-}
-
-int watchdog_disable(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
-
- /* UserManual, once the wdog is disabled, wdog cannot be re-enabled */
- /* halted watchdog timer */
- setbits_be16(&wdp->cr, WTM_WCR_HALTED);
-
- puts("WATCHDOG:disabled\n");
- return (0);
-}
-
-int watchdog_init(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
- u32 wdog_module = 0;
-
- /* set timeout and enable watchdog */
- wdog_module = ((CFG_SYS_CLK / CONFIG_SYS_HZ) * CONFIG_WATCHDOG_TIMEOUT_MSECS);
- wdog_module |= (wdog_module / 8192);
- out_be16(&wdp->mr, wdog_module);
-
- out_be16(&wdp->cr, WTM_WCR_EN);
- puts("WATCHDOG:enabled\n");
-
- return (0);
-}
-#endif /* CONFIG_WATCHDOG */
-
#if defined(CONFIG_MCFFEC)
/* Default initializations for MCFFEC controllers. To override,
* create a board-specific function called:
diff --git a/arch/m68k/cpu/mcf52x2/cpu.c b/arch/m68k/cpu/mcf52x2/cpu.c
index d7cbf11e25..5042a38b3e 100644
--- a/arch/m68k/cpu/mcf52x2/cpu.c
+++ b/arch/m68k/cpu/mcf52x2/cpu.c
@@ -17,7 +17,6 @@
#include <init.h>
#include <net.h>
#include <vsprintf.h>
-#include <watchdog.h>
#include <command.h>
#include <asm/global_data.h>
#include <asm/immap.h>
@@ -53,51 +52,7 @@ int print_cpuinfo(void)
return 0;
};
#endif /* CONFIG_DISPLAY_CPUINFO */
-
-#if defined(CONFIG_WATCHDOG)
-/* Called by macro WATCHDOG_RESET */
-void watchdog_reset(void)
-{
- wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
-
- out_be16(&wdt->sr, 0x5555);
- out_be16(&wdt->sr, 0xaaaa);
-}
-
-int watchdog_disable(void)
-{
- wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
-
- /* reset watchdog counter */
- out_be16(&wdt->sr, 0x5555);
- out_be16(&wdt->sr, 0xaaaa);
- /* disable watchdog timer */
- out_be16(&wdt->cr, 0);
-
- puts("WATCHDOG:disabled\n");
- return (0);
-}
-
-int watchdog_init(void)
-{
- wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
-
- /* disable watchdog */
- out_be16(&wdt->cr, 0);
-
- /* set timeout and enable watchdog */
- out_be16(&wdt->mr,
- (CONFIG_WATCHDOG_TIMEOUT_MSECS * CONFIG_SYS_HZ) / (32768 * 1000) - 1);
-
- /* reset watchdog counter */
- out_be16(&wdt->sr, 0x5555);
- out_be16(&wdt->sr, 0xaaaa);
-
- puts("WATCHDOG:enabled\n");
- return (0);
-}
-#endif /* #ifdef CONFIG_WATCHDOG */
-#endif /* #ifdef CONFIG_M5208 */
+#endif /* #ifdef CONFIG_M5208 */
#ifdef CONFIG_M5271
#if defined(CONFIG_DISPLAY_CPUINFO)
diff --git a/arch/m68k/cpu/mcf532x/cpu.c b/arch/m68k/cpu/mcf532x/cpu.c
index 548cbca36a..18d20a8926 100644
--- a/arch/m68k/cpu/mcf532x/cpu.c
+++ b/arch/m68k/cpu/mcf532x/cpu.c
@@ -12,7 +12,6 @@
#include <init.h>
#include <net.h>
#include <vsprintf.h>
-#include <watchdog.h>
#include <command.h>
#include <netdev.h>
#include <asm/global_data.h>
@@ -102,49 +101,6 @@ int print_cpuinfo(void)
};
#endif /* CONFIG_DISPLAY_CPUINFO */
-#if defined(CONFIG_WATCHDOG)
-/* Called by macro WATCHDOG_RESET */
-void watchdog_reset(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
-
- /* Count register */
- out_be16(&wdp->sr, 0x5555);
- out_be16(&wdp->sr, 0xaaaa);
-}
-
-int watchdog_disable(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
-
- /* UserManual, once the wdog is disabled, wdog cannot be re-enabled */
- /* halted watchdog timer */
- setbits_be16(&wdp->cr, WTM_WCR_HALTED);
-
- puts("WATCHDOG:disabled\n");
- return (0);
-}
-
-int watchdog_init(void)
-{
- wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
- u32 wdog_module = 0;
-
- /* set timeout and enable watchdog */
- wdog_module = ((CFG_SYS_CLK / 1000) * CONFIG_WATCHDOG_TIMEOUT_MSECS);
-#ifdef CONFIG_M5329
- out_be16(&wdp->mr, wdog_module / 8192);
-#else
- out_be16(&wdp->mr, wdog_module / 4096);
-#endif
-
- out_be16(&wdp->cr, WTM_WCR_EN);
- puts("WATCHDOG:enabled\n");
-
- return (0);
-}
-#endif /* CONFIG_WATCHDOG */
-
#if defined(CONFIG_MCFFEC)
/* Default initializations for MCFFEC controllers. To override,
* create a board-specific function called:
--
2.41.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v3 2/5] m68k: move watchdog functions in mcf_wdt driver
2023-07-24 6:49 ` [PATCH v3 2/5] m68k: move watchdog functions in mcf_wdt driver Angelo Dureghello
@ 2023-07-24 7:41 ` Stefan Roese
0 siblings, 0 replies; 11+ messages in thread
From: Stefan Roese @ 2023-07-24 7:41 UTC (permalink / raw)
To: Angelo Dureghello, trini; +Cc: u-boot
On 7/24/23 08:49, Angelo Dureghello wrote:
> Move watchdog functions inside a separate watchdog driver.
>
> Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> Changes for v2:
> - none
> Changes for v3:
> - none
> ---
> arch/m68k/cpu/mcf523x/cpu.c | 42 ---------------------------------
> arch/m68k/cpu/mcf52x2/cpu.c | 47 +------------------------------------
> arch/m68k/cpu/mcf532x/cpu.c | 44 ----------------------------------
> 3 files changed, 1 insertion(+), 132 deletions(-)
>
> diff --git a/arch/m68k/cpu/mcf523x/cpu.c b/arch/m68k/cpu/mcf523x/cpu.c
> index ba2c228911..bef67767b4 100644
> --- a/arch/m68k/cpu/mcf523x/cpu.c
> +++ b/arch/m68k/cpu/mcf523x/cpu.c
> @@ -12,7 +12,6 @@
> #include <init.h>
> #include <net.h>
> #include <vsprintf.h>
> -#include <watchdog.h>
> #include <command.h>
> #include <netdev.h>
> #include <asm/global_data.h>
> @@ -62,47 +61,6 @@ int print_cpuinfo(void)
> };
> #endif /* CONFIG_DISPLAY_CPUINFO */
>
> -#if defined(CONFIG_WATCHDOG)
> -/* Called by macro WATCHDOG_RESET */
> -void watchdog_reset(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> -
> - /* Count register */
> - out_be16(&wdp->sr, 0x5555);
> - asm("nop");
> - out_be16(&wdp->sr, 0xaaaa);
> -}
> -
> -int watchdog_disable(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> -
> - /* UserManual, once the wdog is disabled, wdog cannot be re-enabled */
> - /* halted watchdog timer */
> - setbits_be16(&wdp->cr, WTM_WCR_HALTED);
> -
> - puts("WATCHDOG:disabled\n");
> - return (0);
> -}
> -
> -int watchdog_init(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> - u32 wdog_module = 0;
> -
> - /* set timeout and enable watchdog */
> - wdog_module = ((CFG_SYS_CLK / CONFIG_SYS_HZ) * CONFIG_WATCHDOG_TIMEOUT_MSECS);
> - wdog_module |= (wdog_module / 8192);
> - out_be16(&wdp->mr, wdog_module);
> -
> - out_be16(&wdp->cr, WTM_WCR_EN);
> - puts("WATCHDOG:enabled\n");
> -
> - return (0);
> -}
> -#endif /* CONFIG_WATCHDOG */
> -
> #if defined(CONFIG_MCFFEC)
> /* Default initializations for MCFFEC controllers. To override,
> * create a board-specific function called:
> diff --git a/arch/m68k/cpu/mcf52x2/cpu.c b/arch/m68k/cpu/mcf52x2/cpu.c
> index d7cbf11e25..5042a38b3e 100644
> --- a/arch/m68k/cpu/mcf52x2/cpu.c
> +++ b/arch/m68k/cpu/mcf52x2/cpu.c
> @@ -17,7 +17,6 @@
> #include <init.h>
> #include <net.h>
> #include <vsprintf.h>
> -#include <watchdog.h>
> #include <command.h>
> #include <asm/global_data.h>
> #include <asm/immap.h>
> @@ -53,51 +52,7 @@ int print_cpuinfo(void)
> return 0;
> };
> #endif /* CONFIG_DISPLAY_CPUINFO */
> -
> -#if defined(CONFIG_WATCHDOG)
> -/* Called by macro WATCHDOG_RESET */
> -void watchdog_reset(void)
> -{
> - wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
> -
> - out_be16(&wdt->sr, 0x5555);
> - out_be16(&wdt->sr, 0xaaaa);
> -}
> -
> -int watchdog_disable(void)
> -{
> - wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
> -
> - /* reset watchdog counter */
> - out_be16(&wdt->sr, 0x5555);
> - out_be16(&wdt->sr, 0xaaaa);
> - /* disable watchdog timer */
> - out_be16(&wdt->cr, 0);
> -
> - puts("WATCHDOG:disabled\n");
> - return (0);
> -}
> -
> -int watchdog_init(void)
> -{
> - wdog_t *wdt = (wdog_t *)(MMAP_WDOG);
> -
> - /* disable watchdog */
> - out_be16(&wdt->cr, 0);
> -
> - /* set timeout and enable watchdog */
> - out_be16(&wdt->mr,
> - (CONFIG_WATCHDOG_TIMEOUT_MSECS * CONFIG_SYS_HZ) / (32768 * 1000) - 1);
> -
> - /* reset watchdog counter */
> - out_be16(&wdt->sr, 0x5555);
> - out_be16(&wdt->sr, 0xaaaa);
> -
> - puts("WATCHDOG:enabled\n");
> - return (0);
> -}
> -#endif /* #ifdef CONFIG_WATCHDOG */
> -#endif /* #ifdef CONFIG_M5208 */
> +#endif /* #ifdef CONFIG_M5208 */
>
> #ifdef CONFIG_M5271
> #if defined(CONFIG_DISPLAY_CPUINFO)
> diff --git a/arch/m68k/cpu/mcf532x/cpu.c b/arch/m68k/cpu/mcf532x/cpu.c
> index 548cbca36a..18d20a8926 100644
> --- a/arch/m68k/cpu/mcf532x/cpu.c
> +++ b/arch/m68k/cpu/mcf532x/cpu.c
> @@ -12,7 +12,6 @@
> #include <init.h>
> #include <net.h>
> #include <vsprintf.h>
> -#include <watchdog.h>
> #include <command.h>
> #include <netdev.h>
> #include <asm/global_data.h>
> @@ -102,49 +101,6 @@ int print_cpuinfo(void)
> };
> #endif /* CONFIG_DISPLAY_CPUINFO */
>
> -#if defined(CONFIG_WATCHDOG)
> -/* Called by macro WATCHDOG_RESET */
> -void watchdog_reset(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> -
> - /* Count register */
> - out_be16(&wdp->sr, 0x5555);
> - out_be16(&wdp->sr, 0xaaaa);
> -}
> -
> -int watchdog_disable(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> -
> - /* UserManual, once the wdog is disabled, wdog cannot be re-enabled */
> - /* halted watchdog timer */
> - setbits_be16(&wdp->cr, WTM_WCR_HALTED);
> -
> - puts("WATCHDOG:disabled\n");
> - return (0);
> -}
> -
> -int watchdog_init(void)
> -{
> - wdog_t *wdp = (wdog_t *) (MMAP_WDOG);
> - u32 wdog_module = 0;
> -
> - /* set timeout and enable watchdog */
> - wdog_module = ((CFG_SYS_CLK / 1000) * CONFIG_WATCHDOG_TIMEOUT_MSECS);
> -#ifdef CONFIG_M5329
> - out_be16(&wdp->mr, wdog_module / 8192);
> -#else
> - out_be16(&wdp->mr, wdog_module / 4096);
> -#endif
> -
> - out_be16(&wdp->cr, WTM_WCR_EN);
> - puts("WATCHDOG:enabled\n");
> -
> - return (0);
> -}
> -#endif /* CONFIG_WATCHDOG */
> -
> #if defined(CONFIG_MCFFEC)
> /* Default initializations for MCFFEC controllers. To override,
> * create a board-specific function called:
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: sr@denx.de
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 3/5] m68k: dts: add watchdog node
2023-07-24 6:49 [PATCH v3 0/5] m68k: add ColdFire watchdog driver Angelo Dureghello
2023-07-24 6:49 ` [PATCH v3 1/5] drivers: watchdog: add mcf watchdog support Angelo Dureghello
2023-07-24 6:49 ` [PATCH v3 2/5] m68k: move watchdog functions in mcf_wdt driver Angelo Dureghello
@ 2023-07-24 6:49 ` Angelo Dureghello
2023-07-24 7:41 ` Stefan Roese
2023-07-24 6:49 ` [PATCH v3 4/5] configs: m68k: add watchdog driver Angelo Dureghello
2023-07-24 6:49 ` [PATCH v3 5/5] MAINTAINERS: add myself as mcf_wdt.c maintainer Angelo Dureghello
4 siblings, 1 reply; 11+ messages in thread
From: Angelo Dureghello @ 2023-07-24 6:49 UTC (permalink / raw)
To: trini, sr; +Cc: u-boot, Angelo Dureghello
Add watchdog node for the implemented mcf_wdt driver.
Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
---
Changes for v2:
- remove unnecessary big-endian property
Changes for v3:
- none
---
arch/m68k/dts/M5208EVBE.dts | 5 +++++
arch/m68k/dts/mcf5208.dtsi | 6 ++++++
arch/m68k/dts/mcf523x.dtsi | 6 ++++++
arch/m68k/dts/mcf5271.dtsi | 6 ++++++
arch/m68k/dts/mcf5275.dtsi | 6 ++++++
arch/m68k/dts/mcf5282.dtsi | 6 ++++++
arch/m68k/dts/mcf5329.dtsi | 6 ++++++
arch/m68k/dts/mcf537x.dtsi | 6 ++++++
8 files changed, 47 insertions(+)
diff --git a/arch/m68k/dts/M5208EVBE.dts b/arch/m68k/dts/M5208EVBE.dts
index 1c32718af4..ec203e8b69 100644
--- a/arch/m68k/dts/M5208EVBE.dts
+++ b/arch/m68k/dts/M5208EVBE.dts
@@ -15,6 +15,11 @@
};
};
+&wdog0 {
+ timeout-sec = <32>;
+ status = "okay";
+};
+
&uart0 {
bootph-all;
status = "okay";
diff --git a/arch/m68k/dts/mcf5208.dtsi b/arch/m68k/dts/mcf5208.dtsi
index 9392facfa8..c61dbf3557 100644
--- a/arch/m68k/dts/mcf5208.dtsi
+++ b/arch/m68k/dts/mcf5208.dtsi
@@ -16,6 +16,12 @@
#address-cells = <1>;
#size-cells = <1>;
+ wdog0: watchdog@fc08c000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0xfc08c000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@fc060000 {
compatible = "fsl,mcf-uart";
reg = <0xfc060000 0x40>;
diff --git a/arch/m68k/dts/mcf523x.dtsi b/arch/m68k/dts/mcf523x.dtsi
index 41c7b9b2d1..cae7b4f861 100644
--- a/arch/m68k/dts/mcf523x.dtsi
+++ b/arch/m68k/dts/mcf523x.dtsi
@@ -23,6 +23,12 @@
ranges = <0x00000000 0x40000000 0x40000000>;
reg = <0x40000000 0x40000000>;
+ wdog0: watchdog@140000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0x140000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@200 {
compatible = "fsl,mcf-uart";
reg = <0x200 0x40>;
diff --git a/arch/m68k/dts/mcf5271.dtsi b/arch/m68k/dts/mcf5271.dtsi
index fc82bd3c24..5acf6994af 100644
--- a/arch/m68k/dts/mcf5271.dtsi
+++ b/arch/m68k/dts/mcf5271.dtsi
@@ -23,6 +23,12 @@
ranges = <0x00000000 0x40000000 0x40000000>;
reg = <0x40000000 0x40000000>;
+ wdog0: watchdog@140000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0x140000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@200 {
compatible = "fsl,mcf-uart";
reg = <0x200 0x40>;
diff --git a/arch/m68k/dts/mcf5275.dtsi b/arch/m68k/dts/mcf5275.dtsi
index 402517cdec..3df17b445c 100644
--- a/arch/m68k/dts/mcf5275.dtsi
+++ b/arch/m68k/dts/mcf5275.dtsi
@@ -24,6 +24,12 @@
ranges = <0x00000000 0x40000000 0x40000000>;
reg = <0x40000000 0x40000000>;
+ wdog0: watchdog@140000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0x140000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@200 {
compatible = "fsl,mcf-uart";
reg = <0x200 0x40>;
diff --git a/arch/m68k/dts/mcf5282.dtsi b/arch/m68k/dts/mcf5282.dtsi
index 883c0d0324..640c1a32e4 100644
--- a/arch/m68k/dts/mcf5282.dtsi
+++ b/arch/m68k/dts/mcf5282.dtsi
@@ -23,6 +23,12 @@
ranges = <0x00000000 0x40000000 0x40000000>;
reg = <0x40000000 0x40000000>;
+ wdog0: watchdog@140000 {
+ compatible = "fsl,mcf5282-wdt";
+ reg = <0x140000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@200 {
compatible = "fsl,mcf-uart";
reg = <0x200 0x40>;
diff --git a/arch/m68k/dts/mcf5329.dtsi b/arch/m68k/dts/mcf5329.dtsi
index 7501cc4b01..991985e48c 100644
--- a/arch/m68k/dts/mcf5329.dtsi
+++ b/arch/m68k/dts/mcf5329.dtsi
@@ -16,6 +16,12 @@
#address-cells = <1>;
#size-cells = <1>;
+ wdog0: watchdog@fc098000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0xfc08c000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@fc060000 {
compatible = "fsl,mcf-uart";
reg = <0xfc060000 0x40>;
diff --git a/arch/m68k/dts/mcf537x.dtsi b/arch/m68k/dts/mcf537x.dtsi
index 338b8b4583..4fa0f4bc3d 100644
--- a/arch/m68k/dts/mcf537x.dtsi
+++ b/arch/m68k/dts/mcf537x.dtsi
@@ -16,6 +16,12 @@
#address-cells = <1>;
#size-cells = <1>;
+ wdog0: watchdog@fc098000 {
+ compatible = "fsl,mcf5208-wdt";
+ reg = <0xfc08c000 0x10>;
+ status = "disabled";
+ };
+
uart0: uart@fc060000 {
compatible = "fsl,mcf-uart";
reg = <0xfc060000 0x40>;
--
2.41.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v3 3/5] m68k: dts: add watchdog node
2023-07-24 6:49 ` [PATCH v3 3/5] m68k: dts: add watchdog node Angelo Dureghello
@ 2023-07-24 7:41 ` Stefan Roese
0 siblings, 0 replies; 11+ messages in thread
From: Stefan Roese @ 2023-07-24 7:41 UTC (permalink / raw)
To: Angelo Dureghello, trini; +Cc: u-boot
On 7/24/23 08:49, Angelo Dureghello wrote:
> Add watchdog node for the implemented mcf_wdt driver.
>
> Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> Changes for v2:
> - remove unnecessary big-endian property
> Changes for v3:
> - none
> ---
> arch/m68k/dts/M5208EVBE.dts | 5 +++++
> arch/m68k/dts/mcf5208.dtsi | 6 ++++++
> arch/m68k/dts/mcf523x.dtsi | 6 ++++++
> arch/m68k/dts/mcf5271.dtsi | 6 ++++++
> arch/m68k/dts/mcf5275.dtsi | 6 ++++++
> arch/m68k/dts/mcf5282.dtsi | 6 ++++++
> arch/m68k/dts/mcf5329.dtsi | 6 ++++++
> arch/m68k/dts/mcf537x.dtsi | 6 ++++++
> 8 files changed, 47 insertions(+)
>
> diff --git a/arch/m68k/dts/M5208EVBE.dts b/arch/m68k/dts/M5208EVBE.dts
> index 1c32718af4..ec203e8b69 100644
> --- a/arch/m68k/dts/M5208EVBE.dts
> +++ b/arch/m68k/dts/M5208EVBE.dts
> @@ -15,6 +15,11 @@
> };
> };
>
> +&wdog0 {
> + timeout-sec = <32>;
> + status = "okay";
> +};
> +
> &uart0 {
> bootph-all;
> status = "okay";
> diff --git a/arch/m68k/dts/mcf5208.dtsi b/arch/m68k/dts/mcf5208.dtsi
> index 9392facfa8..c61dbf3557 100644
> --- a/arch/m68k/dts/mcf5208.dtsi
> +++ b/arch/m68k/dts/mcf5208.dtsi
> @@ -16,6 +16,12 @@
> #address-cells = <1>;
> #size-cells = <1>;
>
> + wdog0: watchdog@fc08c000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0xfc08c000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@fc060000 {
> compatible = "fsl,mcf-uart";
> reg = <0xfc060000 0x40>;
> diff --git a/arch/m68k/dts/mcf523x.dtsi b/arch/m68k/dts/mcf523x.dtsi
> index 41c7b9b2d1..cae7b4f861 100644
> --- a/arch/m68k/dts/mcf523x.dtsi
> +++ b/arch/m68k/dts/mcf523x.dtsi
> @@ -23,6 +23,12 @@
> ranges = <0x00000000 0x40000000 0x40000000>;
> reg = <0x40000000 0x40000000>;
>
> + wdog0: watchdog@140000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0x140000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@200 {
> compatible = "fsl,mcf-uart";
> reg = <0x200 0x40>;
> diff --git a/arch/m68k/dts/mcf5271.dtsi b/arch/m68k/dts/mcf5271.dtsi
> index fc82bd3c24..5acf6994af 100644
> --- a/arch/m68k/dts/mcf5271.dtsi
> +++ b/arch/m68k/dts/mcf5271.dtsi
> @@ -23,6 +23,12 @@
> ranges = <0x00000000 0x40000000 0x40000000>;
> reg = <0x40000000 0x40000000>;
>
> + wdog0: watchdog@140000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0x140000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@200 {
> compatible = "fsl,mcf-uart";
> reg = <0x200 0x40>;
> diff --git a/arch/m68k/dts/mcf5275.dtsi b/arch/m68k/dts/mcf5275.dtsi
> index 402517cdec..3df17b445c 100644
> --- a/arch/m68k/dts/mcf5275.dtsi
> +++ b/arch/m68k/dts/mcf5275.dtsi
> @@ -24,6 +24,12 @@
> ranges = <0x00000000 0x40000000 0x40000000>;
> reg = <0x40000000 0x40000000>;
>
> + wdog0: watchdog@140000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0x140000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@200 {
> compatible = "fsl,mcf-uart";
> reg = <0x200 0x40>;
> diff --git a/arch/m68k/dts/mcf5282.dtsi b/arch/m68k/dts/mcf5282.dtsi
> index 883c0d0324..640c1a32e4 100644
> --- a/arch/m68k/dts/mcf5282.dtsi
> +++ b/arch/m68k/dts/mcf5282.dtsi
> @@ -23,6 +23,12 @@
> ranges = <0x00000000 0x40000000 0x40000000>;
> reg = <0x40000000 0x40000000>;
>
> + wdog0: watchdog@140000 {
> + compatible = "fsl,mcf5282-wdt";
> + reg = <0x140000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@200 {
> compatible = "fsl,mcf-uart";
> reg = <0x200 0x40>;
> diff --git a/arch/m68k/dts/mcf5329.dtsi b/arch/m68k/dts/mcf5329.dtsi
> index 7501cc4b01..991985e48c 100644
> --- a/arch/m68k/dts/mcf5329.dtsi
> +++ b/arch/m68k/dts/mcf5329.dtsi
> @@ -16,6 +16,12 @@
> #address-cells = <1>;
> #size-cells = <1>;
>
> + wdog0: watchdog@fc098000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0xfc08c000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@fc060000 {
> compatible = "fsl,mcf-uart";
> reg = <0xfc060000 0x40>;
> diff --git a/arch/m68k/dts/mcf537x.dtsi b/arch/m68k/dts/mcf537x.dtsi
> index 338b8b4583..4fa0f4bc3d 100644
> --- a/arch/m68k/dts/mcf537x.dtsi
> +++ b/arch/m68k/dts/mcf537x.dtsi
> @@ -16,6 +16,12 @@
> #address-cells = <1>;
> #size-cells = <1>;
>
> + wdog0: watchdog@fc098000 {
> + compatible = "fsl,mcf5208-wdt";
> + reg = <0xfc08c000 0x10>;
> + status = "disabled";
> + };
> +
> uart0: uart@fc060000 {
> compatible = "fsl,mcf-uart";
> reg = <0xfc060000 0x40>;
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: sr@denx.de
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 4/5] configs: m68k: add watchdog driver
2023-07-24 6:49 [PATCH v3 0/5] m68k: add ColdFire watchdog driver Angelo Dureghello
` (2 preceding siblings ...)
2023-07-24 6:49 ` [PATCH v3 3/5] m68k: dts: add watchdog node Angelo Dureghello
@ 2023-07-24 6:49 ` Angelo Dureghello
2023-07-24 7:42 ` Stefan Roese
2023-07-24 6:49 ` [PATCH v3 5/5] MAINTAINERS: add myself as mcf_wdt.c maintainer Angelo Dureghello
4 siblings, 1 reply; 11+ messages in thread
From: Angelo Dureghello @ 2023-07-24 6:49 UTC (permalink / raw)
To: trini, sr; +Cc: u-boot, Angelo Dureghello
Add config options for mcf_wdt driver.
Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
---
Changes for v2:
- none
Changes for v3:
- none
---
configs/M5208EVBE_defconfig | 2 ++
configs/astro_mcf5373l_defconfig | 4 ++--
configs/eb_cpu5282_defconfig | 1 +
configs/eb_cpu5282_internal_defconfig | 1 +
4 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/configs/M5208EVBE_defconfig b/configs/M5208EVBE_defconfig
index 0c5afe869b..aa054f753f 100644
--- a/configs/M5208EVBE_defconfig
+++ b/configs/M5208EVBE_defconfig
@@ -50,3 +50,5 @@ CONFIG_MCFFEC=y
CONFIG_MII=y
CONFIG_MCFUART=y
CONFIG_WATCHDOG_TIMEOUT_MSECS=5000
+CONFIG_WDT=y
+CONFIG_WDT_MCF=y
diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
index aade1f98be..f4dad3bcc8 100644
--- a/configs/astro_mcf5373l_defconfig
+++ b/configs/astro_mcf5373l_defconfig
@@ -46,5 +46,5 @@ CONFIG_DM_RTC=y
CONFIG_MCFRTC=y
CONFIG_SYS_MCFRTC_BASE=0xFC0A8000
CONFIG_MCFUART=y
-CONFIG_WATCHDOG=y
-CONFIG_WATCHDOG_TIMEOUT_MSECS=3355
+CONFIG_WDT=y
+CONFIG_WDT_MCF=y
diff --git a/configs/eb_cpu5282_defconfig b/configs/eb_cpu5282_defconfig
index 24edecb510..2873322598 100644
--- a/configs/eb_cpu5282_defconfig
+++ b/configs/eb_cpu5282_defconfig
@@ -52,3 +52,4 @@ CONFIG_MII=y
CONFIG_DM_RTC=y
CONFIG_RTC_DS1338=y
CONFIG_MCFUART=y
+CONFIG_WDT=y
diff --git a/configs/eb_cpu5282_internal_defconfig b/configs/eb_cpu5282_internal_defconfig
index 44e22eb01d..bd780034ba 100644
--- a/configs/eb_cpu5282_internal_defconfig
+++ b/configs/eb_cpu5282_internal_defconfig
@@ -50,3 +50,4 @@ CONFIG_MII=y
CONFIG_DM_RTC=y
CONFIG_RTC_DS1338=y
CONFIG_MCFUART=y
+CONFIG_WDT=y
--
2.41.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v3 4/5] configs: m68k: add watchdog driver
2023-07-24 6:49 ` [PATCH v3 4/5] configs: m68k: add watchdog driver Angelo Dureghello
@ 2023-07-24 7:42 ` Stefan Roese
0 siblings, 0 replies; 11+ messages in thread
From: Stefan Roese @ 2023-07-24 7:42 UTC (permalink / raw)
To: Angelo Dureghello, trini; +Cc: u-boot
On 7/24/23 08:49, Angelo Dureghello wrote:
> Add config options for mcf_wdt driver.
>
> Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> Changes for v2:
> - none
> Changes for v3:
> - none
> ---
> configs/M5208EVBE_defconfig | 2 ++
> configs/astro_mcf5373l_defconfig | 4 ++--
> configs/eb_cpu5282_defconfig | 1 +
> configs/eb_cpu5282_internal_defconfig | 1 +
> 4 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/configs/M5208EVBE_defconfig b/configs/M5208EVBE_defconfig
> index 0c5afe869b..aa054f753f 100644
> --- a/configs/M5208EVBE_defconfig
> +++ b/configs/M5208EVBE_defconfig
> @@ -50,3 +50,5 @@ CONFIG_MCFFEC=y
> CONFIG_MII=y
> CONFIG_MCFUART=y
> CONFIG_WATCHDOG_TIMEOUT_MSECS=5000
> +CONFIG_WDT=y
> +CONFIG_WDT_MCF=y
> diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
> index aade1f98be..f4dad3bcc8 100644
> --- a/configs/astro_mcf5373l_defconfig
> +++ b/configs/astro_mcf5373l_defconfig
> @@ -46,5 +46,5 @@ CONFIG_DM_RTC=y
> CONFIG_MCFRTC=y
> CONFIG_SYS_MCFRTC_BASE=0xFC0A8000
> CONFIG_MCFUART=y
> -CONFIG_WATCHDOG=y
> -CONFIG_WATCHDOG_TIMEOUT_MSECS=3355
> +CONFIG_WDT=y
> +CONFIG_WDT_MCF=y
> diff --git a/configs/eb_cpu5282_defconfig b/configs/eb_cpu5282_defconfig
> index 24edecb510..2873322598 100644
> --- a/configs/eb_cpu5282_defconfig
> +++ b/configs/eb_cpu5282_defconfig
> @@ -52,3 +52,4 @@ CONFIG_MII=y
> CONFIG_DM_RTC=y
> CONFIG_RTC_DS1338=y
> CONFIG_MCFUART=y
> +CONFIG_WDT=y
> diff --git a/configs/eb_cpu5282_internal_defconfig b/configs/eb_cpu5282_internal_defconfig
> index 44e22eb01d..bd780034ba 100644
> --- a/configs/eb_cpu5282_internal_defconfig
> +++ b/configs/eb_cpu5282_internal_defconfig
> @@ -50,3 +50,4 @@ CONFIG_MII=y
> CONFIG_DM_RTC=y
> CONFIG_RTC_DS1338=y
> CONFIG_MCFUART=y
> +CONFIG_WDT=y
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: sr@denx.de
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3 5/5] MAINTAINERS: add myself as mcf_wdt.c maintainer
2023-07-24 6:49 [PATCH v3 0/5] m68k: add ColdFire watchdog driver Angelo Dureghello
` (3 preceding siblings ...)
2023-07-24 6:49 ` [PATCH v3 4/5] configs: m68k: add watchdog driver Angelo Dureghello
@ 2023-07-24 6:49 ` Angelo Dureghello
2023-07-24 7:42 ` Stefan Roese
4 siblings, 1 reply; 11+ messages in thread
From: Angelo Dureghello @ 2023-07-24 6:49 UTC (permalink / raw)
To: trini, sr; +Cc: u-boot, Angelo Dureghello
Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
---
Changes for v2:
- none
Changes for v3:
- none
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 87991cccdd..2b157214cb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -867,6 +867,7 @@ S: Maintained
T: git https://source.denx.de/u-boot/custodians/u-boot-coldfire.git
F: arch/m68k/
F: doc/arch/m68k.rst
+F: drivers/watchdog/mcf_wdt.c
CYCLIC
M: Stefan Roese <sr@denx.de>
--
2.41.0
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v3 5/5] MAINTAINERS: add myself as mcf_wdt.c maintainer
2023-07-24 6:49 ` [PATCH v3 5/5] MAINTAINERS: add myself as mcf_wdt.c maintainer Angelo Dureghello
@ 2023-07-24 7:42 ` Stefan Roese
0 siblings, 0 replies; 11+ messages in thread
From: Stefan Roese @ 2023-07-24 7:42 UTC (permalink / raw)
To: Angelo Dureghello, trini; +Cc: u-boot
On 7/24/23 08:49, Angelo Dureghello wrote:
> Signed-off-by: Angelo Dureghello <angelo@kernel-space.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Thanks,
Stefan
> ---
> Changes for v2:
> - none
> Changes for v3:
> - none
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 87991cccdd..2b157214cb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -867,6 +867,7 @@ S: Maintained
> T: git https://source.denx.de/u-boot/custodians/u-boot-coldfire.git
> F: arch/m68k/
> F: doc/arch/m68k.rst
> +F: drivers/watchdog/mcf_wdt.c
>
> CYCLIC
> M: Stefan Roese <sr@denx.de>
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: sr@denx.de
^ permalink raw reply [flat|nested] 11+ messages in thread