[PATCH] dmaengine: at_xdmac: fix debug string

Subsystems: dma generic offload engine subsystem, microchip at91 dma drivers, the rest

STALE3729d

2 messages, 2 authors, 2016-06-30 · open the first message on its own page

[PATCH] dmaengine: at_xdmac: fix debug string

From: Alexandre Belloni <hidden>
Date: 2016-06-29 17:45:01

mbr_ds is an integer, don't use %pad to print it.

Fixes: commit 268914f4e7a0 ("dmaengine: at_xdmac: use %pad format string for dma_addr_t")
Reported-by: Dan Carpenter <redacted>
Signed-off-by: Alexandre Belloni <redacted>
---
 drivers/dma/at_xdmac.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
index 8e304b1befc5..e4c5522f9dcd 100644
--- a/drivers/dma/at_xdmac.c
+++ b/drivers/dma/at_xdmac.c
@@ -1195,8 +1195,8 @@ static struct at_xdmac_desc *at_xdmac_memset_create_desc(struct dma_chan *chan,
 	desc->lld.mbr_cfg = chan_cc;
 
 	dev_dbg(chan2dev(chan),
-		"%s: lld: mbr_da=%pad, mbr_ds=%pad, mbr_ubc=0x%08x, mbr_cfg=0x%08x\n",
-		__func__, &desc->lld.mbr_da, &desc->lld.mbr_ds, desc->lld.mbr_ubc,
+		"%s: lld: mbr_da=%pad, mbr_ds=0x%08x, mbr_ubc=0x%08x, mbr_cfg=0x%08x\n",
+		__func__, &desc->lld.mbr_da, desc->lld.mbr_ds, desc->lld.mbr_ubc,
 		desc->lld.mbr_cfg);
 
 	return desc;
-- 
2.8.1

Re: [PATCH] dmaengine: at_xdmac: fix debug string

From: Vinod Koul <hidden>
Date: 2016-06-30 03:15:40

On Wed, Jun 29, 2016 at 07:44:51PM +0200, Alexandre Belloni wrote:
mbr_ds is an integer, don't use %pad to print it.
Applied, thanks

-- 
~Vinod
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help