linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning.
@ 2017-08-07  7:45 Arvind Yadav
  2017-08-07  7:45 ` [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable Arvind Yadav
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Arvind Yadav @ 2017-08-07  7:45 UTC (permalink / raw)
  To: ludovic.desroches, dan.j.williams, vinod.koul
  Cc: linux-kernel, linux-arm-kernel, dmaengine

Replace '%d' by '%zu' to fix the compilation warning:-
   "format ‘%d’ expects argument of type ‘int’,but argument has type ‘size_t’ [-Wformat=]"

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/dma/at_xdmac.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
index 7d4e0bc..6a3cf97 100644
--- a/drivers/dma/at_xdmac.c
+++ b/drivers/dma/at_xdmac.c
@@ -875,7 +875,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
 	dwidth = at_xdmac_align_width(chan, src | dst | chunk->size);
 	if (chunk->size >= (AT_XDMAC_MBR_UBC_UBLEN_MAX << dwidth)) {
 		dev_dbg(chan2dev(chan),
-			"%s: chunk too big (%d, max size %lu)...\n",
+			"%s: chunk too big (%zu, max size %lu)...\n",
 			__func__, chunk->size,
 			AT_XDMAC_MBR_UBC_UBLEN_MAX << dwidth);
 		return NULL;
@@ -956,7 +956,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
 	if ((xt->numf > 1) && (xt->frame_size > 1))
 		return NULL;
 
-	dev_dbg(chan2dev(chan), "%s: src=%pad, dest=%pad, numf=%d, frame_size=%d, flags=0x%lx\n",
+	dev_dbg(chan2dev(chan), "%s: src=%pad, dest=%pad, numf=%zu, frame_size=%zu, flags=0x%lx\n",
 		__func__, &xt->src_start, &xt->dst_start,	xt->numf,
 		xt->frame_size, flags);
 
@@ -990,7 +990,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
 			dst_skip = chunk->size + dst_icg;
 
 			dev_dbg(chan2dev(chan),
-				"%s: chunk size=%d, src icg=%d, dst icg=%d\n",
+				"%s: chunk size=%zu, src icg=%zu, dst icg=%zu\n",
 				__func__, chunk->size, src_icg, dst_icg);
 
 			desc = at_xdmac_interleaved_queue_desc(chan, atchan,
@@ -1207,7 +1207,7 @@ static struct at_xdmac_desc *at_xdmac_memset_create_desc(struct dma_chan *chan,
 	struct at_xdmac_chan	*atchan = to_at_xdmac_chan(chan);
 	struct at_xdmac_desc	*desc;
 
-	dev_dbg(chan2dev(chan), "%s: dest=%pad, len=%d, pattern=0x%x, flags=0x%lx\n",
+	dev_dbg(chan2dev(chan), "%s: dest=%pad, len=%zu, pattern=0x%x, flags=0x%lx\n",
 		__func__, &dest, len, value, flags);
 
 	if (unlikely(!len))
-- 
1.9.1

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

* [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable.
  2017-08-07  7:45 [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Arvind Yadav
@ 2017-08-07  7:45 ` Arvind Yadav
  2017-08-16  6:19   ` Ludovic Desroches
  2017-08-16  6:17 ` [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Ludovic Desroches
  2017-08-21 16:33 ` Vinod Koul
  2 siblings, 1 reply; 5+ messages in thread
From: Arvind Yadav @ 2017-08-07  7:45 UTC (permalink / raw)
  To: ludovic.desroches, dan.j.williams, vinod.koul
  Cc: linux-kernel, linux-arm-kernel, dmaengine

clk_prepare_enable() can fail here and we must check its return value.

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
---
 drivers/dma/at_xdmac.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
index 6a3cf97..c00e392 100644
--- a/drivers/dma/at_xdmac.c
+++ b/drivers/dma/at_xdmac.c
@@ -1883,8 +1883,11 @@ static int atmel_xdmac_resume(struct device *dev)
 	struct at_xdmac_chan	*atchan;
 	struct dma_chan		*chan, *_chan;
 	int			i;
+	int ret;
 
-	clk_prepare_enable(atxdmac->clk);
+	ret = clk_prepare_enable(atxdmac->clk);
+	if (ret)
+		return ret;
 
 	/* Clear pending interrupts. */
 	for (i = 0; i < atxdmac->dma.chancnt; i++) {
-- 
1.9.1

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

* Re: [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning.
  2017-08-07  7:45 [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Arvind Yadav
  2017-08-07  7:45 ` [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable Arvind Yadav
@ 2017-08-16  6:17 ` Ludovic Desroches
  2017-08-21 16:33 ` Vinod Koul
  2 siblings, 0 replies; 5+ messages in thread
From: Ludovic Desroches @ 2017-08-16  6:17 UTC (permalink / raw)
  To: Arvind Yadav
  Cc: ludovic.desroches, dan.j.williams, vinod.koul, dmaengine,
	linux-kernel, linux-arm-kernel

On Mon, Aug 07, 2017 at 01:15:18PM +0530, Arvind Yadav wrote:
> Replace '%d' by '%zu' to fix the compilation warning:-
>    "format ‘%d’ expects argument of type ‘int’,but argument has type ‘size_t’ [-Wformat=]"
> 
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>

Thanks
> ---
>  drivers/dma/at_xdmac.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
> index 7d4e0bc..6a3cf97 100644
> --- a/drivers/dma/at_xdmac.c
> +++ b/drivers/dma/at_xdmac.c
> @@ -875,7 +875,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
>  	dwidth = at_xdmac_align_width(chan, src | dst | chunk->size);
>  	if (chunk->size >= (AT_XDMAC_MBR_UBC_UBLEN_MAX << dwidth)) {
>  		dev_dbg(chan2dev(chan),
> -			"%s: chunk too big (%d, max size %lu)...\n",
> +			"%s: chunk too big (%zu, max size %lu)...\n",
>  			__func__, chunk->size,
>  			AT_XDMAC_MBR_UBC_UBLEN_MAX << dwidth);
>  		return NULL;
> @@ -956,7 +956,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
>  	if ((xt->numf > 1) && (xt->frame_size > 1))
>  		return NULL;
>  
> -	dev_dbg(chan2dev(chan), "%s: src=%pad, dest=%pad, numf=%d, frame_size=%d, flags=0x%lx\n",
> +	dev_dbg(chan2dev(chan), "%s: src=%pad, dest=%pad, numf=%zu, frame_size=%zu, flags=0x%lx\n",
>  		__func__, &xt->src_start, &xt->dst_start,	xt->numf,
>  		xt->frame_size, flags);
>  
> @@ -990,7 +990,7 @@ static inline u32 at_xdmac_align_width(struct dma_chan *chan, dma_addr_t addr)
>  			dst_skip = chunk->size + dst_icg;
>  
>  			dev_dbg(chan2dev(chan),
> -				"%s: chunk size=%d, src icg=%d, dst icg=%d\n",
> +				"%s: chunk size=%zu, src icg=%zu, dst icg=%zu\n",
>  				__func__, chunk->size, src_icg, dst_icg);
>  
>  			desc = at_xdmac_interleaved_queue_desc(chan, atchan,
> @@ -1207,7 +1207,7 @@ static struct at_xdmac_desc *at_xdmac_memset_create_desc(struct dma_chan *chan,
>  	struct at_xdmac_chan	*atchan = to_at_xdmac_chan(chan);
>  	struct at_xdmac_desc	*desc;
>  
> -	dev_dbg(chan2dev(chan), "%s: dest=%pad, len=%d, pattern=0x%x, flags=0x%lx\n",
> +	dev_dbg(chan2dev(chan), "%s: dest=%pad, len=%zu, pattern=0x%x, flags=0x%lx\n",
>  		__func__, &dest, len, value, flags);
>  
>  	if (unlikely(!len))
> -- 
> 1.9.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable.
  2017-08-07  7:45 ` [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable Arvind Yadav
@ 2017-08-16  6:19   ` Ludovic Desroches
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Desroches @ 2017-08-16  6:19 UTC (permalink / raw)
  To: Arvind Yadav
  Cc: ludovic.desroches, dan.j.williams, vinod.koul, linux-kernel,
	linux-arm-kernel, dmaengine

On Mon, Aug 07, 2017 at 01:15:19PM +0530, Arvind Yadav wrote:
> clk_prepare_enable() can fail here and we must check its return value.
> 
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com>

Thanks
> ---
>  drivers/dma/at_xdmac.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/dma/at_xdmac.c b/drivers/dma/at_xdmac.c
> index 6a3cf97..c00e392 100644
> --- a/drivers/dma/at_xdmac.c
> +++ b/drivers/dma/at_xdmac.c
> @@ -1883,8 +1883,11 @@ static int atmel_xdmac_resume(struct device *dev)
>  	struct at_xdmac_chan	*atchan;
>  	struct dma_chan		*chan, *_chan;
>  	int			i;
> +	int ret;
>  
> -	clk_prepare_enable(atxdmac->clk);
> +	ret = clk_prepare_enable(atxdmac->clk);
> +	if (ret)
> +		return ret;
>  
>  	/* Clear pending interrupts. */
>  	for (i = 0; i < atxdmac->dma.chancnt; i++) {
> -- 
> 1.9.1
> 

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

* Re: [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning.
  2017-08-07  7:45 [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Arvind Yadav
  2017-08-07  7:45 ` [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable Arvind Yadav
  2017-08-16  6:17 ` [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Ludovic Desroches
@ 2017-08-21 16:33 ` Vinod Koul
  2 siblings, 0 replies; 5+ messages in thread
From: Vinod Koul @ 2017-08-21 16:33 UTC (permalink / raw)
  To: Arvind Yadav
  Cc: ludovic.desroches, dan.j.williams, linux-kernel,
	linux-arm-kernel, dmaengine

On Mon, Aug 07, 2017 at 01:15:18PM +0530, Arvind Yadav wrote:
> Replace '%d' by '%zu' to fix the compilation warning:-
>    "format ‘%d’ expects argument of type ‘int’,but argument has type ‘size_t’ [-Wformat=]"

Applied both, thanks

-- 
~Vinod

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

end of thread, other threads:[~2017-08-21 16:31 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-07  7:45 [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Arvind Yadav
2017-08-07  7:45 ` [PATCH 2/2] dmaengine: at_xdmac: Handle return value of clk_prepare_enable Arvind Yadav
2017-08-16  6:19   ` Ludovic Desroches
2017-08-16  6:17 ` [PATCH 1/2] dmaengine: at_xdmac: Fix compilation warning Ludovic Desroches
2017-08-21 16:33 ` Vinod Koul

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).