linux-kernel - Re: [PATCH 4/5] arm64: dts: ti: k3-j721s2-main: Add main domain watchdog entries

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <78431d87-ea56-138b-d2f7-6b27b7d83dc1@ti.com>
Date:   Thu, 31 Aug 2023 19:10:38 +0530
From:   "J, KEERTHY" <j-keerthy@...com>
To:     Nishanth Menon <nm@...com>
CC:     <vigneshr@...com>, <robh+dt@...nel.org>,
        <krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
        <kristo@...nel.org>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 4/5] arm64: dts: ti: k3-j721s2-main: Add main domain
 watchdog entries



On 8/31/2023 5:18 PM, Nishanth Menon wrote:
> On 12:56-20230830, Keerthy wrote:
>> Add DT entries for main domain watchdog instances.
>>
>> Signed-off-by: Keerthy <j-keerthy@...com>
>> ---
>>   arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi | 18 ++++++++++++++++++
>>   1 file changed, 18 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
>> index 87841bcc509d..33276d31faa1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi
>> @@ -1513,4 +1513,22 @@
>>   		reg = <0x0 0x700000 0x0 0x1000>;
>>   		ti,esm-pins = <344>, <345>;
>>   	};
>> +
>> +	watchdog0: watchdog@...0000 {
>> +		compatible = "ti,j7-rti-wdt";
>> +		reg = <0x0 0x2200000 0x0 0x100>;
>> +		clocks = <&k3_clks 286 1>;
>> +		power-domains = <&k3_pds 286 TI_SCI_PD_EXCLUSIVE>;
>> +		assigned-clocks = <&k3_clks 286 1>;
>> +		assigned-clock-parents = <&k3_clks 286 5>;
>> +	};
>> +
>> +	watchdog1: watchdog@...0000 {
>> +		compatible = "ti,j7-rti-wdt";
>> +		reg = <0x0 0x2210000 0x0 0x100>;
>> +		clocks = <&k3_clks 287 1>;
>> +		power-domains = <&k3_pds 287 TI_SCI_PD_EXCLUSIVE>;
>> +		assigned-clocks = <&k3_clks 287 1>;
>> +		assigned-clock-parents = <&k3_clks 287 5>;
>> +	};
> 
> Are these all the wdts in main domain - do define and leave the other wdts
> disabled as they are tightly coupled to other processors (and must be
> enabled by the OS running on them), but defining them allows us to reuse
> the definitions in OSes running on R5s (for instance) to reuse the same
> device definitions.

Hello Nishanth,

Okay. I will add all the remaining ones & keep them disabled.

Thanks for the feedback.

Regards,
Keerthy

> 
>>   };
>> -- 
>> 2.17.1
>>
>