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.
