Re: [PATCH 2/2] arm64: dts: ti: k3-am62: Add watchdog nodes

From: Bryan Brattlof
Date: Mon Mar 20 2023 - 11:46:44 EST


On March 11, 2023 thus sayeth Nishanth Menon:
> From: Julien Panis <jpanis@xxxxxxxxxxxx>
>
> Add nodes for watchdogs :
> - 5 in main domain
> - 1 in MCU domain
> - 1 in wakeup domain
>
> Signed-off-by: Julien Panis <jpanis@xxxxxxxxxxxx>
> Signed-off-by: Nishanth Menon <nm@xxxxxx>
> ---
> Changes since V3:
> - Dropped the board level wdt reservation, do that with overlays based on
> modifiable firmware dependencies.
>
> V3: https://lore.kernel.org/all/20221109093026.103790-2-jpanis@xxxxxxxxxxxx/
>
> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 45 ++++++++++++++++++++++
> arch/arm64/boot/dts/ti/k3-am62-mcu.dtsi | 9 +++++
> arch/arm64/boot/dts/ti/k3-am62-wakeup.dtsi | 11 ++++++
> 3 files changed, 65 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> index ea683fd77d6a..90d5f145de4f 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> @@ -758,6 +758,51 @@ main_mcan0: can@20701000 {
> status = "disabled";
> };
>
> + main_rti0: watchdog@e000000 {
> + compatible = "ti,j7-rti-wdt";
> + reg = <0x00 0x0e000000 0x00 0x100>;
> + clocks = <&k3_clks 125 0>;
> + power-domains = <&k3_pds 125 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 125 0>;
> + assigned-clock-parents = <&k3_clks 125 2>;
> + };
> +
> + main_rti1: watchdog@e010000 {
> + compatible = "ti,j7-rti-wdt";
> + reg = <0x00 0x0e010000 0x00 0x100>;
> + clocks = <&k3_clks 126 0>;
> + power-domains = <&k3_pds 126 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 126 0>;
> + assigned-clock-parents = <&k3_clks 126 2>;
> + };
> +
> + main_rti2: watchdog@e020000 {
> + compatible = "ti,j7-rti-wdt";
> + reg = <0x00 0x0e020000 0x00 0x100>;
> + clocks = <&k3_clks 127 0>;
> + power-domains = <&k3_pds 127 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 127 0>;
> + assigned-clock-parents = <&k3_clks 127 2>;
> + };
> +
> + main_rti3: watchdog@e030000 {
> + compatible = "ti,j7-rti-wdt";
> + reg = <0x00 0x0e030000 0x00 0x100>;
> + clocks = <&k3_clks 128 0>;
> + power-domains = <&k3_pds 128 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 128 0>;
> + assigned-clock-parents = <&k3_clks 128 2>;
> + };
> +
> + main_rti4: watchdog@e0f0000 {
> + compatible = "ti,j7-rti-wdt";
> + reg = <0x00 0x0e0f0000 0x00 0x100>;
> + clocks = <&k3_clks 130 0>;
> + power-domains = <&k3_pds 130 TI_SCI_PD_EXCLUSIVE>;
> + assigned-clocks = <&k3_clks 130 0>;
> + assigned-clock-parents = <&k3_clks 130 2>;
> + };
> +

This may be a dumb question, though the ti-sci and TRM documentation is
labeling this as rti15? idk if we should label this the same? it might
make grepping a little easier :)

~Bryan