5 files changed, 26 insertions, 14 deletions

diff --git a/drivers/media/video/cx18/cx18-driver.h b/drivers/media/video/cx18/cx18-driver.h
index 183420723060..b9a94fc5146d 100644
--- a/drivers/media/video/cx18/cx18-driver.h
+++ b/drivers/media/video/cx18/cx18-driver.h
@@ -409,6 +409,7 @@ struct cx18_stream {
 
         /* Videobuf for YUV video */
         u32 pixelformat;
+        u32 vb_bytes_per_frame;
         struct list_head vb_capture;    /* video capture queue */
         spinlock_t vb_lock;
         struct timer_list vb_timeout;
@@ -430,10 +431,6 @@ struct cx18_open_id {
         u32 open_id;
         int type;
         struct cx18 *cx;
-
-        struct videobuf_queue vbuf_q;
-        spinlock_t s_lock; /* Protect vbuf_q */
-        enum v4l2_buf_type vb_type;
 };
 
 static inline struct cx18_open_id *fh2id(struct v4l2_fh *fh)
diff --git a/drivers/media/video/cx18/cx18-fileops.c b/drivers/media/video/cx18/cx18-fileops.c
index 07411f34885a..14cb961c22bd 100644
--- a/drivers/media/video/cx18/cx18-fileops.c
+++ b/drivers/media/video/cx18/cx18-fileops.c
@@ -784,8 +784,6 @@ int cx18_v4l2_close(struct file *filp)
                 cx18_release_stream(s);
         } else {
                 cx18_stop_capture(id, 0);
-                if (id->type == CX18_ENC_STREAM_TYPE_YUV)
-                        videobuf_mmap_free(&id->vbuf_q);
         }
         kfree(id);
         mutex_unlock(&cx->serialize_lock);
diff --git a/drivers/media/video/cx18/cx18-ioctl.c b/drivers/media/video/cx18/cx18-ioctl.c
index afe0a29e7200..66b1c15c3541 100644
--- a/drivers/media/video/cx18/cx18-ioctl.c
+++ b/drivers/media/video/cx18/cx18-ioctl.c
@@ -160,12 +160,7 @@ static int cx18_g_fmt_vid_cap(struct file *file, void *fh,
         pixfmt->priv = 0;
         if (id->type == CX18_ENC_STREAM_TYPE_YUV) {
                 pixfmt->pixelformat = s->pixelformat;
-                /* HM12 YUV size is (Y=(h*720) + UV=(h*(720/2)))
-                   UYUV YUV size is (Y=(h*720) + UV=(h*(720))) */
-                if (s->pixelformat == V4L2_PIX_FMT_HM12)
-                        pixfmt->sizeimage = pixfmt->height * 720 * 3 / 2;
-                else
-                        pixfmt->sizeimage = pixfmt->height * 720 * 2;
+                pixfmt->sizeimage = s->vb_bytes_per_frame;
                 pixfmt->bytesperline = 720;
         } else {
                 pixfmt->pixelformat = V4L2_PIX_FMT_MPEG;
@@ -296,6 +291,12 @@ static int cx18_s_fmt_vid_cap(struct file *file, void *fh,
                 return -EBUSY;
 
         s->pixelformat = fmt->fmt.pix.pixelformat;
+        /* HM12 YUV size is (Y=(h*720) + UV=(h*(720/2)))
+           UYUV YUV size is (Y=(h*720) + UV=(h*(720))) */
+        if (s->pixelformat == V4L2_PIX_FMT_HM12)
+                s->vb_bytes_per_frame = h * 720 * 3 / 2;
+        else
+                s->vb_bytes_per_frame = h * 720 * 2;
 
         mbus_fmt.width = cx->cxhdl.width = w;
         mbus_fmt.height = cx->cxhdl.height = h;
@@ -463,13 +464,16 @@ static int cx18_s_register(struct file *file, void *fh,
 static int cx18_querycap(struct file *file, void *fh,
                                 struct v4l2_capability *vcap)
 {
-        struct cx18 *cx = fh2id(fh)->cx;
+        struct cx18_open_id *id = fh2id(fh);
+        struct cx18 *cx = id->cx;
 
         strlcpy(vcap->driver, CX18_DRIVER_NAME, sizeof(vcap->driver));
         strlcpy(vcap->card, cx->card_name, sizeof(vcap->card));
         snprintf(vcap->bus_info, sizeof(vcap->bus_info),
                  "PCI:%s", pci_name(cx->pci_dev));
         vcap->capabilities = cx->v4l2_cap;          /* capabilities */
+        if (id->type == CX18_ENC_STREAM_TYPE_YUV)
+                vcap->capabilities |= V4L2_CAP_STREAMING;
         return 0;
 }
 
diff --git a/drivers/media/video/cx18/cx18-mailbox.c b/drivers/media/video/cx18/cx18-mailbox.c
index c07191e09fcb..0c7796e76ac0 100644
--- a/drivers/media/video/cx18/cx18-mailbox.c
+++ b/drivers/media/video/cx18/cx18-mailbox.c
@@ -196,7 +196,7 @@ static void cx18_mdl_send_to_videobuf(struct cx18_stream *s,
         }
 
         /* If we've filled the buffer as per the callers res then dispatch it */
-        if (vb_buf->bytes_used >= (vb_buf->vb.width * vb_buf->vb.height * 2)) {
+        if (vb_buf->bytes_used >= s->vb_bytes_per_frame) {
                 dispatch = 1;
                 vb_buf->bytes_used = 0;
         }
diff --git a/drivers/media/video/cx18/cx18-streams.c b/drivers/media/video/cx18/cx18-streams.c
index 852f420fd271..638cca156b58 100644
--- a/drivers/media/video/cx18/cx18-streams.c
+++ b/drivers/media/video/cx18/cx18-streams.c
@@ -138,6 +138,12 @@ static int cx18_prepare_buffer(struct videobuf_queue *q,
                 buf->tvnorm    = cx->std;
                 s->pixelformat = pixelformat;
 
+                /* HM12 YUV size is (Y=(h*720) + UV=(h*(720/2)))
+                   UYUV YUV size is (Y=(h*720) + UV=(h*(720))) */
+                if (s->pixelformat == V4L2_PIX_FMT_HM12)
+                        s->vb_bytes_per_frame = height * 720 * 3 / 2;
+                else
+                        s->vb_bytes_per_frame = height * 720 * 2;
                 cx18_dma_free(q, s, buf);
         }
 
@@ -154,6 +160,12 @@ static int cx18_prepare_buffer(struct videobuf_queue *q,
                 buf->tvnorm    = cx->std;
                 s->pixelformat = pixelformat;
 
+                /* HM12 YUV size is (Y=(h*720) + UV=(h*(720/2)))
+                   UYUV YUV size is (Y=(h*720) + UV=(h*(720))) */
+                if (s->pixelformat == V4L2_PIX_FMT_HM12)
+                        s->vb_bytes_per_frame = height * 720 * 3 / 2;
+                else
+                        s->vb_bytes_per_frame = height * 720 * 2;
                 rc = videobuf_iolock(q, &buf->vb, NULL);
                 if (rc != 0)
                         goto fail;
@@ -287,6 +299,7 @@ static void cx18_stream_init(struct cx18 *cx, int type)
 
                 /* Assume the previous pixel default */
                 s->pixelformat = V4L2_PIX_FMT_HM12;
+                s->vb_bytes_per_frame = cx->cxhdl.height * 720 * 3 / 2;
         }
 }