alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Sangbeom Kim <sbkim73@samsung.com>
Cc: alsa-devel@alsa-project.org, sbkim01@gmail.com,
	sangsu4u.park@samsung.com, 'Liam Girdwood' <lrg@ti.com>
Subject: Re: WM8994 playback delay
Date: Thu, 29 Mar 2012 11:50:16 +0100	[thread overview]
Message-ID: <20120329105015.GA3668@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <031201cd0d36$d9b78310$8d268930$@com>


[-- Attachment #1.1: Type: text/plain, Size: 1297 bytes --]

On Thu, Mar 29, 2012 at 08:02:29AM +0900, Sangbeom Kim wrote:

> Currently, We are developing SMDK with Exynos5250 and  WM8994 on 3.3-rc6.
> We try to enable CONFIG_PM_RUNTIME and on/off control of power domain.
> But It has playback delay in the first playback.(3~4sec and 6/10times)

> In that time, Log maessage is 
> 'wm8994-codec wm8994-codec: Timed out waiting for DC Servo 300'
> wait_for_dc_servo() function generate above message on sound/soc/codecs/wm_hubs.c

This issue usually indicates that the clocking has an issue - are you
following a similar pattern to that seen in the lowland machine driver
in mainline?  This is the best practice for WM8994 class parts, it
should minimise power consumption when idle.

> After removing wait_for_dc_servo for testing, playback delay is disappeared.
> So, Could you please explain the  purporse of wait_for_dc_servo()?
> If wait_for_dc_servo() is removed, What kind of problem can be happened?

The DC servo is used to reduce pops on the headphone output.  If the DC
servo is not run the headphone outputs will produce audible artifacts
when headphone is enabled which are generally considered unacceptable by
users.  There's a more detailed explanation of the feature in the WM8994
datasheet but that's the high level.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2012-03-29 10:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-28 19:53 [PATCH 1/2] ASoC: sgtl5000: Convert mic bias to a supply widget Mark Brown
2012-03-28 19:53 ` [PATCH 2/2] ASoC: dapm: Remove SND_SOC_DAPM_MICBIAS_E() Mark Brown
2012-03-28 23:02   ` WM8994 playback delay Sangbeom Kim
2012-03-29 10:50     ` Mark Brown [this message]
2012-03-29  3:40 ` [PATCH 1/2] ASoC: sgtl5000: Convert mic bias to a supply widget Dong Aisheng
2012-03-29  5:24 ` Shawn Guo
2012-03-29 10:56   ` Mark Brown
2012-03-29 14:11     ` Shawn Guo

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=20120329105015.GA3668@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=lrg@ti.com \
    --cc=sangsu4u.park@samsung.com \
    --cc=sbkim01@gmail.com \
    --cc=sbkim73@samsung.com \
    /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).