alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: David Henningsson <david.henningsson@canonical.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org, "Dâniel Fraga" <fragabr@gmail.com>
Subject: Re: Realtek ACL892: audio gap ~ each 10 seconds? [SOLVED]
Date: Fri, 11 May 2012 07:25:12 -0700	[thread overview]
Message-ID: <4FAD2148.4080701@canonical.com> (raw)
In-Reply-To: <s5h1umqakkg.wl%tiwai@suse.de>

2012-05-11 06:55, Takashi Iwai skrev:
> At Fri, 11 May 2012 06:45:30 -0700,
> David Henningsson wrote:
>> 2012-05-11 00:43, Takashi Iwai skrev:
>>> At Fri, 11 May 2012 04:01:23 -0300,
>>> Dâniel Fraga wrote:
>>>> On Fri, 11 May 2012 07:39:46 +0200
>>>> Takashi Iwai<tiwai@suse.de>   wrote:
>>>>
>>>>> Ah, it's the case...  Yes, some mobos are badly designed not to handle
>>>>> the jack detection properly.
>>>>>
>>>>> Did the auto-mute actually work on your machine?  If not, we can
>>>>> blacklist the device.  Please give alsa-info.sh output.
>>>> 	Hi Takashi! Disabling the auto-mute solved the issue. Regarding
>>>> your question: in fact, I don't know why auto-mute was enabled, because
>>>> I never use a headphone.
>>> It's a driver feature that is enabled as default, no matter whether
>>> you use or not :)
>>>
>>>> So I can't test it.
>>> Does it mean that you have no headphone, or does the machine have no
>>> headphone jack?
>>>
>>>> But you can be sure that
>>>> with this motherboard (Asus P8Z68-V Pro/gen3) there's this "pop and
>>>> click" problem when Auto-mute is enabled.
>>> That's why I'm asking to test.  Usually when such a noise occurs due
>>> to the auto-mute feature, it's because the bogus unsolicited events
>>> are fired up too much although no jack is plugged actually.  Thus
>>> usually the auto-mute feature itself doesn't work in such a case
>>> (either no hardware implementation or the hardware has its own
>>>    switching mechanism.)
>> In the long term, I think we should filter out short jack sense events.
>> That is, when we get an unsol jack event, check the current status and
>> set a timer for 200 ms.
>> 200 ms later, in the timer callback, we would only accept the jack sense
>> event as valid (and do automute/autoswitch/kcontrol update) if
>>    - The current status is the same as 200 ms earlier
>>    - If there has not been more unsol events in the mean time
>> ,
>> Question is whether to do this by default, or only for devices we know
>> are flickering.
>>
>> I haven't seen this a lot, but the few I have seen have had times with
>> the wrong state below 100 ms, so I think 200 ms could be a good value to
>> start with.
> Well, if many events are triggered at plugging/unplugging, we can
> filter them out.  The events are already handled in the workqueue, and
> in the case of jack-detection, we can filter out the too frequent
> ones, and even requeue it a certain delay to be sure.
>
> But like this case, if the problem happens even without touching the
> jack, it's a different issue.
>

I'm not talking about flickering at physical plug/unplug, even if my 
proposal would help in such cases also.

A small "click", as Dâniel is experiencing, would indicate that the 
hardware (incorrectly) reports as the jack being plugged in, then a few 
milliseconds later, it is reported as no longer being plugged in. This 
applies to both getting the unsol event and the actual GET_PIN_SENSE 
result, i e, get_pin_sense would return the wrong value for a few 
milliseconds.
By filtering out these events, we remove the occasional clicks.

You instead suggested some blacklisting, would that break headphone 
support completely, at least for the jack you would blacklist?

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

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

  reply	other threads:[~2012-05-11 14:25 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-03 17:11 Realtek ACL892: audio gap ~ each 10 seconds? Dâniel Fraga
2012-05-06  5:51 ` Alexander E. Patrakov
2012-05-08 15:11   ` Dâniel Fraga
2012-05-11  1:39     ` Realtek ACL892: audio gap ~ each 10 seconds? [SOLVED] Dâniel Fraga
2012-05-11  5:39       ` Takashi Iwai
2012-05-11  7:01         ` Dâniel Fraga
2012-05-11  7:43           ` Takashi Iwai
2012-05-11 13:45             ` David Henningsson
2012-05-11 13:55               ` Takashi Iwai
2012-05-11 14:25                 ` David Henningsson [this message]
2012-05-11 14:29                   ` Takashi Iwai
2012-05-11 15:20                     ` David Henningsson
     [not found]             ` <4fad162b.07b3340a.14cb.5a06@mx.google.com>
2012-05-11 13:52               ` Takashi Iwai
2012-05-11 14:53                 ` Dâniel Fraga
2012-05-11 15:05                   ` Takashi Iwai
2012-05-11 15:51                     ` Dâniel Fraga
2012-05-11 15:54                       ` Takashi Iwai
2012-05-11 16:11                         ` Dâniel Fraga

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=4FAD2148.4080701@canonical.com \
    --to=david.henningsson@canonical.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=fragabr@gmail.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).