2023-10-13 15:28:21

by Iuliana Prodan (OSS)

Subject: [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP

From: Iuliana Prodan <[email protected]>

Add the reserve-memory nodes used by DSP when the rpmsg
feature is enabled.

Signed-off-by: Iuliana Prodan <[email protected]>
---
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

2023-10-23 17:24:51

by Mathieu Poirier

Subject: Re: [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP

Hey guys,

On Fri, Oct 13, 2023 at 06:27:31PM +0300, Iuliana Prodan (OSS) wrote:
> From: Iuliana Prodan <[email protected]>
>
> Add the reserve-memory nodes used by DSP when the rpmsg
> feature is enabled.
>
> Signed-off-by: Iuliana Prodan <[email protected]>
> ---
> 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
>

2023-10-24 05:38:02

by Alexander Stein

Subject: Re: [PATCH v4 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP

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 <[email protected]>
> >
> > Add the reserve-memory nodes used by DSP when the rpmsg
> > feature is enabled.
> >
> > Signed-off-by: Iuliana Prodan <[email protected]>
> > ---
> >
> > 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 <[email protected]>

> 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/