* intel-hda patch for sigmatel codec volume knob
@ 2007-10-15 7:29 zhejiang
2007-10-15 10:02 ` Maxim Levitsky
2007-10-15 11:27 ` Takashi Iwai
0 siblings, 2 replies; 3+ messages in thread
From: zhejiang @ 2007-10-15 7:29 UTC (permalink / raw)
To: alsa-devel; +Cc: tiwai
Hi,
I found that STAC_VOLKNOB hardwired the KNOB nid to 0x24.
It is okay for stac9205 and stac927x.
But the VolumeKnob nid of stac9220-9221 is 0x16.
diff -Nur a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
--- a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
2007-09-21 16:41:28.000000000 +0800
+++ b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
2007-10-15 15:13:14.000000000 +0800
@@ -351,7 +351,7 @@
static int stac92xx_volknob_get(struct snd_kcontrol *kcontrol,
struct snd_ctl_elem_value *ucontrol)
{
- ucontrol->value.integer.value[0] = kcontrol->private_value;
+ ucontrol->value.integer.value[0] = kcontrol->private_value & 0xFFFF;
return 0;
}
@@ -360,14 +360,15 @@
{
struct hda_codec *codec = snd_kcontrol_chip(kcontrol);
- if (kcontrol->private_value == ucontrol->value.integer.value[0])
+ if ((kcontrol->private_value & 0xFFFF) ==
ucontrol->value.integer.value[0])
return 0;
- kcontrol->private_value = ucontrol->value.integer.value[0];
+ kcontrol->private_value &= 0xFFFF0000;
+ kcontrol->private_value += ucontrol->value.integer.value[0];
- snd_hda_codec_write_cache(codec, 0x24, 0,
+ snd_hda_codec_write_cache(codec, kcontrol->private_value >> 16, 0,
AC_VERB_SET_VOLUME_KNOB_CONTROL,
- kcontrol->private_value | 0x80);
+ (kcontrol->private_value & 0xFFFF) | 0x80);
return 1;
}
@@ -433,7 +434,7 @@
.private_value = verb_read | (verb_write << 16), \
}
-#define STAC_VOLKNOB \
+#define STAC_VOLKNOB(knob_nid) \
{ \
.iface = SNDRV_CTL_ELEM_IFACE_MIXER, \
.name = "Master Playback Volume", \
@@ -441,7 +442,7 @@
.info = stac92xx_volknob_info, \
.get = stac92xx_volknob_get, \
.put = stac92xx_volknob_put, \
- .private_value = 127, \
+ .private_value = 127 | (knob_nid << 16), \
}
@@ -474,7 +475,7 @@
},
STAC_INPUT_SOURCE(2),
STAC_ANALOG_LOOPBACK(0xFE0, 0x7E0),
- STAC_VOLKNOB,
+ STAC_VOLKNOB(0x24),
HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x1b, 0x0, HDA_INPUT),
HDA_CODEC_MUTE_IDX("Capture Switch", 0x0, 0x1d, 0x0, HDA_OUTPUT),
@@ -490,7 +491,7 @@
/* This needs to be generated dynamically based on sequence */
static struct snd_kcontrol_new stac922x_mixer[] = {
STAC_INPUT_SOURCE(2),
- STAC_VOLKNOB,
+ STAC_VOLKNOB(0x16),
HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x17, 0x0, HDA_INPUT),
HDA_CODEC_MUTE_IDX("Capture Switch", 0x0, 0x17, 0x0, HDA_INPUT),
HDA_CODEC_VOLUME_IDX("Mux Capture Volume", 0x0, 0x12, 0x0,
HDA_OUTPUT),
@@ -504,7 +505,7 @@
static struct snd_kcontrol_new stac927x_mixer[] = {
STAC_INPUT_SOURCE(3),
- STAC_VOLKNOB,
+ STAC_VOLKNOB(0x24),
STAC_ANALOG_LOOPBACK(0xFEB, 0x7EB),
HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x18, 0x0, HDA_INPUT),
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: intel-hda patch for sigmatel codec volume knob
2007-10-15 7:29 intel-hda patch for sigmatel codec volume knob zhejiang
@ 2007-10-15 10:02 ` Maxim Levitsky
2007-10-15 11:27 ` Takashi Iwai
1 sibling, 0 replies; 3+ messages in thread
From: Maxim Levitsky @ 2007-10-15 10:02 UTC (permalink / raw)
To: alsa-devel; +Cc: tiwai, zhejiang
On Monday 15 October 2007 09:29:30 zhejiang wrote:
> Hi,
>
> I found that STAC_VOLKNOB hardwired the KNOB nid to 0x24.
> It is okay for stac9205 and stac927x.
> But the VolumeKnob nid of stac9220-9221 is 0x16.
>
>
> diff -Nur a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> --- a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> 2007-09-21 16:41:28.000000000 +0800
> +++ b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> 2007-10-15 15:13:14.000000000 +0800
> @@ -351,7 +351,7 @@
Hi,
Thank you very much, I am the author of patch that added the volume knob
Although I checked carefully the datasheets I missed that one.
Thanks a lot.
Maxim Levitsky
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: intel-hda patch for sigmatel codec volume knob
2007-10-15 7:29 intel-hda patch for sigmatel codec volume knob zhejiang
2007-10-15 10:02 ` Maxim Levitsky
@ 2007-10-15 11:27 ` Takashi Iwai
1 sibling, 0 replies; 3+ messages in thread
From: Takashi Iwai @ 2007-10-15 11:27 UTC (permalink / raw)
To: zhejiang; +Cc: alsa-devel
At Mon, 15 Oct 2007 15:29:30 +0800,
zhejiang wrote:
>
> Hi,
>
> I found that STAC_VOLKNOB hardwired the KNOB nid to 0x24.
> It is okay for stac9205 and stac927x.
> But the VolumeKnob nid of stac9220-9221 is 0x16.
Thanks for report. Since we'll need to release 1.0.15 really soon,
I'll queue the fix up onto my local tree first, then commit it to the
upstream. We might have a fixed release soon later.
Takashi
>
>
> diff -Nur a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> --- a/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> 2007-09-21 16:41:28.000000000 +0800
> +++ b/alsa-driver-1.0.15rc3/alsa-kernel/pci/hda/patch_sigmatel.c
> 2007-10-15 15:13:14.000000000 +0800
> @@ -351,7 +351,7 @@
> static int stac92xx_volknob_get(struct snd_kcontrol *kcontrol,
> struct snd_ctl_elem_value *ucontrol)
> {
> - ucontrol->value.integer.value[0] = kcontrol->private_value;
> + ucontrol->value.integer.value[0] = kcontrol->private_value & 0xFFFF;
> return 0;
> }
>
> @@ -360,14 +360,15 @@
> {
> struct hda_codec *codec = snd_kcontrol_chip(kcontrol);
>
> - if (kcontrol->private_value == ucontrol->value.integer.value[0])
> + if ((kcontrol->private_value & 0xFFFF) ==
> ucontrol->value.integer.value[0])
> return 0;
>
> - kcontrol->private_value = ucontrol->value.integer.value[0];
> + kcontrol->private_value &= 0xFFFF0000;
> + kcontrol->private_value += ucontrol->value.integer.value[0];
>
> - snd_hda_codec_write_cache(codec, 0x24, 0,
> + snd_hda_codec_write_cache(codec, kcontrol->private_value >> 16, 0,
> AC_VERB_SET_VOLUME_KNOB_CONTROL,
> - kcontrol->private_value | 0x80);
> + (kcontrol->private_value & 0xFFFF) | 0x80);
> return 1;
> }
>
> @@ -433,7 +434,7 @@
> .private_value = verb_read | (verb_write << 16), \
> }
>
> -#define STAC_VOLKNOB \
> +#define STAC_VOLKNOB(knob_nid) \
> { \
> .iface = SNDRV_CTL_ELEM_IFACE_MIXER, \
> .name = "Master Playback Volume", \
> @@ -441,7 +442,7 @@
> .info = stac92xx_volknob_info, \
> .get = stac92xx_volknob_get, \
> .put = stac92xx_volknob_put, \
> - .private_value = 127, \
> + .private_value = 127 | (knob_nid << 16), \
> }
>
>
> @@ -474,7 +475,7 @@
> },
> STAC_INPUT_SOURCE(2),
> STAC_ANALOG_LOOPBACK(0xFE0, 0x7E0),
> - STAC_VOLKNOB,
> + STAC_VOLKNOB(0x24),
>
> HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x1b, 0x0, HDA_INPUT),
> HDA_CODEC_MUTE_IDX("Capture Switch", 0x0, 0x1d, 0x0, HDA_OUTPUT),
> @@ -490,7 +491,7 @@
> /* This needs to be generated dynamically based on sequence */
> static struct snd_kcontrol_new stac922x_mixer[] = {
> STAC_INPUT_SOURCE(2),
> - STAC_VOLKNOB,
> + STAC_VOLKNOB(0x16),
> HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x17, 0x0, HDA_INPUT),
> HDA_CODEC_MUTE_IDX("Capture Switch", 0x0, 0x17, 0x0, HDA_INPUT),
> HDA_CODEC_VOLUME_IDX("Mux Capture Volume", 0x0, 0x12, 0x0,
> HDA_OUTPUT),
> @@ -504,7 +505,7 @@
>
> static struct snd_kcontrol_new stac927x_mixer[] = {
> STAC_INPUT_SOURCE(3),
> - STAC_VOLKNOB,
> + STAC_VOLKNOB(0x24),
> STAC_ANALOG_LOOPBACK(0xFEB, 0x7EB),
>
> HDA_CODEC_VOLUME_IDX("Capture Volume", 0x0, 0x18, 0x0, HDA_INPUT),
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-10-15 11:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-15 7:29 intel-hda patch for sigmatel codec volume knob zhejiang
2007-10-15 10:02 ` Maxim Levitsky
2007-10-15 11:27 ` Takashi Iwai
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.