Split function audio_mux():
move the mute setting part to function audio_mute() and the input setting part
to function audio_input().

Signed-off-by: Frank Schäfer <fschaefer....@googlemail.com>
Acked-by: Hans Verkuil <hans.verk...@cisco.com>
---
 drivers/media/pci/bt8xx/bttv-driver.c |   51 ++++++++++++++++-----------------
 1 Datei geändert, 24 Zeilen hinzugefügt(+), 27 Zeilen entfernt(-)

diff --git a/drivers/media/pci/bt8xx/bttv-driver.c 
b/drivers/media/pci/bt8xx/bttv-driver.c
index f1cb0db..0df4a16 100644
--- a/drivers/media/pci/bt8xx/bttv-driver.c
+++ b/drivers/media/pci/bt8xx/bttv-driver.c
@@ -1022,18 +1022,37 @@ audio_mux_gpio(struct bttv *btv, int input, int mute)
 }
 
 static int
-audio_mux(struct bttv *btv, int input, int mute)
+audio_mute(struct bttv *btv, int mute)
 {
        struct v4l2_ctrl *ctrl;
 
-       audio_mux_gpio(btv, input, mute);
+       audio_mux_gpio(btv, btv->audio_input, mute);
 
        if (btv->sd_msp34xx) {
-               u32 in;
-
                ctrl = v4l2_ctrl_find(btv->sd_msp34xx->ctrl_handler, 
V4L2_CID_AUDIO_MUTE);
                if (ctrl)
                        v4l2_ctrl_s_ctrl(ctrl, mute);
+       }
+       if (btv->sd_tvaudio) {
+               ctrl = v4l2_ctrl_find(btv->sd_tvaudio->ctrl_handler, 
V4L2_CID_AUDIO_MUTE);
+               if (ctrl)
+                       v4l2_ctrl_s_ctrl(ctrl, mute);
+       }
+       if (btv->sd_tda7432) {
+               ctrl = v4l2_ctrl_find(btv->sd_tda7432->ctrl_handler, 
V4L2_CID_AUDIO_MUTE);
+               if (ctrl)
+                       v4l2_ctrl_s_ctrl(ctrl, mute);
+       }
+       return 0;
+}
+
+static int
+audio_input(struct bttv *btv, int input)
+{
+       audio_mux_gpio(btv, input, btv->mute);
+
+       if (btv->sd_msp34xx) {
+               u32 in;
 
                /* Note: the inputs tuner/radio/extern/intern are translated
                   to msp routings. This assumes common behavior for all msp3400
@@ -1079,34 +1098,12 @@ audio_mux(struct bttv *btv, int input, int mute)
                               in, MSP_OUTPUT_DEFAULT, 0);
        }
        if (btv->sd_tvaudio) {
-               ctrl = v4l2_ctrl_find(btv->sd_tvaudio->ctrl_handler, 
V4L2_CID_AUDIO_MUTE);
-
-               if (ctrl)
-                       v4l2_ctrl_s_ctrl(ctrl, mute);
                v4l2_subdev_call(btv->sd_tvaudio, audio, s_routing,
-                               input, 0, 0);
-       }
-       if (btv->sd_tda7432) {
-               ctrl = v4l2_ctrl_find(btv->sd_tda7432->ctrl_handler, 
V4L2_CID_AUDIO_MUTE);
-
-               if (ctrl)
-                       v4l2_ctrl_s_ctrl(ctrl, mute);
+                                input, 0, 0);
        }
        return 0;
 }
 
-static inline int
-audio_mute(struct bttv *btv, int mute)
-{
-       return audio_mux(btv, btv->audio_input, mute);
-}
-
-static inline int
-audio_input(struct bttv *btv, int input)
-{
-       return audio_mux(btv, input, btv->mute);
-}
-
 static void
 bttv_crop_calc_limits(struct bttv_crop *c)
 {
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to