[PATCH 07/10] Makefile: Align fit-dtb.blob and u-boot.itb by 64bits

Andre Przywara andre.przywara at arm.com
Mon Aug 23 13:19:48 CEST 2021


On Mon, 23 Aug 2021 09:37:54 +0200
Michal Simek <michal.simek at xilinx.com> wrote:

> Enabling MULTI_DTB_FIT and DTB_RESELECT can end up with multi DTBs in FIT
> image placed and aligned only by 32bits (4bytes). Based on device tree
> specification:
> "Specifically, the memory reservation block shall be aligned to an 8-byte boundary
> and the structure block to a 4-byte boundary."
> is 64bit (8bytes) alignment required. That's why make sure that
> fit-dtb.blob and u-boot.itb as our primary target images for Xilinx ZynqMP
> are all 64bit aligned.

Indeed, thanks for fixing this!

> 
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>

Reviewed-by: Andre Przywara <andre.przywara at arm.com>

Cheers,
Andre

> ---
> 
> Changes in v2:
> - Update subject and commit message
> - Enable 64bit alignment for all - reported by Andre
> 
>  Makefile | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Makefile b/Makefile
> index 269e353a28ad..0de909123cf9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1169,6 +1169,8 @@ MKIMAGEFLAGS_fit-dtb.blob = -f auto -A $(ARCH) -T firmware -C none -O u-boot \
>  	-a 0 -e 0 -E \
>  	$(patsubst %,-b arch/$(ARCH)/dts/%.dtb,$(subst ",,$(CONFIG_OF_LIST))) -d /dev/null
>  
> +MKIMAGEFLAGS_fit-dtb.blob += -B 0x8
> +
>  ifneq ($(EXT_DTB),)
>  u-boot-fit-dtb.bin: u-boot-nodtb.bin $(EXT_DTB)
>  		$(call if_changed,cat)
> @@ -1431,6 +1433,7 @@ MKIMAGEFLAGS_u-boot.itb =
>  else
>  MKIMAGEFLAGS_u-boot.itb = -E
>  endif
> +MKIMAGEFLAGS_u-boot.itb += -B 0x8
>  
>  ifdef U_BOOT_ITS
>  u-boot.itb: u-boot-nodtb.bin \



More information about the U-Boot mailing list