All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] hda: fix incorrect mixer index values for 92hd83xx
@ 2008-12-20 22:47 Matthew Ranostay
  2008-12-20 22:53 ` Takashi Iwai
  0 siblings, 1 reply; 3+ messages in thread
From: Matthew Ranostay @ 2008-12-20 22:47 UTC (permalink / raw)
  To: alsa-devel; +Cc: Takashi Iwai

Fixed incorrect mixer index values for 92hd83xx codec's audio
input mixer.

Signed-off-by: Matthew Ranostay <mranostay@embeddedalley.com>
---

diff --git a/pci/hda/patch_sigmatel.c b/pci/hda/patch_sigmatel.c
index 70181d5..efa1fb0 100644
--- a/pci/hda/patch_sigmatel.c
+++ b/pci/hda/patch_sigmatel.c
@@ -1101,21 +1101,21 @@ static struct snd_kcontrol_new stac92hd83xxx_mixer[] = {
 	HDA_CODEC_VOLUME_IDX("Capture Volume", 0x1, 0x18, 0x0, HDA_OUTPUT),
 	HDA_CODEC_MUTE_IDX("Capture Switch", 0x1, 0x18, 0x0, HDA_OUTPUT),
 
-	HDA_CODEC_VOLUME("DAC0 Capture Volume", 0x1b, 0, HDA_INPUT),
-	HDA_CODEC_MUTE("DAC0 Capture Switch", 0x1b, 0, HDA_INPUT),
+	HDA_CODEC_VOLUME("DAC0 Capture Volume", 0x1b, 0x3, HDA_INPUT),
+	HDA_CODEC_MUTE("DAC0 Capture Switch", 0x1b, 0x3, HDA_INPUT),
 
-	HDA_CODEC_VOLUME("DAC1 Capture Volume", 0x1b, 0x1, HDA_INPUT),
-	HDA_CODEC_MUTE("DAC1 Capture Switch", 0x1b, 0x1, HDA_INPUT),
+	HDA_CODEC_VOLUME("DAC1 Capture Volume", 0x1b, 0x4, HDA_INPUT),
+	HDA_CODEC_MUTE("DAC1 Capture Switch", 0x1b, 0x4, HDA_INPUT),
 
-	HDA_CODEC_VOLUME("Front Mic Capture Volume", 0x1b, 0x2, HDA_INPUT),
-	HDA_CODEC_MUTE("Front Mic Capture Switch", 0x1b, 0x2, HDA_INPUT),
+	HDA_CODEC_VOLUME("Front Mic Capture Volume", 0x1b, 0x0, HDA_INPUT),
+	HDA_CODEC_MUTE("Front Mic Capture Switch", 0x1b, 0x0, HDA_INPUT),
 
-	HDA_CODEC_VOLUME("Line In Capture Volume", 0x1b, 0x3, HDA_INPUT),
-	HDA_CODEC_MUTE("Line In Capture Switch", 0x1b, 0x3, HDA_INPUT),
+	HDA_CODEC_VOLUME("Line In Capture Volume", 0x1b, 0x2, HDA_INPUT),
+	HDA_CODEC_MUTE("Line In Capture Switch", 0x1b, 0x2, HDA_INPUT),
 
 	/*
-	HDA_CODEC_VOLUME("Mic Capture Volume", 0x1b, 0x4, HDA_INPUT),
-	HDA_CODEC_MUTE("Mic Capture Switch", 0x1b 0x4, HDA_INPUT),
+	HDA_CODEC_VOLUME("Mic Capture Volume", 0x1b, 0x1, HDA_INPUT),
+	HDA_CODEC_MUTE("Mic Capture Switch", 0x1b 0x1, HDA_INPUT),
 	*/
 	{ } /* end */
 };

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] hda: fix incorrect mixer index values for 92hd83xx
  2008-12-20 22:47 [PATCH] hda: fix incorrect mixer index values for 92hd83xx Matthew Ranostay
@ 2008-12-20 22:53 ` Takashi Iwai
  2008-12-20 22:56   ` Matthew Ranostay
  0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2008-12-20 22:53 UTC (permalink / raw)
  To: Matthew Ranostay; +Cc: alsa-devel

At Sat, 20 Dec 2008 17:47:24 -0500,
Matthew Ranostay wrote:
> 
> Fixed incorrect mixer index values for 92hd83xx codec's audio
> input mixer.
> 
> Signed-off-by: Matthew Ranostay <mranostay@embeddedalley.com>

Applied this one, too.

Regarding this change: what I've been concerned by the recent
patch_sigmatel.c are that the input routes are fixed, and capture
mixer elements are often too messy and not intuitive.

Hopefully we can sort out these later...


Thanks,

Takashi

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] hda: fix incorrect mixer index values for 92hd83xx
  2008-12-20 22:53 ` Takashi Iwai
@ 2008-12-20 22:56   ` Matthew Ranostay
  0 siblings, 0 replies; 3+ messages in thread
From: Matthew Ranostay @ 2008-12-20 22:56 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: alsa-devel

Takashi Iwai wrote:
> At Sat, 20 Dec 2008 17:47:24 -0500,
> Matthew Ranostay wrote:
>> Fixed incorrect mixer index values for 92hd83xx codec's audio
>> input mixer.
>>
>> Signed-off-by: Matthew Ranostay <mranostay@embeddedalley.com>
> 
> Applied this one, too.
> 
> Regarding this change: what I've been concerned by the recent
> patch_sigmatel.c are that the input routes are fixed, and capture
> mixer elements are often too messy and not intuitive.
> 
Yeah, there has to be better way to do this.

> Hopefully we can sort out these later...
> 
> 
> Thanks,
> 
> Takashi
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-12-20 22:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-20 22:47 [PATCH] hda: fix incorrect mixer index values for 92hd83xx Matthew Ranostay
2008-12-20 22:53 ` Takashi Iwai
2008-12-20 22:56   ` Matthew Ranostay

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.