From: Sergey Lapin <slapin@ossfans.org>
To: Ryan Mallon <ryan@bluewatersys.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
gwossum@acm.org, Mark Brown <broonie@opensource.wolfsonmicro.com>,
Nicolas Ferre <nicolas.ferre@atmel.com>,
Sedji Gaouaou <sedji.gaouaou@atmel.com>,
Liam Girdwood <lrg@slimlogic.co.uk>
Subject: Re: [RFC PATCH] Add combined clock support to Atmel SoC audio
Date: Tue, 7 Jun 2011 06:03:21 -0400 [thread overview]
Message-ID: <20110607100321.GB14633@build.ihdev.net> (raw)
In-Reply-To: <4CEC3A97.4040400@bluewatersys.com>
Hi, Ryan!
On Wed, Nov 24, 2010 at 11:05:11AM +1300, Ryan Mallon wrote:
> The following patch is one that has been floating around in various
> forms in our own internal trees for a while.
>
> The Atmel SSC peripheral has seperate TX and RX clocks which use
> separate pins from the the micro. TF (frame) and TK (clock) for transmit
> and RF and RK for receive. Not all codecs have separate frame and bit
> clocks for transmit and receive so we want to be able to do both
> playback and capture using a single set of pins.
>
> This patch introduces a combined clock mode for the Atmel SSC
> peripheral. Which allows playback and capture to use a single set of
> pins. Currently combined clock is only supported on the TF/TK pins (some
> incomplete support exists for using RF/RK).
>
> I have tested this patch on our AT91SAM9G45 + TLV320AIC26 platform.
> Playback and capture work individually. Simultaneous playback and
> capture have been tested by connecting a loopback cable on the linein
> and lineout jacks and then doing:
>
> arecord -c 2 -f S16_LE -r 44100 > recording.wav &
> aplay 500hz_sine.wav
>
> This patch is posted as RFC since the approach is incomplete and a bit
> hackish. I am mostly interested in knowing if this is a sensible
> approach, and could be cleaned up for mainline inclusion, or if there is
> a better way to do this.
>
> Signed-off-by: Ryan Mallon <ryan@bluewatersys.com>
I think it is also important to submit code, which uses it.
If you can't do it I might try to do this in a few days.
Also worth mentioning codec slave mode requirement for this to work.
All the best,
S.
next prev parent reply other threads:[~2011-06-07 10:03 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-23 22:05 [RFC PATCH] Add combined clock support to Atmel SoC audio Ryan Mallon
2010-11-23 23:29 ` Mark Brown
2010-11-24 4:02 ` Ryan Mallon
2010-11-24 12:58 ` Mark Brown
2011-06-07 9:51 ` Sergey Lapin
2011-06-07 10:03 ` Sergey Lapin [this message]
2011-06-07 12:29 ` Ryan Mallon
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=20110607100321.GB14633@build.ihdev.net \
--to=slapin@ossfans.org \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=gwossum@acm.org \
--cc=lrg@slimlogic.co.uk \
--cc=nicolas.ferre@atmel.com \
--cc=ryan@bluewatersys.com \
--cc=sedji.gaouaou@atmel.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.