diff mbox series

[2/2] dmaengine: sh: rz-dmac: Fix dma_set_max_seg_size

Message ID 20220307191211.12087-2-biju.das.jz@bp.renesas.com (mailing list archive)
State Changes Requested
Headers show
Series [1/2] dmaengine: sh: rz-dmac: Set DMA transfer size for src/dst based on the transfer direction | expand

Commit Message

Biju Das March 7, 2022, 7:12 p.m. UTC
As per Hardware manual, maximum transfer count is  2^32 − 1 bytes.
This patch fixes this issue by replacing 'U32_MAX'->'U32_MAX - 1'.

Fixes: 5000d37042a61ca55 ("dmaengine: sh: Add DMAC driver for RZ/G2L SoC")
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/dma/sh/rz-dmac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Biju Das March 8, 2022, 4:49 a.m. UTC | #1
Hi All,

Oops. I would like to drop this patch as 2^32 − 1 = 'U32_MAX'.

Sorry for the noise.

Regards,
Biju

> Subject: [PATCH 2/2] dmaengine: sh: rz-dmac: Fix dma_set_max_seg_size
> 
> As per Hardware manual, maximum transfer count is  2^32 − 1 bytes.
> This patch fixes this issue by replacing 'U32_MAX'->'U32_MAX - 1'.
> 
> Fixes: 5000d37042a61ca55 ("dmaengine: sh: Add DMAC driver for RZ/G2L SoC")
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> ---
>  drivers/dma/sh/rz-dmac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c index
> 52d82f67d3dd..b35bea56e475 100644
> --- a/drivers/dma/sh/rz-dmac.c
> +++ b/drivers/dma/sh/rz-dmac.c
> @@ -913,7 +913,7 @@ static int rz_dmac_probe(struct platform_device *pdev)
>  	engine->device_issue_pending = rz_dmac_issue_pending;
> 
>  	engine->copy_align = DMAENGINE_ALIGN_1_BYTE;
> -	dma_set_max_seg_size(engine->dev, U32_MAX);
> +	dma_set_max_seg_size(engine->dev, U32_MAX - 1);
> 
>  	ret = dma_async_device_register(engine);
>  	if (ret < 0) {
> --
> 2.17.1
diff mbox series

Patch

diff --git a/drivers/dma/sh/rz-dmac.c b/drivers/dma/sh/rz-dmac.c
index 52d82f67d3dd..b35bea56e475 100644
--- a/drivers/dma/sh/rz-dmac.c
+++ b/drivers/dma/sh/rz-dmac.c
@@ -913,7 +913,7 @@  static int rz_dmac_probe(struct platform_device *pdev)
 	engine->device_issue_pending = rz_dmac_issue_pending;
 
 	engine->copy_align = DMAENGINE_ALIGN_1_BYTE;
-	dma_set_max_seg_size(engine->dev, U32_MAX);
+	dma_set_max_seg_size(engine->dev, U32_MAX - 1);
 
 	ret = dma_async_device_register(engine);
 	if (ret < 0) {