[PATCH v2 3/4] test: fdt: fix run_commandf() warnings

Simon Glass sjg at chromium.org
Mon Mar 20 19:40:01 CET 2023


On Mon, 20 Mar 2023 at 21:23, Evgeny Bachinin <EABachinin at sberdevices.ru> wrote:
>
> Fix warnings both for 32bit and 64bit architecture after adding
> printf-like attribute format for run_commandf():
> warning: format ‘%x’ expects argument of type ‘unsigned int’, but
>   argument 2 has type ‘ulong {aka long unsigned int}’ [-Wformat=]
>   ret = run_commandf("fdt addr -c %08x", addr);
>                      ^
> Signed-off-by: Evgeny Bachinin <EABachinin at sberdevices.ru>
> Cc: Marek Vasut <marek.vasut+renesas at mailbox.org>
> ---
> Changes for v2:
> - fdt_test_move(): due to addition of __printf(), fix compilation
> after rebase atop 79bcd809f49 ("test: cmd: fdt: Test fdt move")
> - not apply Reviewed-by due to changes above
>
>  test/cmd/fdt.c | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list