On 12/4/19 11:42 PM, Heinrich Schuchardt wrote:
Function fetch_tftp_file() in test/py/tests/test_efi_loader.py expects that the dictionary describing a file contains an entry 'addr' specifying the loading address. Otherwise it defaults to the start of RAM. On qemu_arm64_defconfig and qemu_arm_defconfig this collides with the hardware supplied device tree.
Add an optional parameter in function file2env() to set the 'addr' entry.
Set the load address for files to the value of $kernel_addr_r for the qemu_arm64_defconfig and qemu_arm_defconfig boards to avoid collisions with the device tree.
The series looks fine to me; I hope to apply it tomorrow just in case there are other comments.
P.S. It might have been useful to mention the target repo in the patch subject or at least at the top of the cover letter, since it took me a few extra seconds to realize this was a patch to my hooks repo not U-Boot's test/py directory, which was confusing! "[hooks PATCH] ..." might have helped there.