ALSA: hda - Refactor init_extra_out() in hda_generic.c
authorTakashi Iwai <tiwai@suse.de>
Thu, 20 Dec 2012 14:27:24 +0000 (15:27 +0100)
committerTakashi Iwai <tiwai@suse.de>
Sat, 12 Jan 2013 07:34:16 +0000 (08:34 +0100)
Just a small clean up by splitting a function.
No functional changes at all.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_generic.c

index b7b8d7eee7c6b6b421906dee3d72bdbf26613ecc..96c779b6ca25ad6cd3ffc78151554c4961deff22 100644 (file)
@@ -3389,45 +3389,44 @@ static void init_multi_out(struct hda_codec *codec)
        }
 }
 
-/* initialize hp and speaker paths */
-static void init_extra_out(struct hda_codec *codec)
+
+static void __init_extra_out(struct hda_codec *codec, int num_outs,
+                            hda_nid_t *pins, hda_nid_t *dacs, int type)
 {
        struct hda_gen_spec *spec = codec->spec;
        int i;
        hda_nid_t pin, dac;
 
-       for (i = 0; i < spec->autocfg.hp_outs; i++) {
-               if (spec->autocfg.line_out_type == AUTO_PIN_HP_OUT)
-                       break;
-               pin = spec->autocfg.hp_pins[i];
-               if (!pin)
-                       break;
-               dac = spec->multiout.hp_out_nid[i];
-               if (!dac) {
-                       if (i > 0 && spec->multiout.hp_out_nid[0])
-                               dac = spec->multiout.hp_out_nid[0];
-                       else
-                               dac = spec->multiout.dac_nids[0];
-               }
-               set_output_and_unmute(codec, pin, PIN_HP, dac);
-       }
-       for (i = 0; i < spec->autocfg.speaker_outs; i++) {
-               if (spec->autocfg.line_out_type == AUTO_PIN_SPEAKER_OUT)
-                       break;
-               pin = spec->autocfg.speaker_pins[i];
+       for (i = 0; i < num_outs; i++) {
+               pin = pins[i];
                if (!pin)
                        break;
-               dac = spec->multiout.extra_out_nid[i];
+               dac = dacs[i];
                if (!dac) {
-                       if (i > 0 && spec->multiout.extra_out_nid[0])
-                               dac = spec->multiout.extra_out_nid[0];
+                       if (i > 0 && dacs[0])
+                               dac = dacs[0];
                        else
                                dac = spec->multiout.dac_nids[0];
                }
-               set_output_and_unmute(codec, pin, PIN_OUT, dac);
+               set_output_and_unmute(codec, pin, type, dac);
        }
 }
 
+/* initialize hp and speaker paths */
+static void init_extra_out(struct hda_codec *codec)
+{
+       struct hda_gen_spec *spec = codec->spec;
+
+       if (spec->autocfg.line_out_type != AUTO_PIN_HP_OUT)
+               __init_extra_out(codec, spec->autocfg.hp_outs,
+                                spec->autocfg.hp_pins,
+                                spec->multiout.hp_out_nid, PIN_HP);
+       if (spec->autocfg.line_out_type != AUTO_PIN_SPEAKER_OUT)
+               __init_extra_out(codec, spec->autocfg.speaker_outs,
+                                spec->autocfg.speaker_pins,
+                                spec->multiout.extra_out_nid, PIN_OUT);
+}
+
 /* initialize multi-io paths */
 static void init_multi_io(struct hda_codec *codec)
 {