[PATCH 1/1] cmd: upl: correct printf code

[Neha Malcom Francis]

Hi Heinrich

On 18/10/24 08:16, Heinrich Schuchardt wrote:
> Building on 32-bit results in a build failure:
> 
>      cmd/upl.c:75:51: warning: format ‘%lx’ expects argument of type
>      ‘long unsigned int’, but argument 3 has type ‘size_t’
>      {aka ‘unsigned int’} [-Wformat=]
>      75 | printf("UPL handoff written to %lx size %lx\n", addr, abuf_size(&buf));
>         |                                         ~~^           ~~~~~~~~~~~~~~~
>         |                                           |           |
>         |                                           |           size_t {aka unsigned int}
>         |                                           long unsigned int
>         |                                         %x
> 
> Fixes: 264f4b0b34c0 ("upl: Add a command")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>   cmd/upl.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/cmd/upl.c b/cmd/upl.c
> index c9745886507..4996f36c787 100644
> --- a/cmd/upl.c
> +++ b/cmd/upl.c
> @@ -72,7 +72,7 @@ static int do_upl_write(struct cmd_tbl *cmdtp, int flag, int argc,
>   		return CMD_RET_FAILURE;
>   	}
>   	addr = map_to_sysmem(abuf_data(&buf));
> -	printf("UPL handoff written to %lx size %lx\n", addr, abuf_size(&buf));
> +	printf("UPL handoff written to %lx size %zx\n", addr, abuf_size(&buf));
>   	if (env_set_hex("upladdr", addr) ||
>   	    env_set_hex("uplsize", abuf_size(&buf))) {
>   		printf("Cannot set env var\n");

Reviewed-by: Neha Malcom Francis <n-francis at ti.com>

-- 
Thanking You
Neha Malcom Francis