All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Timur Tabi <timur@tabi.org>
Cc: Fabio Estevam <fabio.estevam@freescale.com>,
	alsa-devel@alsa-project.org, devicetree-discuss@lists.ozlabs.org,
	Liam Girdwood <lgirdwood@gmail.com>,
	Grant Likely <grant.likely@secretlab.ca>,
	Sascha Hauer <kernel@pengutronix.de>,
	Markus Pargmann <mpa@pengutronix.de>,
	Shawn Guo <shawn.guo@linaro.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 05/11] ASoC: fsl-ssi: Add support for imx-pcm-fiq
Date: Mon, 15 Apr 2013 17:15:45 +0100	[thread overview]
Message-ID: <20130415161544.GH15837@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <516ACA7D.60807@tabi.org>


[-- Attachment #1.1: Type: text/plain, Size: 615 bytes --]

On Sun, Apr 14, 2013 at 10:25:49AM -0500, Timur Tabi wrote:

> Ah, so this is a work-around for a bug, where the codec is
> incompatible with the SoC?  Mark, are you familiar with this issue?
> Is there a way to reconfigure the codec so that it works better with
> the SoC?  It seems ridiculous to add all this code just because an
> incompatible codec was chosen for some board.

It's a limitation in the SoC - it sort of tries to bodge AC'97 in but
doesn't do a great job of it and can't handle the idea that there's
multiple streams of data on the bus.  This isn't something that can be
configured in the CODEC.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



WARNING: multiple messages have this Message-ID (diff)
From: broonie@kernel.org (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 05/11] ASoC: fsl-ssi: Add support for imx-pcm-fiq
Date: Mon, 15 Apr 2013 17:15:45 +0100	[thread overview]
Message-ID: <20130415161544.GH15837@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <516ACA7D.60807@tabi.org>

On Sun, Apr 14, 2013 at 10:25:49AM -0500, Timur Tabi wrote:

> Ah, so this is a work-around for a bug, where the codec is
> incompatible with the SoC?  Mark, are you familiar with this issue?
> Is there a way to reconfigure the codec so that it works better with
> the SoC?  It seems ridiculous to add all this code just because an
> incompatible codec was chosen for some board.

It's a limitation in the SoC - it sort of tries to bodge AC'97 in but
doesn't do a great job of it and can't handle the idea that there's
multiple streams of data on the bus.  This isn't something that can be
configured in the CODEC.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130415/17c01bc8/attachment.sig>

  parent reply	other threads:[~2013-04-15 16:15 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-07 19:25 [PATCH v2 00/11] ASoC: fsl-ssi: ac97-slave support Markus Pargmann
2013-04-07 19:25 ` Markus Pargmann
2013-04-07 19:25 ` [PATCH v2 02/11] ASoC: imx-pcm-dma: Add support for DMA init by device node Markus Pargmann
2013-04-07 19:25   ` Markus Pargmann
     [not found] ` <1365362721-3731-1-git-send-email-mpa-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2013-04-07 19:25   ` [PATCH v2 01/11] ASoC: phycore-ac97: Add DT support Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-08  2:52     ` Timur Tabi
2013-04-08  2:52       ` Timur Tabi
2013-04-10 11:03       ` Markus Pargmann
2013-04-10 11:03         ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 03/11] ASoC: imx-pcm-fiq: Introduce pcm-fiq-params Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 04/11] ASoC: fsl-ssi: Add SACNT definitions Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-08  0:13     ` Timur Tabi
2013-04-08  0:13       ` Timur Tabi
2013-04-07 19:25   ` [PATCH v2 05/11] ASoC: fsl-ssi: Add support for imx-pcm-fiq Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-08  0:18     ` Timur Tabi
2013-04-08  0:18       ` Timur Tabi
     [not found]       ` <51620CEA.3010703-N01EOCouUvQ@public.gmane.org>
2013-04-14 10:38         ` Markus Pargmann
2013-04-14 10:38           ` Markus Pargmann
2013-04-14 13:37           ` Timur Tabi
2013-04-14 13:37             ` Timur Tabi
2013-04-14 14:10             ` Markus Pargmann
2013-04-14 14:10               ` Markus Pargmann
2013-04-14 14:53               ` Timur Tabi
2013-04-14 14:53                 ` Timur Tabi
     [not found]                 ` <516AC305.2000800-N01EOCouUvQ@public.gmane.org>
2013-04-14 15:20                   ` Markus Pargmann
2013-04-14 15:20                     ` Markus Pargmann
2013-04-14 15:25                     ` Timur Tabi
2013-04-14 15:25                       ` Timur Tabi
2013-04-14 15:44                       ` Markus Pargmann
2013-04-14 15:44                         ` Markus Pargmann
2013-04-15 16:15                       ` Mark Brown [this message]
2013-04-15 16:15                         ` Mark Brown
2013-04-07 19:25   ` [PATCH v2 06/11] ASoC: fsl-ssi: Setup generic imx dma params Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-08  2:50     ` Timur Tabi
2013-04-08  2:50       ` Timur Tabi
2013-04-13 13:52       ` Markus Pargmann
2013-04-13 13:52         ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 07/11] ARM: imx: Export ac97 reset functions Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 08/11] ASoC: fsl-ssi: imx ac97 support Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-08  2:49     ` Timur Tabi
2013-04-08  2:49       ` Timur Tabi
2013-04-10 11:33       ` Markus Pargmann
2013-04-10 11:33         ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 09/11] ASoC: fsl: Kconfig: Use fsl-ssi for phycore-ac97 Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-07 19:25   ` [PATCH v2 11/11] ASoC: fsl: Update fsl-ssi binding doc Markus Pargmann
2013-04-07 19:25     ` Markus Pargmann
2013-04-07 19:25 ` [PATCH v2 10/11] ASoC: fsl: Move fsl-ssi binding doc to sound/ Markus Pargmann
2013-04-07 19:25   ` Markus Pargmann
2013-04-08  0:20   ` Timur Tabi
2013-04-08  0:20     ` Timur Tabi
2013-04-07 19:39 ` [PATCH v2 00/11] ASoC: fsl-ssi: ac97-slave support Lars-Peter Clausen
2013-04-07 19:39   ` [alsa-devel] " Lars-Peter Clausen
2013-04-07 20:08   ` Markus Pargmann
2013-04-07 20:08     ` [alsa-devel] " Markus Pargmann
     [not found]     ` <20130407200843.GA8350-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2013-04-07 20:35       ` Lars-Peter Clausen
2013-04-07 20:35         ` Lars-Peter Clausen
2013-04-13 13:32         ` Markus Pargmann
2013-04-13 13:32           ` [alsa-devel] " Markus Pargmann
2013-04-13 13:40           ` Lars-Peter Clausen
2013-04-13 13:40             ` [alsa-devel] " Lars-Peter Clausen
     [not found]             ` <51696034.6060709-Qo5EllUWu/uELgA04lAiVw@public.gmane.org>
2013-04-13 13:55               ` Markus Pargmann
2013-04-13 13:55                 ` Markus Pargmann

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=20130415161544.GH15837@opensource.wolfsonmicro.com \
    --to=broonie@kernel.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=fabio.estevam@freescale.com \
    --cc=grant.likely@secretlab.ca \
    --cc=kernel@pengutronix.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=mpa@pengutronix.de \
    --cc=shawn.guo@linaro.org \
    --cc=timur@tabi.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 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.