From mboxrd@z Thu Jan  1 00:00:00 1970
From: Takashi Iwai <tiwai@suse.de>
Subject: Re: [PATCH] ALSA: HDA: Pinfix quirk for HP Z200
	Workstation
Date: Thu, 23 Jun 2011 14:15:11 +0200
Message-ID: <s5h62nwzquo.wl%tiwai@suse.de>
References: <4E02510B.5090404@canonical.com> <s5hd3i5ypjw.wl%tiwai@suse.de>
	<4E02F929.40407@canonical.com>
Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <alsa-devel-bounces@alsa-project.org>
Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15])
	by alsa0.perex.cz (Postfix) with ESMTP id EE994103846
	for <alsa-devel@alsa-project.org>;
	Thu, 23 Jun 2011 14:15:13 +0200 (CEST)
In-Reply-To: <4E02F929.40407@canonical.com>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
	<mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Sender: alsa-devel-bounces@alsa-project.org
Errors-To: alsa-devel-bounces@alsa-project.org
To: David Henningsson <david.henningsson@canonical.com>
Cc: ALSA Development Mailing List <alsa-devel@alsa-project.org>
List-Id: alsa-devel@alsa-project.org

At Thu, 23 Jun 2011 10:28:25 +0200,
David Henningsson wrote:
> 
> On 2011-06-23 09:28, Takashi Iwai wrote:
> > At Wed, 22 Jun 2011 22:31:07 +0200,
> > David Henningsson wrote:
> >>
> >> BIOS lists the internal speaker as an internal line-out. Change to
> >> internal speaker + model=auto for better auto-mute capabilities.
> >>
> >> Relevant alsa-info is here:
> >> https://bugs.launchpad.net/ubuntu/+source/alsa-driver/+bug/754964/+attachment/2002204/+files/alsa-info.txt
> >
> > Thanks, applied now.
> >
> > 3.0 should work even without this patch since line-out can be
> > auto-muted now, but it's still more correct to show the speaker as
> > speaker (e.g. the right mixer name).  So, it's still good to have
> > this kind of fix.
> 
> I think it was not muted at all, because it is a mono pin, so it does 
> not show up in the autocfg->line_out_pins array. (Perhaps this should 
> change in the auto-mute behaviour?)

Ah, right, these have the real line-out, too.
(The auto-mute can handle mono as well, but before that, the mono pin
 has to be parsed properly :)

> Also, the user wanted the real line out to mute the speaker.

Yes, so the patch does the right thing.


thanks,

Takashi