Re: [PATCH] dmaengine: add dma_slave_get_caps api

From: Lars-Peter Clausen
Date: Mon Jul 15 2013 - 06:51:47 EST


On 07/15/2013 11:57 AM, Vinod Koul wrote:
> add new device callback .device_slave_caps api which can be used by clients to
> query the dma channel capablties before they program the channel. This can help
> is removing errors during the channel programming. Also add helper
> dma_slave_get_caps API
>
> This patch folds the work done by Matt earlier
> https://patchwork.kernel.org/patch/2094891/
>
> Signed-off-by: Vinod Koul <vinod.koul@xxxxxxxxx>
> --
> this version
> changes api to include struct dma_slave_caps as arg
> removed burst lengths, can be added when we have users

Hi,

it looks like you did send out v1 by accident.

- Lars

> --
>
> include/linux/dmaengine.h | 45 +++++++++++++++++++++++++++++++++++++++++++++
> 1 files changed, 45 insertions(+), 0 deletions(-)
>
> diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
> index cb286b1..f247660 100644
> --- a/include/linux/dmaengine.h
> +++ b/include/linux/dmaengine.h
> @@ -370,6 +370,37 @@ struct dma_slave_config {
> unsigned int slave_id;
> };
>
> +/* struct dma_slave_caps - expose capabilities of a slave channel only
> + *
> + * @src_addr_widths: bit mask of src addr widths the channel supports
> + * @dstn_addr_widths: bit mask of dstn addr widths the channel supports
> + * @src_burst_lengths: bit mask of src slave burst lengths supported
> + * @dstn_burst_lengths: bit mask of dstn slave burst lengths supported
> + * @directions: bit mask of slave direction the channel supported
> + * since the enum dma_transfer_direction is not defined as bits for each
> + * type of direction, the dma controller should fill (1 << <TYPE>) and same
> + * should be checked by controller as well
> + * @cmd_pause: true, if pause and thereby resume is supported
> + * @cmd_terminate: true, if terminate cmd is supported
> + *
> + * @max_seg_nr: maximum number of SG segments supported
> + * 0 for no maximum
> + * @max_seg_len: maximum length of a SG segment supported
> + * 0 for no maximum
> + */
> +struct dma_slave_caps {
> + u32 src_addr_widths;
> + u32 dstn_addr_widths;
> + u32 src_burst_lengths;
> + u32 dstn_burst_lengths;
> + u32 directions;
> + bool cmd_pause;
> + bool cmd_terminate;
> +
> + u32 max_sg_nr;
> + u32 max_sg_len;
> +};
> +
> static inline const char *dma_chan_name(struct dma_chan *chan)
> {
> return dev_name(&chan->dev->device);
> @@ -532,6 +563,7 @@ struct dma_tx_state {
> * struct with auxiliary transfer status information, otherwise the call
> * will just return a simple status code
> * @device_issue_pending: push pending transactions to hardware
> + * @device_slave_caps: return the slave channel capabilities
> */
> struct dma_device {
>
> @@ -597,6 +629,7 @@ struct dma_device {
> dma_cookie_t cookie,
> struct dma_tx_state *txstate);
> void (*device_issue_pending)(struct dma_chan *chan);
> + struct dma_slave_caps *(*device_slave_caps)(struct dma_chan *chan);
> };
>
> static inline int dmaengine_device_control(struct dma_chan *chan,
> @@ -670,6 +703,18 @@ static inline struct dma_async_tx_descriptor *dmaengine_prep_interleaved_dma(
> return chan->device->device_prep_interleaved_dma(chan, xt, flags);
> }
>
> +static inline struct dma_slave_caps *dma_get_slave_caps(struct dma_chan *chan)
> +{
> + /* check if the channel supports slave transactions */
> + if (!test_bit(DMA_SLAVE, chan->device->cap_mask.bits))
> + return NULL;
> +
> + if (chan->device->device_slave_caps)
> + return chan->device->device_slave_caps(chan);
> +
> + return NULL;
> +}
> +
> static inline int dmaengine_terminate_all(struct dma_chan *chan)
> {
> return dmaengine_device_control(chan, DMA_TERMINATE_ALL, 0);
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/