Re: [PATCH v2 3/4] arm: mediatek: mt8390: build boot images with binman

[David Lechner <dlechner@baylibre.com>]

On 7/22/26 4:35 PM, Carlo Caione wrote:
> Generate the Genio 700 boot-chain images from the U-Boot build.
> 
> mtk-boot.bin wraps an external DDR loader, padded to its fixed
> 0x4b000-byte ABI, followed by a CONFIG_SPL_MAX_SIZE U-Boot SPL region
> in a MediaTek eMMC BootROM image. The external loader copies that
> complete fixed-size SPL window, so pad the region to keep every source
> byte inside the BootROM-loaded payload.
> 
> bootloaders.img is a conventional FIT containing BL31, U-Boot proper,
> OP-TEE, and the U-Boot control devicetree.
> 
> Use the standard BL31 and TEE binman inputs and obtain ddr-loader.bin
> through BINMAN_INDIRS. Disable SPL binman symbols because SPL and
> U-Boot proper live in separate storage images. SPL locates U-Boot
> through the GPT FIT partition, so there are no in-image U-Boot offsets
> to publish.
> 
> Signed-off-by: Vitor Sato Eschholz <vsatoes@baylibre.com>
> Signed-off-by: Julien Stephan <jstephan@baylibre.com>
> Signed-off-by: Carlo Caione <ccaione@baylibre.com>
> ---
>  arch/arm/dts/mt8390-genio-700-evk-u-boot.dtsi | 120 ++++++++++++++++++++++++++

Similar to Julien's comment on patch 2, I would suggest to put
as much of this as possible in mt8390-genio-common-u-boot.dtsi (following
upstream naming convention) so that we don't have to repeat it
on Genio 510 or move it later.


>  arch/arm/mach-mediatek/Kconfig                |   1 +
>  configs/mt8390_genio_700_evk_defconfig        |   1 +
>  3 files changed, 122 insertions(+)
>