* [PATCH v4 1/2] remoteproc: imx_dsp_rproc: add mandatory find_loaded_rsc_table op
2023-10-13 15:27 [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table Iuliana Prodan (OSS)
@ 2023-10-13 15:27 ` Iuliana Prodan (OSS)
2023-10-13 15:27 ` [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP Iuliana Prodan (OSS)
2023-11-20 21:49 ` [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table Mathieu Poirier
2 siblings, 0 replies; 6+ messages in thread
From: Iuliana Prodan (OSS) @ 2023-10-13 15:27 UTC (permalink / raw)
To: Mathieu Poirier, robh+dt, krzysztof.kozlowski+dt, conor+dt,
Shawn Guo, Bjorn Andersson, Sascha Hauer, alexander.stein,
S.J. Wang, Fabio Estevam, devicetree, Daniel Baluta, Mpuaudiosw,
Iuliana Prodan
Cc: linux-imx, linux-remoteproc, linux-arm-kernel, linux-kernel,
Pengutronix Kernel Team, LnxRevLi
From: Iuliana Prodan <iuliana.prodan@nxp.com>
Add the .find_loaded_rsc_table operation for i.MX DSP.
We need it for inter-process communication between DSP
and main core.
This callback is used to find the resource table (defined
in remote processor linker script) where the address of the
vrings along with the other allocated resources (carveouts etc)
are stored.
If this is not found, the vrings are not allocated and
the IPC between cores will not work.
Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>
---
drivers/remoteproc/imx_dsp_rproc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/remoteproc/imx_dsp_rproc.c b/drivers/remoteproc/imx_dsp_rproc.c
index 8fcda9b74545..a1c62d15f16c 100644
--- a/drivers/remoteproc/imx_dsp_rproc.c
+++ b/drivers/remoteproc/imx_dsp_rproc.c
@@ -940,6 +940,7 @@ static const struct rproc_ops imx_dsp_rproc_ops = {
.kick = imx_dsp_rproc_kick,
.load = imx_dsp_rproc_elf_load_segments,
.parse_fw = imx_dsp_rproc_parse_fw,
+ .find_loaded_rsc_table = rproc_elf_find_loaded_rsc_table,
.sanity_check = rproc_elf_sanity_check,
.get_boot_addr = rproc_elf_get_boot_addr,
};
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP
2023-10-13 15:27 [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table Iuliana Prodan (OSS)
2023-10-13 15:27 ` [PATCH v4 1/2] remoteproc: imx_dsp_rproc: add mandatory find_loaded_rsc_table op Iuliana Prodan (OSS)
@ 2023-10-13 15:27 ` Iuliana Prodan (OSS)
2023-10-23 17:24 ` Mathieu Poirier
2023-11-20 21:49 ` [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table Mathieu Poirier
2 siblings, 1 reply; 6+ messages in thread
From: Iuliana Prodan (OSS) @ 2023-10-13 15:27 UTC (permalink / raw)
To: Mathieu Poirier, robh+dt, krzysztof.kozlowski+dt, conor+dt,
Shawn Guo, Bjorn Andersson, Sascha Hauer, alexander.stein,
S.J. Wang, Fabio Estevam, devicetree, Daniel Baluta, Mpuaudiosw,
Iuliana Prodan
Cc: linux-imx, linux-remoteproc, linux-arm-kernel, linux-kernel,
Pengutronix Kernel Team, LnxRevLi
From: Iuliana Prodan <iuliana.prodan@nxp.com>
Add the reserve-memory nodes used by DSP when the rpmsg
feature is enabled.
Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 22 ++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
index fa37ce89f8d3..b677ad8ef042 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
@@ -125,6 +125,28 @@
};
};
+
+ reserved-memory {
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
+
+ dsp_vdev0vring0: vdev0vring0@942f0000 {
+ reg = <0 0x942f0000 0 0x8000>;
+ no-map;
+ };
+
+ dsp_vdev0vring1: vdev0vring1@942f8000 {
+ reg = <0 0x942f8000 0 0x8000>;
+ no-map;
+ };
+
+ dsp_vdev0buffer: vdev0buffer@94300000 {
+ compatible = "shared-dma-pool";
+ reg = <0 0x94300000 0 0x100000>;
+ no-map;
+ };
+ };
};
&flexspi {
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP
2023-10-13 15:27 ` [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP Iuliana Prodan (OSS)
@ 2023-10-23 17:24 ` Mathieu Poirier
2023-10-24 5:37 ` Alexander Stein
0 siblings, 1 reply; 6+ messages in thread
From: Mathieu Poirier @ 2023-10-23 17:24 UTC (permalink / raw)
To: Iuliana Prodan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, Shawn Guo,
Bjorn Andersson, Sascha Hauer, alexander.stein, S.J. Wang,
Fabio Estevam, devicetree, Daniel Baluta, Mpuaudiosw,
Iuliana Prodan, linux-imx, linux-remoteproc, linux-arm-kernel,
linux-kernel, Pengutronix Kernel Team, LnxRevLi
Hey guys,
On Fri, Oct 13, 2023 at 06:27:31PM +0300, Iuliana Prodan (OSS) wrote:
> From: Iuliana Prodan <iuliana.prodan@nxp.com>
>
> Add the reserve-memory nodes used by DSP when the rpmsg
> feature is enabled.
>
> Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
> ---
> arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 22 ++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> index fa37ce89f8d3..b677ad8ef042 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> @@ -125,6 +125,28 @@
> };
>
> };
> +
> + reserved-memory {
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> +
> + dsp_vdev0vring0: vdev0vring0@942f0000 {
> + reg = <0 0x942f0000 0 0x8000>;
> + no-map;
> + };
> +
> + dsp_vdev0vring1: vdev0vring1@942f8000 {
> + reg = <0 0x942f8000 0 0x8000>;
> + no-map;
> + };
> +
> + dsp_vdev0buffer: vdev0buffer@94300000 {
> + compatible = "shared-dma-pool";
> + reg = <0 0x94300000 0 0x100000>;
> + no-map;
> + };
> + };
Alexander: Are you good with the refactoring?
Rob and Krzysztof: I'm not sure if you want to ack this patch but giving you the
benefit of the doubt.
Shawn and Sascha: Did you plan on picking up this patch or shoud I?
Thanks,
Mathieu
> };
>
> &flexspi {
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP
2023-10-23 17:24 ` Mathieu Poirier
@ 2023-10-24 5:37 ` Alexander Stein
0 siblings, 0 replies; 6+ messages in thread
From: Alexander Stein @ 2023-10-24 5:37 UTC (permalink / raw)
To: Iuliana Prodan (OSS), Mathieu Poirier
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, Shawn Guo,
Bjorn Andersson, Sascha Hauer, S.J. Wang, Fabio Estevam,
devicetree, Daniel Baluta, Mpuaudiosw, Iuliana Prodan, linux-imx,
linux-remoteproc, linux-arm-kernel, linux-kernel,
Pengutronix Kernel Team, LnxRevLi
Hi Mathieu,
Am Montag, 23. Oktober 2023, 19:24:28 CEST schrieb Mathieu Poirier:
> Hey guys,
>
> On Fri, Oct 13, 2023 at 06:27:31PM +0300, Iuliana Prodan (OSS) wrote:
> > From: Iuliana Prodan <iuliana.prodan@nxp.com>
> >
> > Add the reserve-memory nodes used by DSP when the rpmsg
> > feature is enabled.
> >
> > Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
> > ---
> >
> > arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 22 ++++++++++++++++++++
> > 1 file changed, 22 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts index
> > fa37ce89f8d3..b677ad8ef042 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > @@ -125,6 +125,28 @@
> >
> > };
> >
> > };
> >
> > +
> > + reserved-memory {
> > + #address-cells = <2>;
> > + #size-cells = <2>;
> > + ranges;
> > +
> > + dsp_vdev0vring0: vdev0vring0@942f0000 {
> > + reg = <0 0x942f0000 0 0x8000>;
> > + no-map;
> > + };
> > +
> > + dsp_vdev0vring1: vdev0vring1@942f8000 {
> > + reg = <0 0x942f8000 0 0x8000>;
> > + no-map;
> > + };
> > +
> > + dsp_vdev0buffer: vdev0buffer@94300000 {
> > + compatible = "shared-dma-pool";
> > + reg = <0 0x94300000 0 0x100000>;
> > + no-map;
> > + };
> > + };
>
> Alexander: Are you good with the refactoring?
Yes, adding this to EVK is good for me.
Acked-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> Rob and Krzysztof: I'm not sure if you want to ack this patch but giving you
> the benefit of the doubt.
>
> Shawn and Sascha: Did you plan on picking up this patch or shoud I?
>
> Thanks,
> Mathieu
>
> > };
> >
> > &flexspi {
--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table
2023-10-13 15:27 [PATCH v4 0/2] Rpmsg support for i.MX DSP with resource table Iuliana Prodan (OSS)
2023-10-13 15:27 ` [PATCH v4 1/2] remoteproc: imx_dsp_rproc: add mandatory find_loaded_rsc_table op Iuliana Prodan (OSS)
2023-10-13 15:27 ` [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP Iuliana Prodan (OSS)
@ 2023-11-20 21:49 ` Mathieu Poirier
2 siblings, 0 replies; 6+ messages in thread
From: Mathieu Poirier @ 2023-11-20 21:49 UTC (permalink / raw)
To: Iuliana Prodan (OSS)
Cc: robh+dt, krzysztof.kozlowski+dt, conor+dt, Shawn Guo,
Bjorn Andersson, Sascha Hauer, alexander.stein, S.J. Wang,
Fabio Estevam, devicetree, Daniel Baluta, Mpuaudiosw,
Iuliana Prodan, linux-imx, linux-remoteproc, linux-arm-kernel,
linux-kernel, Pengutronix Kernel Team, LnxRevLi
On Fri, Oct 13, 2023 at 06:27:29PM +0300, Iuliana Prodan (OSS) wrote:
> From: Iuliana Prodan <iuliana.prodan@nxp.com>
>
> These patches are needed in order to support rpmsg on DSP when a
> resource table is available.
>
> Changes since v3:
> - add reserve-memory nodes in imx8mp-evk.dts rather than .dtsi (patch 2/2)
>
> Changes since v2:
> - add newline between nodes in dtsi (patch 2/2)
>
> Changes since v1:
> - add missing bracket in dtsi (patch 2/2)
>
> Iuliana Prodan (2):
> remoteproc: imx_dsp_rproc: add mandatory find_loaded_rsc_table op
> arm64: dts: imx8mp: add reserve-memory nodes for DSP
>
> arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 22 ++++++++++++++++++++
> drivers/remoteproc/imx_dsp_rproc.c | 1 +
> 2 files changed, 23 insertions(+)
>
I have applied this set.
Thanks,
Mathieu
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread