From: ckeepax@opensource.wolfsonmicro.com (Charles Keepax)
To: linux-arm-kernel@lists.infradead.org
Subject: [alsa-devel] [PATCH 0/3] ASoC: Enable a new IC master mode: bcm2835<=>IC<=>cs42xx8
Date: Wed, 22 Mar 2017 09:43:01 +0000 [thread overview]
Message-ID: <20170322094301.GE6986@localhost.localdomain> (raw)
In-Reply-To: <579f0763-a5bd-f820-b36f-b6437e331b4b@flatmax.org>
On Wed, Mar 22, 2017 at 10:29:33AM +1100, Matt Flax wrote:
> On 22/03/17 09:11, Matthias Reichl wrote:
> >On Tue, Mar 21, 2017 at 10:21:04PM +0100, Emmanuel Fust? wrote:
> >>Le 16/03/2017 ? 23:14, Matt Flax a ?crit :
> >>>On 17/03/17 08:27, Lars-Peter Clausen wrote:
> >>>>On 03/16/2017 09:51 PM, Matt Flax wrote:
> >>>>>On 16/03/17 06:01, Mark Brown wrote:
> >>>>>>On Tue, Feb 28, 2017 at 09:59:29AM +0000, Charles Keepax wrote:
> >>>>>>>On Mon, Feb 27, 2017 at 12:51:08PM +0100, Matthias Reichl wrote:
> >>Re-reading this document, the bcm2835 PCM IP block SHOULD support real DSP
> >>mode, with one BCLK pulsed LRCLK, zero BCLK delay etc...
> >>It just need to be properly setup.
> >I've re-read the document, too, last week and noticed the framesync
> >registers - sorry, I had completely forgotten about these. I guess it
> >should be possible to configure the bcm2835 to DSP mode but it'd still be
> >limited to 2 channel setups - the hardware only has 2 channel position
> >registers for each direction.
> >
> >>According to the same document, you could program the bmc up to 16 32bits
> >>channels when in master mode, so I suspect that you could go up to this
> >>limit in slave mode.
> >>But as it is designed, it could only use up to two of any channels among the
> >>16.
> >I'm not quite sure if I can follow you on this - how would you
> >configure 16 channels when there are only 2 channel position registers?
> >
> >With bclk ratio eg set to 16*32=512 BCM2835 will only transmit 2*32
> >bits of data (at configurable bit positions), the remaining 448
> >bits will be zero.
> >
> The document seems to stipulate that the PCM audio device is an AMBA device
> with 2 APB data channels. The first sync edge marks the beginning of the two
> data words. Their frame lengths can be up to 1024+32 bits in length !
>
> I think the point is that they intended their PCM audio interface to be
> configurable, they say in their document "It supports many classic PCM
> formats".
>
> The important point here is that in ALSA we can only have I2S or DSP modes -
> right ?
> Unless we want to create a new ALSA mode (which clearly worries people) then
> we need to support the versatility of the bcm2835 PCM hardware using either
> DSP or I2S modes. Now, we have already implemented the I2S mode, so
> logically the only available mode left is the DSP mode. Using this mode, we
> can implement more features of this device.
>
> People seem to want to reserve DSP and I2S modes for strictly I2S and DSP
> protocols. At the same time people don't want to allow a looser "APB" mode
> into ALSA. For that reason, we have a lack of functionality for perfectly
> versatile hardware - the bcm2835 hardware.
>
Apologies but I am still a little unclear as to what actually happens
on the bus here.
Are we saying that what gets transmitted on the bus is neither valid
I2S or DSP mode data? But as you have your custom hardware block in
the middle it interprets this data correctly and converts it to a
regular bus format on the other side that goes to the CODEC?
Thanks,
Charles
next prev parent reply other threads:[~2017-03-22 9:43 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-25 5:03 [PATCH 0/3] ASoC: Enable a new IC master mode: bcm2835<=>IC<=>cs42xx8 Matt Flax
2017-02-25 5:03 ` [PATCH 1/3] ASoC : Add an IC bit and frame master mode (SoC and Codec slave) Matt Flax
2017-03-15 19:02 ` Mark Brown
2017-02-25 5:03 ` [PATCH 2/3] ASoC: cs42xx8: allow IC master mode Matt Flax
2017-02-25 5:03 ` [PATCH 3/3] ASoC: bcm2835: Add mutichannel mode in DSP and IC master modes Matt Flax
2017-02-25 13:39 ` [PATCH 0/3] ASoC: Enable a new IC master mode: bcm2835<=>IC<=>cs42xx8 Matthias Reichl
2017-02-25 22:13 ` [alsa-devel] " Matt Flax
2017-02-26 14:49 ` Matthias Reichl
2017-02-26 20:21 ` Matt Flax
2017-02-26 22:16 ` Matthias Reichl
2017-02-26 22:35 ` Matt Flax
2017-02-27 8:04 ` Matthias Reichl
2017-02-27 10:08 ` Matt Flax
2017-02-27 10:30 ` Matthias Reichl
2017-02-27 11:21 ` Matt Flax
2017-02-27 11:51 ` Matthias Reichl
2017-02-28 9:59 ` Charles Keepax
2017-03-15 19:01 ` Mark Brown
2017-03-16 20:51 ` Matt Flax
2017-03-16 21:27 ` Lars-Peter Clausen
2017-03-16 22:14 ` Matt Flax
2017-03-21 21:21 ` Emmanuel Fusté
2017-03-21 22:11 ` Matthias Reichl
2017-03-21 23:29 ` Matt Flax
2017-03-22 9:43 ` Charles Keepax [this message]
2017-03-22 12:04 ` Matt Flax
2017-03-22 12:34 ` Charles Keepax
2017-03-22 15:38 ` Stephen Warren
2017-03-24 19:11 ` Mark Brown
2017-03-24 19:09 ` Mark Brown
2017-03-25 5:45 ` Matt Flax
2017-03-27 10:01 ` Mark Brown
2017-03-27 10:35 ` Matt Flax
2017-03-27 11:30 ` Mark Brown
2017-03-26 19:02 ` Emmanuel Fusté
2017-02-28 10:10 ` Charles Keepax
2017-02-26 20:41 ` Emmanuel Fusté
2017-02-27 9:14 ` Matthias Reichl
2017-02-27 19:12 ` Emmanuel Fusté
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=20170322094301.GE6986@localhost.localdomain \
--to=ckeepax@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).