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

Wolfgang Denk wd at denx.de
Sun Nov 19 11:41:11 UTC 2017


Dear Heinrich,

In message <20171118110946.26014-1-xypron.glpk at gmx.de> you 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);
...

> +#ifdef CONFIG_EFI_LOADER
> +static char *device_path_string(char *buf, char *end, void *dp, int field_width,
> +				int precision, int flags)
> +{
> +	u16 *str = efi_dp_str((struct efi_device_path *)dp);
> +
> +	buf = string16(buf, end, str, field_width, precision, flags);
> +	efi_free_pool(str);

efi_dp_str() can return NULL.  Should thhis not be handled?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Old programmers never die, they just become managers.


More information about the U-Boot mailing list