Re: [PATCH] arm64: dts: ti: k3-j7xx: Change timer nodes status to reserved

["Kumar, Udit" <u-kumar1@ti.com>]

On 6/7/2024 4:25 PM, Beleswar Padhi wrote:
> The remoteproc firmware like of R5F and DSPs in the MAIN voltage domain
> use timers. Therefore, change the status of the timer nodes to
> "reserved" to avoid any clash.
>
> This change is already incorporated for timer nodes in the MCU voltage
> domain.
>
> Fixes: 835d04422f9d ("arm64: dts: ti: k3-j721s2: Add general purpose timers")
>
> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
> ---
>   arch/arm64/boot/dts/ti/k3-j7200-main.dtsi  |  2 ++
>   arch/arm64/boot/dts/ti/k3-j721e-main.dtsi  |  7 +++++++
>   arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi |  6 ++++++
>   arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 10 ++++++++++
>   4 files changed, 25 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi
> index 9386bf3ef9f68..22351a4f3da6e 100644
> --- a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi
> @@ -1254,6 +1254,7 @@ main_timer0: timer@2400000 {
>   		assigned-clock-parents = <&k3_clks 49 2>;
>   		power-domains = <&k3_pds 49 TI_SCI_PD_EXCLUSIVE>;
>   		ti,timer-pwm;
> +		status = "reserved";
>   	};
>   
>   	main_timer1: timer@2410000 {
> @@ -1266,6 +1267,7 @@ main_timer1: timer@2410000 {
>   		assigned-clock-parents = <&k3_clks 50 2>, <&k3_clks 313 1>;
>   		power-domains = <&k3_pds 50 TI_SCI_PD_EXCLUSIVE>;
>   		ti,timer-pwm;
> +		status = "reserved";
>   	};
>   
>   	main_timer2: timer@2420000 {

checked only on J7200, for Main MCU 1-0, Looks you need main_timer2 as 
well to get it working

Please verify on other SOCs as well for all firmwares, if some extra 
nodes needs to be added


> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> index 0da785be80ff4..944bdbb98e910 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
> [..]