Re: [PATCH 3/5] xilinx: zynqmp: move fdt_addr so we can use devices with less memory

[Michal Simek]

On 8/30/23 16:03, Steffen Dirkwinkel wrote:

From: Steffen Dirkwinkel <s.dirkwin...@beckhoff.com>

With the current config we'd put the fdt outside of memory.

Signed-off-by: Steffen Dirkwinkel <s.dirkwin...@beckhoff.com>
---

  include/configs/xilinx_zynqmp.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h
index 74264b7bee..334fe95d66 100644
--- a/include/configs/xilinx_zynqmp.h
+++ b/include/configs/xilinx_zynqmp.h
@@ -50,7 +50,7 @@
  #endif

#define ENV_MEM_LAYOUT_SETTINGS \

-       "fdt_addr_r=0x40000000\0" \
+       "fdt_addr_r=0x28000000\0" \
        "fdt_size_r=0x400000\0" \
        "pxefile_addr_r=0x10000000\0" \
        "kernel_addr_r=0x18000000\0" \

This is not the right way to go.

I have started to move these variables setting to DT.
You can take a look at
https://github.com/devicetree-org/dt-schema/commit/3815da51a138619f443abcf2f821ca0a6fd57949

And fdt_addr_r is the same case. It should be moved to DT and remove it from 
here.

Thanks,
Michal