ALSA: hda - Give more comments to hda_gen_spec flags
authorTakashi Iwai <tiwai@suse.de>
Wed, 16 Jan 2013 17:28:38 +0000 (18:28 +0100)
committerTakashi Iwai <tiwai@suse.de>
Wed, 16 Jan 2013 17:28:38 +0000 (18:28 +0100)
Since we have many bit flags in hda_gen_spec, rearrange in sections
and give more comments there.

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

index 59d08c60ac47fe815240cd04045cf773d6a09aa4..dd0818b8963bac45b0105a94869b9fc8d456f5dd 100644 (file)
@@ -164,27 +164,34 @@ struct hda_gen_spec {
        struct automic_entry am_entry[MAX_AUTO_MIC_PINS];
 
        /* for pin sensing */
+       /* current status; set in hda_geneic.c */
        unsigned int hp_jack_present:1;
        unsigned int line_jack_present:1;
-       unsigned int master_mute:1;
+       unsigned int speaker_muted:1; /* current status of speaker mute */
+       unsigned int line_out_muted:1; /* current status of LO mute */
+
+       /* internal states of automute / autoswitch behavior */
        unsigned int auto_mic:1;
        unsigned int automute_speaker:1; /* automute speaker outputs */
        unsigned int automute_lo:1; /* automute LO outputs */
-       unsigned int speaker_muted:1; /* current status of speaker mute */
-       unsigned int line_out_muted:1; /* current status of LO mute */
+
+       /* capabilities detected by parser */
        unsigned int detect_hp:1;       /* Headphone detection enabled */
        unsigned int detect_lo:1;       /* Line-out detection enabled */
        unsigned int automute_speaker_possible:1; /* there are speakers and either LO or HP */
        unsigned int automute_lo_possible:1;      /* there are line outs and HP */
+
+       /* additional parameters set by codec drivers */
+       unsigned int master_mute:1;     /* master mute over all */
        unsigned int keep_vref_in_automute:1; /* Don't clear VREF in automute */
+       unsigned int line_in_auto_switch:1; /* allow line-in auto switch */
+
+       /* parser behavior flags; set before snd_hda_gen_parse_auto_config() */
        unsigned int suppress_auto_mute:1; /* suppress input jack auto mute */
        unsigned int suppress_auto_mic:1; /* suppress input jack auto switch */
-       unsigned int line_in_auto_switch:1; /* allow line-in auto switch */
 
-       /* other flags */
+       /* other parse behavior flags */
        unsigned int need_dac_fix:1; /* need to limit DACs for multi channels */
-       unsigned int no_analog:1; /* digital I/O only */
-       unsigned int dyn_adc_switch:1; /* switch ADCs (for ALC275) */
        unsigned int shared_mic_hp:1; /* HP/Mic-in sharing */
        unsigned int no_primary_hp:1; /* Don't prefer HP pins to speaker pins */
        unsigned int multi_cap_vol:1; /* allow multiple capture xxx volumes */
@@ -192,11 +199,15 @@ struct hda_gen_spec {
        unsigned int own_eapd_ctl:1; /* set EAPD by own function */
        unsigned int vmaster_mute_enum:1; /* add vmaster mute mode enum */
        unsigned int indep_hp:1; /* independent HP supported */
-       unsigned int indep_hp_enabled:1; /* independent HP enabled */
        unsigned int prefer_hp_amp:1; /* enable HP amp for speaker if any */
        unsigned int add_stereo_mix_input:1; /* add aamix as a capture src */
        unsigned int add_out_jack_modes:1; /* add output jack mode enum ctls */
 
+       /* other internal flags */
+       unsigned int no_analog:1; /* digital I/O only */
+       unsigned int dyn_adc_switch:1; /* switch ADCs (for ALC275) */
+       unsigned int indep_hp_enabled:1; /* independent HP enabled */
+
        /* loopback mixing mode */
        bool aamix_mode;