Pavel Hofman wrote:
> Hi,
>
> Following your discussion on moving low-level controls from MIXER to PCM
> types, I would like to suggest the following two patches:
>
> diff --git a/sound/pci/ice1712/ice1724.c b/sound/pci/ice1712/ice1724.c
> index 3896fb9..28ca8a4 100644
> --- a/sound/pci/ice1712/ice1724.c
> +++ b/sound/pci/ice1712/ice1724.c
> @@ -2160,7 +2160,7 @@ static int snd_vt1724_pro_peak_get(struct
> snd_kcontrol *kcontrol,
> }
>
> static struct snd_kcontrol_new snd_vt1724_mixer_pro_peak __devinitdata = {
> - .iface = SNDRV_CTL_ELEM_IFACE_MIXER,
> + .iface = SNDRV_CTL_ELEM_IFACE_PCM,
> .name = "Multi Track Peak",
> .access = SNDRV_CTL_ELEM_ACCESS_READ |
> SNDRV_CTL_ELEM_ACCESS_VOLATILE,
> .info = snd_vt1724_pro_peak_info,
>
>
> diff --git a/envy24control/levelmeters.c b/envy24control/levelmeters.c
> index 3bf1a94..9bdaace 100644
> --- a/envy24control/levelmeters.c
> +++ b/envy24control/levelmeters.c
> @@ -262,7 +262,7 @@ void level_meters_reset_peaks(GtkButton *button,
> gpointer da
> void level_meters_init(void)
> {
> snd_ctl_elem_value_malloc(&peaks);
> - snd_ctl_elem_value_set_interface(peaks, SND_CTL_ELEM_IFACE_MIXER);
> + snd_ctl_elem_value_set_interface(peaks, SND_CTL_ELEM_IFACE_PCM);
> snd_ctl_elem_value_set_name(peaks, "Multi Track Peak");
> }
>
>
>
>
> The point is these controls are read-only, display incorrectly in
> alsamixer, and are very confusing for majority of users who do not read
> envy24 datasheet and the source code.
>
> Thanks for considering this IMO useful change.
>
> Regards,
>
Sorry for a mistake - envy24control is used for ice1712 only, i.e. the
change should happen in ice1712.c too. No further implications for
change in ice1724.c.
Should you approve the change, I will send proper patch emails.
Thanks,
Pavel
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
