public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de, trini@konsulko.com, sjg@chromium.org,
	ilias.apalodimas@linaro.org, joe.hershberger@ni.com,
	marex@denx.de, mkorpershoek@baylibre.com, maximmosk4@gmail.com,
	raymond.mao@linaro.org, sughosh.ganu@linaro.org
Subject: Re: [PATCH 16/34] image: Add a new type for EFI apps
Date: Fri, 18 Oct 2024 11:03:35 +0200	[thread overview]
Message-ID: <877ca5hjfc.fsf@bloch.sibelius.xs4all.nl> (raw)
In-Reply-To: <20241017232413.724808-17-sjg@chromium.org> (message from Simon Glass on Thu, 17 Oct 2024 17:23:55 -0600)

> From: Simon Glass <sjg@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

Cheers,

Mark

> Signed-off-by: Simon Glass <sjg@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
> 
> 

  reply	other threads:[~2024-10-18  9:03 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-17 23:23 [PATCH 00/34] bootstd: Support recording images Simon Glass
2024-10-17 23:23 ` [PATCH 01/34] alist: Mention the error condition in alist_add_placeholder() Simon Glass
2024-10-17 23:23 ` [PATCH 02/34] alist: Add a comment for alist_init_struct() Simon Glass
2024-10-17 23:23 ` [PATCH 03/34] alist: Expand the comment for alist_get() Simon Glass
2024-10-17 23:23 ` [PATCH 04/34] alist: Add a way to get the next element Simon Glass
2024-10-17 23:23 ` [PATCH 05/34] alist: Add for-loop helpers Simon Glass
2024-10-17 23:23 ` [PATCH 06/34] alist: Add a function to empty the list Simon Glass
2024-10-17 23:23 ` [PATCH 07/34] alist: Add a way to efficiently filter an alist Simon Glass
2024-10-17 23:23 ` [PATCH 08/34] dm: core: Add a function to see if a device exists Simon Glass
2024-10-17 23:23 ` [PATCH 09/34] test: boot: Use a consistent name for the script bootmeth Simon Glass
2024-10-17 23:23 ` [PATCH 10/34] bootstd: Move bootflow-adding to bootstd Simon Glass
2024-10-17 23:23 ` [PATCH 11/34] bootstd: Move bootflow-clearing " Simon Glass
2024-10-17 23:23 ` [PATCH 12/34] bootstd: Add a function to get bootstd only if available Simon Glass
2024-10-17 23:23 ` [PATCH 13/34] bootstd: Drop the bootdev-specific list of bootflows Simon Glass
2024-10-17 23:23 ` [PATCH 14/34] bootstd: Move the bootflow list into an alist Simon Glass
2024-10-17 23:23 ` [PATCH 15/34] image: Add a new type for extlinux Simon Glass
2024-10-18  3:14   ` Tom Rini
2024-10-18 14:57     ` Simon Glass
2024-10-18 15:17       ` Tom Rini
2024-10-18 17:20         ` Simon Glass
2024-10-18 17:54           ` Tom Rini
2024-10-18 18:19             ` Simon Glass
2024-10-18 18:31               ` Tom Rini
2024-10-17 23:23 ` [PATCH 16/34] image: Add a new type for EFI apps Simon Glass
2024-10-18  9:03   ` Mark Kettenis [this message]
2024-10-18 14:55     ` Simon Glass
2024-10-17 23:23 ` [PATCH 17/34] image: Add a new type for logo images Simon Glass
2024-10-17 23:23 ` [PATCH 18/34] image: Add a new type for a command-line string Simon Glass
2024-10-17 23:23 ` [PATCH 19/34] test: Expand implementation of ut_list_has_dm_tests() Simon Glass
2024-10-17 23:23 ` [PATCH 20/34] test: Drop the duplicate line in setup_bootmenu_image() Simon Glass
2024-10-17 23:24 ` [PATCH 21/34] test: boot: Update bootflow_iter() for console checking Simon Glass
2024-10-17 23:24 ` [PATCH 22/34] bootstd: cros: Correct the x86-setup address Simon Glass
2024-10-17 23:24 ` [PATCH 23/34] bootstd: Maintain a list of images Simon Glass
2024-10-18  3:14   ` Tom Rini
2024-10-18  4:01   ` Heinrich Schuchardt
2024-10-18 15:01     ` Simon Glass
2024-10-18 16:33       ` Tom Rini
2024-10-18 17:20         ` Simon Glass
2024-10-18 18:04           ` Tom Rini
2024-10-18 18:48             ` Simon Glass
2024-10-18 21:30               ` Tom Rini
2024-10-19 11:49                 ` Simon Glass
2024-10-19 16:30                   ` Tom Rini
2024-10-22 12:16                     ` Simon Glass
2024-10-22 15:01                       ` Tom Rini
2024-10-22 17:00                         ` Simon Glass
2024-10-23 18:41                           ` Tom Rini
2024-10-31 18:04                             ` Simon Glass
2024-10-17 23:24 ` [PATCH 24/34] bootstd: Update bootmeth_alloc_file() to record images Simon Glass
2024-10-17 23:24 ` [PATCH 25/34] boot: pxe: Drop the duplicate comment on get_pxe_file() Simon Glass
2024-10-17 23:24 ` [PATCH 26/34] efi: Simplify reading files by using the common function Simon Glass
2024-10-18  3:14   ` Tom Rini
2024-10-17 23:24 ` [PATCH 27/34] bootmeth: Update the read_file() method to include a type Simon Glass
2024-10-17 23:24 ` [PATCH 28/34] efi: Check the filename-allocation in the network path Simon Glass
2024-10-17 23:24 ` [PATCH 29/34] boot: Update extlinux pxe_getfile_func() to include type Simon Glass
2024-10-17 23:24 ` [PATCH 30/34] boot: Update pxe bootmeth to record images Simon Glass
2024-10-17 23:24 ` [PATCH 31/34] Update bootmeth_alloc_other() " Simon Glass
2024-10-17 23:24 ` [PATCH 32/34] bootstd: Avoid showing an invalid buffer address Simon Glass
2024-10-17 23:24 ` [PATCH 33/34] bootstd: Update cros bootmeth to record images Simon Glass
2024-10-17 23:24 ` [PATCH 34/34] bootstd: Add a simple command to list images Simon Glass

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=877ca5hjfc.fsf@bloch.sibelius.xs4all.nl \
    --to=mark.kettenis@xs4all.nl \
    --cc=ilias.apalodimas@linaro.org \
    --cc=joe.hershberger@ni.com \
    --cc=marex@denx.de \
    --cc=maximmosk4@gmail.com \
    --cc=mkorpershoek@baylibre.com \
    --cc=raymond.mao@linaro.org \
    --cc=sjg@chromium.org \
    --cc=sughosh.ganu@linaro.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox