Hi Niklas,
Thank you for the patch.
On Wednesday 11 Jan 2017 15:39:31 Niklas Söderlund wrote:
> The slave mapping should be removed together with other channel
> resources when the channel is freed. If it's not unmapped it will hang
> around forever after the channel is freed.
>
> Fixes: 9f878603dbdb7db3 ("dmaengine: rcar-dmac: add iommu support for slave
> transfers") Reported-by: Laurent Pinchart
> <laurent.pinchart@xxxxxxxxxxxxxxxx>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> ---
> drivers/dma/sh/rcar-dmac.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/dma/sh/rcar-dmac.c b/drivers/dma/sh/rcar-dmac.c
> index 2e441d0ccd79..4c357d475465 100644
> --- a/drivers/dma/sh/rcar-dmac.c
> +++ b/drivers/dma/sh/rcar-dmac.c
> @@ -986,6 +986,7 @@ static void rcar_dmac_free_chan_resources(struct
> dma_chan *chan) {
> struct rcar_dmac_chan *rchan = to_rcar_dmac_chan(chan);
> struct rcar_dmac *dmac = to_rcar_dmac(chan->device);
> + struct rcar_dmac_chan_map *map = &rchan->map;
> struct rcar_dmac_desc_page *page, *_page;
> struct rcar_dmac_desc *desc;
> LIST_HEAD(list);
> @@ -1019,6 +1020,13 @@ static void rcar_dmac_free_chan_resources(struct
> dma_chan *chan) free_page((unsigned long)page);
> }
>
> + /* Remove slave mapping if present. */
> + if (map->slave.xfer_size) {
> + dma_unmap_resource(chan->device->dev, map->addr,
> + map->slave.xfer_size, map->dir, 0);
> + map->slave.xfer_size = 0;
> + }
> +
> pm_runtime_put(chan->device->dev);
> }
--
Regards,
Laurent Pinchart
