All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org, Liam Girdwood <lrg@ti.com>,
	Misael Lopez Cruz <misael.lopez@ti.com>
Subject: Re: [PATCH] ASoC: twl6040: Remove HS/HF gain ramp feature
Date: Tue, 08 May 2012 14:46:54 +0300	[thread overview]
Message-ID: <4FA907AE.4060409@ti.com> (raw)
In-Reply-To: <20120507172732.GP17002@opensource.wolfsonmicro.com>

Hi,

On 05/07/2012 08:27 PM, Mark Brown wrote:
> On Fri, May 04, 2012 at 03:17:20PM +0300, Peter Ujfalusi wrote:
>> None of the machines uses the gain ramp possibility for HS/HF.
>> This code path is mostly unused and it does not reduces the pop
>> noise on the output (it alters it to sound a bit different).
>> The preferred method to reduce pop noise is to use ABE.
> 
> ...to implement gain changes?

To fix up the pop noise.

After this feature was implemented to reduce the pop noise it become
clear that the underlying issue can be better solved with the offset
cancellation feature within ABE.
It turned out that the ramp code (which was implemented and deployed at
the same time as the ABE offset cancellation) alone just alters the
quality of the pop. With the ABE solution alone we are fine.

> Applied, thanks.

Thank you,
Péter

      reply	other threads:[~2012-05-08 11:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-04 12:17 [PATCH] ASoC: twl6040: Remove HS/HF gain ramp feature Peter Ujfalusi
2012-05-07 17:27 ` Mark Brown
2012-05-08 11:46   ` Peter Ujfalusi [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=4FA907AE.4060409@ti.com \
    --to=peter.ujfalusi@ti.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=lrg@ti.com \
    --cc=misael.lopez@ti.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 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.