Commit 373459ee authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Vinod Koul

edma: do not waste memory for dma_mask

Accordingly to commentary in the platform_device_register_full the memory
allocated for dma_mask will not going to be freed. That's why is better to
assign dma_mask afterwards.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarVinod Koul <vinod.koul@intel.com>
parent 9b562639
...@@ -620,13 +620,11 @@ static struct platform_device *pdev0, *pdev1; ...@@ -620,13 +620,11 @@ static struct platform_device *pdev0, *pdev1;
static const struct platform_device_info edma_dev_info0 = { static const struct platform_device_info edma_dev_info0 = {
.name = "edma-dma-engine", .name = "edma-dma-engine",
.id = 0, .id = 0,
.dma_mask = DMA_BIT_MASK(32),
}; };
static const struct platform_device_info edma_dev_info1 = { static const struct platform_device_info edma_dev_info1 = {
.name = "edma-dma-engine", .name = "edma-dma-engine",
.id = 1, .id = 1,
.dma_mask = DMA_BIT_MASK(32),
}; };
static int edma_init(void) static int edma_init(void)
...@@ -640,6 +638,8 @@ static int edma_init(void) ...@@ -640,6 +638,8 @@ static int edma_init(void)
ret = PTR_ERR(pdev0); ret = PTR_ERR(pdev0);
goto out; goto out;
} }
pdev0->dev.dma_mask = &pdev0->dev.coherent_dma_mask;
pdev0->dev.coherent_dma_mask = DMA_BIT_MASK(32);
} }
if (EDMA_CTLRS == 2) { if (EDMA_CTLRS == 2) {
...@@ -649,6 +649,8 @@ static int edma_init(void) ...@@ -649,6 +649,8 @@ static int edma_init(void)
platform_device_unregister(pdev0); platform_device_unregister(pdev0);
ret = PTR_ERR(pdev1); ret = PTR_ERR(pdev1);
} }
pdev1->dev.dma_mask = &pdev1->dev.coherent_dma_mask;
pdev1->dev.coherent_dma_mask = DMA_BIT_MASK(32);
} }
out: out:
......
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