Hi Daniel,
On 12/16/25 10:35 AM, Daniel Palmer wrote:
[You don't often get email from [email protected]. Learn why this is important
at https://aka.ms/LearnAboutSenderIdentification ]
I guess someone had EFI on the mind when writing the comment
and wrote EFI instead of ELF.
Looking at the code, I believe it should be "EI" instead?
https://elixir.bootlin.com/u-boot/v2025.10/source/lib/elf.c#L201
Also please add the following trailer to your commit:
Fixes: 839c4e9c5bb0 ("elf: Add a very simple ELF64 loader")
which should automatically Cc the author of the patch, here Bin Meng
(added manually in Cc) who could tell us what it should actually be.
Cheers,
Quentin
