[FFmpeg-cvslog] r26225 - trunk/libavdevice/v4l2.c

[stefano]

Author: stefano
Date: Wed Jan  5 15:15:13 2011
New Revision: 26225

Log:
Factorize common code in v4l2_set_parameters().

Modified:
   trunk/libavdevice/v4l2.c

Modified: trunk/libavdevice/v4l2.c
==============================================================================
--- trunk/libavdevice/v4l2.c	Wed Jan  5 15:15:11 2011	(r26224)
+++ trunk/libavdevice/v4l2.c	Wed Jan  5 15:15:13 2011	(r26225)
@@ -442,8 +442,12 @@ static int v4l2_set_parameters(AVFormatC
     struct video_data *s = s1->priv_data;
     struct v4l2_input input;
     struct v4l2_standard standard;
+    struct v4l2_streamparm streamparm = { 0 };
+    struct v4l2_fract *tpf = &streamparm.parm.capture.timeperframe;
     int i;
 
+    streamparm.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
+
     if (ap->channel>=0) {
         /* set tv video input */
         memset (&input, 0, sizeof (input));
@@ -490,12 +494,8 @@ static int v4l2_set_parameters(AVFormatC
     }
 
     if (ap->time_base.num && ap->time_base.den) {
-        struct v4l2_streamparm streamparm = { 0 };
-        struct v4l2_fract *tpf = &streamparm.parm.capture.timeperframe;
-
         av_log(s1, AV_LOG_DEBUG, "Setting time per frame to %d/%d\n",
                ap->time_base.num, ap->time_base.den);
-        streamparm.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
         tpf->numerator = ap->time_base.num;
         tpf->denominator = ap->time_base.den;
         if (ioctl(s->fd, VIDIOC_S_PARM, &streamparm) != 0) {
@@ -511,23 +511,16 @@ static int v4l2_set_parameters(AVFormatC
                    "The driver changed the time per frame from %d/%d to %d/%d\n",
                    ap->time_base.num, ap->time_base.den,
                    tpf->numerator, tpf->denominator);
-            ap->time_base.num = tpf->numerator;
-            ap->time_base.den = tpf->denominator;
         }
     } else {
-        /* if timebase value is not set in ap, read the timebase value
-         * from the driver and set it in ap */
-        struct v4l2_streamparm streamparm = { 0 };
-        struct v4l2_fract *tpf = &streamparm.parm.capture.timeperframe;
-
-        streamparm.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
+        /* if timebase value is not set in ap, read the timebase value from the driver */
         if (ioctl(s->fd, VIDIOC_G_PARM, &streamparm) != 0) {
             av_log(s1, AV_LOG_ERROR, "ioctl(VIDIOC_G_PARM): %s\n", strerror(errno));
             return AVERROR(errno);
         }
-        ap->time_base.num = tpf->numerator;
-        ap->time_base.den = tpf->denominator;
     }
+    ap->time_base.num = tpf->numerator;
+    ap->time_base.den = tpf->denominator;
 
     return 0;
 }