[PATCH 16/34] image: Add a new type for EFI apps

[Simon Glass]

Hi Mark,

On Fri, 18 Oct 2024 at 03:03, Mark Kettenis <mark.kettenis at xs4all.nl> wrote:
>
> > From: Simon Glass <sjg at chromium.org>
> > Date: Thu, 17 Oct 2024 17:23:55 -0600
>
> Hi Simon,
>
> > Add an image type for EFI applications, which U-Boot supports reading.
>
> Not necessarily a bad idea, but what would the image type be of a
> Linux kernel that is also an EFI application?  See:
>
>   https://docs.kernel.org/admin-guide/efi-stub.html

Hmm that's a good question. That is one of the challenges with EFI,
that we don't know what we are booting. Is there a way to tell whether
an EFI app is a kernel, or not?

For PXE/extlinux we have a 'kernel' line and in that case I set it to
IH_TYPE_KERNEL

>
> Cheers,
>
> Mark
>
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >  boot/image.c    | 1 +
> >  include/image.h | 1 +
> >  2 files changed, 2 insertions(+)
> >
> > diff --git a/boot/image.c b/boot/image.c
> > index cf16fb7fbf7..b13644b03dc 100644
> > --- a/boot/image.c
> > +++ b/boot/image.c
> > @@ -184,6 +184,7 @@ static const table_entry_t uimage_type[] = {
> >       {       IH_TYPE_RENESAS_SPKG, "spkgimage", "Renesas SPKG Image" },
> >       {       IH_TYPE_STARFIVE_SPL, "sfspl", "StarFive SPL Image" },
> >       {       IH_TYPE_EXTLINUX_CFG, "extlinux_cfg", "Extlinux configuration" },
> > +     {       IH_TYPE_EFI, "efi", "EFI applicaiton" },
> >       {       -1,                 "",           "",                   },
> >  };
> >
> > diff --git a/include/image.h b/include/image.h
> > index 606e7d49169..a04925aa829 100644
> > --- a/include/image.h
> > +++ b/include/image.h
> > @@ -233,6 +233,7 @@ enum image_type_t {
> >       IH_TYPE_RENESAS_SPKG,           /* Renesas SPKG image */
> >       IH_TYPE_STARFIVE_SPL,           /* StarFive SPL image */
> >       IH_TYPE_EXTLINUX_CFG,           /* extlinux configuration-file */
> > +     IH_TYPE_EFI,                    /* EFI PE image */
> >
> >       IH_TYPE_COUNT,                  /* Number of image types */
> >  };
> > --
> > 2.34.1
> >
> >

Regards,
Simon