]> Pileus Git - ~andy/linux/commitdiff
[media] bttv: audio_mux(): use a local variable "gpio_mute" instead of modifying...
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Thu, 21 Mar 2013 16:51:13 +0000 (13:51 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 23 Mar 2013 14:29:10 +0000 (11:29 -0300)
Function audio_mux() actually deals with two types of mute: gpio mute and
subdevice muting.
This patch claryfies the meaning of these values, but mainly prepares the code for
the next patch.

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/pci/bt8xx/bttv-driver.c

index 8610b6a8b8fb1be69cee70ac8804036a99b1fdc1..a584d82ab90900d6d0d95e1b20360a9a0db4e6ab 100644 (file)
@@ -992,7 +992,7 @@ static char *audio_modes[] = {
 static int
 audio_mux(struct bttv *btv, int input, int mute)
 {
-       int gpio_val, signal;
+       int gpio_val, signal, mute_gpio;
        struct v4l2_ctrl *ctrl;
 
        gpio_inout(bttv_tvcards[btv->c.type].gpiomask,
@@ -1003,10 +1003,10 @@ audio_mux(struct bttv *btv, int input, int mute)
        btv->audio = input;
 
        /* automute */
-       mute = mute || (btv->opt_automute && (!signal || !btv->users)
+       mute_gpio = mute || (btv->opt_automute && (!signal || !btv->users)
                                && !btv->has_radio_tuner);
 
-       if (mute)
+       if (mute_gpio)
                gpio_val = bttv_tvcards[btv->c.type].gpiomute;
        else
                gpio_val = bttv_tvcards[btv->c.type].gpiomux[input];
@@ -1022,7 +1022,7 @@ audio_mux(struct bttv *btv, int input, int mute)
        }
 
        if (bttv_gpio)
-               bttv_gpio_tracking(btv, audio_modes[mute ? 4 : input]);
+               bttv_gpio_tracking(btv, audio_modes[mute_gpio ? 4 : input]);
        if (in_interrupt())
                return 0;