All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Henningsson <david.henningsson@canonical.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH] ALSA: HDA: Fix front mic on Dell	Precision M6500
Date: Fri, 03 Sep 2010 05:43:57 +0200	[thread overview]
Message-ID: <4C806EFD.8030106@canonical.com> (raw)
In-Reply-To: <s5hpqwzm9kz.wl%tiwai@suse.de>

2010-08-31 11:10, Takashi Iwai skrev:
> At Tue, 31 Aug 2010 11:01:40 +0200,
> David Henningsson wrote:
>>
>> 2010-08-30 18:58, Takashi Iwai skrev:
>>> At Thu, 26 Aug 2010 12:22:42 +0200,
>>> I wrote:
>>>>
>>>> At Mon, 23 Aug 2010 21:25:55 +0200,
>>>> David Henningsson wrote:
>>>>>
>>>>> I admit that the best thing would be to make the driver support all
>>>>> three mics, but at this point, the alsa driver can only support two, one
>>>>> called "mic" and one called "front mic". I believe the best choice is to
>>>>> select the internal mic as "front mic" and the mic jack on the laptop as
>>>>> "mic", thus leaving the docking station mic jack non-functional.
>>>>>
>>>>> The behavior without this patch is to select the mic jack on the laptop
>>>>> as "mic" and the mic on the docking station as "front mic", thus leaving
>>>>> the internal mic non-functional in all cases, and that regardless of you
>>>>> have a docking station or not.
>>>>
>>>> OK, here is a test patch for fixing more intensively.  (I just did
>>>> build-test, totally untested with real machines :)
>>>>
>>>> It basically extends the config parser to allow multiple input pins
>>>> assigned for a single type.  The old input_pins field is kept for
>>>> other drivers, so that it won't give regressions.  It'll be removed
>>>> in future.
>>>
>>> As my tests with hda-emu through all codec data passed well, I
>>> committed the fixes to sound git tree now.
>>> Give the latest alsa-driver snapshot try.
>>
>> Thanks, I'll try to get someone to test it :-)
> 
> Great, thanks.

Got a response saying: "Working, had to select microphone 2."
https://launchpad.net/bugs/519066

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

      reply	other threads:[~2010-09-03  3:43 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-23 19:03 [PATCH] ALSA: HDA: Fix front mic on Dell Precision M6500 David Henningsson
2010-08-23 19:07 ` Jerone Young
2010-08-23 19:25   ` David Henningsson
2010-08-24  0:57     ` Raymond Yau
2010-08-24  6:04       ` David Henningsson
2010-08-24 13:20         ` Raymond Yau
2010-08-26 10:20     ` Takashi Iwai
2010-08-26 10:22     ` Takashi Iwai
2010-08-30 16:58       ` Takashi Iwai
2010-08-31  9:01         ` David Henningsson
2010-08-31  9:10           ` Takashi Iwai
2010-09-03  3:43             ` David Henningsson [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C806EFD.8030106@canonical.com \
    --to=david.henningsson@canonical.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.