Re: [PATCH v2 00/15] Various toolchain compatibility fixes/improvements

[Heinrich Schuchardt]

Am 15. März 2025 23:17:58 MEZ schrieb Sam Edwards <cfsworks at gmail.com>:
>Hello again U-Boot list,
>
>This is v2 of my "misc. fixes" series, sent to prepare the codebase for more
>direct LLVM support in the near future. This series contains several fixes that
>I found in the process of preparing that support and which address issues
>independent of any future feature or enhancement. I am sending these now, both
>so that their inclusion is not delayed by discussion on my upcoming series and
>to make the latter more manageable.
>
>PLEASE APPLY PARTIALLY! I have tried to sort these so that the most
>straightforward changes come first. If any patch proves contentious, you can
>either skip it or stop applying the series at that point. Any patches that need
>revisions will be included in a future series, but I'm trying to avoid there
>being a v3 of *this* series. :)
>
>Happy Saturday,
>Sam

Hello Sam,

I wonder why there are only ARM and X86 specific patches in this series . Wouldn't some of the changes also be needed for RISC-V and other architectures?

For EFI we have carved out a linker script include file to harmonize the linker scripts used by ARM and RISC-V. Shouldn't we do the same for main U-Boot?

Best regards

Heinrich

>
>Changes v1->v2:
>- Carried forward acked/reviewed tags
>- Dropped two patches (one was incorrect, the other can be fixed differently)
>- Minor phrasing changes to commit messages
>- Instead of removing `ENTRY(_start)`, I instead added `.globl _start` where
>  appropriate, in the EFI apps
>
>Sam Edwards (15):
>  arm: Remove stray .mmutable reference in linker script
>  arm: Exclude eabi_compat from LTO
>  arm: Add __aeabi_memclr in eabi_compat
>  arm: Add aligned-memory aliases to eabi_compat
>  arm: Discard unwanted sections in linker script
>  arm: Replace 'adrl' in EFI crt0
>  x86: Fix call64's section flags
>  makefile: Avoid objcopy --gap-fill for .hex/.srec
>  makefile: Add `norelro` linker option
>  makefile: Add READELF command variable
>  arm: riscv: efi: Export _start symbol from crt0_*_efi stubs
>  efi_loader: Move .dynamic out of .text in EFI
>  scripts/Makefile.lib: efi: Preserve the .dynstr section as well
>  spl: riscv: opensbi: Error on misaligned FDT
>  spl: Align FDT load address
>
> Makefile                        | 14 ++++++++------
> arch/arm/cpu/u-boot.lds         | 29 ++++++++++-------------------
> arch/arm/lib/Makefile           |  1 +
> arch/arm/lib/crt0_aarch64_efi.S |  1 +
> arch/arm/lib/crt0_arm_efi.S     |  4 +++-
> arch/arm/lib/eabi_compat.c      | 17 +++++++++++++++++
> arch/riscv/lib/crt0_riscv_efi.S |  1 +
> arch/x86/cpu/i386/call64.S      |  2 +-
> common/spl/spl_fit.c            |  2 +-
> common/spl/spl_opensbi.c        |  5 +++++
> lib/efi_loader/elf_efi.ldsi     |  6 +++---
> scripts/Makefile.lib            |  4 ++--
> 12 files changed, 53 insertions(+), 33 deletions(-)
>