2024-01-17 06:51:01

by Li, Hua Qian

[permalink] [raw]
Subject: [PATCH] arm64: dts: ti: Add reserved memory for watchdog

From: Li Hua Qian <[email protected]>

This patch adds a reserved memory for the TI AM65X platform watchdog
to reserve the specific info, triggering the watchdog reset in last
boot, to know if the board reboot is due to a watchdog reset.

Signed-off-by: Li Hua Qian <[email protected]>
---
The driver and binding have hit the master, which could be found in the
following link.
- watchdog:rit_wdt: Add support for WDIOF_CARDRESET
- dt-bindings: watchdog: ti,rti-wdt: Add support for WDIOF_CARDRESET
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=29057cc5bddc785ea0a11534d7ad2546fa0872d3

arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index 61a634afaa4f..88321799d468 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -75,6 +75,12 @@ rtos_ipc_memory_region: ipc-memories@a2000000 {
alignment = <0x1000>;
no-map;
};
+
+ /* To reserve the power-on(PON) reason for watchdog reset */
+ wdt_reset_memory_region: wdt-memory@a2200000 {
+ reg = <0x00 0xa2200000 0x00 0x1000>;
+ no-map;
+ };
};

leds {
@@ -1427,6 +1433,10 @@ &mcu_r5fss0_core1 {
mboxes = <&mailbox0_cluster1>, <&mbox_mcu_r5fss0_core1>;
};

+&mcu_rti1 {
+ memory-region = <&wdt_reset_memory_region>;
+};
+
&icssg0_mdio {
status = "okay";
pinctrl-names = "default";
--
2.34.1



2024-02-16 16:49:08

by Jan Kiszka

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: Add reserved memory for watchdog

On 17.01.24 07:06, [email protected] wrote:
> From: Li Hua Qian <[email protected]>
>
> This patch adds a reserved memory for the TI AM65X platform watchdog
> to reserve the specific info, triggering the watchdog reset in last
> boot, to know if the board reboot is due to a watchdog reset.
>
> Signed-off-by: Li Hua Qian <[email protected]>
> ---
> The driver and binding have hit the master, which could be found in the
> following link.
> - watchdog:rit_wdt: Add support for WDIOF_CARDRESET
> - dt-bindings: watchdog: ti,rti-wdt: Add support for WDIOF_CARDRESET
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=29057cc5bddc785ea0a11534d7ad2546fa0872d3
>
> arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> index 61a634afaa4f..88321799d468 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> @@ -75,6 +75,12 @@ rtos_ipc_memory_region: ipc-memories@a2000000 {
> alignment = <0x1000>;
> no-map;
> };
> +
> + /* To reserve the power-on(PON) reason for watchdog reset */
> + wdt_reset_memory_region: wdt-memory@a2200000 {
> + reg = <0x00 0xa2200000 0x00 0x1000>;
> + no-map;
> + };
> };
>
> leds {
> @@ -1427,6 +1433,10 @@ &mcu_r5fss0_core1 {
> mboxes = <&mailbox0_cluster1>, <&mbox_mcu_r5fss0_core1>;
> };
>
> +&mcu_rti1 {
> + memory-region = <&wdt_reset_memory_region>;
> +};
> +
> &icssg0_mdio {
> status = "okay";
> pinctrl-names = "default";

Is this not forgotten (now that my IOT2050 series was merged)?

Thanks,
Jan

--
Siemens AG, Technology
Linux Expert Center


2024-02-19 13:20:55

by Vignesh Raghavendra

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: Add reserved memory for watchdog

Hi [email protected],

On Wed, 17 Jan 2024 14:06:56 +0800, [email protected] wrote:
> This patch adds a reserved memory for the TI AM65X platform watchdog
> to reserve the specific info, triggering the watchdog reset in last
> boot, to know if the board reboot is due to a watchdog reset.
>
>

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/1] arm64: dts: ti: Add reserved memory for watchdog
commit: 14a65ea5fee71d716d246b8cc494a95bc9924306

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Vignesh