Commit b2c843a1 authored by Peter Ujfalusi's avatar Peter Ujfalusi Committed by Vinod Koul

ARM/dmaengine: edma: Remove limitation on the number of eDMA controllers

Since the driver stack no longer depends on lookup with id number in a
global array of pointers, the limitation for the number of eDMAs are no
longer needed. We can handle as many eDMAs in legacy and DT boot as we have
memory for them to allocate the needed structures.
Signed-off-by: default avatarPeter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: default avatarVinod Koul <vinod.koul@intel.com>
parent ca304fa9
...@@ -1227,24 +1227,7 @@ static int edma_probe(struct platform_device *pdev) ...@@ -1227,24 +1227,7 @@ static int edma_probe(struct platform_device *pdev)
.parent = &pdev->dev, .parent = &pdev->dev,
}; };
/* When booting with DT the pdev->id is -1 */
if (dev_id < 0)
dev_id = arch_num_cc;
if (dev_id >= EDMA_MAX_CC) {
dev_err(dev,
"eDMA3 with device id 0 and 1 is supported (id: %d)\n",
dev_id);
return -EINVAL;
}
if (node) { if (node) {
/* Check if this is a second instance registered */
if (arch_num_cc) {
dev_err(dev, "only one EDMA instance is supported via DT\n");
return -ENODEV;
}
info = edma_setup_info_from_dt(dev, node); info = edma_setup_info_from_dt(dev, node);
if (IS_ERR(info)) { if (IS_ERR(info)) {
dev_err(dev, "failed to get DT data\n"); dev_err(dev, "failed to get DT data\n");
...@@ -1278,6 +1261,11 @@ static int edma_probe(struct platform_device *pdev) ...@@ -1278,6 +1261,11 @@ static int edma_probe(struct platform_device *pdev)
cc->dev = dev; cc->dev = dev;
cc->id = dev_id; cc->id = dev_id;
/* When booting with DT the pdev->id is -1 */
if (dev_id < 0) {
cc->id = 0;
dev_id = arch_num_cc;
}
dev_set_drvdata(dev, cc); dev_set_drvdata(dev, cc);
cc->base = devm_ioremap_resource(dev, mem); cc->base = devm_ioremap_resource(dev, mem);
......
...@@ -991,14 +991,12 @@ static void edma_dma_init(struct edma_cc *ecc, struct dma_device *dma, ...@@ -991,14 +991,12 @@ static void edma_dma_init(struct edma_cc *ecc, struct dma_device *dma,
INIT_LIST_HEAD(&dma->channels); INIT_LIST_HEAD(&dma->channels);
} }
static struct of_dma_filter_info edma_filter_info = {
.filter_fn = edma_filter_fn,
};
static int edma_probe(struct platform_device *pdev) static int edma_probe(struct platform_device *pdev)
{ {
struct edma_cc *ecc; struct edma_cc *ecc;
struct device_node *parent_node = pdev->dev.parent->of_node; struct device_node *parent_node = pdev->dev.parent->of_node;
struct platform_device *parent_pdev =
to_platform_device(pdev->dev.parent);
int ret; int ret;
ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32)); ret = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
...@@ -1015,7 +1013,10 @@ static int edma_probe(struct platform_device *pdev) ...@@ -1015,7 +1013,10 @@ static int edma_probe(struct platform_device *pdev)
if (!ecc->cc) if (!ecc->cc)
return -ENODEV; return -ENODEV;
ecc->ctlr = pdev->id; ecc->ctlr = parent_pdev->id;
if (ecc->ctlr < 0)
ecc->ctlr = 0;
ecc->dummy_slot = edma_alloc_slot(ecc->cc, EDMA_SLOT_ANY); ecc->dummy_slot = edma_alloc_slot(ecc->cc, EDMA_SLOT_ANY);
if (ecc->dummy_slot < 0) { if (ecc->dummy_slot < 0) {
dev_err(&pdev->dev, "Can't allocate PaRAM dummy slot\n"); dev_err(&pdev->dev, "Can't allocate PaRAM dummy slot\n");
...@@ -1038,10 +1039,8 @@ static int edma_probe(struct platform_device *pdev) ...@@ -1038,10 +1039,8 @@ static int edma_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, ecc); platform_set_drvdata(pdev, ecc);
if (parent_node) { if (parent_node) {
dma_cap_set(DMA_SLAVE, edma_filter_info.dma_cap); of_dma_controller_register(parent_node, of_dma_xlate_by_chan_id,
dma_cap_set(DMA_CYCLIC, edma_filter_info.dma_cap); &ecc->dma_slave);
of_dma_controller_register(parent_node, of_dma_simple_xlate,
&edma_filter_info);
} }
dev_info(&pdev->dev, "TI EDMA DMA engine driver\n"); dev_info(&pdev->dev, "TI EDMA DMA engine driver\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment