On Wed, 6 Apr 2022, Miquel Raynal wrote:
> The Renesas RZN1 DMA IP is based on a DW core, with eg. an additional
> dmamux register located in the system control area which can take up to
> 32 requests (16 per DMA controller). Each DMA channel can be wired to
> two different peripherals.
>
> We need two additional information from the 'dmas' property: the channel
> (bit in the dmamux register) that must be accessed and the value of the
> mux for this channel.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> ---
> +static void *rzn1_dmamux_route_allocate(struct of_phandle_args *dma_spec,
> + struct of_dma *ofdma)
> +{
> + struct platform_device *pdev = of_find_device_by_node(ofdma->of_node);
> + struct rzn1_dmamux_data *dmamux = platform_get_drvdata(pdev);
> + struct rzn1_dmamux_map *map;
> + unsigned int dmac_idx, chan, val;
> + u32 mask;
> + int ret;
> +
> + if (dma_spec->args_count != 6)
> + return ERR_PTR(-EINVAL);
> +
> + map = kzalloc(sizeof(*map), GFP_KERNEL);
> + if (!map)
> + return ERR_PTR(-ENOMEM);
> +
> + chan = dma_spec->args[0];
> + map->req_idx = dma_spec->args[4];
> + val = dma_spec->args[5];
> + dma_spec->args_count -= 2;
> +
> + if (chan >= RZN1_DMAMUX_MAX_LINES) {
> + dev_err(&pdev->dev, "Invalid DMA request line: %u\n", chan);
> + ret = -EINVAL;
> + goto free_map;
> + }
> +
> + if (map->req_idx >= RZN1_DMAMUX_LINES ||
> + (map->req_idx % RZN1_DMAMUX_MAX_LINES) != chan) {
> + dev_err(&pdev->dev, "Invalid MUX request line: %u\n", map->req_idx);
> + ret = -EINVAL;
> + goto free_map;
> + }
> +
> + dmac_idx = map->req_idx >= RZN1_DMAMUX_MAX_LINES ? 1 : 0;
> + dma_spec->np = of_parse_phandle(ofdma->of_node, "dma-masters", dmac_idx);
> + if (!dma_spec->np) {
> + dev_err(&pdev->dev, "Can't get DMA master\n");
> + ret = -EINVAL;
> + goto free_map;
> + }
> +
> + dev_dbg(&pdev->dev, "Mapping DMAMUX request %u to DMAC%u request %u\n",
> + map->req_idx, dmac_idx, chan);
> +
> + mask = BIT(map->req_idx);
> + mutex_lock(&dmamux->lock);
> + dmamux->used_chans |= mask;
> + ret = r9a06g032_sysctrl_set_dmamux(mask, val ? mask : 0);
> + if (ret)
> + goto release_chan_and_unlock;
> +
> + mutex_unlock(&dmamux->lock);
> +
> + return map;
> +
> +release_chan_and_unlock:
> + dmamux->used_chans &= ~mask;
Now that I check this again, I'm not sure why dmamux->used_chans |= mask;
couldn't be done after r9a06g032_sysctrl_set_dmamux() call so this
rollback of it wouldn't be necessary.
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
--
i.
