On 01/08/2018 04:52 AM, Simon Glass wrote:
On 26 December 2017 at 03:07, Heinrich Schuchardt <xypron.g...@gmx.de> wrote:
For debugging efi_loader we need the capability to print EFI
device paths. With this patch we can write:

     debug("device path: %pD", dp);

A possible output would be

     device path: /MemoryMapped(0x0,0x3ff93a82,0x3ff93a82)

This enhancement is not available when building without EFI support
and neither in the SPL nor in the API example.

Cc: Wolfgang Denk <w...@denx.de>
Cc: Simon Glass <s...@chromium.org>
Suggested-by: Rob Clark <robdcl...@gmail.com>
Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
---
I propose Alex picks up this patch for the EFI tree.

v3:
         Return -ENOMEM if out of memory.
         Avoid missing dependency error when building the SPL of the
         API example.
v2:
         Panic if out of memory.
         Wolfgang suggested not to silently ignore an out of memory
         situation.
---
  examples/api/Makefile |  3 +++
  lib/vsprintf.c        | 47 +++++++++++++++++++++++++++++++++++++++++------
  2 files changed, 44 insertions(+), 6 deletions(-)

Can you please add a test for this? It could go in the print unit
tests, perhaps?

Regards,
Simon

test/print_ut.c is only compiled with CONFIG_SANDBOX.

It seems your patches to enable EFI_LOADER in the sandbox did not make it into v2018.01. Are you still working on these? Do you have a repo with a working copy?

Best regards

Heinrich

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to