[alsa-devel] New code for conexant chipsets in -mm tree

[Tobin Davis]

The other thing to check is that you are not specifying a "model="
parameter in the modprobe.conf file.  The module should be allowed to
autodetect your system.  Your previous comment about commenting out the
"spec->init_verbs[1] = cxt5045_mic_sense_init_verbs;" line indicates
that your system is using the "model=fujitsu" option for some reason.


The diff below was actually a test for a user, and is not intended to be
added to the main driver, sorry.

Tobin

On Tue, 2007-03-27 at 14:33 -0300, Rodrigo Luiz wrote:

> This file in your home page is more recent in that -mm tree.
> 
> However, the changes is just for cxt5047 chipsets.
> 
> But I tried to load the hda-intel with the last one patch_conexant.c and 
> I have the same issue.
> 
> 
> Rodrigo.
> 
> 
> The diff:
> 
> --- sound/pci/hda/patch_conexant.c      2007-03-27 13:21:30.000000000 -0300
> +++ patch_conexant.c    2007-03-27 14:10:08.000000000 -0300
> @@ -978,22 +978,10 @@
>          HDA_CODEC_MUTE("PCM Switch", 0x10, 0x00, HDA_OUTPUT),
>          HDA_CODEC_VOLUME("PCM-2 Volume", 0x1c, 0x00, HDA_OUTPUT),
>          HDA_CODEC_MUTE("PCM-2 Switch", 0x1c, 0x00, HDA_OUTPUT),
> -       {
> -               .iface = SNDRV_CTL_ELEM_IFACE_MIXER,
> -               .name = "Master Playback Volume",
> -               .info = snd_hda_mixer_amp_volume_info,
> -               .get = snd_hda_mixer_amp_volume_get,
> -               .put = cxt5047_hp_master_vol_put,
> -               .private_value = HDA_COMPOSE_AMP_VAL(0x13, 3, 0, 
> HDA_OUTPUT),
> -       },
> -       {
> -               .iface = SNDRV_CTL_ELEM_IFACE_MIXER,
> -               .name = "Master Playback Switch",
> -               .info = cxt_eapd_info,
> -               .get = cxt_eapd_get,
> -               .put = cxt5047_hp_master_sw_put,
> -               .private_value = 0x13,
> -       },
> +       HDA_CODEC_VOLUME("Speaker Volume", 0x1d, 0x00, HDA_OUTPUT),
> +       HDA_CODEC_MUTE("Speaker Switch", 0x1d, 0x00, HDA_OUTPUT),
> +       HDA_CODEC_VOLUME("Headphone Volume", 0x13, 0x00, HDA_OUTPUT),
> +       HDA_CODEC_MUTE("Headphone Switch", 0x13, 0x00, HDA_OUTPUT),
> 
>          {}
>   };
> 
> 
> 
> Tobin Davis wrote:
> > I'm not entirely sure that the code you have is the latest.  Could you 
> > compare your patch_conexant.c with this one:  
> > http://members.dsl-only.net/~tdavis/patch_conexant.c
> > 
> > This is my latest version (it's also in the alsa hg tree).
> > 
> > Tobin
> > 
> 

-- 
Tobin Davis <tdavis at dsl-only.net>