On 12/17/18 10:22 AM, Heinrich Schuchardt wrote:
> core is of type uint64_t. So for printing we need "%"PRIu64 (not "%lu").
> 
> Without the patch a warning is issued when building on a 32bit system.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.g...@gmx.de>
> ---
>  tools/imx8image.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/imx8image.c b/tools/imx8image.c
> index 6e8ac464e7..f3158bc434 100644
> --- a/tools/imx8image.c
> +++ b/tools/imx8image.c
> @@ -555,7 +555,8 @@ static void set_image_array_entry(flash_header_v3_t 
> *container,
>               } else if (soc == QM && core == CORE_CA72) {
>                       meta = IMAGE_A72_DEFAULT_META(custom_partition);
>               } else {
> -                     fprintf(stderr, "Error: invalid AP core id: %lu\n",
> +                     fprintf(stderr,
> +                             "Error: invalid AP core id: %" PRIu64 "\n",
>                               core);
>                       exit(EXIT_FAILURE);
>               }
> @@ -577,7 +578,9 @@ static void set_image_array_entry(flash_header_v3_t 
> *container,
>                       core = CORE_CM4_1;
>                       meta = IMAGE_M4_1_DEFAULT_META(custom_partition);
>               } else {
> -                     fprintf(stderr, "Error: invalid m4 core id: %lu\n", 
> core);
> +                     fprintf(stderr,
> +                             "Error: invalid m4 core id: %" PRIu64 "\n",
> +                             core);
>                       exit(EXIT_FAILURE);
>               }
>               img->hab_flags |= IMG_TYPE_EXEC;
> 

Hello Stefano,

in patchwork this patch has been assigned to you. Could you, please,
review it.

Best regards

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

Reply via email to