All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Mack <daniel@caiaq.de>
To: Philipp Zabel <philipp.zabel@gmail.com>
Cc: alsa-devel@alsa-project.org,
	Mark Brown <broonie@opensource.wolfsonmicro.com>
Subject: Re: [PATCH 1/4] ASoC: Use network mode with 2 slots	for 16-bit stereo in pxa-ssp/Zylonite
Date: Wed, 4 Mar 2009 02:34:24 +0100	[thread overview]
Message-ID: <20090304013424.GA5165@buzzloop.caiaq.de> (raw)
In-Reply-To: <1236093054-1952-1-git-send-email-philipp.zabel@gmail.com>

Hi Phillip,

glad to see someone is wasting time with that issue, too :)

On Tue, Mar 03, 2009 at 04:10:51PM +0100, Philipp Zabel wrote:
> For consistency with 24-bit and 32-bit modes, don't send 16-bit stereo
> in one 32-bit transfer. Use 2 slots instead on Zylonite. It should result
> in exactly the same behaviour.
> Now it is possible to use 16-bit single slot transfers in pxa-ssp, which
> are needed for Magician to get two frame clock pulses per sample
> (one for each channel).

We've been fiddling around with these modes and registers for
another two days in a row now and figured out that literally all the
documentation about these registers is entirly bogus. In particular,
the network mode with the associated timeslots simply does not work at
all, according to our measurements.

Hence the question: how does your I2S signal look like at the moment?
How many clocks do you measure in one frame cylce, how many of them are
actually filled with data? And which format does the userspace use to
send samples in?

We finally got a mode now that outputs 2x16 bit on a 2x32 bits I2S
frame, padded with zeros. But that is in non-network modes, more
about that tomorrow ...

Thanks,
Daniel

  parent reply	other threads:[~2009-03-04  1:34 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-03 15:10 [PATCH 1/4] ASoC: Use network mode with 2 slots for 16-bit stereo in pxa-ssp/Zylonite Philipp Zabel
2009-03-03 15:10 ` [PATCH 2/4] ASoC: UDA1380: DATAI is slave only Philipp Zabel
2009-03-03 15:50   ` Mark Brown
2009-03-03 15:56     ` pHilipp Zabel
2009-03-03 15:58       ` Mark Brown
2009-03-03 15:10 ` [PATCH 3/4] ASoC: Remove version display from the UDA1380 driver Philipp Zabel
2009-03-03 15:52   ` Mark Brown
2009-03-03 15:10 ` [PATCH 4/4] ASoC: UDA1380: change decimator/interpolator register handling Philipp Zabel
2009-03-03 15:48 ` [PATCH 1/4] ASoC: Use network mode with 2 slots for 16-bit stereo in pxa-ssp/Zylonite Mark Brown
2009-03-04  1:34 ` Daniel Mack [this message]
2009-03-04 14:29   ` Mark Brown
2009-03-04 20:45   ` pHilipp Zabel
2009-03-05 16:06     ` pHilipp Zabel

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=20090304013424.GA5165@buzzloop.caiaq.de \
    --to=daniel@caiaq.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=philipp.zabel@gmail.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.