Re: [PATCH] arm64: dts: ti: k3-am625-sk: Add ECAP APWM nodes

[Aswath Govindraju <a-govindraju@ti.com>]

On 19/04/22 11:59, Vignesh Raghavendra wrote:
> AM62 has 3 ECAP instances with 1 APWM each. Add DT nodes for the same.
> Keep them disabled in am625-sk dts as these pins can be repurposed in
> user exp connector.
> 
> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> ---

Reviewed-by: Aswath Govindraju <a-govindraju@ti.com>

Thanks,
Aswath

>  arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 27 ++++++++++++++++++++++++
>  arch/arm64/boot/dts/ti/k3-am625-sk.dts   | 12 +++++++++++
>  2 files changed, 39 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> index 4b6ba98dd0a2..dd972fcdaedb 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> @@ -530,4 +530,31 @@ mailbox0_cluster0: mailbox@29000000 {
>  		ti,mbox-num-users = <4>;
>  		ti,mbox-num-fifos = <16>;
>  	};
> +
> +	ecap0: pwm@23100000 {
> +		compatible = "ti,am3352-ecap";
> +		#pwm-cells = <3>;
> +		reg = <0x00 0x23100000 0x00 0x100>;
> +		power-domains = <&k3_pds 51 TI_SCI_PD_EXCLUSIVE>;
> +		clocks = <&k3_clks 51 0>;
> +		clock-names = "fck";
> +	};
> +
> +	ecap1: pwm@23110000 {
> +		compatible = "ti,am3352-ecap";
> +		#pwm-cells = <3>;
> +		reg = <0x00 0x23110000 0x00 0x100>;
> +		power-domains = <&k3_pds 52 TI_SCI_PD_EXCLUSIVE>;
> +		clocks = <&k3_clks 52 0>;
> +		clock-names = "fck";
> +	};
> +
> +	ecap2: pwm@23120000 {
> +		compatible = "ti,am3352-ecap";
> +		#pwm-cells = <3>;
> +		reg = <0x00 0x23120000 0x00 0x100>;
> +		power-domains = <&k3_pds 53 TI_SCI_PD_EXCLUSIVE>;
> +		clocks = <&k3_clks 53 0>;
> +		clock-names = "fck";
> +	};
>  };
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-sk.dts b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> index 5fc35898a1e2..2c5e0e5b826b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am625-sk.dts
> @@ -477,3 +477,15 @@ partition@3fc0000 {
>  		};
>  	};
>  };
> +
> +&ecap0 {
> +	status = "disabled";
> +};
> +
> +&ecap1 {
> +	status = "disabled";
> +};
> +
> +&ecap2 {
> +	status = "disabled";
> +};