[media] cx231xx: do not call video_mux as part of isoc setup
authorDevin Heitmueller <dheitmueller@hauppauge.com>
Mon, 12 Jul 2010 18:34:57 +0000 (15:34 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 21 Oct 2010 03:17:23 +0000 (01:17 -0200)
You cannot call the video_mux routine from within the isoc setup, because that
code is shared with the digital isoc handler.  This was causing the GPIOs and
power control to be put into the wrong state when starting up digital mode.

Signed-off-by: Devin Heitmueller <dheitmueller@hauppauge.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx231xx/cx231xx-core.c
drivers/media/video/cx231xx/cx231xx-video.c

index 7beb201..4fc62c4 100644 (file)
@@ -1002,12 +1002,6 @@ int cx231xx_init_isoc(struct cx231xx *dev, int max_packets,
        int j, k;
        int rc;
 
-       dev->video_input = dev->video_input > 2 ? 2 : dev->video_input;
-
-       cx231xx_coredbg("Setting Video mux to %d\n", dev->video_input);
-
-       video_mux(dev, dev->video_input);
-
        /* De-allocates all pending stuff */
        cx231xx_uninit_isoc(dev);
 
index 03a94e6..06f492e 100644 (file)
@@ -2667,7 +2667,9 @@ int cx231xx_register_analog_devices(struct cx231xx *dev)
 
        /* Analog specific initialization */
        dev->format = &format[0];
-       /* video_mux(dev, dev->video_input); */
+
+       /* Set the initial input */
+       video_mux(dev, dev->video_input);
 
        /* Audio defaults */
        dev->mute = 1;