[media] cx88: Fix regression in initial video standard setting
authorDevin Heitmueller <dheitmueller@kernellabs.com>
Sat, 20 Sep 2014 12:23:44 +0000 (09:23 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 6 Jun 2017 10:26:25 +0000 (07:26 -0300)
Setting initial standard at the top of cx8800_initdev would cause the
first call to cx88_set_tvnorm() to return without programming any
registers (leaving the driver saying it's set to NTSC but the hardware
isn't programmed).  Even worse, any subsequent attempt to explicitly
set it to NTSC-M will return success but actually fail to program the
underlying registers unless first changing the standard to something
other than NTSC-M.

Set the initial standard later in the process, and make sure the field
is zero at the beginning to ensure that the call always goes through.

This regression was introduced in the following commit:

commit ccd6f1d488e7 ("[media] cx88: move width, height and field to core
struct")

Author: Hans Verkuil <hans.verkuil@cisco.com>

[media] cx88: move width, height and field to core struct

Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Cc: <stable@vger.kernel.org> # for v3.19 and up
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/pci/cx88/cx88-cards.c
drivers/media/pci/cx88/cx88-video.c

index 73cc7a67a8bce0e82f42448a2306557d7894f599..6df21b29ea17f8dd294394d66cabfd1dff275a8e 100644 (file)
@@ -3681,7 +3681,14 @@ struct cx88_core *cx88_core_create(struct pci_dev *pci, int nr)
        core->nr = nr;
        sprintf(core->name, "cx88[%d]", core->nr);
 
-       core->tvnorm = V4L2_STD_NTSC_M;
+       /*
+        * Note: Setting initial standard here would cause first call to
+        * cx88_set_tvnorm() to return without programming any registers.  Leave
+        * it blank for at this point and it will get set later in
+        * cx8800_initdev()
+        */
+       core->tvnorm  = 0;
+
        core->width   = 320;
        core->height  = 240;
        core->field   = V4L2_FIELD_INTERLACED;
index 36b4fb908e31b377da4f2f7b9b49cefaeb266be2..7d25ecd4404bfa61443117241ed4dfcd8d89e9f1 100644 (file)
@@ -1435,7 +1435,7 @@ static int cx8800_initdev(struct pci_dev *pci_dev,
 
        /* initial device configuration */
        mutex_lock(&core->lock);
-       cx88_set_tvnorm(core, core->tvnorm);
+       cx88_set_tvnorm(core, V4L2_STD_NTSC_M);
        v4l2_ctrl_handler_setup(&core->video_hdl);
        v4l2_ctrl_handler_setup(&core->audio_hdl);
        cx88_video_mux(core, 0);