fsldma: Set correct dma_mask based on hw capability
authorBen Collins <bcollins@kernel.org>
Mon, 5 May 2025 17:53:07 +0000 (13:53 -0400)
committerVinod Koul <vkoul@kernel.org>
Wed, 14 May 2025 14:25:15 +0000 (15:25 +0100)
The driver currently hardcodes DMA_BIT_MASK to 36-bits, which is only
correct on eloplus:

elo3 supports 40-bits
eloplus supports 36-bits
elo supports 32-bits

This is based on 0x08 cdar register documention in the respective
reference manuals. Set the dma mask accordingly.

Feedback from Arnd Bergmann:

- Use match data to set address bit mask

Signed-off-by: Ben Collins <bcollins@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: linuxppc-dev@lists.ozlabs.org
Cc: dmaengine@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Link: https://lore.kernel.org/r/2025050513-complex-crane-2babb6@boujee-and-buff
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/fsldma.c
drivers/dma/fsldma.h

index b5e7d18b97669467b5367ad9c21c4e78f3e189ac..9b126a26026751ea23602a7ba37c4e67d6e136ce 100644 (file)
@@ -1226,6 +1226,8 @@ static int fsldma_of_probe(struct platform_device *op)
 
        fdev->dev = &op->dev;
        INIT_LIST_HEAD(&fdev->common.channels);
+       /* The DMA address bits supported for this device. */
+       fdev->addr_bits = (long)device_get_match_data(fdev->dev);
 
        /* ioremap the registers for use */
        fdev->regs = of_iomap(op->dev.of_node, 0);
@@ -1254,7 +1256,7 @@ static int fsldma_of_probe(struct platform_device *op)
        fdev->common.directions = BIT(DMA_DEV_TO_MEM) | BIT(DMA_MEM_TO_DEV);
        fdev->common.residue_granularity = DMA_RESIDUE_GRANULARITY_DESCRIPTOR;
 
-       dma_set_mask(&(op->dev), DMA_BIT_MASK(36));
+       dma_set_mask(&(op->dev), DMA_BIT_MASK(fdev->addr_bits));
 
        platform_set_drvdata(op, fdev);
 
@@ -1387,10 +1389,20 @@ static const struct dev_pm_ops fsldma_pm_ops = {
 };
 #endif
 
+/* The .data field is used for dma-bit-mask. */
 static const struct of_device_id fsldma_of_ids[] = {
-       { .compatible = "fsl,elo3-dma", },
-       { .compatible = "fsl,eloplus-dma", },
-       { .compatible = "fsl,elo-dma", },
+       {
+               .compatible = "fsl,elo3-dma",
+               .data = (void *)40,
+       },
+       {
+               .compatible = "fsl,eloplus-dma",
+               .data = (void *)36,
+       },
+       {
+               .compatible = "fsl,elo-dma",
+               .data = (void *)32,
+       },
        {}
 };
 MODULE_DEVICE_TABLE(of, fsldma_of_ids);
index 308bed0a560acdf6016f3438f3c9e4e26c5ddebb..d7b7a3138b85cad4ece6e75f5ccc36970f7fb92c 100644 (file)
@@ -124,6 +124,7 @@ struct fsldma_device {
        struct fsldma_chan *chan[FSL_DMA_MAX_CHANS_PER_DEVICE];
        u32 feature;            /* The same as DMA channels */
        int irq;                /* Channel IRQ */
+       int addr_bits;          /* DMA addressing bits supported */
 };
 
 /* Define macros for fsldma_chan->feature property */