diff mbox series

[RESEND,v6,7/7] arm64: dts: ti: k3-j712s2-mcu: Add the mcu domain watchdog instances

Message ID 20230928011416.2900-1-j-keerthy@ti.com (mailing list archive)
State New, archived
Headers show
Series None | expand

Commit Message

J, KEERTHY Sept. 28, 2023, 1:14 a.m. UTC
There are totally 2 instances of watchdog module in MCU domain.
These instances are coupled with the MCU domain R5F instances.
Disabling them as they are not used by Linux.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---

Changes:

  * Corrected subject.

 .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     | 24 +++++++++++++++++++
 1 file changed, 24 insertions(+)

Comments

Nishanth Menon Sept. 28, 2023, 4:10 a.m. UTC | #1
On 06:44-20230928, Keerthy wrote:
> There are totally 2 instances of watchdog module in MCU domain.
> These instances are coupled with the MCU domain R5F instances.
> Disabling them as they are not used by Linux.
> 
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
> 
> Changes:
> 
>   * Corrected subject.

Thank you. But, I'd suggest a cooling off period of a week before you
post another version to the list (and that will allow others to comment
as well if required).

While you do that, please don't send a partial patch resend unless
requested by maintainer and the maintainer is willing to handle the
extra step of hand cherry-picking the patch.

At least in my personal workflow with mutt scripts integrated to b4 the
way I do it.. it breaks my work flow.

https://gist.github.com/nmenon/98b936633bb738cf867ae33ccf5ddd5f

https://lpc.events/event/11/contributions/983/attachments/759/1421/Doing%20more%20with%20lore%20and%20b4.pdf

If you haven't converted to b4 and lei, I suggest looking at it ;)
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
index 4b29418a6076..83377c47e709 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi
@@ -669,4 +669,28 @@ 
 		ti,esm-pins = <63>;
 		bootph-pre-ram;
 	};
+
+	/*
+	 * The 2 RTI instances are couple with MCU R5Fs so keeping them
+	 * disabled as these will be used by their respective firmware
+	 */
+	mcu_watchdog0: watchdog@40600000 {
+		compatible = "ti,j7-rti-wdt";
+		reg = <0x00 0x40600000 0x00 0x100>;
+		clocks = <&k3_clks 295 1>;
+		power-domains = <&k3_pds 295 TI_SCI_PD_EXCLUSIVE>;
+		assigned-clocks = <&k3_clks 295 1>;
+		assigned-clock-parents = <&k3_clks 295 5>;
+		status = "disabled";
+	};
+
+	mcu_watchdog1: watchdog@40610000 {
+		compatible = "ti,j7-rti-wdt";
+		reg = <0x00 0x40610000 0x00 0x100>;
+		clocks = <&k3_clks 296 1>;
+		power-domains = <&k3_pds 296 TI_SCI_PD_EXCLUSIVE>;
+		assigned-clocks = <&k3_clks 296 1>;
+		assigned-clock-parents = <&k3_clks 296 5>;
+		status = "disabled";
+	};
 };