[media] em28xx: use common urb data copying function for vbi and non-vbi data streams
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Sun, 25 Nov 2012 09:37:37 +0000 (06:37 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 22 Dec 2012 21:17:37 +0000 (19:17 -0200)
em28xx_urb_data_copy_vbi() is actually an extended version of
em28xx_urb_data_copy(). With the preceding fixes and improvements, it works
fine with both, vbi and non-vbi data streams without performance impacts.
So rename em28xx_urb_data_copy_vbi() to em28xx_urb_data_copy(), delete the
the old implementation of em28xx_urb_data_copy() and change the code to use
this function for both data stream types.
Tested with "SilverCrest 1.3 MPix webcam" (progressive, non-vbi) and
"Hauppauge HVR-900 (65008/A1C0)" (interlaced, vbi enabled and disabled).

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/usb/em28xx/em28xx-video.c
drivers/media/usb/em28xx/em28xx.h

index c397aa213601eca124dce1dc529974cedba002fb..a1436a453bc895d9d2b60b469ce8a365a49d86ec 100644 (file)
@@ -376,7 +376,6 @@ static inline void get_next_buf(struct em28xx_dmaqueue *dma_q,
 
        /* Get the next buffer */
        *buf = list_entry(dma_q->active.next, struct em28xx_buffer, vb.queue);
-
        /* Cleans up buffer - Useful for testing for frame/URB loss */
        outp = videobuf_to_vmalloc(&(*buf)->vb);
        memset(outp, 0, (*buf)->vb.size);
@@ -413,114 +412,8 @@ static inline void vbi_get_next_buf(struct em28xx_dmaqueue *dma_q,
        return;
 }
 
-/* Processes and copies the URB data content to a frame buffer queue */
+/* Processes and copies the URB data content (video and VBI data) */
 static inline int em28xx_urb_data_copy(struct em28xx *dev, struct urb *urb)
-{
-       struct em28xx_buffer    *buf;
-       struct em28xx_dmaqueue  *dma_q = &dev->vidq;
-       int xfer_bulk, num_packets, i, rc = 1;
-       unsigned int actual_length, len = 0;
-       unsigned char *p, *outp = NULL;
-
-       if (!dev)
-               return 0;
-
-       if ((dev->state & DEV_DISCONNECTED) || (dev->state & DEV_MISCONFIGURED))
-               return 0;
-
-       if (urb->status < 0)
-               print_err_status(dev, -1, urb->status);
-
-       xfer_bulk = usb_pipebulk(urb->pipe);
-
-       buf = dev->usb_ctl.vid_buf;
-       if (buf != NULL)
-               outp = videobuf_to_vmalloc(&buf->vb);
-
-       if (xfer_bulk) /* bulk */
-               num_packets = 1;
-       else /* isoc */
-               num_packets = urb->number_of_packets;
-
-       for (i = 0; i < num_packets; i++) {
-               if (xfer_bulk) { /* bulk */
-                       actual_length = urb->actual_length;
-
-                       p = urb->transfer_buffer;
-               } else { /* isoc */
-                       if (urb->iso_frame_desc[i].status < 0) {
-                               print_err_status(dev, i,
-                                                urb->iso_frame_desc[i].status);
-                               if (urb->iso_frame_desc[i].status != -EPROTO)
-                                       continue;
-                       }
-
-                       actual_length = urb->iso_frame_desc[i].actual_length;
-                       if (actual_length > dev->max_pkt_size) {
-                               em28xx_isocdbg("packet bigger than packet size");
-                               continue;
-                       }
-
-                       p = urb->transfer_buffer +
-                           urb->iso_frame_desc[i].offset;
-               }
-
-               if (actual_length <= 0) {
-                       /* NOTE: happens very often with isoc transfers */
-                       /* em28xx_usbdbg("packet %d is empty",i); - spammy */
-                       continue;
-               }
-
-               /* FIXME: incomplete buffer checks where removed to make
-                  logic simpler. Impacts of those changes should be evaluated
-                */
-               if (p[0] == 0x33 && p[1] == 0x95 && p[2] == 0x00) {
-                       em28xx_isocdbg("VBI HEADER!!!\n");
-                       /* FIXME: Should add vbi copy */
-                       continue;
-               }
-               if (p[0] == 0x22 && p[1] == 0x5a) {
-                       em28xx_isocdbg("Video frame %d, length=%i, %s\n", p[2],
-                                      len, (p[2] & 1) ? "odd" : "even");
-
-                       if (dev->progressive || !(p[2] & 1)) {
-                               if (buf != NULL)
-                                       buffer_filled(dev, dma_q, buf);
-                               get_next_buf(dma_q, &buf);
-                               if (buf == NULL)
-                                       outp = NULL;
-                               else
-                                       outp = videobuf_to_vmalloc(&buf->vb);
-                       }
-
-                       if (buf != NULL) {
-                               if (p[2] & 1)
-                                       buf->top_field = 0;
-                               else
-                                       buf->top_field = 1;
-                       }
-
-                       dma_q->pos = 0;
-               }
-               if (buf != NULL) {
-                       if (p[0] != 0x88 && p[0] != 0x22) {
-                               /* NOTE: no intermediate data packet header
-                                * 88 88 88 88 when using bulk transfers */
-                               em28xx_isocdbg("frame is not complete\n");
-                               len = actual_length;
-                       } else {
-                               len = actual_length - 4;
-                               p += 4;
-                       }
-                       em28xx_copy_video(dev, dma_q, buf, p, outp, len);
-               }
-       }
-       return rc;
-}
-
-/* Version of the urb data handler that takes into account a mixture of
-   video and VBI data */
-static inline int em28xx_urb_data_copy_vbi(struct em28xx *dev, struct urb *urb)
 {
        struct em28xx_buffer    *buf, *vbi_buf;
        struct em28xx_dmaqueue  *dma_q = &dev->vidq;
@@ -767,20 +660,12 @@ buffer_prepare(struct videobuf_queue *vq, struct videobuf_buffer *vb,
 
        if (urb_init) {
                dev->capture_type = -1;
-               if (em28xx_vbi_supported(dev) == 1)
-                       rc = em28xx_init_usb_xfer(dev, EM28XX_ANALOG_MODE,
-                                                 dev->analog_xfer_bulk,
-                                                 EM28XX_NUM_BUFS,
-                                                 dev->max_pkt_size,
-                                                 dev->packet_multiplier,
-                                                 em28xx_urb_data_copy_vbi);
-               else
-                       rc = em28xx_init_usb_xfer(dev, EM28XX_ANALOG_MODE,
-                                                 dev->analog_xfer_bulk,
-                                                 EM28XX_NUM_BUFS,
-                                                 dev->max_pkt_size,
-                                                 dev->packet_multiplier,
-                                                 em28xx_urb_data_copy);
+               rc = em28xx_init_usb_xfer(dev, EM28XX_ANALOG_MODE,
+                                         dev->analog_xfer_bulk,
+                                         EM28XX_NUM_BUFS,
+                                         dev->max_pkt_size,
+                                         dev->packet_multiplier,
+                                         em28xx_urb_data_copy);
                if (rc < 0)
                        goto fail;
        }
index 09df56a4570c6e716f8d2ce09129d340519cb513..304896de4879a102d02f887d37a5e1e9d91a7481 100644 (file)
@@ -560,10 +560,10 @@ struct em28xx {
        /* states */
        enum em28xx_dev_state state;
 
-       /* vbi related state tracking */
+       /* capture state tracking */
        int capture_type;
-       int vbi_read;
        unsigned char top_field:1;
+       int vbi_read;
        unsigned int vbi_width;
        unsigned int vbi_height; /* lines per field */