All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jarkko Nikula <jarkko.nikula@nokia.com>
To: ext Troy Kisky <troy.kisky@boundarydevices.com>
Cc: Arun KS <getarunks@gmail.com>,
	"Ujfalusi Peter (Nokia-D/Tampere)" <peter.ujfalusi@nokia.com>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	ext Mark Brown <broonie@sirena.org.uk>
Subject: Re: [2.6.30-rc2][PATCH 1/2] ASoC: OMAP: Fix DSP_B format in OMAP McBSP DAI driver
Date: Fri, 17 Apr 2009 09:59:23 +0300	[thread overview]
Message-ID: <20090417095923.6392f82e.jarkko.nikula@nokia.com> (raw)
In-Reply-To: <49E7A498.8060804@boundarydevices.com>

On Thu, 16 Apr 2009 23:35:20 +0200
ext Troy Kisky <troy.kisky@boundarydevices.com> wrote:

> Arun KS wrote:
> > But when I2S is used there is no audio on Right channel(left channel
> > is working fine).
> > Thinks something is missing on tlv320aic23 driver.
> > 
> This sounds like the same problem Davinci has. From a comment in
> davinci-i2s.c
> 
> case SND_SOC_DAIFMT_I2S:
>         /* Davinci doesn't support TRUE I2S, but some codecs will have
>          * the left and right channels contiguous. This allows
>          * dsp_a mode to be used with an inverted normal frame clk.
>          * If your codec is master and does not have contiguous
>          * channels, then you will have sound on only one channel.
>          * Try using a different mode, or codec as slave.
>          *
>          * The TLV320AIC33 is an example of a codec where this works.
>          * It has a variable bit clock frequency allowing it to have
>          * valid data on every bit clock.
>          *
>          * The TLV320AIC23 is an example of a codec where this does
> not
>          * work. It has a fixed bit clock frequency with progressively
>          * more empty bit clock slots between channels as the sample
>          * rate is lowered.
>          */

Interesting codec. Arun, how is the right channel working with I2S if
you try to run with higher rate? E.g. aplay -r [48000 |
96000] /dev/urandom

The I2S mode setup in tlv320aic23_set_dai_fmt looks correct.


Jarkko

  reply	other threads:[~2009-04-17  6:58 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-15 10:48 [2.6.30-rc2][PATCH 0/2] Yet another DSP format fix into OMAP McBSP DAI Jarkko Nikula
2009-04-15 10:48 ` [2.6.30-rc2][PATCH 1/2] ASoC: OMAP: Fix DSP_B format in OMAP McBSP DAI driver Jarkko Nikula
2009-04-15 11:04   ` Peter Ujfalusi
2009-04-15 11:56     ` Mark Brown
2009-04-16 10:08       ` Arun KS
2009-04-16 10:12         ` Mark Brown
2009-04-16 10:17           ` Arun KS
2009-04-16 10:19             ` Mark Brown
2009-04-16 10:23               ` Arun KS
2009-04-16 10:31               ` Jarkko Nikula
2009-04-16 10:33                 ` Mark Brown
2009-04-16 10:47                   ` Jarkko Nikula
2009-04-16 11:20                     ` Arun KS
2009-04-16 11:27                       ` Peter Ujfalusi
2009-04-16 13:05                         ` Arun KS
2009-04-16 13:42                           ` Peter Ujfalusi
2009-04-16 13:53                             ` Arun KS
2009-04-16 21:35                           ` Troy Kisky
2009-04-17  6:59                             ` Jarkko Nikula [this message]
2009-04-17  9:16                               ` Arun KS
2009-04-17  9:43                                 ` Jarkko Nikula
2009-04-16 11:35                       ` Jarkko Nikula
2009-04-16 11:37                       ` Mark Brown
2009-04-16 10:36             ` Peter Ujfalusi
2009-04-15 10:48 ` [2.6.30-rc2][PATCH 2/2] ASoC: OMAP: Fix FS polarity in OSK5912 machine driver Jarkko Nikula
2009-04-16 11:24   ` Arun KS

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=20090417095923.6392f82e.jarkko.nikula@nokia.com \
    --to=jarkko.nikula@nokia.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@sirena.org.uk \
    --cc=getarunks@gmail.com \
    --cc=peter.ujfalusi@nokia.com \
    --cc=troy.kisky@boundarydevices.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.