[U-Boot] [PATCH v16 02/10] cmd_pxe: remove compiling warnings

FengHua fenghua at phytium.com.cn
Thu Dec 19 15:27:26 CET 2013


> On Sat, Dec 14, 2013 at 9:17 AM,  <fenghua at phytium.com.cn> wrote:
> > From: David Feng <fenghua at phytium.com.cn>
> >
> > Signed-off-by: David Feng <fenghua at phytium.com.cn>
> > ---
> >  common/cmd_pxe.c |    4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/common/cmd_pxe.c b/common/cmd_pxe.c
> > index db6b156..c27ec35 100644
> > --- a/common/cmd_pxe.c
> > +++ b/common/cmd_pxe.c
> > @@ -59,7 +59,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 (%zd < 21)\n", outbuf_len);
> >
> >                 return -EINVAL;
> >         }
> > @@ -103,7 +103,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. (%zd < %zd)\n",
> >                                 bootfile_path_size, path_len);
> 
> CHECK: Alignment should match open parenthesis
> #40: FILE: common/cmd_pxe.c:107:
> +        printf("bootfile_path too small. (%zd < %zd)\n",
>                  bootfile_path_size, path_len);

Yes, this should be fixed. I only replace %d with %z and did not notice the alignment.









More information about the U-Boot mailing list