From: Barry Song <21cnbao@gmail.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: uclinux-dist-devel@blackfin.uclinux.org, alsa-devel@alsa-project.org
Subject: Re: [PATCH V2 2/3] add set_tdm_slot in tdm dai to define the relationship between audio channel and tdm slot
Date: Mon, 31 Aug 2009 11:14:54 +0800 [thread overview]
Message-ID: <3c17e3570908302014n2cdd62fdga2642b7f968b0e65@mail.gmail.com> (raw)
In-Reply-To: <1251463341.5039.28.camel@finisterre.sirena.org.uk>
On Fri, Aug 28, 2009 at 8:42 PM, Mark
Brown<broonie@opensource.wolfsonmicro.com> wrote:
> On Thu, 2009-08-20 at 10:37 +0800, Barry Song wrote:
>> On Wed, Aug 19, 2009 at 7:34 PM, Mark
>> Brown<broonie@opensource.wolfsonmicro.com> wrote:
>
>> > Yup, that DSP mode with I2S like channel layout (does it also do TDM in
>> > I2S mode by any chance?). Could you submit a version of your patch
>> > which uses the default TDM layout - an API to rewrite things is likely
>> > to be a separate API?
>
>> I don't know what is your exact meaning. Do you mean I send a patch
>> whose tdm slot is not changed, then send a patch which change the tdm
>> slot?
>
> Send a patch that allows selection of the active slots without rewriting
>
Sorry. I am not sure about your meaning yet. The problem is the slot
order, the match between slot number and channel number, not only
which slot is active, and which is not active.
> Although thinking about it I'm not sure if the driver is actually
> implementing TDM or not - does the CPU actually get configured to stop
> driving the data line during an inactive slot (so something else can
> drive it) or is it simply sending zeros on those slots?
It's the later. The hardware signal is still active but no valid data
in inactive slot.
>
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2009-08-31 3:15 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-13 4:25 [PATCH V2 2/3] add set_tdm_slot in tdm dai to define the relationship between audio channel and tdm slot Barry Song
2009-08-17 17:50 ` Mark Brown
2009-08-18 2:53 ` Barry Song
2009-08-18 13:11 ` Mark Brown
2009-08-19 1:34 ` Barry Song
2009-08-19 11:34 ` Mark Brown
2009-08-20 2:37 ` Barry Song
2009-08-28 12:42 ` Mark Brown
2009-08-31 3:14 ` Barry Song [this message]
2009-08-31 12:01 ` 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=3c17e3570908302014n2cdd62fdga2642b7f968b0e65@mail.gmail.com \
--to=21cnbao@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=uclinux-dist-devel@blackfin.uclinux.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