[PATCH 14/32] efi: Rename ImageBase to image_base
Simon Glass
sjg at chromium.org
Thu Feb 6 22:27:30 CET 2025
Hi Tom,
On Mon, 3 Feb 2025 at 13:09, Tom Rini <trini at konsulko.com> wrote:
>
> On Mon, Feb 03, 2025 at 10:42:07AM -0700, Simon Glass wrote:
>
> > It is confusing to have a camel-case indentifier in the link scripts.
> > Rename it to fit with U-Boot's snake-case style.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
>
> This isn't the linker script, it's the crt0 file.
Yes, it's referred to in the x86 linker scripts. Probably we should
rationalise some of these names.
> Given that this is intended to match up with
> https://learn.microsoft.com/en-us/windows/win32/debug/pe-format (which I
> got from looking at the commit message on commit 216459d2cf36
> ("efi_loader: correct SizeOfCode, SizeOfInitializedData") and
> include/pe.h I don't think this is a good change. Heinrich?
Well, pe.h is a file format so I don't believe we need to use the same
code style. We had that discussion when the EFI code was coming in,
and decided that it wasn't a good idea (as I recall).
Regards,
Simon
More information about the U-Boot
mailing list