Aw: Re: [RFC v1 12/14] arm64: dts: mediatek: mt7988: add reserved memory

[Frank Wunderlich]

> Gesendet: Mittwoch, 27. November 2024 um 14:34
> Von: "AngeloGioacchino Del Regno" <angelogioacchino.delregno at collabora.com>
> An: "Frank Wunderlich" <linux at fw-web.de>, "Rob Herring" <robh at kernel.org>, "Krzysztof Kozlowski" <krzk+dt at kernel.org>, "Conor Dooley" <conor+dt at kernel.org>, "Matthias Brugger" <matthias.bgg at gmail.com>
> CC: "Frank Wunderlich" <frank-w at public-files.de>, devicetree at vger.kernel.org, linux-kernel at vger.kernel.org, linux-arm-kernel at lists.infradead.org, linux-mediatek at lists.infradead.org
> Betreff: Re: [RFC v1 12/14] arm64: dts: mediatek: mt7988: add reserved memory
>
> Il 29/10/24 11:39, Frank Wunderlich ha scritto:
> > From: Frank Wunderlich <frank-w at public-files.de>
> >
> > Add memory range handled by ATF to not be touched by linux kernel.
> >
> > Signed-off-by: Frank Wunderlich <frank-w at public-files.de>
>
> If you are really sure that this is always the same for all boards based on MT7988,
> I will trust you and give you my R-b once this series gets out of RFC state.

I'm not sure, but ATF is SoC specific and so it should be a SoC specific range, not board specific

> Cheers,
> Angelo
>
> > ---
> >   arch/arm64/boot/dts/mediatek/mt7988a.dtsi | 12 ++++++++++++
> >   1 file changed, 12 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt7988a.dtsi b/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
> > index a3a7deda35a5..7721d2ec06ee 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
> > +++ b/arch/arm64/boot/dts/mediatek/mt7988a.dtsi
> > @@ -81,6 +81,18 @@ reg_3p3v: regulator-3p3v {
> >   		regulator-always-on;
> >   	};
> >
> > +	reserved-memory {
> > +		#address-cells = <2>;
> > +		#size-cells = <2>;
> > +		ranges;
> > +
> > +		/* 320 KiB reserved for ARM Trusted Firmware (BL31 and BL32) */
> > +		secmon_reserved: secmon at 43000000 {
> > +			reg = <0 0x43000000 0 0x50000>;
> > +			no-map;
> > +		};
> > +	};
> > +
> >   	soc {
> >   		compatible = "simple-bus";
> >   		ranges;
>
> </frank-w at public-files.de></frank-w at public-files.de></frank-w at public-files.de></matthias.bgg at gmail.com></conor+dt at kernel.org></krzk+dt at kernel.org></robh at kernel.org></linux at fw-web.de></angelogioacchino.delregno at collabora.com>