[U-Boot] [PATCH v3 1/1] vsprintf.c: add EFI device path printing

Simon Glass sjg at chromium.org
Mon Jan 8 03:52:17 UTC 2018


On 26 December 2017 at 03:07, Heinrich Schuchardt <xypron.glpk at 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 <wd at denx.de>
> Cc: Simon Glass <sjg at chromium.org>
> Suggested-by: Rob Clark <robdclark at gmail.com>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at 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


More information about the U-Boot mailing list