diff mbox series

[3/5] ARM: dts: rmobile: Reserve space in R-Car Gen3 DTs

Message ID 20200411200119.309400-3-marek.vasut+renesas@gmail.com
State Superseded
Headers show
Series [1/5] libfdt: Export overlay_apply_node() as fdt_overlay_apply_node() | expand

Commit Message

Marek Vasut April 11, 2020, 8:01 p.m. UTC
Reserve 4 kiB of space in R-Car Gen3 DTs when those DTs are compiled
to permit patching in OpTee-OS /firmware node, /reserved-memory node
and possibly also additional /memory@ nodes.

Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
Cc: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
Cc: Simon Glass <sjg at chromium.org>
Cc: Tom Rini <trini at konsulko.com>
---
 arch/arm/dts/Makefile | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Simon Glass April 19, 2020, 11:37 p.m. UTC | #1
Hi Marek,

On Sat, 11 Apr 2020 at 14:01, Marek Vasut <marek.vasut at gmail.com> wrote:
>
> Reserve 4 kiB of space in R-Car Gen3 DTs when those DTs are compiled
> to permit patching in OpTee-OS /firmware node, /reserved-memory node
> and possibly also additional /memory@ nodes.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> Cc: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Tom Rini <trini at konsulko.com>
> ---
>  arch/arm/dts/Makefile | 4 ++++
>  1 file changed, 4 insertions(+)

Reviewed-by: Simon Glass <sjg at chromium.org>

Perhaps we should have a CONFIG_DTC_RESERVE option to set the reserved size?

>
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 1ce44204ec..b9aa44b5d8 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -749,6 +749,10 @@ dtb-$(CONFIG_RCAR_GEN3) += \
>         r8a77990-ebisu-u-boot.dtb \
>         r8a77995-draak-u-boot.dtb
>
> +ifdef CONFIG_RCAR_GEN3
> +DTC_FLAGS += -R 4 -p 0x1000
> +endif
> +
>  dtb-$(CONFIG_RZA1) += \
>         r7s72100-gr-peach-u-boot.dtb
>
> --
> 2.25.1
>

Regards,
Simon
Tom Rini April 20, 2020, 4:31 p.m. UTC | #2
On Sun, Apr 19, 2020 at 05:37:17PM -0600, Simon Glass wrote:
> Hi Marek,
> 
> On Sat, 11 Apr 2020 at 14:01, Marek Vasut <marek.vasut at gmail.com> wrote:
> >
> > Reserve 4 kiB of space in R-Car Gen3 DTs when those DTs are compiled
> > to permit patching in OpTee-OS /firmware node, /reserved-memory node
> > and possibly also additional /memory@ nodes.
> >
> > Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> > Cc: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
> > Cc: Simon Glass <sjg at chromium.org>
> > Cc: Tom Rini <trini at konsulko.com>
> > ---
> >  arch/arm/dts/Makefile | 4 ++++
> >  1 file changed, 4 insertions(+)
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> Perhaps we should have a CONFIG_DTC_RESERVE option to set the reserved size?

Perhaps this is also related to the patch Michal posted about modifying
the reserve size?  Thanks!

> 
> >
> > diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> > index 1ce44204ec..b9aa44b5d8 100644
> > --- a/arch/arm/dts/Makefile
> > +++ b/arch/arm/dts/Makefile
> > @@ -749,6 +749,10 @@ dtb-$(CONFIG_RCAR_GEN3) += \
> >         r8a77990-ebisu-u-boot.dtb \
> >         r8a77995-draak-u-boot.dtb
> >
> > +ifdef CONFIG_RCAR_GEN3
> > +DTC_FLAGS += -R 4 -p 0x1000
> > +endif
> > +
> >  dtb-$(CONFIG_RZA1) += \
> >         r7s72100-gr-peach-u-boot.dtb
> >
> > --
> > 2.25.1
> >
> 
> Regards,
> Simon
diff mbox series

Patch

diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 1ce44204ec..b9aa44b5d8 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -749,6 +749,10 @@  dtb-$(CONFIG_RCAR_GEN3) += \
 	r8a77990-ebisu-u-boot.dtb \
 	r8a77995-draak-u-boot.dtb
 
+ifdef CONFIG_RCAR_GEN3
+DTC_FLAGS += -R 4 -p 0x1000
+endif
+
 dtb-$(CONFIG_RZA1) += \
 	r7s72100-gr-peach-u-boot.dtb