Re: [PATCH] Re: ALSA HDA Intel stoped to work in 2.6.16-*

From: Takashi Iwai
Date: Wed Mar 11 2009 - 09:27:57 EST


At Wed, 11 Mar 2009 06:07:20 -0700,
Daniel Gimpelevich wrote:
>
> (Re-adding mistakenly snipped CC'sâ)
>
> On Wed, 2009-03-11 at 09:33 +0100, Takashi Iwai wrote:
> > At Sun, 8 Mar 2009 23:21:13 +0000 (UTC),
> > Daniel Gimpelevich wrote:
> > >
> > > Takashi Iwai <tiwai <at> suse.de> writes:
> > >
> > > > Daniel, how "incorrect" do you mean exactly?
> > > >
> > > > As you cited, the above patch was added for the request for the
> > > > specific model, so the patch itself is correct per definition. What
> > > > wrong could be the choice of the model option by the original poster,
> > > > which I cannot judge.
> > > >
> > > > Of course I have no objection to fix the model entry at all, but I
> > > > need a more proper justification.
> > >
> > > The master volume control appeared to be affecting the wrong control line,
> >
> > Is it so with the latest 2.6.29 kernel?
> > (Also you aren't accessing pulse plugin, right?)
>
> Was using alsamixer to test, without pulseaudio running. The underlying
> issue is unchanged in three years of commits.

No, the master volume behavior did change recently.

Doesn't it really work with model=basic on 2.6.29?
Its master should change both the widget 0x08 and 0x09, so it should
influence on the volume.


> > > and
> > > there was no way to turn off IEC958, which appeared to be on by default.
> >
> > This should work.
> >
> > % amixer -Dhw:0 set IEC958 mute
>
> It does work, but only after the patch I submitted (or by using the
> equivalent module argument).

Ah, you mean there is no IEC958 mixer as is, right? Then yes, there
is no control with model=basic.

But, I basically wonder whether model=auto works or not.
Choosing an existing model for a device of another vendor is often
wrong in small corner cases.

Anyway, it'd be helpful if you attach the output of alsa-info.sh (with
--no-upload option) on your device. The script is found at
http://www.alsa-project.org/alsa-info.sh


thanks,

Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/