All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dma: pl08x: Use correct specifier for size_t values
@ 2014-08-01 17:09 Mark Brown
  2014-08-04  8:15 ` Vinod Koul
  0 siblings, 1 reply; 2+ messages in thread
From: Mark Brown @ 2014-08-01 17:09 UTC (permalink / raw)
  To: Vinod Koul, Dan Williams, Tomasz Figa
  Cc: linux-samsung-soc, dmaengine, linaro-kernel, Mark Brown

From: Mark Brown <broonie@linaro.org>

When printing size_t values we should use the %zd or %zx format specifier
in order to ensure the value is displayed correctly and avoid warnings from
sparse.

Signed-off-by: Mark Brown <broonie@linaro.org>
---
 drivers/dma/amba-pl08x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
index 8114731..8ff32b1 100644
--- a/drivers/dma/amba-pl08x.c
+++ b/drivers/dma/amba-pl08x.c
@@ -1040,7 +1040,7 @@ static int pl08x_fill_llis_for_desc(struct pl08x_driver_data *pl08x,
 
 		if (early_bytes) {
 			dev_vdbg(&pl08x->adev->dev,
-				"%s byte width LLIs (remain 0x%08x)\n",
+				"%s byte width LLIs (remain 0x%08zx)\n",
 				__func__, bd.remainder);
 			prep_byte_width_lli(pl08x, &bd, &cctl, early_bytes,
 				num_llis++, &total_bytes);
@@ -1662,7 +1662,7 @@ static struct dma_async_tx_descriptor *pl08x_prep_dma_cyclic(
 	dma_addr_t slave_addr;
 
 	dev_dbg(&pl08x->adev->dev,
-		"%s prepare cyclic transaction of %d/%d bytes %s %s\n",
+		"%s prepare cyclic transaction of %zd/%zd bytes %s %s\n",
 		__func__, period_len, buf_len,
 		direction == DMA_MEM_TO_DEV ? "to" : "from",
 		plchan->name);
-- 
2.0.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dma: pl08x: Use correct specifier for size_t values
  2014-08-01 17:09 [PATCH] dma: pl08x: Use correct specifier for size_t values Mark Brown
@ 2014-08-04  8:15 ` Vinod Koul
  0 siblings, 0 replies; 2+ messages in thread
From: Vinod Koul @ 2014-08-04  8:15 UTC (permalink / raw)
  To: Mark Brown
  Cc: Dan Williams, Tomasz Figa, linux-samsung-soc, dmaengine,
	linaro-kernel, Mark Brown

On Fri, Aug 01, 2014 at 06:09:48PM +0100, Mark Brown wrote:
> From: Mark Brown <broonie@linaro.org>
> 
> When printing size_t values we should use the %zd or %zx format specifier
> in order to ensure the value is displayed correctly and avoid warnings from
> sparse.

Applied, thanks

-- 
~Vinod

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-08-04  8:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-01 17:09 [PATCH] dma: pl08x: Use correct specifier for size_t values Mark Brown
2014-08-04  8:15 ` Vinod Koul

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.