Hi Tom, On Sat, 8 Feb 2025 at 14:29, Tom Rini <[email protected]> wrote: > > On Sat, Feb 08, 2025 at 10:11:20AM -0700, Simon Glass wrote: > > [snip] > > Changes in v2: > > - Depend on CONFIG_EFI_APP instead > > - Drop change to u-boot-elf rule > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Drop the word 'Sadly' > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Add new patch to rename VENDOR_EFI to ARCH_EFI > > - Update to use ARCH_EFI instead of VENDOR_EFI > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Rework the commit message to clarify the relationship to link-scripts > > - Expand commit message to explains that EFI_APP skips relocation > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Document the x86 field better and add a comment for ARM too > > - Mention '64-bit ARM' here, rather than just 'ARM' > > - Put the EFI-app case first in setup_mon_len(), for clarity > > - Use ARCH_EFI instead of VENDOR_EFI > > - Merge the linker-script rules into Kconfig in this patch > > - Drop patch 'Select the EFI linker script for the app' > > There's things that were mentioned for v1 that aren't in this list, did > you miss them or disagree with them? >
The only one I think I didn't do was ' Drop exception code'. I'm not actually using / enabling an SPL symbol so I think you have the wrong end of the stick. Regards, SImon

