From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heinrich Schuchardt Date: Sun, 14 Mar 2021 10:42:06 +0100 Subject: [PATCH 5/6 v2] efidebug: add multiple device path instances on Boot#### In-Reply-To: <379384cb-810b-e16f-6c74-30dba5a6d32f@gmx.de> References: <20210313214738.3257922-1-ilias.apalodimas@linaro.org> <20210313214738.3257922-6-ilias.apalodimas@linaro.org> <379384cb-810b-e16f-6c74-30dba5a6d32f@gmx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 3/14/21 10:27 AM, Heinrich Schuchardt wrote: > On 3/13/21 10:47 PM, Ilias Apalodimas wrote: >> The UEFI spec allow a packed array of UEFI device paths in the > > %s/allow/allows/ > >> FilePathList[] of an EFI_LOAD_OPTION. The first file path must >> describe the loaded 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. >> >> Signed-off-by: Ilias Apalodimas >> index 70d6be00e8a8..0849572a5143 100644 >> --- a/test/py/tests/test_efi_secboot/test_signed_intca.py >> +++ b/test/py/tests/test_efi_secboot/test_signed_intca.py >> @@ -39,7 +39,7 @@ class TestEfiSignedImageIntca(object): >> ????????????? assert 'Failed to set EFI variable' not in ''.join(output) >> >> ????????????? output = u_boot_console.run_command_list([ >> -??????????????? 'efidebug boot add 1 HELLO_a host 0:1 >> /helloworld.efi.signed_a ""', >> +??????????????? 'efidebug boot add -b 1 HELLO_a host 0:1 >> /helloworld.efi.signed_a ""', >> 'efidebug boot next 1', >> 'efidebug test bootmgr']) Why don't we use as syntax 'efidebug boot add HELLO_a -b 1 host 0:1 helloworld.efi.signed_a'? The boot option number and label are not a property of the binary. Best regards Heinrich