Hi Heinrich, On Tue, 21 Nov 2023 at 04:36, Simon Glass <s...@chromium.org> wrote: > > The test coverage for the EFI bootmeth is incomplete since it does not > actually boot the application. > > This series makes an attempt at this. > > However there are at least two problems: > 1. The test does not set up the loaded image properly, so it isn't > found by the app. > 2. The test needs to use USB instead of mmc, so that usb_stop() is > called
I think you may have missed the point of this series. I am not sure how to fix problem #1 above. Do you know what is going on, or could you help figure it out, please? > > > Simon Glass (7): > efi: Use puts() in cout so that console recording works > efi: Show the vendor in helloworld > efi: test: Create a disk image with and EFI app in it > WIP: efi: Disable ANSI output > WIP: efi: Add a test for the EFI bootmeth > WIP: efi: Allow helloworld to exit boot services > WIP: efi: debugging > > arch/sandbox/dts/test.dts | 10 ++++++- > boot/bootflow.c | 2 ++ > boot/bootmeth_efi.c | 1 + > cmd/bootefi.c | 1 + > include/efi_loader.h | 14 +++++++-- > lib/efi_loader/efi_boottime.c | 5 ++++ > lib/efi_loader/efi_console.c | 19 +++++------- > lib/efi_loader/efi_setup.c | 2 +- > lib/efi_loader/helloworld.c | 39 ++++++++++++++++++++++++- > test/boot/bootflow.c | 55 ++++++++++++++++++++++++++++++++++- > test/py/tests/test_ut.py | 32 ++++++++++++++++++++ > 11 files changed, 163 insertions(+), 17 deletions(-) > > -- > 2.43.0.rc1.413.gea7ed67945-goog > Regards, Simon
