| Message ID | 20220126044954.19069-2-christianshewitt@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: meson: add ATF BL32 reserved-memory regions | expand |
Hi! 26.01.2022 07:49, Christian Hewitt wrote: > Add an additional reserved memory region for the BL32 trusted firmware > present in many devices that boot from Amlogic vendor u-boot. > > Suggested-by: Mateusz Krzak <kszaquitto@gmail.com> > Signed-off-by: Christian Hewitt <christianshewitt@gmail.com> > --- > arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > index 6b457b2c30a4..aa14ea017a61 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > @@ -49,6 +49,12 @@ > no-map; > }; > > + /* 32 MiB reserved for ARM Trusted Firmware (BL32) */ > + secmon_reserved_bl32: secmon@5300000 { > + reg = <0x0 0x05300000 0x0 0x2000000>; > + no-map; > + }; > + How do I check if we need a similar patch for axg boards? > linux,cma { > compatible = "shared-dma-pool"; > reusable; >
> On 26 Jan 2022, at 9:35 am, Vyacheslav <adeep@lexina.in> wrote: > > Hi! > > 26.01.2022 07:49, Christian Hewitt wrote: >> Add an additional reserved memory region for the BL32 trusted firmware >> present in many devices that boot from Amlogic vendor u-boot. >> Suggested-by: Mateusz Krzak <kszaquitto@gmail.com> >> Signed-off-by: Christian Hewitt <christianshewitt@gmail.com> >> --- >> arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ >> 1 file changed, 6 insertions(+) >> diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> index 6b457b2c30a4..aa14ea017a61 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> @@ -49,6 +49,12 @@ >> no-map; >> }; >> + /* 32 MiB reserved for ARM Trusted Firmware (BL32) */ >> + secmon_reserved_bl32: secmon@5300000 { >> + reg = <0x0 0x05300000 0x0 0x2000000>; >> + no-map; >> + }; >> + > How do I check if we need a similar patch for axg boards? Are they booting using Amlogic (vendor) u-boot sources that include bl32.img in the FIP signing recipe? If booting from upstream u-boot, like JetHome boards, it’s nothing to worry about. Christian
Hi, On 26/01/2022 06:35, Vyacheslav wrote: > Hi! > > 26.01.2022 07:49, Christian Hewitt wrote: >> Add an additional reserved memory region for the BL32 trusted firmware >> present in many devices that boot from Amlogic vendor u-boot. >> >> Suggested-by: Mateusz Krzak <kszaquitto@gmail.com> >> Signed-off-by: Christian Hewitt <christianshewitt@gmail.com> >> --- >> arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> index 6b457b2c30a4..aa14ea017a61 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi >> @@ -49,6 +49,12 @@ >> no-map; >> }; >> + /* 32 MiB reserved for ARM Trusted Firmware (BL32) */ >> + secmon_reserved_bl32: secmon@5300000 { >> + reg = <0x0 0x05300000 0x0 0x2000000>; >> + no-map; >> + }; >> + > How do I check if we need a similar patch for axg boards? > > >> linux,cma { >> compatible = "shared-dma-pool"; >> reusable; >> > For AXG board with BL32 booting from vendor u-boot, yes. Neil
On 26/01/2022 05:49, Christian Hewitt wrote: > Add an additional reserved memory region for the BL32 trusted firmware > present in many devices that boot from Amlogic vendor u-boot. > > Suggested-by: Mateusz Krzak <kszaquitto@gmail.com> > Signed-off-by: Christian Hewitt <christianshewitt@gmail.com> > --- > arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > index 6b457b2c30a4..aa14ea017a61 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi > @@ -49,6 +49,12 @@ > no-map; > }; > > + /* 32 MiB reserved for ARM Trusted Firmware (BL32) */ > + secmon_reserved_bl32: secmon@5300000 { > + reg = <0x0 0x05300000 0x0 0x2000000>; > + no-map; > + }; > + > linux,cma { > compatible = "shared-dma-pool"; > reusable; > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi index 6b457b2c30a4..aa14ea017a61 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi @@ -49,6 +49,12 @@ no-map; }; + /* 32 MiB reserved for ARM Trusted Firmware (BL32) */ + secmon_reserved_bl32: secmon@5300000 { + reg = <0x0 0x05300000 0x0 0x2000000>; + no-map; + }; + linux,cma { compatible = "shared-dma-pool"; reusable;
Add an additional reserved memory region for the BL32 trusted firmware present in many devices that boot from Amlogic vendor u-boot. Suggested-by: Mateusz Krzak <kszaquitto@gmail.com> Signed-off-by: Christian Hewitt <christianshewitt@gmail.com> --- arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 6 ++++++ 1 file changed, 6 insertions(+)