[U-Boot] [PATCH v6 5/5] remove compiling warnings

Wolfgang Denk wd at denx.de
Mon Sep 9 16:22:01 CEST 2013


Dear fenghua at phytium.com.cn,

In message <1378734616-37558-6-git-send-email-fenghua at phytium.com.cn> you wrote:
> From: David Feng <fenghua at phytium.com.cn>
> 
> Signed-off-by: David Feng <fenghua at phytium.com.cn>
> ---
>  common/cmd_pxe.c |    4 ++--
>  lib/hashtable.c  |    1 +
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/common/cmd_pxe.c b/common/cmd_pxe.c
> index a2fb50a..52375a4 100644
> --- a/common/cmd_pxe.c
> +++ b/common/cmd_pxe.c
> @@ -57,7 +57,7 @@ static int format_mac_pxe(char *outbuf, size_t outbuf_len)
>  	uchar ethaddr[6];
>  
>  	if (outbuf_len < 21) {
> -		printf("outbuf is too small (%d < 21)\n", outbuf_len);
> +		printf("outbuf is too small (%ld < 21)\n", outbuf_len);

This does not look right to me.  outbuf_len is of type "size_t", which
may or may not be the same as "long".

You should use "%z" here.

>  		return -EINVAL;
>  	}
> @@ -100,7 +100,7 @@ static int get_bootfile_path(const char *file_path, char *bootfile_path,
>  	path_len = (last_slash - bootfile) + 1;
>  
>  	if (bootfile_path_size < path_len) {
> -		printf("bootfile_path too small. (%d < %d)\n",
> +		printf("bootfile_path too small. (%ld < %ld)\n",

Ditto.

> --- a/lib/hashtable.c
> +++ b/lib/hashtable.c
> @@ -13,6 +13,7 @@
>   * SPDX-License-Identifier:	LGPL-2.1+
>   */
>  
> +#include <config.h>
>  #include <errno.h>
>  #include <malloc.h>

Why would that be needed?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
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
 The software required `Windows 95 or better', so I installed Linux.


More information about the U-Boot mailing list