Commit 14ef64eb authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Mark Brown

spi: stm32-qspi: fix debug format string

Printing size_t needs a special %zx format modifier to avoid a
warning like:

drivers/spi/spi-stm32-qspi.c:481:41: note: format string is defined here
  481 |         dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);

Patrice already tried to fix this, but picked %lx instead of %zx,
which fixed some architectures but broke others in the same way.
Using %zx works everywhere.

Fixes: 18674dee ("spi: stm32-qspi: Add dirmap support")
Fixes: 1b8a7d42 ("spi: stm32-qspi: Fix compilation warning in ARM64")
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20210422134955.1988316-1-arnd@kernel.orgSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent bc2e9578
...@@ -478,7 +478,7 @@ static ssize_t stm32_qspi_dirmap_read(struct spi_mem_dirmap_desc *desc, ...@@ -478,7 +478,7 @@ static ssize_t stm32_qspi_dirmap_read(struct spi_mem_dirmap_desc *desc,
* all needed transfer information into struct spi_mem_op * all needed transfer information into struct spi_mem_op
*/ */
memcpy(&op, &desc->info.op_tmpl, sizeof(struct spi_mem_op)); memcpy(&op, &desc->info.op_tmpl, sizeof(struct spi_mem_op));
dev_dbg(qspi->dev, "%s len = 0x%lx offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf); dev_dbg(qspi->dev, "%s len = 0x%zx offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
op.data.nbytes = len; op.data.nbytes = len;
op.addr.val = desc->info.offset + offs; op.addr.val = desc->info.offset + offs;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment