From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Date: Wed, 13 Jan 2021 15:21:39 +0200
Subject: [RFC PATCH 3/3] efidebug: add multiple device path instances on
 Boot####
In-Reply-To: <b16e457a-ac87-05da-0b74-dff12179e937@gmx.de>
References: <20210113111149.64567-1-ilias.apalodimas@linaro.org>
 <20210113111149.64567-4-ilias.apalodimas@linaro.org>
 <b16e457a-ac87-05da-0b74-dff12179e937@gmx.de>
Message-ID: <X/7z417LLhNweHan@apalos.home>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.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
> >  );
> >
>