* [RESEND v2 PATCH] v4l: omap_vout: vrfb: Convert to dmaengine
@ 2017-05-03 11:08 ` Peter Ujfalusi
0 siblings, 0 replies; 4+ messages in thread
From: Peter Ujfalusi @ 2017-05-03 11:08 UTC (permalink / raw)
To: laurent.pinchart
Cc: hans.verkuil, linux-media, linux-kernel, arnd, linux, tony, linux-omap
The dmaengine driver for sDMA now have support for interleaved transfer.
This trasnfer type was open coded with the legacy omap-dma API, but now
we can move it to dmaengine.
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
Hi,
changes since RESEND (27.10.2016):
- rebased on next-20170503
I can not test it on real HW (still), but I have validated [1] that the change
is correct and should not cause any regression.
Laurent: can you verify the patch on a real hardware?
Regards,
Peter
drivers/media/platform/omap/omap_vout_vrfb.c | 133 ++++++++++++++++-----------
drivers/media/platform/omap/omap_voutdef.h | 6 +-
2 files changed, 83 insertions(+), 56 deletions(-)
diff --git a/drivers/media/platform/omap/omap_vout_vrfb.c b/drivers/media/platform/omap/omap_vout_vrfb.c
index 92c4e1826356..45a553d4f5b2 100644
--- a/drivers/media/platform/omap/omap_vout_vrfb.c
+++ b/drivers/media/platform/omap/omap_vout_vrfb.c
@@ -16,7 +16,6 @@
#include <media/videobuf-dma-contig.h>
#include <media/v4l2-device.h>
-#include <linux/omap-dma.h>
#include <video/omapvrfb.h>
#include "omap_voutdef.h"
@@ -63,7 +62,7 @@ static int omap_vout_allocate_vrfb_buffers(struct omap_vout_device *vout,
/*
* Wakes up the application once the DMA transfer to VRFB space is completed.
*/
-static void omap_vout_vrfb_dma_tx_callback(int lch, u16 ch_status, void *data)
+static void omap_vout_vrfb_dma_tx_callback(void *data)
{
struct vid_vrfb_dma *t = (struct vid_vrfb_dma *) data;
@@ -94,6 +93,7 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
int ret = 0, i, j;
struct omap_vout_device *vout;
struct video_device *vfd;
+ dma_cap_mask_t mask;
int image_width, image_height;
int vrfb_num_bufs = VRFB_NUM_BUFS;
struct v4l2_device *v4l2_dev = platform_get_drvdata(pdev);
@@ -131,18 +131,27 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
/*
* Request and Initialize DMA, for DMA based VRFB transfer
*/
- vout->vrfb_dma_tx.dev_id = OMAP_DMA_NO_DEVICE;
- vout->vrfb_dma_tx.dma_ch = -1;
- vout->vrfb_dma_tx.req_status = DMA_CHAN_ALLOTED;
- ret = omap_request_dma(vout->vrfb_dma_tx.dev_id, "VRFB DMA TX",
- omap_vout_vrfb_dma_tx_callback,
- (void *) &vout->vrfb_dma_tx, &vout->vrfb_dma_tx.dma_ch);
- if (ret < 0) {
+ dma_cap_zero(mask);
+ dma_cap_set(DMA_INTERLEAVE, mask);
+ vout->vrfb_dma_tx.chan = dma_request_chan_by_mask(&mask);
+ if (IS_ERR(vout->vrfb_dma_tx.chan)) {
vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
+ } else {
+ size_t xt_size = sizeof(struct dma_interleaved_template) +
+ sizeof(struct data_chunk);
+
+ vout->vrfb_dma_tx.xt = kzalloc(xt_size, GFP_KERNEL);
+ if (!vout->vrfb_dma_tx.xt) {
+ dma_release_channel(vout->vrfb_dma_tx.chan);
+ vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
+ }
+ }
+
+ if (vout->vrfb_dma_tx.req_status == DMA_CHAN_NOT_ALLOTED)
dev_info(&pdev->dev,
": failed to allocate DMA Channel for video%d\n",
vfd->minor);
- }
+
init_waitqueue_head(&vout->vrfb_dma_tx.wait);
/* statically allocated the VRFB buffer is done through
@@ -177,7 +186,9 @@ void omap_vout_release_vrfb(struct omap_vout_device *vout)
if (vout->vrfb_dma_tx.req_status == DMA_CHAN_ALLOTED) {
vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
- omap_free_dma(vout->vrfb_dma_tx.dma_ch);
+ kfree(vout->vrfb_dma_tx.xt);
+ dmaengine_terminate_sync(vout->vrfb_dma_tx.chan);
+ dma_release_channel(vout->vrfb_dma_tx.chan);
}
}
@@ -219,70 +230,84 @@ int omap_vout_vrfb_buffer_setup(struct omap_vout_device *vout,
}
int omap_vout_prepare_vrfb(struct omap_vout_device *vout,
- struct videobuf_buffer *vb)
+ struct videobuf_buffer *vb)
{
- dma_addr_t dmabuf;
- struct vid_vrfb_dma *tx;
+ struct dma_async_tx_descriptor *tx;
+ enum dma_ctrl_flags flags;
+ struct dma_chan *chan = vout->vrfb_dma_tx.chan;
+ struct dma_device *dmadev = chan->device;
+ struct dma_interleaved_template *xt = vout->vrfb_dma_tx.xt;
+ dma_cookie_t cookie;
+ enum dma_status status;
enum dss_rotation rotation;
- u32 dest_frame_index = 0, src_element_index = 0;
- u32 dest_element_index = 0, src_frame_index = 0;
- u32 elem_count = 0, frame_count = 0, pixsize = 2;
+ size_t dst_icg;
+ u32 pixsize;
if (!is_rotation_enabled(vout))
return 0;
- dmabuf = vout->buf_phy_addr[vb->i];
/* If rotation is enabled, copy input buffer into VRFB
* memory space using DMA. We are copying input buffer
* into VRFB memory space of desired angle and DSS will
* read image VRFB memory for 0 degree angle
*/
+
pixsize = vout->bpp * vout->vrfb_bpp;
- /*
- * DMA transfer in double index mode
- */
+ dst_icg = ((MAX_PIXELS_PER_LINE * pixsize) -
+ (vout->pix.width * vout->bpp)) + 1;
+
+ xt->src_start = vout->buf_phy_addr[vb->i];
+ xt->dst_start = vout->vrfb_context[vb->i].paddr[0];
+
+ xt->numf = vout->pix.height;
+ xt->frame_size = 1;
+ xt->sgl[0].size = vout->pix.width * vout->bpp;
+ xt->sgl[0].icg = dst_icg;
+
+ xt->dir = DMA_MEM_TO_MEM;
+ xt->src_sgl = false;
+ xt->src_inc = true;
+ xt->dst_sgl = true;
+ xt->dst_inc = true;
+
+ tx = dmadev->device_prep_interleaved_dma(chan, xt, flags);
+ if (tx == NULL) {
+ pr_err("%s: DMA interleaved prep error\n", __func__);
+ return -EINVAL;
+ }
- /* Frame index */
- dest_frame_index = ((MAX_PIXELS_PER_LINE * pixsize) -
- (vout->pix.width * vout->bpp)) + 1;
-
- /* Source and destination parameters */
- src_element_index = 0;
- src_frame_index = 0;
- dest_element_index = 1;
- /* Number of elements per frame */
- elem_count = vout->pix.width * vout->bpp;
- frame_count = vout->pix.height;
- tx = &vout->vrfb_dma_tx;
- tx->tx_status = 0;
- omap_set_dma_transfer_params(tx->dma_ch, OMAP_DMA_DATA_TYPE_S32,
- (elem_count / 4), frame_count, OMAP_DMA_SYNC_ELEMENT,
- tx->dev_id, 0x0);
- /* src_port required only for OMAP1 */
- omap_set_dma_src_params(tx->dma_ch, 0, OMAP_DMA_AMODE_POST_INC,
- dmabuf, src_element_index, src_frame_index);
- /*set dma source burst mode for VRFB */
- omap_set_dma_src_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
- rotation = calc_rotation(vout);
+ tx->callback = omap_vout_vrfb_dma_tx_callback;
+ tx->callback_param = &vout->vrfb_dma_tx;
+
+ cookie = dmaengine_submit(tx);
+ if (dma_submit_error(cookie)) {
+ pr_err("%s: dmaengine_submit failed (%d)\n", __func__, cookie);
+ return -EINVAL;
+ }
- /* dest_port required only for OMAP1 */
- omap_set_dma_dest_params(tx->dma_ch, 0, OMAP_DMA_AMODE_DOUBLE_IDX,
- vout->vrfb_context[vb->i].paddr[0], dest_element_index,
- dest_frame_index);
- /*set dma dest burst mode for VRFB */
- omap_set_dma_dest_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
- omap_dma_set_global_params(DMA_DEFAULT_ARB_RATE, 0x20, 0);
+ vout->vrfb_dma_tx.tx_status = 0;
+ dma_async_issue_pending(chan);
- omap_start_dma(tx->dma_ch);
- wait_event_interruptible_timeout(tx->wait, tx->tx_status == 1,
+ wait_event_interruptible_timeout(vout->vrfb_dma_tx.wait,
+ vout->vrfb_dma_tx.tx_status == 1,
VRFB_TX_TIMEOUT);
- if (tx->tx_status == 0) {
- omap_stop_dma(tx->dma_ch);
+ status = dma_async_is_tx_complete(chan, cookie, NULL, NULL);
+
+ if (vout->vrfb_dma_tx.tx_status == 0) {
+ pr_err("%s: Timeout while waiting for DMA\n", __func__);
+ dmaengine_terminate_sync(chan);
+ return -EINVAL;
+ } else if (status != DMA_COMPLETE) {
+ pr_err("%s: DMA completion %s status\n", __func__,
+ status == DMA_ERROR ? "error" : "busy");
+ dmaengine_terminate_sync(chan);
return -EINVAL;
}
+
/* Store buffers physical address into an array. Addresses
* from this array will be used to configure DSS */
+ rotation = calc_rotation(vout);
vout->queued_buf_addr[vb->i] = (u8 *)
vout->vrfb_context[vb->i].paddr[rotation];
return 0;
diff --git a/drivers/media/platform/omap/omap_voutdef.h b/drivers/media/platform/omap/omap_voutdef.h
index 80c79fabdf95..56b630b1c8b4 100644
--- a/drivers/media/platform/omap/omap_voutdef.h
+++ b/drivers/media/platform/omap/omap_voutdef.h
@@ -14,6 +14,7 @@
#include <media/v4l2-ctrls.h>
#include <video/omapfb_dss.h>
#include <video/omapvrfb.h>
+#include <linux/dmaengine.h>
#define YUYV_BPP 2
#define RGB565_BPP 2
@@ -81,8 +82,9 @@ enum vout_rotaion_type {
* for VRFB hidden buffer
*/
struct vid_vrfb_dma {
- int dev_id;
- int dma_ch;
+ struct dma_chan *chan;
+ struct dma_interleaved_template *xt;
+
int req_status;
int tx_status;
wait_queue_head_t wait;
--
2.12.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [RESEND v2 PATCH] v4l: omap_vout: vrfb: Convert to dmaengine
@ 2017-05-03 11:08 ` Peter Ujfalusi
0 siblings, 0 replies; 4+ messages in thread
From: Peter Ujfalusi @ 2017-05-03 11:08 UTC (permalink / raw)
To: laurent.pinchart
Cc: hans.verkuil, linux-media, linux-kernel, arnd, linux, tony, linux-omap
The dmaengine driver for sDMA now have support for interleaved transfer.
This trasnfer type was open coded with the legacy omap-dma API, but now
we can move it to dmaengine.
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
---
Hi,
changes since RESEND (27.10.2016):
- rebased on next-20170503
I can not test it on real HW (still), but I have validated [1] that the change
is correct and should not cause any regression.
Laurent: can you verify the patch on a real hardware?
Regards,
Peter
drivers/media/platform/omap/omap_vout_vrfb.c | 133 ++++++++++++++++-----------
drivers/media/platform/omap/omap_voutdef.h | 6 +-
2 files changed, 83 insertions(+), 56 deletions(-)
diff --git a/drivers/media/platform/omap/omap_vout_vrfb.c b/drivers/media/platform/omap/omap_vout_vrfb.c
index 92c4e1826356..45a553d4f5b2 100644
--- a/drivers/media/platform/omap/omap_vout_vrfb.c
+++ b/drivers/media/platform/omap/omap_vout_vrfb.c
@@ -16,7 +16,6 @@
#include <media/videobuf-dma-contig.h>
#include <media/v4l2-device.h>
-#include <linux/omap-dma.h>
#include <video/omapvrfb.h>
#include "omap_voutdef.h"
@@ -63,7 +62,7 @@ static int omap_vout_allocate_vrfb_buffers(struct omap_vout_device *vout,
/*
* Wakes up the application once the DMA transfer to VRFB space is completed.
*/
-static void omap_vout_vrfb_dma_tx_callback(int lch, u16 ch_status, void *data)
+static void omap_vout_vrfb_dma_tx_callback(void *data)
{
struct vid_vrfb_dma *t = (struct vid_vrfb_dma *) data;
@@ -94,6 +93,7 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
int ret = 0, i, j;
struct omap_vout_device *vout;
struct video_device *vfd;
+ dma_cap_mask_t mask;
int image_width, image_height;
int vrfb_num_bufs = VRFB_NUM_BUFS;
struct v4l2_device *v4l2_dev = platform_get_drvdata(pdev);
@@ -131,18 +131,27 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
/*
* Request and Initialize DMA, for DMA based VRFB transfer
*/
- vout->vrfb_dma_tx.dev_id = OMAP_DMA_NO_DEVICE;
- vout->vrfb_dma_tx.dma_ch = -1;
- vout->vrfb_dma_tx.req_status = DMA_CHAN_ALLOTED;
- ret = omap_request_dma(vout->vrfb_dma_tx.dev_id, "VRFB DMA TX",
- omap_vout_vrfb_dma_tx_callback,
- (void *) &vout->vrfb_dma_tx, &vout->vrfb_dma_tx.dma_ch);
- if (ret < 0) {
+ dma_cap_zero(mask);
+ dma_cap_set(DMA_INTERLEAVE, mask);
+ vout->vrfb_dma_tx.chan = dma_request_chan_by_mask(&mask);
+ if (IS_ERR(vout->vrfb_dma_tx.chan)) {
vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
+ } else {
+ size_t xt_size = sizeof(struct dma_interleaved_template) +
+ sizeof(struct data_chunk);
+
+ vout->vrfb_dma_tx.xt = kzalloc(xt_size, GFP_KERNEL);
+ if (!vout->vrfb_dma_tx.xt) {
+ dma_release_channel(vout->vrfb_dma_tx.chan);
+ vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
+ }
+ }
+
+ if (vout->vrfb_dma_tx.req_status == DMA_CHAN_NOT_ALLOTED)
dev_info(&pdev->dev,
": failed to allocate DMA Channel for video%d\n",
vfd->minor);
- }
+
init_waitqueue_head(&vout->vrfb_dma_tx.wait);
/* statically allocated the VRFB buffer is done through
@@ -177,7 +186,9 @@ void omap_vout_release_vrfb(struct omap_vout_device *vout)
if (vout->vrfb_dma_tx.req_status == DMA_CHAN_ALLOTED) {
vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
- omap_free_dma(vout->vrfb_dma_tx.dma_ch);
+ kfree(vout->vrfb_dma_tx.xt);
+ dmaengine_terminate_sync(vout->vrfb_dma_tx.chan);
+ dma_release_channel(vout->vrfb_dma_tx.chan);
}
}
@@ -219,70 +230,84 @@ int omap_vout_vrfb_buffer_setup(struct omap_vout_device *vout,
}
int omap_vout_prepare_vrfb(struct omap_vout_device *vout,
- struct videobuf_buffer *vb)
+ struct videobuf_buffer *vb)
{
- dma_addr_t dmabuf;
- struct vid_vrfb_dma *tx;
+ struct dma_async_tx_descriptor *tx;
+ enum dma_ctrl_flags flags;
+ struct dma_chan *chan = vout->vrfb_dma_tx.chan;
+ struct dma_device *dmadev = chan->device;
+ struct dma_interleaved_template *xt = vout->vrfb_dma_tx.xt;
+ dma_cookie_t cookie;
+ enum dma_status status;
enum dss_rotation rotation;
- u32 dest_frame_index = 0, src_element_index = 0;
- u32 dest_element_index = 0, src_frame_index = 0;
- u32 elem_count = 0, frame_count = 0, pixsize = 2;
+ size_t dst_icg;
+ u32 pixsize;
if (!is_rotation_enabled(vout))
return 0;
- dmabuf = vout->buf_phy_addr[vb->i];
/* If rotation is enabled, copy input buffer into VRFB
* memory space using DMA. We are copying input buffer
* into VRFB memory space of desired angle and DSS will
* read image VRFB memory for 0 degree angle
*/
+
pixsize = vout->bpp * vout->vrfb_bpp;
- /*
- * DMA transfer in double index mode
- */
+ dst_icg = ((MAX_PIXELS_PER_LINE * pixsize) -
+ (vout->pix.width * vout->bpp)) + 1;
+
+ xt->src_start = vout->buf_phy_addr[vb->i];
+ xt->dst_start = vout->vrfb_context[vb->i].paddr[0];
+
+ xt->numf = vout->pix.height;
+ xt->frame_size = 1;
+ xt->sgl[0].size = vout->pix.width * vout->bpp;
+ xt->sgl[0].icg = dst_icg;
+
+ xt->dir = DMA_MEM_TO_MEM;
+ xt->src_sgl = false;
+ xt->src_inc = true;
+ xt->dst_sgl = true;
+ xt->dst_inc = true;
+
+ tx = dmadev->device_prep_interleaved_dma(chan, xt, flags);
+ if (tx == NULL) {
+ pr_err("%s: DMA interleaved prep error\n", __func__);
+ return -EINVAL;
+ }
- /* Frame index */
- dest_frame_index = ((MAX_PIXELS_PER_LINE * pixsize) -
- (vout->pix.width * vout->bpp)) + 1;
-
- /* Source and destination parameters */
- src_element_index = 0;
- src_frame_index = 0;
- dest_element_index = 1;
- /* Number of elements per frame */
- elem_count = vout->pix.width * vout->bpp;
- frame_count = vout->pix.height;
- tx = &vout->vrfb_dma_tx;
- tx->tx_status = 0;
- omap_set_dma_transfer_params(tx->dma_ch, OMAP_DMA_DATA_TYPE_S32,
- (elem_count / 4), frame_count, OMAP_DMA_SYNC_ELEMENT,
- tx->dev_id, 0x0);
- /* src_port required only for OMAP1 */
- omap_set_dma_src_params(tx->dma_ch, 0, OMAP_DMA_AMODE_POST_INC,
- dmabuf, src_element_index, src_frame_index);
- /*set dma source burst mode for VRFB */
- omap_set_dma_src_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
- rotation = calc_rotation(vout);
+ tx->callback = omap_vout_vrfb_dma_tx_callback;
+ tx->callback_param = &vout->vrfb_dma_tx;
+
+ cookie = dmaengine_submit(tx);
+ if (dma_submit_error(cookie)) {
+ pr_err("%s: dmaengine_submit failed (%d)\n", __func__, cookie);
+ return -EINVAL;
+ }
- /* dest_port required only for OMAP1 */
- omap_set_dma_dest_params(tx->dma_ch, 0, OMAP_DMA_AMODE_DOUBLE_IDX,
- vout->vrfb_context[vb->i].paddr[0], dest_element_index,
- dest_frame_index);
- /*set dma dest burst mode for VRFB */
- omap_set_dma_dest_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
- omap_dma_set_global_params(DMA_DEFAULT_ARB_RATE, 0x20, 0);
+ vout->vrfb_dma_tx.tx_status = 0;
+ dma_async_issue_pending(chan);
- omap_start_dma(tx->dma_ch);
- wait_event_interruptible_timeout(tx->wait, tx->tx_status == 1,
+ wait_event_interruptible_timeout(vout->vrfb_dma_tx.wait,
+ vout->vrfb_dma_tx.tx_status == 1,
VRFB_TX_TIMEOUT);
- if (tx->tx_status == 0) {
- omap_stop_dma(tx->dma_ch);
+ status = dma_async_is_tx_complete(chan, cookie, NULL, NULL);
+
+ if (vout->vrfb_dma_tx.tx_status == 0) {
+ pr_err("%s: Timeout while waiting for DMA\n", __func__);
+ dmaengine_terminate_sync(chan);
+ return -EINVAL;
+ } else if (status != DMA_COMPLETE) {
+ pr_err("%s: DMA completion %s status\n", __func__,
+ status == DMA_ERROR ? "error" : "busy");
+ dmaengine_terminate_sync(chan);
return -EINVAL;
}
+
/* Store buffers physical address into an array. Addresses
* from this array will be used to configure DSS */
+ rotation = calc_rotation(vout);
vout->queued_buf_addr[vb->i] = (u8 *)
vout->vrfb_context[vb->i].paddr[rotation];
return 0;
diff --git a/drivers/media/platform/omap/omap_voutdef.h b/drivers/media/platform/omap/omap_voutdef.h
index 80c79fabdf95..56b630b1c8b4 100644
--- a/drivers/media/platform/omap/omap_voutdef.h
+++ b/drivers/media/platform/omap/omap_voutdef.h
@@ -14,6 +14,7 @@
#include <media/v4l2-ctrls.h>
#include <video/omapfb_dss.h>
#include <video/omapvrfb.h>
+#include <linux/dmaengine.h>
#define YUYV_BPP 2
#define RGB565_BPP 2
@@ -81,8 +82,9 @@ enum vout_rotaion_type {
* for VRFB hidden buffer
*/
struct vid_vrfb_dma {
- int dev_id;
- int dma_ch;
+ struct dma_chan *chan;
+ struct dma_interleaved_template *xt;
+
int req_status;
int tx_status;
wait_queue_head_t wait;
--
2.12.2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [RESEND v2 PATCH] v4l: omap_vout: vrfb: Convert to dmaengine
2017-05-03 11:08 ` Peter Ujfalusi
@ 2017-05-03 11:09 ` Peter Ujfalusi
-1 siblings, 0 replies; 4+ messages in thread
From: Peter Ujfalusi @ 2017-05-03 11:09 UTC (permalink / raw)
To: laurent.pinchart
Cc: hans.verkuil, linux-media, linux-kernel, arnd, linux, tony, linux-omap
On 2017-05-03 14:08, Peter Ujfalusi wrote:
> The dmaengine driver for sDMA now have support for interleaved transfer.
> This trasnfer type was open coded with the legacy omap-dma API, but now
> we can move it to dmaengine.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> ---
> Hi,
>
> changes since RESEND (27.10.2016):
> - rebased on next-20170503
>
> I can not test it on real HW (still), but I have validated [1] that the change
> is correct and should not cause any regression.
>
> Laurent: can you verify the patch on a real hardware?
[1]
https://github.com/omap-audio/linux-audio/blob/peter/linux-next-wip/drivers/misc/ovv_dmaengine.c
>
> Regards,
> Peter
>
> drivers/media/platform/omap/omap_vout_vrfb.c | 133 ++++++++++++++++-----------
> drivers/media/platform/omap/omap_voutdef.h | 6 +-
> 2 files changed, 83 insertions(+), 56 deletions(-)
>
> diff --git a/drivers/media/platform/omap/omap_vout_vrfb.c b/drivers/media/platform/omap/omap_vout_vrfb.c
> index 92c4e1826356..45a553d4f5b2 100644
> --- a/drivers/media/platform/omap/omap_vout_vrfb.c
> +++ b/drivers/media/platform/omap/omap_vout_vrfb.c
> @@ -16,7 +16,6 @@
> #include <media/videobuf-dma-contig.h>
> #include <media/v4l2-device.h>
>
> -#include <linux/omap-dma.h>
> #include <video/omapvrfb.h>
>
> #include "omap_voutdef.h"
> @@ -63,7 +62,7 @@ static int omap_vout_allocate_vrfb_buffers(struct omap_vout_device *vout,
> /*
> * Wakes up the application once the DMA transfer to VRFB space is completed.
> */
> -static void omap_vout_vrfb_dma_tx_callback(int lch, u16 ch_status, void *data)
> +static void omap_vout_vrfb_dma_tx_callback(void *data)
> {
> struct vid_vrfb_dma *t = (struct vid_vrfb_dma *) data;
>
> @@ -94,6 +93,7 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
> int ret = 0, i, j;
> struct omap_vout_device *vout;
> struct video_device *vfd;
> + dma_cap_mask_t mask;
> int image_width, image_height;
> int vrfb_num_bufs = VRFB_NUM_BUFS;
> struct v4l2_device *v4l2_dev = platform_get_drvdata(pdev);
> @@ -131,18 +131,27 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
> /*
> * Request and Initialize DMA, for DMA based VRFB transfer
> */
> - vout->vrfb_dma_tx.dev_id = OMAP_DMA_NO_DEVICE;
> - vout->vrfb_dma_tx.dma_ch = -1;
> - vout->vrfb_dma_tx.req_status = DMA_CHAN_ALLOTED;
> - ret = omap_request_dma(vout->vrfb_dma_tx.dev_id, "VRFB DMA TX",
> - omap_vout_vrfb_dma_tx_callback,
> - (void *) &vout->vrfb_dma_tx, &vout->vrfb_dma_tx.dma_ch);
> - if (ret < 0) {
> + dma_cap_zero(mask);
> + dma_cap_set(DMA_INTERLEAVE, mask);
> + vout->vrfb_dma_tx.chan = dma_request_chan_by_mask(&mask);
> + if (IS_ERR(vout->vrfb_dma_tx.chan)) {
> vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> + } else {
> + size_t xt_size = sizeof(struct dma_interleaved_template) +
> + sizeof(struct data_chunk);
> +
> + vout->vrfb_dma_tx.xt = kzalloc(xt_size, GFP_KERNEL);
> + if (!vout->vrfb_dma_tx.xt) {
> + dma_release_channel(vout->vrfb_dma_tx.chan);
> + vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> + }
> + }
> +
> + if (vout->vrfb_dma_tx.req_status == DMA_CHAN_NOT_ALLOTED)
> dev_info(&pdev->dev,
> ": failed to allocate DMA Channel for video%d\n",
> vfd->minor);
> - }
> +
> init_waitqueue_head(&vout->vrfb_dma_tx.wait);
>
> /* statically allocated the VRFB buffer is done through
> @@ -177,7 +186,9 @@ void omap_vout_release_vrfb(struct omap_vout_device *vout)
>
> if (vout->vrfb_dma_tx.req_status == DMA_CHAN_ALLOTED) {
> vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> - omap_free_dma(vout->vrfb_dma_tx.dma_ch);
> + kfree(vout->vrfb_dma_tx.xt);
> + dmaengine_terminate_sync(vout->vrfb_dma_tx.chan);
> + dma_release_channel(vout->vrfb_dma_tx.chan);
> }
> }
>
> @@ -219,70 +230,84 @@ int omap_vout_vrfb_buffer_setup(struct omap_vout_device *vout,
> }
>
> int omap_vout_prepare_vrfb(struct omap_vout_device *vout,
> - struct videobuf_buffer *vb)
> + struct videobuf_buffer *vb)
> {
> - dma_addr_t dmabuf;
> - struct vid_vrfb_dma *tx;
> + struct dma_async_tx_descriptor *tx;
> + enum dma_ctrl_flags flags;
> + struct dma_chan *chan = vout->vrfb_dma_tx.chan;
> + struct dma_device *dmadev = chan->device;
> + struct dma_interleaved_template *xt = vout->vrfb_dma_tx.xt;
> + dma_cookie_t cookie;
> + enum dma_status status;
> enum dss_rotation rotation;
> - u32 dest_frame_index = 0, src_element_index = 0;
> - u32 dest_element_index = 0, src_frame_index = 0;
> - u32 elem_count = 0, frame_count = 0, pixsize = 2;
> + size_t dst_icg;
> + u32 pixsize;
>
> if (!is_rotation_enabled(vout))
> return 0;
>
> - dmabuf = vout->buf_phy_addr[vb->i];
> /* If rotation is enabled, copy input buffer into VRFB
> * memory space using DMA. We are copying input buffer
> * into VRFB memory space of desired angle and DSS will
> * read image VRFB memory for 0 degree angle
> */
> +
> pixsize = vout->bpp * vout->vrfb_bpp;
> - /*
> - * DMA transfer in double index mode
> - */
> + dst_icg = ((MAX_PIXELS_PER_LINE * pixsize) -
> + (vout->pix.width * vout->bpp)) + 1;
> +
> + xt->src_start = vout->buf_phy_addr[vb->i];
> + xt->dst_start = vout->vrfb_context[vb->i].paddr[0];
> +
> + xt->numf = vout->pix.height;
> + xt->frame_size = 1;
> + xt->sgl[0].size = vout->pix.width * vout->bpp;
> + xt->sgl[0].icg = dst_icg;
> +
> + xt->dir = DMA_MEM_TO_MEM;
> + xt->src_sgl = false;
> + xt->src_inc = true;
> + xt->dst_sgl = true;
> + xt->dst_inc = true;
> +
> + tx = dmadev->device_prep_interleaved_dma(chan, xt, flags);
> + if (tx == NULL) {
> + pr_err("%s: DMA interleaved prep error\n", __func__);
> + return -EINVAL;
> + }
>
> - /* Frame index */
> - dest_frame_index = ((MAX_PIXELS_PER_LINE * pixsize) -
> - (vout->pix.width * vout->bpp)) + 1;
> -
> - /* Source and destination parameters */
> - src_element_index = 0;
> - src_frame_index = 0;
> - dest_element_index = 1;
> - /* Number of elements per frame */
> - elem_count = vout->pix.width * vout->bpp;
> - frame_count = vout->pix.height;
> - tx = &vout->vrfb_dma_tx;
> - tx->tx_status = 0;
> - omap_set_dma_transfer_params(tx->dma_ch, OMAP_DMA_DATA_TYPE_S32,
> - (elem_count / 4), frame_count, OMAP_DMA_SYNC_ELEMENT,
> - tx->dev_id, 0x0);
> - /* src_port required only for OMAP1 */
> - omap_set_dma_src_params(tx->dma_ch, 0, OMAP_DMA_AMODE_POST_INC,
> - dmabuf, src_element_index, src_frame_index);
> - /*set dma source burst mode for VRFB */
> - omap_set_dma_src_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
> - rotation = calc_rotation(vout);
> + tx->callback = omap_vout_vrfb_dma_tx_callback;
> + tx->callback_param = &vout->vrfb_dma_tx;
> +
> + cookie = dmaengine_submit(tx);
> + if (dma_submit_error(cookie)) {
> + pr_err("%s: dmaengine_submit failed (%d)\n", __func__, cookie);
> + return -EINVAL;
> + }
>
> - /* dest_port required only for OMAP1 */
> - omap_set_dma_dest_params(tx->dma_ch, 0, OMAP_DMA_AMODE_DOUBLE_IDX,
> - vout->vrfb_context[vb->i].paddr[0], dest_element_index,
> - dest_frame_index);
> - /*set dma dest burst mode for VRFB */
> - omap_set_dma_dest_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
> - omap_dma_set_global_params(DMA_DEFAULT_ARB_RATE, 0x20, 0);
> + vout->vrfb_dma_tx.tx_status = 0;
> + dma_async_issue_pending(chan);
>
> - omap_start_dma(tx->dma_ch);
> - wait_event_interruptible_timeout(tx->wait, tx->tx_status == 1,
> + wait_event_interruptible_timeout(vout->vrfb_dma_tx.wait,
> + vout->vrfb_dma_tx.tx_status == 1,
> VRFB_TX_TIMEOUT);
>
> - if (tx->tx_status == 0) {
> - omap_stop_dma(tx->dma_ch);
> + status = dma_async_is_tx_complete(chan, cookie, NULL, NULL);
> +
> + if (vout->vrfb_dma_tx.tx_status == 0) {
> + pr_err("%s: Timeout while waiting for DMA\n", __func__);
> + dmaengine_terminate_sync(chan);
> + return -EINVAL;
> + } else if (status != DMA_COMPLETE) {
> + pr_err("%s: DMA completion %s status\n", __func__,
> + status == DMA_ERROR ? "error" : "busy");
> + dmaengine_terminate_sync(chan);
> return -EINVAL;
> }
> +
> /* Store buffers physical address into an array. Addresses
> * from this array will be used to configure DSS */
> + rotation = calc_rotation(vout);
> vout->queued_buf_addr[vb->i] = (u8 *)
> vout->vrfb_context[vb->i].paddr[rotation];
> return 0;
> diff --git a/drivers/media/platform/omap/omap_voutdef.h b/drivers/media/platform/omap/omap_voutdef.h
> index 80c79fabdf95..56b630b1c8b4 100644
> --- a/drivers/media/platform/omap/omap_voutdef.h
> +++ b/drivers/media/platform/omap/omap_voutdef.h
> @@ -14,6 +14,7 @@
> #include <media/v4l2-ctrls.h>
> #include <video/omapfb_dss.h>
> #include <video/omapvrfb.h>
> +#include <linux/dmaengine.h>
>
> #define YUYV_BPP 2
> #define RGB565_BPP 2
> @@ -81,8 +82,9 @@ enum vout_rotaion_type {
> * for VRFB hidden buffer
> */
> struct vid_vrfb_dma {
> - int dev_id;
> - int dma_ch;
> + struct dma_chan *chan;
> + struct dma_interleaved_template *xt;
> +
> int req_status;
> int tx_status;
> wait_queue_head_t wait;
>
- Péter
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RESEND v2 PATCH] v4l: omap_vout: vrfb: Convert to dmaengine
@ 2017-05-03 11:09 ` Peter Ujfalusi
0 siblings, 0 replies; 4+ messages in thread
From: Peter Ujfalusi @ 2017-05-03 11:09 UTC (permalink / raw)
To: laurent.pinchart
Cc: hans.verkuil, linux-media, linux-kernel, arnd, linux, tony, linux-omap
On 2017-05-03 14:08, Peter Ujfalusi wrote:
> The dmaengine driver for sDMA now have support for interleaved transfer.
> This trasnfer type was open coded with the legacy omap-dma API, but now
> we can move it to dmaengine.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> ---
> Hi,
>
> changes since RESEND (27.10.2016):
> - rebased on next-20170503
>
> I can not test it on real HW (still), but I have validated [1] that the change
> is correct and should not cause any regression.
>
> Laurent: can you verify the patch on a real hardware?
[1]
https://github.com/omap-audio/linux-audio/blob/peter/linux-next-wip/drivers/misc/ovv_dmaengine.c
>
> Regards,
> Peter
>
> drivers/media/platform/omap/omap_vout_vrfb.c | 133 ++++++++++++++++-----------
> drivers/media/platform/omap/omap_voutdef.h | 6 +-
> 2 files changed, 83 insertions(+), 56 deletions(-)
>
> diff --git a/drivers/media/platform/omap/omap_vout_vrfb.c b/drivers/media/platform/omap/omap_vout_vrfb.c
> index 92c4e1826356..45a553d4f5b2 100644
> --- a/drivers/media/platform/omap/omap_vout_vrfb.c
> +++ b/drivers/media/platform/omap/omap_vout_vrfb.c
> @@ -16,7 +16,6 @@
> #include <media/videobuf-dma-contig.h>
> #include <media/v4l2-device.h>
>
> -#include <linux/omap-dma.h>
> #include <video/omapvrfb.h>
>
> #include "omap_voutdef.h"
> @@ -63,7 +62,7 @@ static int omap_vout_allocate_vrfb_buffers(struct omap_vout_device *vout,
> /*
> * Wakes up the application once the DMA transfer to VRFB space is completed.
> */
> -static void omap_vout_vrfb_dma_tx_callback(int lch, u16 ch_status, void *data)
> +static void omap_vout_vrfb_dma_tx_callback(void *data)
> {
> struct vid_vrfb_dma *t = (struct vid_vrfb_dma *) data;
>
> @@ -94,6 +93,7 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
> int ret = 0, i, j;
> struct omap_vout_device *vout;
> struct video_device *vfd;
> + dma_cap_mask_t mask;
> int image_width, image_height;
> int vrfb_num_bufs = VRFB_NUM_BUFS;
> struct v4l2_device *v4l2_dev = platform_get_drvdata(pdev);
> @@ -131,18 +131,27 @@ int omap_vout_setup_vrfb_bufs(struct platform_device *pdev, int vid_num,
> /*
> * Request and Initialize DMA, for DMA based VRFB transfer
> */
> - vout->vrfb_dma_tx.dev_id = OMAP_DMA_NO_DEVICE;
> - vout->vrfb_dma_tx.dma_ch = -1;
> - vout->vrfb_dma_tx.req_status = DMA_CHAN_ALLOTED;
> - ret = omap_request_dma(vout->vrfb_dma_tx.dev_id, "VRFB DMA TX",
> - omap_vout_vrfb_dma_tx_callback,
> - (void *) &vout->vrfb_dma_tx, &vout->vrfb_dma_tx.dma_ch);
> - if (ret < 0) {
> + dma_cap_zero(mask);
> + dma_cap_set(DMA_INTERLEAVE, mask);
> + vout->vrfb_dma_tx.chan = dma_request_chan_by_mask(&mask);
> + if (IS_ERR(vout->vrfb_dma_tx.chan)) {
> vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> + } else {
> + size_t xt_size = sizeof(struct dma_interleaved_template) +
> + sizeof(struct data_chunk);
> +
> + vout->vrfb_dma_tx.xt = kzalloc(xt_size, GFP_KERNEL);
> + if (!vout->vrfb_dma_tx.xt) {
> + dma_release_channel(vout->vrfb_dma_tx.chan);
> + vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> + }
> + }
> +
> + if (vout->vrfb_dma_tx.req_status == DMA_CHAN_NOT_ALLOTED)
> dev_info(&pdev->dev,
> ": failed to allocate DMA Channel for video%d\n",
> vfd->minor);
> - }
> +
> init_waitqueue_head(&vout->vrfb_dma_tx.wait);
>
> /* statically allocated the VRFB buffer is done through
> @@ -177,7 +186,9 @@ void omap_vout_release_vrfb(struct omap_vout_device *vout)
>
> if (vout->vrfb_dma_tx.req_status == DMA_CHAN_ALLOTED) {
> vout->vrfb_dma_tx.req_status = DMA_CHAN_NOT_ALLOTED;
> - omap_free_dma(vout->vrfb_dma_tx.dma_ch);
> + kfree(vout->vrfb_dma_tx.xt);
> + dmaengine_terminate_sync(vout->vrfb_dma_tx.chan);
> + dma_release_channel(vout->vrfb_dma_tx.chan);
> }
> }
>
> @@ -219,70 +230,84 @@ int omap_vout_vrfb_buffer_setup(struct omap_vout_device *vout,
> }
>
> int omap_vout_prepare_vrfb(struct omap_vout_device *vout,
> - struct videobuf_buffer *vb)
> + struct videobuf_buffer *vb)
> {
> - dma_addr_t dmabuf;
> - struct vid_vrfb_dma *tx;
> + struct dma_async_tx_descriptor *tx;
> + enum dma_ctrl_flags flags;
> + struct dma_chan *chan = vout->vrfb_dma_tx.chan;
> + struct dma_device *dmadev = chan->device;
> + struct dma_interleaved_template *xt = vout->vrfb_dma_tx.xt;
> + dma_cookie_t cookie;
> + enum dma_status status;
> enum dss_rotation rotation;
> - u32 dest_frame_index = 0, src_element_index = 0;
> - u32 dest_element_index = 0, src_frame_index = 0;
> - u32 elem_count = 0, frame_count = 0, pixsize = 2;
> + size_t dst_icg;
> + u32 pixsize;
>
> if (!is_rotation_enabled(vout))
> return 0;
>
> - dmabuf = vout->buf_phy_addr[vb->i];
> /* If rotation is enabled, copy input buffer into VRFB
> * memory space using DMA. We are copying input buffer
> * into VRFB memory space of desired angle and DSS will
> * read image VRFB memory for 0 degree angle
> */
> +
> pixsize = vout->bpp * vout->vrfb_bpp;
> - /*
> - * DMA transfer in double index mode
> - */
> + dst_icg = ((MAX_PIXELS_PER_LINE * pixsize) -
> + (vout->pix.width * vout->bpp)) + 1;
> +
> + xt->src_start = vout->buf_phy_addr[vb->i];
> + xt->dst_start = vout->vrfb_context[vb->i].paddr[0];
> +
> + xt->numf = vout->pix.height;
> + xt->frame_size = 1;
> + xt->sgl[0].size = vout->pix.width * vout->bpp;
> + xt->sgl[0].icg = dst_icg;
> +
> + xt->dir = DMA_MEM_TO_MEM;
> + xt->src_sgl = false;
> + xt->src_inc = true;
> + xt->dst_sgl = true;
> + xt->dst_inc = true;
> +
> + tx = dmadev->device_prep_interleaved_dma(chan, xt, flags);
> + if (tx == NULL) {
> + pr_err("%s: DMA interleaved prep error\n", __func__);
> + return -EINVAL;
> + }
>
> - /* Frame index */
> - dest_frame_index = ((MAX_PIXELS_PER_LINE * pixsize) -
> - (vout->pix.width * vout->bpp)) + 1;
> -
> - /* Source and destination parameters */
> - src_element_index = 0;
> - src_frame_index = 0;
> - dest_element_index = 1;
> - /* Number of elements per frame */
> - elem_count = vout->pix.width * vout->bpp;
> - frame_count = vout->pix.height;
> - tx = &vout->vrfb_dma_tx;
> - tx->tx_status = 0;
> - omap_set_dma_transfer_params(tx->dma_ch, OMAP_DMA_DATA_TYPE_S32,
> - (elem_count / 4), frame_count, OMAP_DMA_SYNC_ELEMENT,
> - tx->dev_id, 0x0);
> - /* src_port required only for OMAP1 */
> - omap_set_dma_src_params(tx->dma_ch, 0, OMAP_DMA_AMODE_POST_INC,
> - dmabuf, src_element_index, src_frame_index);
> - /*set dma source burst mode for VRFB */
> - omap_set_dma_src_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
> - rotation = calc_rotation(vout);
> + tx->callback = omap_vout_vrfb_dma_tx_callback;
> + tx->callback_param = &vout->vrfb_dma_tx;
> +
> + cookie = dmaengine_submit(tx);
> + if (dma_submit_error(cookie)) {
> + pr_err("%s: dmaengine_submit failed (%d)\n", __func__, cookie);
> + return -EINVAL;
> + }
>
> - /* dest_port required only for OMAP1 */
> - omap_set_dma_dest_params(tx->dma_ch, 0, OMAP_DMA_AMODE_DOUBLE_IDX,
> - vout->vrfb_context[vb->i].paddr[0], dest_element_index,
> - dest_frame_index);
> - /*set dma dest burst mode for VRFB */
> - omap_set_dma_dest_burst_mode(tx->dma_ch, OMAP_DMA_DATA_BURST_16);
> - omap_dma_set_global_params(DMA_DEFAULT_ARB_RATE, 0x20, 0);
> + vout->vrfb_dma_tx.tx_status = 0;
> + dma_async_issue_pending(chan);
>
> - omap_start_dma(tx->dma_ch);
> - wait_event_interruptible_timeout(tx->wait, tx->tx_status == 1,
> + wait_event_interruptible_timeout(vout->vrfb_dma_tx.wait,
> + vout->vrfb_dma_tx.tx_status == 1,
> VRFB_TX_TIMEOUT);
>
> - if (tx->tx_status == 0) {
> - omap_stop_dma(tx->dma_ch);
> + status = dma_async_is_tx_complete(chan, cookie, NULL, NULL);
> +
> + if (vout->vrfb_dma_tx.tx_status == 0) {
> + pr_err("%s: Timeout while waiting for DMA\n", __func__);
> + dmaengine_terminate_sync(chan);
> + return -EINVAL;
> + } else if (status != DMA_COMPLETE) {
> + pr_err("%s: DMA completion %s status\n", __func__,
> + status == DMA_ERROR ? "error" : "busy");
> + dmaengine_terminate_sync(chan);
> return -EINVAL;
> }
> +
> /* Store buffers physical address into an array. Addresses
> * from this array will be used to configure DSS */
> + rotation = calc_rotation(vout);
> vout->queued_buf_addr[vb->i] = (u8 *)
> vout->vrfb_context[vb->i].paddr[rotation];
> return 0;
> diff --git a/drivers/media/platform/omap/omap_voutdef.h b/drivers/media/platform/omap/omap_voutdef.h
> index 80c79fabdf95..56b630b1c8b4 100644
> --- a/drivers/media/platform/omap/omap_voutdef.h
> +++ b/drivers/media/platform/omap/omap_voutdef.h
> @@ -14,6 +14,7 @@
> #include <media/v4l2-ctrls.h>
> #include <video/omapfb_dss.h>
> #include <video/omapvrfb.h>
> +#include <linux/dmaengine.h>
>
> #define YUYV_BPP 2
> #define RGB565_BPP 2
> @@ -81,8 +82,9 @@ enum vout_rotaion_type {
> * for VRFB hidden buffer
> */
> struct vid_vrfb_dma {
> - int dev_id;
> - int dma_ch;
> + struct dma_chan *chan;
> + struct dma_interleaved_template *xt;
> +
> int req_status;
> int tx_status;
> wait_queue_head_t wait;
>
- Péter
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-05-03 11:09 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-03 11:08 [RESEND v2 PATCH] v4l: omap_vout: vrfb: Convert to dmaengine Peter Ujfalusi
2017-05-03 11:08 ` Peter Ujfalusi
2017-05-03 11:09 ` Peter Ujfalusi
2017-05-03 11:09 ` Peter Ujfalusi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.