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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Mon, 1 Jul 2024 13:56:53 +0530
From: "Kumar, Udit" <u-kumar1@...com>
To: Beleswar Padhi <b-padhi@...com>, <nm@...com>, <vigneshr@...com>,
        Jayesh
 Choudhary <j-choudhary@...com>
CC: <kristo@...nel.org>, <robh@...nel.org>, <krzk+dt@...nel.org>,
        <conor+dt@...nel.org>, <tony@...mide.com>,
        <linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: ti: k3-j7xx: Change timer nodes status to
 reserved


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@...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@...0000 {
>   		assigned-clock-parents = <&k3_clks 49 2>;
>   		power-domains = <&k3_pds 49 TI_SCI_PD_EXCLUSIVE>;
>   		ti,timer-pwm;
> +		status = "reserved";
>   	};
>   
>   	main_timer1: timer@...0000 {
> @@ -1266,6 +1267,7 @@ main_timer1: timer@...0000 {
>   		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@...0000 {

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
> [..]