Hi Venkatesh, On Wed, 20 Aug 2025 at 01:15, Abbarapu, Venkatesh <venkatesh.abbar...@amd.com> wrote: > > [AMD Official Use Only - AMD Internal Distribution Only] > > Hi @Raymond Mao, > I was trying to test these patches with the transfer list binary which > contains tl.bin(dtb+dtbo) inside it. Facing issue with the dtbo address and > fdt_check_header(*blob) from fdtdec_apply_dto_blob() gets the invalid magic > num.
Currently, DTO support is not enabled in TF-A qemu - It won't append a DTO via qemu so I am not sure if you had your own patch or platform code to make this work. To verify this U-Boot patch, please simply follow the 'Notes for testing' in my cover letter, my working branch referred by my manifest provides necessary changes for TF-A and OP-TEE to allow loading a DTO file via qemu, place it into a TF-A reserved memory and append it as a TE. Regards, Raymond > fdt_magic=0x38 fdt_chk_version=0x1 > > Attaching the log with details. Let me know if you need any other information. > > Thanks > Venkatesh > > > -----Original Message----- > > From: Raymond Mao <raymond....@linaro.org> > > Sent: Friday, July 18, 2025 7:46 PM > > To: u-boot@lists.denx.de > > Cc: Simek, Michal <michal.si...@amd.com>; Abbarapu, Venkatesh > > <venkatesh.abbar...@amd.com>; Raymond Mao <raymond....@linaro.org>; > > Tom Rini <tr...@konsulko.com>; Tuomas Tynkkynen <tuomas.tynkky...@iki.fi>; > > Liviu Dudau <liviu.du...@foss.arm.com>; Simon Glass <s...@chromium.org>; > > Ilias > > Apalodimas <ilias.apalodi...@linaro.org>; Dan Carpenter > > <dan.carpen...@linaro.org>; Patrick Rudolph <patrick.rudo...@9elements.com>; > > Andrew Goodbody <andrew.goodb...@linaro.org>; Harrison Mutai > > <harrison.mu...@arm.com>; Jerome Forissier <jerome.foriss...@linaro.org>; > > Heinrich Schuchardt <xypron.g...@gmx.de>; Evgeny Bachinin > > <eabachi...@salutedevices.com>; Matthias Brugger <mbrug...@suse.com>; Lad > > Prabhakar <prabhakar.mahadev-lad...@bp.renesas.com> > > Subject: [PATCH v3 0/6] Add support for DT overlays handoff > > > > The series include refactoring on bloblist and fdtdec to support handoff of > > multiple > > DT overlays and applying them into the DT base during setup. > > All changes are aligned to the spec update for supporting DT overlay > > handoff[1]. > > > > Notes for testing: > > > > Currently DT overlay is not yet enabled in TF-A, but with the test patches > > I provided > > for TF-A and OP-TEE build, importing a DT overlay blob file from QEMU to > > TF-A > > reserved memory is supported. > > Follow below instructions to build and run for test: > > $ repo init -u https://github.com/OP-TEE/manifest.git -m qemu_v8.xml > > Replace your > > local qemu_v8.xml with [2], which contains all necessary changes in both > > TF-A and > > OP-TEE build. > > $ repo sync > > $ cd build > > $ make toolchains > > $ make ARM_FIRMWARE_HANDOFF=y all > > Copy and rename your DT overlay blob as 'qemu_v8.dtb' into out/bin $ make > > ARM_FIRMWARE_HANDOFF=y run-only > > > > [1] Add Transfer Entry for Devicetree Overlay > > https://github.com/FirmwareHandoff/firmware_handoff/pull/74 > > > > [2] > > https://github.com/raymo200915/optee_manifest/blob/dt_overlay_handoff/qemu_v8.x > > ml > > > > Raymond Mao (6): > > bloblist: add blob type for DT overlay > > bloblist: add helper functions > > bloblist: fix a potential negative size for memmove > > bloblist: add API for applying blobs with specified tag > > fdtdec: apply DT overlays from bloblist > > configs: Select OF_LIBFDT_OVERLAY to hand over DTO via bloblist > > > > common/bloblist.c | 67 ++++++++++++++++++++- > > configs/qemu_arm64_defconfig | 1 + > > configs/vexpress_fvp_bloblist_defconfig | 1 + > > include/bloblist.h | 34 ++++++++++- > > lib/fdtdec.c | 80 +++++++++++++++++++++++++ > > 5 files changed, 181 insertions(+), 2 deletions(-) > > > > -- > > 2.25.1 >
