Commit 754b995f authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Greg Kroah-Hartman

dmaengine: idma64: Use actual device for DMA transfers

[ Upstream commit 5ba846b1 ]

Intel IOMMU, when enabled, tries to find the domain of the device,
assuming it's a PCI one, during DMA operations, such as mapping or
unmapping. Since we are splitting the actual PCI device to couple of
children via MFD framework (see drivers/mfd/intel-lpss.c for details),
the DMA device appears to be a platform one, and thus not an actual one
that performs DMA. In a such situation IOMMU can't find or allocate
a proper domain for its operations. As a result, all DMA operations are
failed.

In order to fix this, supply parent of the platform device
to the DMA engine framework and fix filter functions accordingly.

We may rely on the fact that parent is a real PCI device, because no
other configuration is present in the wild.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: default avatarMark Brown <broonie@kernel.org>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> [for tty parts]
Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 6e578116
...@@ -592,7 +592,7 @@ static int idma64_probe(struct idma64_chip *chip) ...@@ -592,7 +592,7 @@ static int idma64_probe(struct idma64_chip *chip)
idma64->dma.directions = BIT(DMA_DEV_TO_MEM) | BIT(DMA_MEM_TO_DEV); idma64->dma.directions = BIT(DMA_DEV_TO_MEM) | BIT(DMA_MEM_TO_DEV);
idma64->dma.residue_granularity = DMA_RESIDUE_GRANULARITY_BURST; idma64->dma.residue_granularity = DMA_RESIDUE_GRANULARITY_BURST;
idma64->dma.dev = chip->dev; idma64->dma.dev = chip->sysdev;
dma_set_max_seg_size(idma64->dma.dev, IDMA64C_CTLH_BLOCK_TS_MASK); dma_set_max_seg_size(idma64->dma.dev, IDMA64C_CTLH_BLOCK_TS_MASK);
...@@ -632,6 +632,7 @@ static int idma64_platform_probe(struct platform_device *pdev) ...@@ -632,6 +632,7 @@ static int idma64_platform_probe(struct platform_device *pdev)
{ {
struct idma64_chip *chip; struct idma64_chip *chip;
struct device *dev = &pdev->dev; struct device *dev = &pdev->dev;
struct device *sysdev = dev->parent;
struct resource *mem; struct resource *mem;
int ret; int ret;
...@@ -648,11 +649,12 @@ static int idma64_platform_probe(struct platform_device *pdev) ...@@ -648,11 +649,12 @@ static int idma64_platform_probe(struct platform_device *pdev)
if (IS_ERR(chip->regs)) if (IS_ERR(chip->regs))
return PTR_ERR(chip->regs); return PTR_ERR(chip->regs);
ret = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64)); ret = dma_coerce_mask_and_coherent(sysdev, DMA_BIT_MASK(64));
if (ret) if (ret)
return ret; return ret;
chip->dev = dev; chip->dev = dev;
chip->sysdev = sysdev;
ret = idma64_probe(chip); ret = idma64_probe(chip);
if (ret) if (ret)
......
...@@ -216,12 +216,14 @@ static inline void idma64_writel(struct idma64 *idma64, int offset, u32 value) ...@@ -216,12 +216,14 @@ static inline void idma64_writel(struct idma64 *idma64, int offset, u32 value)
/** /**
* struct idma64_chip - representation of iDMA 64-bit controller hardware * struct idma64_chip - representation of iDMA 64-bit controller hardware
* @dev: struct device of the DMA controller * @dev: struct device of the DMA controller
* @sysdev: struct device of the physical device that does DMA
* @irq: irq line * @irq: irq line
* @regs: memory mapped I/O space * @regs: memory mapped I/O space
* @idma64: struct idma64 that is filed by idma64_probe() * @idma64: struct idma64 that is filed by idma64_probe()
*/ */
struct idma64_chip { struct idma64_chip {
struct device *dev; struct device *dev;
struct device *sysdev;
int irq; int irq;
void __iomem *regs; void __iomem *regs;
struct idma64 *idma64; struct idma64 *idma64;
......
...@@ -1491,12 +1491,7 @@ static int pxa2xx_spi_get_port_id(struct acpi_device *adev) ...@@ -1491,12 +1491,7 @@ static int pxa2xx_spi_get_port_id(struct acpi_device *adev)
static bool pxa2xx_spi_idma_filter(struct dma_chan *chan, void *param) static bool pxa2xx_spi_idma_filter(struct dma_chan *chan, void *param)
{ {
struct device *dev = param; return param == chan->device->dev;
if (dev != chan->device->dev->parent)
return false;
return true;
} }
#endif /* CONFIG_PCI */ #endif /* CONFIG_PCI */
......
...@@ -365,7 +365,7 @@ static bool dw8250_fallback_dma_filter(struct dma_chan *chan, void *param) ...@@ -365,7 +365,7 @@ static bool dw8250_fallback_dma_filter(struct dma_chan *chan, void *param)
static bool dw8250_idma_filter(struct dma_chan *chan, void *param) static bool dw8250_idma_filter(struct dma_chan *chan, void *param)
{ {
return param == chan->device->dev->parent; return param == chan->device->dev;
} }
/* /*
...@@ -434,7 +434,7 @@ static void dw8250_quirks(struct uart_port *p, struct dw8250_data *data) ...@@ -434,7 +434,7 @@ static void dw8250_quirks(struct uart_port *p, struct dw8250_data *data)
data->uart_16550_compatible = true; data->uart_16550_compatible = true;
} }
/* Platforms with iDMA */ /* Platforms with iDMA 64-bit */
if (platform_get_resource_byname(to_platform_device(p->dev), if (platform_get_resource_byname(to_platform_device(p->dev),
IORESOURCE_MEM, "lpss_priv")) { IORESOURCE_MEM, "lpss_priv")) {
data->dma.rx_param = p->dev->parent; data->dma.rx_param = p->dev->parent;
......
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