From: timur@tabi.org (Timur Tabi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 05/11] ASoC: fsl-ssi: Add support for imx-pcm-fiq
Date: Sun, 14 Apr 2013 10:25:49 -0500 [thread overview]
Message-ID: <516ACA7D.60807@tabi.org> (raw)
In-Reply-To: <20130414152016.GE27394@pengutronix.de>
Markus Pargmann wrote:
> I only ported the imx-ssi ac97 driver code and tested twice using DMA
> without success. But there is a good description of the problem in
> imx-ssi:
>
> " The WM9712 with which this driver
> was developed with always sends GPIO status data in slot 12 which
> we receive in our (PCM-) data stream. The only chance we have is to
> manually skip this data in the FIQ handler."
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.
Either way, the board DTS needs to clearly state *why* we're enabling FIQ
mode when DMA mode should work. And the patch that adds FIQ support needs
to say that it's for boards with broken DMA support.
--
Timur Tabi
next prev parent reply other threads:[~2013-04-14 15:25 UTC|newest]
Thread overview: 35+ 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 ` [PATCH v2 01/11] ASoC: phycore-ac97: Add DT support Markus Pargmann
2013-04-08 2:52 ` Timur Tabi
2013-04-10 11:03 ` 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 ` [PATCH v2 03/11] ASoC: imx-pcm-fiq: Introduce pcm-fiq-params Markus Pargmann
2013-04-07 19:25 ` [PATCH v2 04/11] ASoC: fsl-ssi: Add SACNT definitions Markus Pargmann
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-08 0:18 ` Timur Tabi
2013-04-14 10:38 ` Markus Pargmann
2013-04-14 13:37 ` Timur Tabi
2013-04-14 14:10 ` Markus Pargmann
2013-04-14 14:53 ` Timur Tabi
2013-04-14 15:20 ` Markus Pargmann
2013-04-14 15:25 ` Timur Tabi [this message]
2013-04-14 15:44 ` Markus Pargmann
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-08 2:50 ` Timur Tabi
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 ` [PATCH v2 08/11] ASoC: fsl-ssi: imx ac97 support Markus Pargmann
2013-04-08 2:49 ` Timur Tabi
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 ` [PATCH v2 10/11] ASoC: fsl: Move fsl-ssi binding doc to sound/ Markus Pargmann
2013-04-08 0:20 ` Timur Tabi
2013-04-07 19:25 ` [PATCH v2 11/11] ASoC: fsl: Update fsl-ssi binding doc Markus Pargmann
2013-04-07 19:39 ` [alsa-devel] [PATCH v2 00/11] ASoC: fsl-ssi: ac97-slave support Lars-Peter Clausen
2013-04-07 20:08 ` Markus Pargmann
2013-04-07 20:35 ` Lars-Peter Clausen
2013-04-13 13:32 ` Markus Pargmann
2013-04-13 13:40 ` Lars-Peter Clausen
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=516ACA7D.60807@tabi.org \
--to=timur@tabi.org \
--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).