[patch] v4l: videobuf update

From: Gerd Knorr (kraxel@bytesex.org)
Date: Wed Apr 02 2003 - 11:36:47 EST


  Hi,

This is a minor update for the video-buf module. It does
export the videobuf_next_field symbol and adds a few debug
printks.

  Gerd

diff -u linux-2.5.66/drivers/media/video/video-buf.c linux/drivers/media/video/video-buf.c
--- linux-2.5.66/drivers/media/video/video-buf.c 2003-04-02 11:42:51.957723625 +0200
+++ linux/drivers/media/video/video-buf.c 2003-04-02 11:49:35.894628822 +0200
@@ -16,6 +16,7 @@
  * (at your option) any later version.
  */
 
+#include <linux/version.h>
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/vmalloc.h>
@@ -362,21 +363,33 @@
 {
         int i;
         
- if (q->reading)
+ if (q->streaming) {
+ dprintk(1,"busy: streaming active\n");
                 return 1;
- if (q->streaming)
+ }
+ if (q->reading) {
+ dprintk(1,"busy: pending read #1\n");
                 return 1;
- if (q->read_buf)
+ }
+ if (q->read_buf) {
+ dprintk(1,"busy: pending read #2\n");
                 return 1;
+ }
         for (i = 0; i < VIDEO_MAX_FRAME; i++) {
                 if (NULL == q->bufs[i])
                         continue;
- if (q->bufs[i]->map)
+ if (q->bufs[i]->map) {
+ dprintk(1,"busy: buffer #%d mapped\n",i);
                         return 1;
- if (q->bufs[i]->state == STATE_QUEUED)
+ }
+ if (q->bufs[i]->state == STATE_QUEUED) {
+ dprintk(1,"busy: buffer #%d queued\n",i);
                         return 1;
- if (q->bufs[i]->state == STATE_ACTIVE)
+ }
+ if (q->bufs[i]->state == STATE_ACTIVE) {
+ dprintk(1,"busy: buffer #%d active\n",i);
                         return 1;
+ }
         }
         return 0;
 }
@@ -569,7 +582,7 @@
         if (list_empty(&q->stream))
                 goto done;
         buf = list_entry(q->stream.next, struct videobuf_buffer, stream);
- retval = videobuf_waiton(buf,1,1);
+ retval = videobuf_waiton(buf, file->f_flags & O_NONBLOCK, 1);
         if (retval < 0)
                 goto done;
         switch (buf->state) {
@@ -925,6 +938,9 @@
 videobuf_vm_open(struct vm_area_struct *vma)
 {
         struct videobuf_mapping *map = vma->vm_private_data;
+
+ dprintk(2,"vm_open %p [count=%d,vma=%08lx-%08lx]\n",map,
+ map->count,vma->vm_start,vma->vm_end);
         map->count++;
 }
 
@@ -934,6 +950,9 @@
         struct videobuf_mapping *map = vma->vm_private_data;
         int i;
 
+ dprintk(2,"vm_close %p [count=%d,vma=%08lx-%08lx]\n",map,
+ map->count,vma->vm_start,vma->vm_end);
+
         /* down(&fh->lock); FIXME */
         map->count--;
         if (0 == map->count) {
@@ -1081,11 +1100,11 @@
                 q->bufs[i]->map = map;
                 q->bufs[i]->baddr = vma->vm_start + size;
         }
- map->count = 1;
- map->start = vma->vm_start;
- map->end = vma->vm_end;
- map->q = q;
- vma->vm_ops = &videobuf_vm_ops;
+ map->count = 1;
+ map->start = vma->vm_start;
+ map->end = vma->vm_end;
+ map->q = q;
+ vma->vm_ops = &videobuf_vm_ops;
         vma->vm_flags |= VM_DONTEXPAND;
         vma->vm_flags &= ~VM_IO; /* using shared anonymous pages */
         vma->vm_private_data = map;
@@ -1119,6 +1138,7 @@
 EXPORT_SYMBOL_GPL(videobuf_queue_cancel);
 EXPORT_SYMBOL_GPL(videobuf_queue_is_busy);
 
+EXPORT_SYMBOL_GPL(videobuf_next_field);
 EXPORT_SYMBOL_GPL(videobuf_status);
 EXPORT_SYMBOL_GPL(videobuf_reqbufs);
 EXPORT_SYMBOL_GPL(videobuf_querybuf);
diff -u linux-2.5.66/include/media/video-buf.h linux/include/media/video-buf.h
--- linux-2.5.66/include/media/video-buf.h 2003-04-02 11:42:55.103208272 +0200
+++ linux/include/media/video-buf.h 2003-04-02 11:49:35.895628670 +0200
@@ -196,6 +196,7 @@
 int videobuf_queue_is_busy(struct videobuf_queue *q);
 void videobuf_queue_cancel(struct file *file, struct videobuf_queue *q);
 
+enum v4l2_field videobuf_next_field(struct videobuf_queue *q);
 void videobuf_status(struct v4l2_buffer *b, struct videobuf_buffer *vb,
                      enum v4l2_buf_type type);
 int videobuf_reqbufs(struct file *file, struct videobuf_queue *q,

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



This archive was generated by hypermail 2b29 : Mon Apr 07 2003 - 22:00:16 EST