On 7/4/23 22:18, Heinrich Schuchardt wrote:
Compiling on armv7 results in:

tools/renesas_spkgimage.c: In function ‘spkgimage_parse_config_line’:
tools/renesas_spkgimage.c:76:66: warning: format ‘%ld’ expects
argument of type ‘long int’, but argument 3 has type ‘size_t’
{aka ‘unsigned int’} [-Wformat=]
    76 |         "config error: unknown keyword on line %ld\n",
       |                                                ~~^
       |                                                  |
       |                                                  long int
       |                                                %d
    77 |         line_num);
       |         ~~~~~~~~
       |         |
       |         size_t {aka unsigned int}

The correct printf specifier for size_t is '%zu'.

Fixes: afdfcb11f97c ("tools: spkgimage: add Renesas SPKG format")
Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
---
  tools/renesas_spkgimage.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/renesas_spkgimage.c b/tools/renesas_spkgimage.c
index fa0a468cc4..605d0ab149 100644
--- a/tools/renesas_spkgimage.c
+++ b/tools/renesas_spkgimage.c
@@ -73,7 +73,7 @@ static int spkgimage_parse_config_line(char *line, size_t 
line_num)
                conf.padding = check_range(name, value, 1, INT_MAX);
        } else {
                fprintf(stderr,
-                       "config error: unknown keyword on line %ld\n",
+                       "config error: unknown keyword on line %zu\n",
                        line_num);
                return -EINVAL;
        }

Reviewed-by: Marek Vasut <marek.vasut+rene...@mailbox.org>

Tom, can you pick this directly for 2023.07 ? It is low impact and fixes a warning.

Reply via email to