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

From: Takashi Iwai <tiwai@suse.de>
To: Daniel Gimpelevich <daniel@gimpelevich.san-francisco.ca.us>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>,
	linux-kernel@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] Re: ALSA HDA Intel stoped to work in 2.6.16-*
Date: Wed, 11 Mar 2009 14:27:43 +0100	[thread overview]
Message-ID: <s5hsklknq9c.wl%tiwai@suse.de> (raw)
In-Reply-To: <1236776849.4624.37.camel@the-omnibook>

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