public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 9/9] v4l: xilinx: dma: Get scaling and padding factor to calculate DMA params
@ 2018-02-15  6:43 Satish Kumar Nagireddy
  2018-02-16 17:06 ` Hyun Kwon
  0 siblings, 1 reply; 2+ messages in thread
From: Satish Kumar Nagireddy @ 2018-02-15  6:43 UTC (permalink / raw)
  To: linux-media, laurent.pinchart, michal.simek, hyun.kwon
  Cc: Satish Kumar Nagireddy

Get multiplying factor to calculate bpp especially
in case of 10 bit formats.
Get multiplying factor to calculate padding width

Signed-off-by: Satish Kumar Nagireddy <satishna@xilinx.com>
---
 drivers/media/platform/xilinx/xilinx-dma.c | 29 ++++++++++++++++++++++++++---
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/drivers/media/platform/xilinx/xilinx-dma.c b/drivers/media/platform/xilinx/xilinx-dma.c
index 664981b..3c2fd02 100644
--- a/drivers/media/platform/xilinx/xilinx-dma.c
+++ b/drivers/media/platform/xilinx/xilinx-dma.c
@@ -417,6 +417,7 @@ static void xvip_dma_buffer_queue(struct vb2_buffer *vb)
        struct xvip_dma_buffer *buf = to_xvip_dma_buffer(vbuf);
        struct dma_async_tx_descriptor *desc;
        u32 flags, luma_size;
+       u32 padding_factor_nume, padding_factor_deno, bpl_nume, bpl_deno;
        dma_addr_t addr = vb2_dma_contig_plane_dma_addr(vb, 0);

        if (dma->queue.type == V4L2_BUF_TYPE_VIDEO_CAPTURE ||
@@ -442,8 +443,15 @@ static void xvip_dma_buffer_queue(struct vb2_buffer *vb)
                struct v4l2_pix_format_mplane *pix_mp;

                pix_mp = &dma->format.fmt.pix_mp;
+               xvip_width_padding_factor(pix_mp->pixelformat,
+                                         &padding_factor_nume,
+                                         &padding_factor_deno);
+               xvip_bpl_scaling_factor(pix_mp->pixelformat, &bpl_nume,
+                                       &bpl_deno);
                dma->xt.frame_size = dma->fmtinfo->num_planes;
-               dma->sgl[0].size = pix_mp->width * dma->fmtinfo->bpl_factor;
+               dma->sgl[0].size = (pix_mp->width * dma->fmtinfo->bpl_factor *
+                                   padding_factor_nume * bpl_nume) /
+                                   (padding_factor_deno * bpl_deno);
                dma->sgl[0].icg = pix_mp->plane_fmt[0].bytesperline -
                                                        dma->sgl[0].size;
                dma->xt.numf = pix_mp->height;
@@ -472,8 +480,15 @@ static void xvip_dma_buffer_queue(struct vb2_buffer *vb)
                struct v4l2_pix_format *pix;

                pix = &dma->format.fmt.pix;
+               xvip_width_padding_factor(pix->pixelformat,
+                                         &padding_factor_nume,
+                                         &padding_factor_deno);
+               xvip_bpl_scaling_factor(pix->pixelformat, &bpl_nume,
+                                       &bpl_deno);
                dma->xt.frame_size = dma->fmtinfo->num_planes;
-               dma->sgl[0].size = pix->width * dma->fmtinfo->bpl_factor;
+               dma->sgl[0].size = (pix->width * dma->fmtinfo->bpl_factor *
+                                   padding_factor_nume * bpl_nume) /
+                                   (padding_factor_deno * bpl_deno);
                dma->sgl[0].icg = pix->bytesperline - dma->sgl[0].size;
                dma->xt.numf = pix->height;
                dma->sgl[0].dst_icg = dma->sgl[0].size;
@@ -682,6 +697,8 @@ __xvip_dma_try_format(struct xvip_dma *dma,
        unsigned int align;
        unsigned int bpl;
        unsigned int i, hsub, vsub, plane_width, plane_height;
+       unsigned int padding_factor_nume, padding_factor_deno;
+       unsigned int bpl_nume, bpl_deno;

        /* Retrieve format information and select the default format if the
         * requested format isn't supported.
@@ -694,6 +711,10 @@ __xvip_dma_try_format(struct xvip_dma *dma,
        if (IS_ERR(info))
                info = xvip_get_format_by_fourcc(XVIP_DMA_DEF_FORMAT);

+       xvip_width_padding_factor(info->fourcc, &padding_factor_nume,
+                                 &padding_factor_deno);
+       xvip_bpl_scaling_factor(info->fourcc, &bpl_nume, &bpl_deno);
+
        /* The transfer alignment requirements are expressed in bytes. Compute
         * the minimum and maximum values, clamp the requested width and convert
         * it back to pixels.
@@ -737,7 +758,9 @@ __xvip_dma_try_format(struct xvip_dma *dma,
                        for (i = 0; i < info->num_planes; i++) {
                                plane_width = pix_mp->width / (i ? hsub : 1);
                                plane_height = pix_mp->height / (i ? vsub : 1);
-                               min_bpl = plane_width * info->bpl_factor;
+                               min_bpl = (plane_width * info->bpl_factor *
+                                          padding_factor_nume * bpl_nume) /
+                                          (padding_factor_deno * bpl_deno);
                                max_bpl = rounddown(XVIP_DMA_MAX_WIDTH,
                                                    dma->align);
                                bpl = pix_mp->plane_fmt[i].bytesperline;
--
2.7.4

This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.

^ permalink raw reply related	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-02-16 17:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-15  6:43 [PATCH v3 9/9] v4l: xilinx: dma: Get scaling and padding factor to calculate DMA params Satish Kumar Nagireddy
2018-02-16 17:06 ` Hyun Kwon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox