From: javier Martin <javier.martin@vista-silicon.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Timur Tabi <timur@freescale.com>,
Sascha Hauer <s.hauer@pengutronix.de>,
alsa-devel@alsa-project.org, Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: [PATCH 4/4] ASoC: Flag the i.MX SSI driver as using symmetric rates
Date: Mon, 11 Jan 2010 12:39:13 +0100 [thread overview]
Message-ID: <eedb5541001110339s6c7af715g2c95c65ab1028898@mail.gmail.com> (raw)
In-Reply-To: <20100111113631.GE3440@sirena.org.uk>
2010/1/11 Mark Brown <broonie@opensource.wolfsonmicro.com>
> On Mon, Jan 11, 2010 at 12:20:47PM +0100, Sascha Hauer wrote:
>
> > Why this limitation? I can successfully record with 16000Hz while
> > playing at 48000Hz on my PCM037 board. What is missing in the driver to
> > 'officially' support asymmetric rates?
>
> At least some of the driver was unconditionally setting both TX and RX
> configuration simultaneously, though looking again hw_params() is OK.
> I'll drop the patch. Like I say, I've never actually seen the driver
> working - even when configured as the master it doesn't seem to do any
> DMA (well, FIQs).
>
However,
I think it would be a good idea to add an optional flag to activate
symmetric rate.
Understanding it as TX and RX SSI sections sharing the same CLKs.
This is necessary for some codecs like wm8974 to work because they can only
provide
one BCLK and FCLK for both directions.
Moreover I don't want to make this requirement blocking.
--
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com
next prev parent reply other threads:[~2010-01-11 11:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-08 16:19 [PATCH 1/4] ASoC: Add a new imx-ssi sound driver Mark Brown
2010-01-08 16:19 ` [PATCH 2/4] ASoC: Fix i.MX audio build for i.MX3x Mark Brown
2010-01-08 16:19 ` [PATCH 3/4] ASoC: Convert new i.MX SSI driver to use static DAI array Mark Brown
2010-01-08 16:19 ` [PATCH 4/4] ASoC: Flag the i.MX SSI driver as using symmetric rates Mark Brown
2010-01-11 11:20 ` Sascha Hauer
2010-01-11 11:36 ` Mark Brown
2010-01-11 11:39 ` javier Martin [this message]
2010-01-11 11:51 ` Mark Brown
2010-01-11 15:51 ` Timur Tabi
2010-01-11 16:25 ` Mark Brown
2010-01-11 17:38 ` Sascha Hauer
2010-01-11 12:14 ` Sascha Hauer
2010-01-11 12:30 ` Mark Brown
2010-01-11 7:33 ` [PATCH 1/4] ASoC: Add a new imx-ssi sound driver javier Martin
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=eedb5541001110339s6c7af715g2c95c65ab1028898@mail.gmail.com \
--to=javier.martin@vista-silicon.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=lrg@slimlogic.co.uk \
--cc=s.hauer@pengutronix.de \
--cc=timur@freescale.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