]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
dmaengine: mediatek: uart-apdma: Get addressing bits from match data
authorAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Thu, 13 Nov 2025 12:22:25 +0000 (13:22 +0100)
committerVinod Koul <vkoul@kernel.org>
Tue, 23 Dec 2025 11:01:24 +0000 (16:31 +0530)
The only SoC that declares mediatek,dma-33bits in its devicetree
currently is MT6795, which obviously also declares a SoC-specific
compatible string: in preparation for adding new SoCs with 34 bits
addressing, replace the parsing of said vendor property with logic
to get the number of addressing bits from platform data associated
to compatible strings.

While at it, also make the bit_mask variable unsigned and move the
`int rc` declaration as last to beautify the code.

Thanks to the correct declaration of the APDMA node is in all of
the MediaTek device trees that are currently upstream, this commit
brings no functional differences.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Link: https://patch.msgid.link/20251113122229.23998-5-angelogioacchino.delregno@collabora.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/dma/mediatek/mtk-uart-apdma.c

index 08e15177427b94246951d38a2a1d76875c1e452e..bbacaa89eb920380cdd0301bc897fbb507140c2b 100644 (file)
@@ -468,7 +468,8 @@ static void mtk_uart_apdma_free(struct mtk_uart_apdmadev *mtkd)
 }
 
 static const struct of_device_id mtk_uart_apdma_match[] = {
-       { .compatible = "mediatek,mt6577-uart-dma", },
+       { .compatible = "mediatek,mt6577-uart-dma", .data = (void *)32 },
+       { .compatible = "mediatek,mt6795-uart-dma", .data = (void *)33 },
        { /* sentinel */ },
 };
 MODULE_DEVICE_TABLE(of, mtk_uart_apdma_match);
@@ -477,9 +478,9 @@ static int mtk_uart_apdma_probe(struct platform_device *pdev)
 {
        struct device_node *np = pdev->dev.of_node;
        struct mtk_uart_apdmadev *mtkd;
-       int bit_mask = 32, rc;
        struct mtk_chan *c;
-       unsigned int i;
+       unsigned int bit_mask, i;
+       int rc;
 
        mtkd = devm_kzalloc(&pdev->dev, sizeof(*mtkd), GFP_KERNEL);
        if (!mtkd)
@@ -492,12 +493,10 @@ static int mtk_uart_apdma_probe(struct platform_device *pdev)
                return rc;
        }
 
-       if (of_property_read_bool(np, "mediatek,dma-33bits"))
+       bit_mask = (unsigned int)(uintptr_t)of_device_get_match_data(&pdev->dev);
+       if (bit_mask > 32)
                mtkd->support_33bits = true;
 
-       if (mtkd->support_33bits)
-               bit_mask = 33;
-
        rc = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(bit_mask));
        if (rc)
                return rc;