From: John Hsu <KCHSU0@nuvoton.com>
To: Mark Brown <broonie@kernel.org>
Cc: AP MS30 Linux ALSA <alsa-devel@alsa-project.org>,
"anatol.pomozov@gmail.com" <anatol.pomozov@gmail.com>,
AC30 YHChuang <YHCHuang@nuvoton.com>,
AC30 WTLi <WTLI@nuvoton.com>,
"lgirdwood@gmail.com" <lgirdwood@gmail.com>,
"benzh@chromium.org" <benzh@chromium.org>,
AC30 CTLin0 <CTLIN0@nuvoton.com>, MS40 MHKuo <MHKuo@nuvoton.com>,
"yong.zhi@intel.com" <yong.zhi@intel.com>
Subject: Re: [PATCH] ASoC: nau8825: disable sinc filter for high THD of ADC
Date: Tue, 6 Dec 2016 10:07:40 +0800 [thread overview]
Message-ID: <58461D6C.2040506@nuvoton.com> (raw)
In-Reply-To: <20161205115009.ri5qpgzlfszumvo7@sirena.org.uk>
Hi,
On 12/5/2016 7:50 PM, Mark Brown wrote:
> On Fri, Dec 02, 2016 at 09:48:58AM +0800, John Hsu wrote:
>
>
>> This bit will enable 4th order SINC filter.
>> =1, filter will enable; but it consumes higher power.
>> =0, the sinc filter is disable, and it should always keep 0 value to
>> get high THD.
>>
>
>
>> Therefor, disable the filter when codec initiation for better
>> performance when recording.
>>
>
> This looks like something that users should be able to control at
> runtime - while some users are going to prefer lower power there will
> most likely also be some users who prefer the better performance (or
> perhaps even users who switch depending on if they're running on battery
> or something).
>
The function is not suitable for the voice application and easy to
make distortion. Therefore, we want to disalbe it as default to
get a high signal to distortion ratio. Yes, I agree it that users
maybe want to eanble it sometimes. I can add a kcontrol for the
function as well.
next prev parent reply other threads:[~2016-12-06 2:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-02 1:48 [PATCH] ASoC: nau8825: disable sinc filter for high THD of ADC John Hsu
2016-12-05 11:50 ` Mark Brown
2016-12-06 2:07 ` John Hsu [this message]
2016-12-06 14:35 ` Applied "ASoC: nau8825: disable sinc filter for high THD of ADC" to the asoc tree Mark Brown
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=58461D6C.2040506@nuvoton.com \
--to=kchsu0@nuvoton.com \
--cc=CTLIN0@nuvoton.com \
--cc=MHKuo@nuvoton.com \
--cc=WTLI@nuvoton.com \
--cc=YHCHuang@nuvoton.com \
--cc=alsa-devel@alsa-project.org \
--cc=anatol.pomozov@gmail.com \
--cc=benzh@chromium.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=yong.zhi@intel.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.