| Message ID | 20230810005629.21738-1-hnagalla@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: ti: k3-j784s4-main: disable remote proc nodes | expand |
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 >
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#
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.
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"; }; };
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(+)