* [PATCH V3] dmaengine: sprd: add dma mask interface in probe
@ 2023-09-19 7:38 Kaiwei Liu
2023-09-20 8:12 ` Baolin Wang
2023-09-28 11:56 ` Vinod Koul
0 siblings, 2 replies; 3+ messages in thread
From: Kaiwei Liu @ 2023-09-19 7:38 UTC (permalink / raw)
To: Vinod Koul, Orson Zhai, Baolin Wang, Chunyan Zhang
Cc: dmaengine, linux-kernel, kaiwei liu, Wenming Wu
In the probe of DMA, the default addressing range is 32 bits,
while the actual DMA hardware addressing range used is 36 bits.
So add dma_set_mask_and_coherent function to match DMA
addressing range.
Signed-off-by: Kaiwei Liu <kaiwei.liu@unisoc.com>
---
Change in V2:
-Change subject line.
Change in V3:
-Modify error message to make it more readable.
---
drivers/dma/sprd-dma.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c
index 20c3cb1ef2f5..c371ce405f1d 100644
--- a/drivers/dma/sprd-dma.c
+++ b/drivers/dma/sprd-dma.c
@@ -1115,6 +1115,15 @@ static int sprd_dma_probe(struct platform_device *pdev)
u32 chn_count;
int ret, i;
+ ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(36));
+ if (ret) {
+ ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
+ if (ret) {
+ dev_err(&pdev->dev, "unable to set coherent mask to 32\n");
+ return ret;
+ }
+ }
+
/* Parse new and deprecated dma-channels properties */
ret = device_property_read_u32(&pdev->dev, "dma-channels", &chn_count);
if (ret)
--
2.17.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V3] dmaengine: sprd: add dma mask interface in probe
2023-09-19 7:38 [PATCH V3] dmaengine: sprd: add dma mask interface in probe Kaiwei Liu
@ 2023-09-20 8:12 ` Baolin Wang
2023-09-28 11:56 ` Vinod Koul
1 sibling, 0 replies; 3+ messages in thread
From: Baolin Wang @ 2023-09-20 8:12 UTC (permalink / raw)
To: Kaiwei Liu, Vinod Koul, Orson Zhai, Chunyan Zhang
Cc: dmaengine, linux-kernel, kaiwei liu, Wenming Wu
On 9/19/2023 3:38 PM, Kaiwei Liu wrote:
> In the probe of DMA, the default addressing range is 32 bits,
> while the actual DMA hardware addressing range used is 36 bits.
> So add dma_set_mask_and_coherent function to match DMA
> addressing range.
>
> Signed-off-by: Kaiwei Liu <kaiwei.liu@unisoc.com>
LGTM.
Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
> ---
> Change in V2:
> -Change subject line.
> Change in V3:
> -Modify error message to make it more readable.
> ---
> drivers/dma/sprd-dma.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c
> index 20c3cb1ef2f5..c371ce405f1d 100644
> --- a/drivers/dma/sprd-dma.c
> +++ b/drivers/dma/sprd-dma.c
> @@ -1115,6 +1115,15 @@ static int sprd_dma_probe(struct platform_device *pdev)
> u32 chn_count;
> int ret, i;
>
> + ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(36));
> + if (ret) {
> + ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
> + if (ret) {
> + dev_err(&pdev->dev, "unable to set coherent mask to 32\n");
> + return ret;
> + }
> + }
> +
> /* Parse new and deprecated dma-channels properties */
> ret = device_property_read_u32(&pdev->dev, "dma-channels", &chn_count);
> if (ret)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V3] dmaengine: sprd: add dma mask interface in probe
2023-09-19 7:38 [PATCH V3] dmaengine: sprd: add dma mask interface in probe Kaiwei Liu
2023-09-20 8:12 ` Baolin Wang
@ 2023-09-28 11:56 ` Vinod Koul
1 sibling, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2023-09-28 11:56 UTC (permalink / raw)
To: Orson Zhai, Baolin Wang, Chunyan Zhang, Kaiwei Liu
Cc: dmaengine, linux-kernel, kaiwei liu, Wenming Wu
On Tue, 19 Sep 2023 15:38:01 +0800, Kaiwei Liu wrote:
> In the probe of DMA, the default addressing range is 32 bits,
> while the actual DMA hardware addressing range used is 36 bits.
> So add dma_set_mask_and_coherent function to match DMA
> addressing range.
>
>
Applied, thanks!
[1/1] dmaengine: sprd: add dma mask interface in probe
commit: 47b077c21590490f5bcb8ee80c66ce7a6c201d11
Best regards,
--
~Vinod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-09-28 11:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-19 7:38 [PATCH V3] dmaengine: sprd: add dma mask interface in probe Kaiwei Liu
2023-09-20 8:12 ` Baolin Wang
2023-09-28 11:56 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®