[media] au0828: fix initial video routing
authorHans Verkuil <hans.verkuil@cisco.com>
Mon, 11 Mar 2013 19:18:31 +0000 (16:18 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 25 Mar 2013 18:10:35 +0000 (15:10 -0300)
After loading the module the initial video routing is not setup.
Explicitly call s_input to get this right.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Reviewed-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/usb/au0828/au0828-video.c

index 6f2a6264d17319b53453d995835f0414d95d3148..a90a0b9771b37b263b3ce0a561999df0b0b593e7 100644 (file)
@@ -1391,20 +1391,10 @@ static int vidioc_g_input(struct file *file, void *priv, unsigned int *i)
        return 0;
 }
 
-static int vidioc_s_input(struct file *file, void *priv, unsigned int index)
+static void au0828_s_input(struct au0828_dev *dev, int index)
 {
-       struct au0828_fh *fh = priv;
-       struct au0828_dev *dev = fh->dev;
        int i;
 
-       dprintk(1, "VIDIOC_S_INPUT in function %s, input=%d\n", __func__,
-               index);
-       if (index >= AU0828_MAX_INPUT)
-               return -EINVAL;
-       if (AUVI_INPUT(index).type == 0)
-               return -EINVAL;
-       dev->ctrl_input = index;
-
        switch (AUVI_INPUT(index).type) {
        case AU0828_VMUX_SVIDEO:
                dev->input_type = AU0828_VMUX_SVIDEO;
@@ -1419,7 +1409,7 @@ static int vidioc_s_input(struct file *file, void *priv, unsigned int index)
                dev->ctrl_ainput = 0;
                break;
        default:
-               dprintk(1, "VIDIOC_S_INPUT unknown input type set [%d]\n",
+               dprintk(1, "unknown input type set [%d]\n",
                        AUVI_INPUT(index).type);
                break;
        }
@@ -1450,6 +1440,21 @@ static int vidioc_s_input(struct file *file, void *priv, unsigned int index)
 
        v4l2_device_call_all(&dev->v4l2_dev, 0, audio, s_routing,
                        AUVI_INPUT(index).amux, 0, 0);
+}
+
+static int vidioc_s_input(struct file *file, void *priv, unsigned int index)
+{
+       struct au0828_fh *fh = priv;
+       struct au0828_dev *dev = fh->dev;
+
+       dprintk(1, "VIDIOC_S_INPUT in function %s, input=%d\n", __func__,
+               index);
+       if (index >= AU0828_MAX_INPUT)
+               return -EINVAL;
+       if (AUVI_INPUT(index).type == 0)
+               return -EINVAL;
+       dev->ctrl_input = index;
+       au0828_s_input(dev, index);
        return 0;
 }
 
@@ -1982,6 +1987,7 @@ int au0828_analog_register(struct au0828_dev *dev,
        dev->ctrl_ainput = 0;
        dev->ctrl_freq = 960;
        dev->std = V4L2_STD_NTSC_M;
+       au0828_s_input(dev, 0);
 
        /* allocate and fill v4l2 video struct */
        dev->vdev = video_device_alloc();