From: broonie@opensource.wolfsonmicro.com (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/4] ASoC: Add TI tlv320aic32x4 codec support.
Date: Tue, 1 Mar 2011 14:11:06 +0000 [thread overview]
Message-ID: <20110301141105.GF9662@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <AANLkTinjj8rZS_Y9qiHbrU2nAYSXTikTNtyU71Bk7UqL@mail.gmail.com>
On Tue, Mar 01, 2011 at 08:59:23AM +0100, javier Martin wrote:
> > What are these "aic32x4_configs" all about? ?They look like you're
> > setting non-default configurations...
> This tlv320aic32x4 codec is kind of a huge monster; it has more than
> 200 registers which provide more flexibility than it is needed.
So what is all this magic stuff and why is it using such an odd way of
implementing whatever it's doing? Part of the problem here is that the
functionality is scattered all the way through the driver so it's less
clear than it should be what's going on.
> For example, BCLK and MCLK are forced here to derive from internal PLL
> which is needed for frequency calculation table to be valid. Bypassing
> this PLL doesn't provide new or better functionality.
I rather suspect it provides lower power consumption.
> The same happens regarding DAC internal connections. I don't think
> allowing the user to swap left and right channels makes sense here.
There are a number of CPUs out there that end up being happier with
channels in the wrong order due to peculiarities of the DMA. This is
also used to get a microphone input connected to the right analogue
input to appear on the left channel of a recording for mono use.
prev parent reply other threads:[~2011-03-01 14:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-25 13:42 [PATCH 1/4] ASoC: Add TI tlv320aic32x4 codec support Javier Martin
2011-02-25 13:42 ` [PATCH 2/4] ASoC: Fix burstsize and DSP_B format problems in imx-ssi Javier Martin
2011-02-25 13:42 ` [PATCH 3/4] ASoC: Add machine driver for Visstrim_M10 board Javier Martin
2011-02-25 13:42 ` [PATCH 4/4] ARM: Add SSI and aic3204 code to Visstrim_M10 boards Javier Martin
2011-02-28 15:00 ` [PATCH 1/4] ASoC: Add TI tlv320aic32x4 codec support Mark Brown
2011-03-01 7:59 ` javier Martin
2011-03-01 14:11 ` Mark Brown [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=20110301141105.GF9662@opensource.wolfsonmicro.com \
--to=broonie@opensource.wolfsonmicro.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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).