[media] media: davinci: vpif_capture: improve vpif_buffer_queue_setup() function
authorLad, Prabhakar <prabhakar.csengg@gmail.com>
Fri, 16 May 2014 13:33:38 +0000 (10:33 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Fri, 23 May 2014 22:22:43 +0000 (19:22 -0300)
this patch sets the sizes[0] of plane according to the fmt passed
or which is being set in the channel, in both MMAP and USERPTR buffer
type.

This patch also move the calculation of offests(vpif_calculate_offsets())
to queue_setup() callback as after queue_setup() callback the
application is no longer allowed to change format, and prepares to
removal of config_params.

Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/platform/davinci/vpif_capture.c

index 025eb2410cde24663bd7536bab00e74d092b7ee1..c77c17695607e27154c623c47b8d367f390e49e0 100644 (file)
@@ -134,45 +134,24 @@ static int vpif_buffer_queue_setup(struct vb2_queue *vq,
 {
        struct channel_obj *ch = vb2_get_drv_priv(vq);
        struct common_obj *common;
-       unsigned long size;
 
        common = &ch->common[VPIF_VIDEO_INDEX];
 
        vpif_dbg(2, debug, "vpif_buffer_setup\n");
 
-       /* If memory type is not mmap, return */
-       if (V4L2_MEMORY_MMAP == common->memory) {
-               /* Calculate the size of the buffer */
-               size = config_params.channel_bufsize[ch->channel_id];
-               /*
-                * Checking if the buffer size exceeds the available buffer
-                * ycmux_mode = 0 means 1 channel mode HD and
-                * ycmux_mode = 1 means 2 channels mode SD
-                */
-               if (ch->vpifparams.std_info.ycmux_mode == 0) {
-                       if (config_params.video_limit[ch->channel_id])
-                               while (size * *nbuffers >
-                                       (config_params.video_limit[0]
-                                               + config_params.video_limit[1]))
-                                       (*nbuffers)--;
-               } else {
-                       if (config_params.video_limit[ch->channel_id])
-                               while (size * *nbuffers >
-                               config_params.video_limit[ch->channel_id])
-                                       (*nbuffers)--;
-               }
-
-       } else {
-               size = common->fmt.fmt.pix.sizeimage;
-       }
+       if (fmt && fmt->fmt.pix.sizeimage < common->fmt.fmt.pix.sizeimage)
+               return -EINVAL;
 
-       if (*nbuffers < config_params.min_numbuffers)
-               *nbuffers = config_params.min_numbuffers;
+       if (vq->num_buffers + *nbuffers < 3)
+               *nbuffers = 3 - vq->num_buffers;
 
        *nplanes = 1;
-       sizes[0] = size;
+       sizes[0] = fmt ? fmt->fmt.pix.sizeimage : common->fmt.fmt.pix.sizeimage;
        alloc_ctxs[0] = common->alloc_ctx;
 
+       /* Calculate the offset for Y and C data in the buffer */
+       vpif_calculate_offsets(ch);
+
        return 0;
 }
 
@@ -214,9 +193,6 @@ static int vpif_start_streaming(struct vb2_queue *vq, unsigned int count)
        ch->field_id = 0;
        common->started = 1;
 
-       /* Calculate the offset for Y and C data in the buffer */
-       vpif_calculate_offsets(ch);
-
        if ((vpif->std_info.frm_fmt &&
            ((common->fmt.fmt.pix.field != V4L2_FIELD_NONE) &&
             (common->fmt.fmt.pix.field != V4L2_FIELD_ANY))) ||