Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
From:   Alexander Stein <alexander.stein@ew.tq-group.com>
To:     Daniel Baluta <daniel.baluta@nxp.com>,
        Mpuaudiosw <Mpuaudiosw@nxp.com>,
        Iuliana Prodan <iuliana.prodan@nxp.com>,
        "Iuliana Prodan (OSS)" <iuliana.prodan@oss.nxp.com>
Cc:     Mathieu Poirier <mathieu.poirier@linaro.org>, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
        Shawn Guo <shawnguo@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        "S.J. Wang" <shengjiu.wang@nxp.com>,
        Fabio Estevam <festevam@gmail.com>, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-imx <linux-imx@nxp.com>, linux-remoteproc@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        LnxRevLi <LnxRevLi@nxp.com>
Subject: Re: [PATCH v3 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP
Date:   Fri, 13 Oct 2023 11:31:39 +0200
Message-ID: <7769811.31r3eYUQgx@steina-w>
Organization: TQ-Systems GmbH
In-Reply-To: <80a6a56b-4a12-99bb-acb3-94a5753b2de4@nxp.com>
References: <20231010090929.4371-1-iuliana.prodan@oss.nxp.com> <2301291.ElGaqSPkdT@steina-w> <80a6a56b-4a12-99bb-acb3-94a5753b2de4@nxp.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"
Precedence: bulk

Hi Iuliana,

Am Freitag, 13. Oktober 2023, 10:35:17 CEST schrieb Iuliana Prodan:
> Hi Alexander,
>=20
> On 10/11/2023 8:37 AM, Alexander Stein wrote:
> > Hi Iuliana,
> >=20
> > Am Dienstag, 10. Oktober 2023, 11:09:29 CEST schrieb Iuliana Prodan (OS=
S):
> >> From: Iuliana Prodan <iuliana.prodan@nxp.com>
> >>=20
> >> Add the reserve-memory nodes used by DSP when the rpmsg
> >> feature is enabled.
> >>=20
> >> Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
> >> ---
> >>=20
> >>   arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++++++
> >>   1 file changed, 16 insertions(+)
> >>=20
> >> diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index
> >> cc406bb338fe..22815b3ea890 100644
> >> --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> @@ -211,6 +211,22 @@
> >>=20
> >>   			reg =3D <0 0x92400000 0 0x2000000>;
> >>   			no-map;
> >>   	=09
> >>   		};
> >>=20
> >> +
> >> +		dsp_vdev0vring0: vdev0vring0@942f0000 {
> >> +			reg =3D <0 0x942f0000 0 0x8000>;
> >> +			no-map;
> >> +		};
> >> +
> >> +		dsp_vdev0vring1: vdev0vring1@942f8000 {
> >> +			reg =3D <0 0x942f8000 0 0x8000>;
> >> +			no-map;
> >> +		};
> >> +
> >> +		dsp_vdev0buffer: vdev0buffer@94300000 {
> >> +			compatible =3D "shared-dma-pool";
> >> +			reg =3D <0 0x94300000 0 0x100000>;
> >> +			no-map;
> >> +		};
> >=20
> > Please configure these reserved memories on board level. Not every i.MX=
8MP
> > based board uses this DSP or has these memory addresses available.
>=20
> Will it be ok in imx8mp-evk.dts?

If that is the board using the DSP and the reserved memory, then yes.

Best regards,
Alexander

> Thanks,
> Iulia
>=20
> > Best regards,
> > Alexander
> >=20
> >>   	};
> >>   =09
> >>   	pmu {


=2D-=20
TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht M=FCnchen, HRB 105018
Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider
http://www.tq-group.com/