Date:	Wed, 11 Mar 2009 06:53:34 -0700
From:	Daniel Gimpelevich <daniel@...pelevich.san-francisco.ca.us>
To:	Takashi Iwai <tiwai@...e.de>
Cc:	"Rafael J. Wysocki" <rjw@...k.pl>, linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] Re: ALSA HDA Intel stoped to work in 2.6.16-*

BTW, Rafael: The reason I originally sent this to the main LKML and not
to Takashi through other channels was that I was replying directly to a
thread already on only this list. I do acknowledge that this list was
not the proper venue for the thread when it was begun, three years ago.

On Wed, 2009-03-11 at 14:27 +0100, Takashi Iwai wrote:
> 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.

Perhaps; I was referring only to the issue of model=basic being chosen
as a lowest common denominator amidst lack of other info. The newest
kernel I have actually tested on that hardware so far is the one on the
Fedora 10 LiveCD.

> > > > 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.

As the OP stated, there is no sound output whatsoever with model=auto.
The controls in alsamixer in that case do not exactly match any of the
existing models, and I currently intend to attach a three-way comparison
of them among model=basic, model=will, and model=auto.

> 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

Unfortunately, the machine has since suffered a hardware failure, which
I will attempt to correct this week. Further testing must wait until
then.

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

Powered by blists - more mailing lists