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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <s5hsklknq9c.wl%tiwai@suse.de>
Date:	Wed, 11 Mar 2009 14:27:43 +0100
From:	Takashi Iwai <tiwai@...e.de>
To:	Daniel Gimpelevich <daniel@...pelevich.san-francisco.ca.us>
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-*

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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/