If a driver does "dma_cap_set(DMA_MEMCPY), ...)", it should advertise DMA_MEM_TO_MEM support in the direction mask. Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> --- Is it true that DMA_MEMCPY implies DMA_MEM_TO_MEM? --- drivers/dma/sh/rcar-dmac.c | 3 ++- drivers/dma/sh/rcar-hpbdma.c | 3 ++- drivers/dma/sh/shdmac.c | 4 +++- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/dma/sh/rcar-dmac.c b/drivers/dma/sh/rcar-dmac.c index 7820d07e7beea7dd..11e5003a6cc27b40 100644 --- a/drivers/dma/sh/rcar-dmac.c +++ b/drivers/dma/sh/rcar-dmac.c @@ -1707,7 +1707,8 @@ static int rcar_dmac_probe(struct platform_device *pdev) engine->src_addr_widths = widths; engine->dst_addr_widths = widths; - engine->directions = BIT(DMA_MEM_TO_DEV) | BIT(DMA_DEV_TO_MEM); + engine->directions = BIT(DMA_MEM_TO_MEM) | BIT(DMA_MEM_TO_DEV) | + BIT(DMA_DEV_TO_MEM); engine->residue_granularity = DMA_RESIDUE_GRANULARITY_BURST; engine->device_alloc_chan_resources = rcar_dmac_alloc_chan_resources; diff --git a/drivers/dma/sh/rcar-hpbdma.c b/drivers/dma/sh/rcar-hpbdma.c index 2e21c1197769d0c8..29d67277415084b4 100644 --- a/drivers/dma/sh/rcar-hpbdma.c +++ b/drivers/dma/sh/rcar-hpbdma.c @@ -599,7 +599,8 @@ static int hpb_dmae_probe(struct platform_device *pdev) dma_cap_set(DMA_SLAVE, dma_dev->cap_mask); dma_dev->src_addr_widths = widths; dma_dev->dst_addr_widths = widths; - dma_dev->directions = BIT(DMA_MEM_TO_DEV) | BIT(DMA_DEV_TO_MEM); + dma_dev->directions = BIT(DMA_MEM_TO_MEM) | BIT(DMA_MEM_TO_DEV) | + BIT(DMA_DEV_TO_MEM); dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_BURST; hpbdev->shdma_dev.ops = &hpb_dmae_ops; diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma/sh/shdmac.c index c9489ae6c0fa7f47..8693fff83d7b3af4 100644 --- a/drivers/dma/sh/shdmac.c +++ b/drivers/dma/sh/shdmac.c @@ -750,8 +750,10 @@ static int sh_dmae_probe(struct platform_device *pdev) dma_dev->directions = BIT(DMA_MEM_TO_DEV) | BIT(DMA_DEV_TO_MEM); dma_dev->residue_granularity = DMA_RESIDUE_GRANULARITY_BURST; - if (!pdata->slave_only) + if (!pdata->slave_only) { dma_cap_set(DMA_MEMCPY, dma_dev->cap_mask); + dma_dev->directions |= BIT(DMA_MEM_TO_MEM); + } if (pdata->slave && pdata->slave_num) dma_cap_set(DMA_SLAVE, dma_dev->cap_mask); -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html