[U-Boot] [PATCH 2/2] cmd: avb: Fix compiler warnings

[Igor Opaniuk]

Hi,

On Wed, Jul 31, 2019 at 7:59 PM Sam Protsenko
<semen.protsenko at linaro.org> wrote:
>
> When building U-Boot with AVB enabled, compiler shows next warnings:
>
>     cmd/avb.c: In function 'do_avb_read_pvalue':
>     cmd/avb.c:371:18: warning: format '%ld' expects argument of type
>                       'long int', but argument 2 has type 'size_t'
>                       {aka 'unsigned int'} [-Wformat=]
>        printf("Read %ld bytes, value = %s\n", bytes_read,
>                     ~~^                       ~~~~~~~~~~
>                     %d
>
>     cmd/avb.c: In function 'do_avb_write_pvalue':
>     cmd/avb.c:404:19: warning: format '%ld' expects argument of type
>                       'long int', but argument 2 has type '__kernel_size_t'
>                       {aka 'unsigned int'} [-Wformat=]
>        printf("Wrote %ld bytes\n", strlen(value) + 1);
>                      ~~^           ~~~~~~~~~~~~~~~~~
>                      %d
>
> Fix those by using "%zu" specified.
>
> Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
> ---
>  cmd/avb.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/avb.c b/cmd/avb.c
> index c5af4a2e46..3f6fd763a0 100644
> --- a/cmd/avb.c
> +++ b/cmd/avb.c
> @@ -368,7 +368,7 @@ int do_avb_read_pvalue(cmd_tbl_t *cmdtp, int flag, int argc,
>
>         if (avb_ops->read_persistent_value(avb_ops, name, bytes, buffer,
>                                            &bytes_read) == AVB_IO_RESULT_OK) {
> -               printf("Read %ld bytes, value = %s\n", bytes_read,
> +               printf("Read %zu bytes, value = %s\n", bytes_read,
>                        (char *)buffer);
>                 free(buffer);
>                 return CMD_RET_SUCCESS;
> @@ -401,7 +401,7 @@ int do_avb_write_pvalue(cmd_tbl_t *cmdtp, int flag, int argc,
>         if (avb_ops->write_persistent_value(avb_ops, name, strlen(value) + 1,
>                                             (const uint8_t *)value) ==
>             AVB_IO_RESULT_OK) {
> -               printf("Wrote %ld bytes\n", strlen(value) + 1);
> +               printf("Wrote %zu bytes\n", strlen(value) + 1);
>                 return CMD_RET_SUCCESS;
>         }
>
> --
> 2.20.1
>

Reviewed-by: Igor Opaniuk <igor.opaniuk at gmail.com>

-- 
Best regards - Freundliche Grüsse - Meilleures salutations

Igor Opaniuk

mailto: igor.opaniuk at gmail.com
skype: igor.opanyuk
+380 (93) 836 40 67
http://ua.linkedin.com/in/iopaniuk