media: avoid use of 'videobuf'
authorHans Verkuil <hverkuil-cisco@xs4all.nl>
Wed, 10 Aug 2022 13:01:27 +0000 (15:01 +0200)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Mon, 29 Aug 2022 14:27:18 +0000 (16:27 +0200)
The term 'videobuf' typically refers to the old videobuf version 1
framework. Avoid using this word in drivers that are converted to
vb2.

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
16 files changed:
Documentation/driver-api/media/drivers/pxa_camera.rst
drivers/media/dvb-frontends/rtl2832_sdr.c
drivers/media/pci/solo6x10/solo6x10-v4l2-enc.c
drivers/media/platform/intel/pxa_camera.c
drivers/media/platform/marvell/mcam-core.h
drivers/media/platform/renesas/vsp1/vsp1_video.c
drivers/media/platform/samsung/exynos4-is/fimc-core.h
drivers/media/platform/samsung/s5p-mfc/s5p_mfc.c
drivers/media/test-drivers/vim2m.c
drivers/media/usb/airspy/airspy.c
drivers/media/usb/au0828/au0828-video.c
drivers/media/usb/cx231xx/cx231xx-vbi.c
drivers/media/usb/cx231xx/cx231xx-video.c
drivers/media/usb/em28xx/em28xx-video.c
drivers/media/usb/msi2500/msi2500.c
drivers/media/usb/pvrusb2/pvrusb2-dvb.c

index ee1bd96..4691991 100644 (file)
@@ -19,7 +19,7 @@ Global video workflow
 
 a) QCI stopped
    Initially, the QCI interface is stopped.
-   When a buffer is queued (pxa_videobuf_ops->buf_queue), the QCI starts.
+   When a buffer is queued, start_streaming is called and the QCI starts.
 
 b) QCI started
    More buffers can be queued while the QCI is started without halting the
index 6a4f299..05f71d1 100644 (file)
@@ -245,7 +245,7 @@ static void rtl2832_sdr_urb_complete(struct urb *urb)
                if (unlikely(fbuf == NULL)) {
                        dev->vb_full++;
                        dev_notice_ratelimited(&pdev->dev,
-                                              "videobuf is full, %d packets dropped\n",
+                                              "video buffer is full, %d packets dropped\n",
                                               dev->vb_full);
                        goto skip;
                }
index 80d20e2..0adf3d8 100644 (file)
@@ -1020,7 +1020,7 @@ static int solo_g_parm(struct file *file, void *priv,
        cp->timeperframe.numerator = solo_enc->interval;
        cp->timeperframe.denominator = solo_enc->solo_dev->fps;
        cp->capturemode = 0;
-       /* XXX: Shouldn't we be able to get/set this from videobuf? */
+       /* XXX: Shouldn't we be able to get/set this from vb2? */
        cp->readbuffers = 2;
 
        return 0;
index 35145e3..54270d6 100644 (file)
@@ -854,7 +854,7 @@ fail:
        return -ENOMEM;
 }
 
-static void pxa_videobuf_set_actdma(struct pxa_camera_dev *pcdev,
+static void pxa_video_buf_set_actdma(struct pxa_camera_dev *pcdev,
                                    struct pxa_buffer *buf)
 {
        buf->active_dma = DMA_Y;
@@ -973,7 +973,7 @@ static void pxa_camera_wakeup(struct pxa_camera_dev *pcdev,
  * stopped. This means the tailed buffer would never be transferred by DMA.
  * This function restarts the capture for this corner case, where :
  *  - DADR() == DADDR_STOP
- *  - a videobuffer is queued on the pcdev->capture list
+ *  - a video buffer is queued on the pcdev->capture list
  *
  * Please check the "DMA hot chaining timeslice issue" in
  *   Documentation/driver-api/media/drivers/pxa_camera.rst
@@ -1163,7 +1163,7 @@ static void pxa_camera_eof(struct tasklet_struct *t)
        pcdev->active = list_first_entry(&pcdev->capture,
                                         struct pxa_buffer, queue);
        buf = pcdev->active;
-       pxa_videobuf_set_actdma(pcdev, buf);
+       pxa_video_buf_set_actdma(pcdev, buf);
 
        pxa_dma_start_channels(pcdev);
 }
@@ -1416,7 +1416,7 @@ static int pxac_vb2_prepare(struct vb2_buffer *vb)
         * the actual buffer is yours
         */
        buf->inwork = 0;
-       pxa_videobuf_set_actdma(pcdev, buf);
+       pxa_video_buf_set_actdma(pcdev, buf);
 
        return ret;
 }
index f324d80..51e66db 100644 (file)
@@ -32,7 +32,7 @@
 
 #if !defined(MCAM_MODE_VMALLOC) && !defined(MCAM_MODE_DMA_CONTIG) && \
        !defined(MCAM_MODE_DMA_SG)
-#error One of the videobuf buffer modes must be selected in the config
+#error One of the vb2 buffer modes must be selected in the config
 #endif
 
 
index e8e0ee5..df1606b 100644 (file)
@@ -305,7 +305,7 @@ static int vsp1_video_pipeline_setup_partitions(struct vsp1_pipeline *pipe)
  * @video: the video node
  *
  * This function completes the current buffer by filling its sequence number,
- * time stamp and payload size, and hands it back to the videobuf core.
+ * time stamp and payload size, and hands it back to the vb2 core.
  *
  * Return the next queued buffer or NULL if the queue is empty.
  */
index 7a058f3..2b0760a 100644 (file)
@@ -215,7 +215,7 @@ struct fimc_addr {
 
 /**
  * struct fimc_vid_buffer - the driver's video buffer
- * @vb:    v4l videobuf buffer
+ * @vb:    v4l vb2 buffer
  * @list:  linked list structure for buffer queue
  * @addr: precalculated DMA address set
  * @index: buffer index for the output DMA engine
index 7613419..219fc02 100644 (file)
@@ -323,7 +323,7 @@ static void s5p_mfc_handle_frame_new(struct s5p_mfc_ctx *ctx, unsigned int err)
        }
        ctx->sequence++;
        /* The MFC returns address of the buffer, now we have to
-        * check which videobuf does it correspond to */
+        * check which vb2_buffer does it correspond to */
        list_for_each_entry(dst_buf, &ctx->dst_queue, list) {
                u32 addr = (u32)vb2_dma_contig_plane_dma_addr(&dst_buf->b->vb2_buf, 0);
 
index 4757549..7964426 100644 (file)
@@ -2,7 +2,7 @@
 /*
  * A virtual v4l2-mem2mem example device.
  *
- * This is a virtual device driver for testing mem-to-mem videobuf framework.
+ * This is a virtual device driver for testing mem-to-mem vb2 framework.
  * It simulates a device that uses memory buffers for both source and
  * destination, processes the data and issues an "irq" (simulated by a delayed
  * workqueue).
index 240a7cc..b8b8824 100644 (file)
@@ -294,7 +294,7 @@ static void airspy_urb_complete(struct urb *urb)
                if (unlikely(fbuf == NULL)) {
                        s->vb_full++;
                        dev_notice_ratelimited(s->dev,
-                                       "videobuf is full, %d packets dropped\n",
+                                       "video buffer is full, %d packets dropped\n",
                                        s->vb_full);
                        goto skip;
                }
index c0f1185..eb303e9 100644 (file)
@@ -384,7 +384,7 @@ static void au0828_copy_video(struct au0828_dev *dev,
 }
 
 /*
- * video-buf generic routine to get the next available buffer
+ * generic routine to get the next available buffer
  */
 static inline void get_next_buf(struct au0828_dmaqueue *dma_q,
                                struct au0828_buffer **buf)
@@ -459,7 +459,7 @@ static void au0828_copy_vbi(struct au0828_dev *dev,
 
 
 /*
- * video-buf generic routine to get the next available VBI buffer
+ * generic routine to get the next available VBI buffer
  */
 static inline void vbi_get_next_buf(struct au0828_dmaqueue *dma_q,
                                    struct au0828_buffer **buf)
index fdc8b7f..33431d9 100644 (file)
@@ -558,7 +558,7 @@ u32 cx231xx_copy_vbi_line(struct cx231xx *dev, struct cx231xx_dmaqueue *dma_q,
 }
 
 /*
- * video-buf generic routine to get the next available buffer
+ * generic routine to get the next available buffer
  */
 static inline void get_next_vbi_buf(struct cx231xx_dmaqueue *dma_q,
                                    struct cx231xx_buffer **buf)
index 425e470..e23b8cc 100644 (file)
@@ -220,7 +220,7 @@ static inline void print_err_status(struct cx231xx *dev, int packet, int status)
 }
 
 /*
- * video-buf generic routine to get the next available buffer
+ * generic routine to get the next available buffer
  */
 static inline void get_next_buf(struct cx231xx_dmaqueue *dma_q,
                                struct cx231xx_buffer **buf)
index 8181c0e..25e0620 100644 (file)
@@ -440,7 +440,7 @@ static inline void finish_buffer(struct em28xx *dev,
 }
 
 /*
- * Copy picture data from USB buffer to videobuf buffer
+ * Copy picture data from USB buffer to video buffer
  */
 static void em28xx_copy_video(struct em28xx *dev,
                              struct em28xx_buffer *buf,
@@ -521,7 +521,7 @@ static void em28xx_copy_video(struct em28xx *dev,
 }
 
 /*
- * Copy VBI data from USB buffer to videobuf buffer
+ * Copy VBI data from USB buffer to video buffer
  */
 static void em28xx_copy_vbi(struct em28xx *dev,
                            struct em28xx_buffer *buf,
index 71de6b4..5a1f269 100644 (file)
@@ -411,7 +411,7 @@ static void msi2500_isoc_handler(struct urb *urb)
                if (unlikely(fbuf == NULL)) {
                        dev->vb_full++;
                        dev_dbg_ratelimited(dev->dev,
-                                           "videobuf is full, %d packets dropped\n",
+                                           "video buffer is full, %d packets dropped\n",
                                            dev->vb_full);
                        continue;
                }
index 6954584..26811ef 100644 (file)
@@ -80,7 +80,7 @@ static int pvr2_dvb_feed_func(struct pvr2_dvb_adapter *adap)
 static int pvr2_dvb_feed_thread(void *data)
 {
        int stat = pvr2_dvb_feed_func(data);
-       /* from videobuf-dvb.c: */
+
        while (!kthread_should_stop()) {
                set_current_state(TASK_INTERRUPTIBLE);
                schedule();