* [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes
@ 2023-08-10 0:56 Hari Nagalla
2023-08-10 1:10 ` Nishanth Menon
0 siblings, 1 reply; 4+ messages in thread
From: Hari Nagalla @ 2023-08-10 0:56 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-kernel, linux-arm-kernel, devicetree
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
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
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes
2023-08-10 0:56 [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes Hari Nagalla
@ 2023-08-10 1:10 ` Nishanth Menon
2023-08-10 15:38 ` Hari Nagalla
0 siblings, 1 reply; 4+ messages in thread
From: Nishanth Menon @ 2023-08-10 1:10 UTC (permalink / raw)
To: Hari Nagalla
Cc: vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt, conor+dt,
linux-kernel, linux-arm-kernel, devicetree
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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes
2023-08-10 1:10 ` Nishanth Menon
@ 2023-08-10 15:38 ` Hari Nagalla
2023-08-10 18:38 ` Nishanth Menon
0 siblings, 1 reply; 4+ messages in thread
From: Hari Nagalla @ 2023-08-10 15:38 UTC (permalink / raw)
To: Nishanth Menon
Cc: vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt, conor+dt,
linux-kernel, linux-arm-kernel, devicetree
On 8/9/23 20:10, Nishanth Menon 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?
As such this yamllint errors for am69-sk.dtb were present in several
earlier tags of linux-next. Checked with latest tag: next-20230809.
Please let me know if you would need v2 with the tag specified in comments.
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@64800000: 'mboxes' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@64800000: 'memory-region' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@65800000: 'mboxes' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@65800000: 'memory-region' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@66800000: 'mboxes' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@66800000: 'memory-region' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@67800000: 'mboxes' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
/home/a0868345/git/upstream/linux-next/arch/arm64/boot/dts/ti/k3-am69-sk.dtb:
dsp@67800000: 'memory-region' is a required property
from schema $id:
http://devicetree.org/schemas/remoteproc/ti,k3-dsp-rproc.yaml#
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes
2023-08-10 15:38 ` Hari Nagalla
@ 2023-08-10 18:38 ` Nishanth Menon
0 siblings, 0 replies; 4+ messages in thread
From: Nishanth Menon @ 2023-08-10 18:38 UTC (permalink / raw)
To: Hari Nagalla
Cc: vigneshr, kristo, robh+dt, krzysztof.kozlowski+dt, conor+dt,
linux-kernel, linux-arm-kernel, devicetree
On 10:38-20230810, Hari Nagalla wrote:
> On 8/9/23 20:10, Nishanth Menon 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?
> As such this yamllint errors for am69-sk.dtb were present in several earlier
> tags of linux-next. Checked with latest tag: next-20230809.
>
> Please let me know if you would need v2 with the tag specified in comments.
>
Please rebase on
https://lore.kernel.org/all/5ec8b817-e63f-3d76-894d-8af4f4e880db@ti.com/
series.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-08-10 18:40 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-10 0:56 [PATCH] arm64: dts: ti: k3-j784s4-main: disable remote proc nodes Hari Nagalla
2023-08-10 1:10 ` Nishanth Menon
2023-08-10 15:38 ` Hari Nagalla
2023-08-10 18:38 ` Nishanth Menon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox