Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 9 Aug 2023 20:10:46 -0500
From:   Nishanth Menon <nm@ti.com>
To:     Hari Nagalla <hnagalla@ti.com>
CC:     <vigneshr@ti.com>, <kristo@kernel.org>, <robh+dt@kernel.org>,
        <krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>,
        <linux-kernel@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <devicetree@vger.kernel.org>
Subject: Re: [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes
Message-ID: <20230810011046.ta3qapj3oj2oqs7o@value>
References: <20230810005629.21738-1-hnagalla@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20230810005629.21738-1-hnagalla@ti.com>
Precedence: bulk

On 19:56-20230809, Hari Nagalla wrote:
> Disable the dsp and r5f subsystem nodes by default. Proper functioning
> of remote processors with ipc need mailbox configurations which can
> vary between board configurations and applications. Hence move enabling
> the remote processor device nodes to where the required configurations
> are complete.
> 
> Signed-off-by: Hari Nagalla <hnagalla@ti.com>
> ---
> This patch fixes the remote proc yamllint errors for am69-sk board

Fixes tag?
> 
>  arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> index a04c44708a09..e7b54338b220 100644
> --- a/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi
> @@ -1402,6 +1402,7 @@ main_r5fss0: r5fss@5c00000 {
>  		ranges = <0x5c00000 0x00 0x5c00000 0x20000>,
>  			 <0x5d00000 0x00 0x5d00000 0x20000>;
>  		power-domains = <&k3_pds 336 TI_SCI_PD_EXCLUSIVE>;
> +		status = "disabled";
>  
>  		main_r5fss0_core0: r5f@5c00000 {
>  			compatible = "ti,j721s2-r5f";
> @@ -1442,6 +1443,7 @@ main_r5fss1: r5fss@5e00000 {
>  		ranges = <0x5e00000 0x00 0x5e00000 0x20000>,
>  			 <0x5f00000 0x00 0x5f00000 0x20000>;
>  		power-domains = <&k3_pds 337 TI_SCI_PD_EXCLUSIVE>;
> +		status = "disabled";
>  
>  		main_r5fss1_core0: r5f@5e00000 {
>  			compatible = "ti,j721s2-r5f";
> @@ -1482,6 +1484,7 @@ main_r5fss2: r5fss@5900000 {
>  		ranges = <0x5900000 0x00 0x5900000 0x20000>,
>  			 <0x5a00000 0x00 0x5a00000 0x20000>;
>  		power-domains = <&k3_pds 338 TI_SCI_PD_EXCLUSIVE>;
> +		status = "disabled";
>  
>  		main_r5fss2_core0: r5f@5900000 {
>  			compatible = "ti,j721s2-r5f";
> @@ -1524,6 +1527,7 @@ c71_0: dsp@64800000 {
>  		ti,sci-proc-ids = <0x30 0xff>;
>  		resets = <&k3_reset 30 1>;
>  		firmware-name = "j784s4-c71_0-fw";
> +		status = "disabled";
>  	};
>  
>  	c71_1: dsp@65800000 {
> @@ -1536,6 +1540,7 @@ c71_1: dsp@65800000 {
>  		ti,sci-proc-ids = <0x31 0xff>;
>  		resets = <&k3_reset 33 1>;
>  		firmware-name = "j784s4-c71_1-fw";
> +		status = "disabled";
>  	};
>  
>  	c71_2: dsp@66800000 {
> @@ -1548,6 +1553,7 @@ c71_2: dsp@66800000 {
>  		ti,sci-proc-ids = <0x32 0xff>;
>  		resets = <&k3_reset 37 1>;
>  		firmware-name = "j784s4-c71_2-fw";
> +		status = "disabled";
>  	};
>  
>  	c71_3: dsp@67800000 {
> @@ -1560,5 +1566,6 @@ c71_3: dsp@67800000 {
>  		ti,sci-proc-ids = <0x33 0xff>;
>  		resets = <&k3_reset 40 1>;
>  		firmware-name = "j784s4-c71_3-fw";
> +		status = "disabled";
>  	};
>  };
> -- 
> 2.34.1
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D