[PATCH 31/32] efi: arm: Simplify the crt0 file and update link script

[Simon Glass]

Hi Tom,

On Mon, 3 Feb 2025 at 19:14, Tom Rini <trini at konsulko.com> wrote:
>
> On Mon, Feb 03, 2025 at 05:38:16PM -0700, Simon Glass wrote:
> > Hi Tom,
> >
> > On Mon, 3 Feb 2025 at 13:10, Tom Rini <trini at konsulko.com> wrote:
> > >
> > > On Mon, Feb 03, 2025 at 10:42:24AM -0700, Simon Glass wrote:
> > > > We don't need to manually add the PE header, since binutils has support
> > > > for this now. Remove it to simplify the file.
> > > >
> > > > Set the link-target to efi-app-aarch64 so that binutils knows what to
> > > > do. Add rules to pick up the arm64 files.
> > > >
> > > > Make some updates to the link-script for arm64, so this all works:
> > > >
> > > > - Pass  .hash .eh_frame and .reloc sections through to objcopy
> > > > - Put RELA pieces into a single section
> > > > - Put linker lists into .data
> > > > - Embed the dtb
> > > >
> > > > Note that it does not seem to be possible to use this approach with arm,
> > > > so this is left alone.
> >
> > Thanks for looking at all this stuff.
>
> You're welcome.
>
> > > Did you mean x86 in this last comment? I'm not sure about the rest,
> > > either way, so I'll leave it to others to comment on that.
> >
> > No, I mean 32-bit ARM. For reasons I don't understand, it seems the
> > toolchain doesn't support PE on 32-bit ARM.
>
> Ah, OK. Were other parts of this series really laying the groundwork for
> EFI_APP with 32bit arm?

Not intentionally, since I was trying to get the series as small as
possible and still actually get something that boots.

I'm not sure that anyone will want a 32-bit ARM EFI-app. I would much
rather that we had it for completeness, but it seems that the binutils
people were not that interested in it. What do you think?

Regards,
Simon