Skip to contentSkip to conversation

ALSA: VIA HDA: Get codec's type in via_new_spec() function

PATCH In review Lydia Wang 2 participants 2 messages Mar 22, 2011 Sound

lore
Share & tools
lore

Lydia Wang

author
#0 lore raw

PATCH 7/8 ALSA: VIA HDA: Get codec's type in via_new_spec() function

From: Lydia Wang <[email protected]>
Subject: ALSA: VIA HDA: Get codec's type in via_new_spec() function.

Add get_codec_type() in via_new_spec() function to make sure getting correct codec type before building mixer controls.

---
 sound/pci/hda/patch_via.c |    9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)
sound/pci/hda/patch_via.c +5 -4
@@ -162,6 +162,7 @@
 #endif
 };
 
+static enum VIA_HDA_CODEC get_codec_type(struct hda_codec *codec);
 static struct via_spec * via_new_spec(struct hda_codec *codec)
 {
 	struct via_spec *spec;
@@ -172,6 +173,10 @@
 
 	codec->spec = spec;
 	spec->codec = codec;
+	spec->codec_type = get_codec_type(codec);
+	/* VT1708BCE & VT1708S are almost same */
+	if (spec->codec_type == VT1708BCE)
+		spec->codec_type = VT1708S;
 	return spec;
 }
 
@@ -1735,10 +1740,6 @@
 	for (i = 0; i < spec->num_iverbs; i++)
 		snd_hda_sequence_write(codec, spec->init_verbs[i]);
 
-	spec->codec_type = get_codec_type(codec);
-	if (spec->codec_type == VT1708BCE)
-		spec->codec_type = VT1708S; /* VT1708BCE & VT1708S are almost
-					       same */
 	/* Lydia Add for EAPD enable */
 	if (!spec->dig_in_nid) { /* No Digital In connection */
 		if (spec->dig_in_pin) {
#1 lore raw

Lydia

Quoted 4 lines · Lydia Wang

spec->codec_type·=·get_codec_type(codec);

»       /*·VT1708BCE·&·VT1708S·are·almost·same·*/
»       if·(spec->codec_type·==·VT1708BCE)
»       »       spec->codec_type·=·VT1708S;

Why you need to do this.Are you doing any specific work related to VT1708S. I think in call cases your codec_type is VT1708S.

Best Rgds
Rajeev