From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Tue, 22 Oct 2019 14:42:05 +0200
Subject: [U-Boot] [PATCH v2 02/10] tiny-printf: Add print_grouped_ull()
In-Reply-To: <20191021232652.132164-3-sjg@chromium.org>
References: <20191021232652.132164-1-sjg@chromium.org>
 <20191021232652.132164-3-sjg@chromium.org>
Message-ID: <3263796a-48c6-7882-363b-e190b006a92a@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 22.10.19 01:26, Simon Glass wrote:
> This function is used in the bootstage report which may be trigged in TPL
> or TPL. Add a very basic implication of this function so that it builds.
> There is no attempt to get the formatting right, since this would add too
> much code size.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
> 
> Changes in v2: None
> 
>   lib/tiny-printf.c | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/lib/tiny-printf.c b/lib/tiny-printf.c
> index 632b4249142..430c3255bc5 100644
> --- a/lib/tiny-printf.c
> +++ b/lib/tiny-printf.c
> @@ -389,3 +389,9 @@ int snprintf(char *buf, size_t size, const char *fmt, ...)
>   
>   	return ret;
>   }
> +
> +void print_grouped_ull(unsigned long long int_val, int digits)
> +{
> +	/* Don't try to ptint the upper 32-bits */

s/ptint/print ?

> +	printf("%ld ", (ulong)int_val);
> +}
> 

Other than that:

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan