common: fdt: Remove additional 4k space for fdt allocation
authorAshok Reddy Soma <ashok.reddy.soma@xilinx.com>
Mon, 6 Apr 2020 13:58:30 +0000 (07:58 -0600)
committerMichal Simek <michal.simek@xilinx.com>
Thu, 25 Jun 2020 08:09:32 +0000 (10:09 +0200)
There is no technical reason to add additional 4k space for FDT. This space
is completely unused and just increase memory requirements. This is
problematic on systems with limited memory resources as Xilinx Zynq
CSE/ZynqMP mini and Versal mini configurations.

The patch is removing additional 4k space.

EFI code is using copy_fdt() which copy FDT to different location.
And all boot commands in case of using U-Boot's FDT pointed by
$fdtcontroladdr are copying FDT to different locations by
image_setup_libfdt().
That's why in proper flow none should modified DTB used by U-Boot that's
why there is no need for additional space.

Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Acked-by: Stephen Warren <swarren@nvidia.com>
common/board_f.c

index 01194eaa0e4d3cf9ba20c00aee0cd24ef69b0a20..dcad551ae434e64bd28a24587c0da6bddbab649d 100644 (file)
@@ -537,7 +537,7 @@ static int reserve_fdt(void)
         * will be relocated with other data.
         */
        if (gd->fdt_blob) {
-               gd->fdt_size = ALIGN(fdt_totalsize(gd->fdt_blob) + 0x1000, 32);
+               gd->fdt_size = ALIGN(fdt_totalsize(gd->fdt_blob), 32);
 
                gd->start_addr_sp = reserve_stack_aligned(gd->fdt_size);
                gd->new_fdt = map_sysmem(gd->start_addr_sp, gd->fdt_size);