U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
To: u-boot@lists.denx.de
Subject: [RFC PATCH 3/3] efidebug: add multiple device path instances on Boot####
Date: Wed, 13 Jan 2021 15:21:39 +0200	[thread overview]
Message-ID: <X/7z417LLhNweHan@apalos.home> (raw)
In-Reply-To: <b16e457a-ac87-05da-0b74-dff12179e937@gmx.de>

On Wed, Jan 13, 2021 at 02:13:44PM +0100, Heinrich Schuchardt wrote:
> On 13.01.21 12:11, Ilias Apalodimas wrote:
> > The UEFI spec allow a packed array of UEFI device paths in the
> > FilePathList[] of an EFI_LOAD_OPTION. The first file path must
> > describe the laoded image but the rest are OS specific.
> > Previous patches parse the device path and try to use the second
> > member of the array as an initrd. So let's modify efidebug slightly
> > and install the second file described in the command line as the
> > initrd device path.
> 
> Please, describe the syntax of the efidebug command in the commit message.

sure 

[...]

> > +	}
> >
> >  	printf("  data:\n");
> >  	print_hex_dump("    ", DUMP_PREFIX_OFFSET, 16, 1,
> > @@ -1583,7 +1656,7 @@ static char efidebug_help_text[] =
> 
> Where is the change to efidebug_help_text[]?

Thanks, I'll add that on the next version

Cheers
/Ilias
> 
> Best regards
> 
> Heinrich
> 
> >  #endif
> >
> >  U_BOOT_CMD(
> > -	efidebug, 10, 0, do_efidebug,
> > +	efidebug, CONFIG_SYS_MAXARGS, 0, do_efidebug,
> >  	"Configure UEFI environment",
> >  	efidebug_help_text
> >  );
> >
> 

  reply	other threads:[~2021-01-13 13:21 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-13 11:11 [RFC 0/3] Change logic of EFI LoadFile2 protocol for initrd loading Ilias Apalodimas
2021-01-13 11:11 ` [RFC PATCH 1/3] efi_loader: Introduce helper functions for EFI Ilias Apalodimas
2021-01-13 12:41   ` Heinrich Schuchardt
2021-01-13 13:30     ` Ilias Apalodimas
2021-01-14  4:48   ` AKASHI Takahiro
2021-01-14  4:57     ` Heinrich Schuchardt
2021-01-14 10:54       ` Ilias Apalodimas
2021-01-14  9:46     ` Ilias Apalodimas
2021-01-13 11:11 ` [RFC PATCH 2/3] efi_loader: efi_loader: Replace config option for initrd loading Ilias Apalodimas
2021-01-13 13:08   ` Heinrich Schuchardt
2021-01-13 13:23     ` Ilias Apalodimas
2021-01-14  4:23   ` AKASHI Takahiro
2021-01-14  9:40     ` Ilias Apalodimas
2021-01-13 11:11 ` [RFC PATCH 3/3] efidebug: add multiple device path instances on Boot#### Ilias Apalodimas
2021-01-13 13:13   ` Heinrich Schuchardt
2021-01-13 13:21     ` Ilias Apalodimas [this message]
2021-01-14  5:11   ` AKASHI Takahiro
2021-01-14  9:55     ` Ilias Apalodimas
2021-01-14 12:07       ` Heinrich Schuchardt
2021-01-14 12:36         ` Ilias Apalodimas
2021-01-15  2:16           ` AKASHI Takahiro

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=X/7z417LLhNweHan@apalos.home \
    --to=ilias.apalodimas@linaro.org \
    --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