At Fri, 16 Sep 2011 11:22:01 +0200,
David Henningsson wrote:
>> [1 <text/plain; ISO-8859-1 (7bit)>]
> On 09/16/2011 10:31 AM, Takashi Iwai wrote:
> > At Fri, 16 Sep 2011 10:01:48 +0200,
> > David Henningsson wrote:
> >>
> >> The ~6 months old functionality for configurable automuting is broken
> >> for the case that the user has only HP and LO (no speakers) - basically
> >> there is an "Automute mode" control, but enabling it does nothing. While
> >> fixing that, I also took the liberty of refactoring/rewriting some of
> >> that code, in order to make it easier to understand and maintain.
> >>
> >> Takashi, given you approve/like these changes, I'll go ahead and fix up
> >> the quirk files to fit the new variable names.
> >
> > I'm fine with renames but let's split patches: first fix the bug
> > of non-working auto-mute control, then refactoring. The former
> > should go to 3.1 while the latter is queued to 3.2.
>> Ok. If it also should go to 3.0, feel free to add cc: stable at kernel.org> (hmm, is that working even though kernel.org is down?)
>> Here comes the 3.1 part of the patch. If you're happy with it, commit it
> and I'll continue with the 3.2 renames.
Hm, it's more complicated than I wanted.
Could you have an example codec proc or alsa-info.sh file to simulate
here? The logic is already there, so a few lines of corrections
should suffice to fix the behavior.
thanks,
Takashi