On 08/06/2017 01:39 PM, Rob Clark wrote:
> This works (roughly) the same way as linux's, but we currently always
> print lower-case (ie. we just keep %pUB and %pUL for compat with linux),
> mostly just because that is what uuid_bin_to_str() supports.
> 
>   %pUb:   01020304-0506-0708-090a-0b0c0d0e0f10
>   %pUl:   04030201-0605-0807-090a-0b0c0d0e0f10
> 
> It will be used by a later efi_loader paths for efi variables and for
> device-path-to-text protocol, and also quite useful for debug prints
> of protocol GUIDs.
> 
> Signed-off-by: Rob Clark <robdcl...@gmail.com>
> ---
> This replaces "efi_loader: add guidstr helper" in my "enough UEFI for
> standard distro boot" patchset, and could also replace Heinrich's
> "efi_loader: write protocol GUID in OpenProtocol" with a single one-
> line debug() print.
> 
>  include/config_fallbacks.h |  1 +
>  lib/vsprintf.c             | 46 
> ++++++++++++++++++++++++++++++++++++++++++++--
>  2 files changed, 45 insertions(+), 2 deletions(-)
> 
> diff --git a/include/config_fallbacks.h b/include/config_fallbacks.h
> index 961a83d758..56b9de09f2 100644
> --- a/include/config_fallbacks.h
> +++ b/include/config_fallbacks.h
> @@ -57,6 +57,7 @@
>  
>  #if (CONFIG_IS_ENABLED(PARTITION_UUIDS) || \
>       CONFIG_IS_ENABLED(EFI_PARTITION) || \
> +     CONFIG_IS_ENABLED(EFI_LOADER) || \
>       defined(CONFIG_RANDOM_UUID) || \
>       defined(CONFIG_CMD_UUID) || \
>       defined(CONFIG_BOOTP_PXE)) && \
> diff --git a/lib/vsprintf.c b/lib/vsprintf.c
> index 0c40f852ce..156fda5ad6 100644
> --- a/lib/vsprintf.c
> +++ b/lib/vsprintf.c
> @@ -18,6 +18,7 @@
>  
>  #include <common.h>
>  #include <charset.h>
> +#include <uuid.h>
>  
>  #include <div64.h>
>  #define noinline __attribute__((noinline))
> @@ -366,6 +367,40 @@ static char *ip4_addr_string(char *buf, char *end, u8 
> *addr, int field_width,
>  }
>  #endif
>  
> +#ifdef CONFIG_LIB_UUID
> +/*
> + * This works (roughly) the same way as linux's, but we currently always
> + * print lower-case (ie. we just keep %pUB and %pUL for compat with linux),
> + * mostly just because that is what uuid_bin_to_str() supports.
> + *
> + *   %pUb:   01020304-0506-0708-090a-0b0c0d0e0f10
> + *   %pUl:   04030201-0605-0807-090a-0b0c0d0e0f10
> + */
> +static char *uuid_string(char *buf, char *end, u8 *addr, int field_width,
> +                      int precision, int flags, const char *fmt)
> +{
> +     char uuid[UUID_STR_LEN + 1];
> +     int str_format = UUID_STR_FORMAT_STD;
> +
> +     switch (*(++fmt)) {
> +     case 'L':
> +     case 'l':
> +             str_format = UUID_STR_FORMAT_GUID;
> +             break;
> +     case 'B':
> +     case 'b':
> +             /* this is the default */
> +             break;
> +     default:
> +             break;
> +     }
> +
> +     uuid_bin_to_str(addr, uuid, str_format);
> +
> +     return string(buf, end, uuid, field_width, precision, flags);
> +}
> +#endif
> +
>  /*
>   * Show a '%p' thing.  A kernel extension is that the '%p' is followed
>   * by an extra set of alphanumeric characters that are extended format
> @@ -399,8 +434,8 @@ static char *pointer(const char *fmt, char *buf, char 
> *end, void *ptr,
>                             flags);
>  #endif
>  
> -#ifdef CONFIG_CMD_NET
>       switch (*fmt) {
> +#ifdef CONFIG_CMD_NET
>       case 'a':
>               flags |= SPECIAL | ZEROPAD;
>  
> @@ -430,8 +465,15 @@ static char *pointer(const char *fmt, char *buf, char 
> *end, void *ptr,
>                                              precision, flags);
>               flags &= ~SPECIAL;
>               break;
> -     }
>  #endif
> +#ifdef CONFIG_LIB_UUID
> +     case 'U':
> +             return uuid_string(buf, end, ptr, field_width, precision,
> +                                flags, fmt);
> +#endif
> +     default:
> +             break;
> +     }
>       flags |= SMALL;
>       if (field_width == -1) {
>               field_width = 2*sizeof(void *);
> 

I was not able to apply the patch to v2017.09-rc1 nor to git HEAD.

Applying: vsprintf.c: add GUID printing
error: patch failed: lib/vsprintf.c:18
error: lib/vsprintf.c: patch does not apply

This is my lib/vsprintf.c:
 14 #include <stdarg.h>
 15 #include <linux/types.h>
 16 #include <linux/string.h>
 17 #include <linux/ctype.h>
 18
 19 #include <common.h>
 20
 21 #include <div64.h>

There is no line with #include <charset.h>.

Best regards

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

Reply via email to