From: mpa@pengutronix.de (Markus Pargmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/9] ASoC: fsl-ssi: offline/online configuration and cleanups
Date: Fri, 20 Dec 2013 14:11:27 +0100 [thread overview]
Message-ID: <1387545096-2334-1-git-send-email-mpa@pengutronix.de> (raw)
Hi,
This patch series fixes some fsl-ssi code that does not act exactly as it is
described in the reference manuals.
The reference manuals before imx51 do mention that some register bits,
including TDMAE/RDMAE, should not be changed while the SSI unit is enabled
(SSIEN). At the same time the SDMA unit has a undefined request handling if
there are DMA requests before the SDMA engine and its channel is configured.
The reference manual states that SSI DMA requests are sent as soon as the FIFO
and the DMAE bits are enabled. It does not mention a dependency to SCR's TE/RE
bit.
To avoid sending DMA requests before a channel is configured, we have to
seperate the fsl-ssi behavior into offline and online configuration. SoCs
before imx51 have to use offline configuration while later SoC versions can use
online reconfiguration.
This series adds support for online configuration and cleans up the
configuration register code especially in fsl_ssi_trigger. At the end we have a
seperation between enable/disable logic and the actual configuration register
values used to enable/disable TX/RX.
v4 is another rebase onto latest topic/fsl branch. It includes the DAI master
mode changes from Nicolin.
@Nicolin: Could you please test this series with your hardware to verify that
your patch still works?
I tested this series on mx53.
The series is based on Mark Brown's topic/fsl branch.
Regards,
Markus
Changes in v4:
- Support for irqstats on all imx SoCs
- Fix offline_config and use switch statement
- Introduce ssi_private->irq_stats bool
Changes in v3:
- Rebase on latest topic/fsl
- Added patch to cleanup probe function error handling
- Fix interrupt stats for imx53
- Replace imx50-ssi by imx51-ssi
Markus Pargmann (9):
ASoC: fsl-ssi: Fix probe error handling
ASoC: fsl-ssi: Move sysfs stats to debugfs
ASoC: fsl-ssi: Add imx51-ssi and of_device_id matching
ASoC: fsl-ssi: Fix interrupt stats for imx
ASoC: fsl-ssi: Add offline_config flag
ASoC: fsl-ssi: Add configuration helper functions
ASoC: fsl-ssi: Move RX/TX configuration to seperate functions
ASoC: fsl-ssi: Drop ac97 specific trigger function
ARM: DTS: imx5* imx6*, use imx51-ssi
arch/arm/boot/dts/imx53.dtsi | 10 +-
arch/arm/boot/dts/imx6qdl.dtsi | 12 +-
arch/arm/boot/dts/imx6sl.dtsi | 12 +-
sound/soc/fsl/fsl_ssi.c | 567 ++++++++++++++++++++++++++++-------------
4 files changed, 411 insertions(+), 190 deletions(-)
--
1.8.5.1
next reply other threads:[~2013-12-20 13:11 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-20 13:11 Markus Pargmann [this message]
2013-12-20 13:11 ` [PATCH v4 1/9] ASoC: fsl-ssi: Fix probe error handling Markus Pargmann
2014-01-08 17:18 ` Mark Brown
2013-12-20 13:11 ` [PATCH v4 2/9] ASoC: fsl-ssi: Move sysfs stats to debugfs Markus Pargmann
2014-01-08 17:18 ` Mark Brown
2013-12-20 13:11 ` [PATCH v4 3/9] ASoC: fsl-ssi: Add imx51-ssi and of_device_id matching Markus Pargmann
2014-01-08 17:18 ` Mark Brown
2013-12-20 13:11 ` [PATCH v4 4/9] ASoC: fsl-ssi: Fix interrupt stats for imx Markus Pargmann
2014-01-08 17:19 ` Mark Brown
2013-12-20 13:11 ` [PATCH v4 5/9] ASoC: fsl-ssi: Add offline_config flag Markus Pargmann
2013-12-20 13:11 ` [PATCH v4 6/9] ASoC: fsl-ssi: Add configuration helper functions Markus Pargmann
2013-12-20 13:11 ` [PATCH v4 7/9] ASoC: fsl-ssi: Move RX/TX configuration to seperate functions Markus Pargmann
2013-12-20 13:11 ` [PATCH v4 8/9] ASoC: fsl-ssi: Drop ac97 specific trigger function Markus Pargmann
2013-12-20 13:11 ` [PATCH v4 9/9] ARM: DTS: imx5* imx6*, use imx51-ssi Markus Pargmann
2014-01-16 15:55 ` Markus Pargmann
2014-01-17 5:32 ` Shawn Guo
2014-01-08 17:21 ` [PATCH v4 0/9] ASoC: fsl-ssi: offline/online configuration and cleanups Mark Brown
2014-01-09 10:16 ` [PATCH 0/2] fsl-ssi fixups Markus Pargmann
2014-01-09 10:16 ` [PATCH 1/2] ASoC: fsl-ssi doc: Add list of supported compatibles Markus Pargmann
2014-01-09 10:16 ` [PATCH 2/2] ASoC: fsl-ssi: Fix stats compile warning Markus Pargmann
2014-01-09 13:55 ` [PATCH 0/2] fsl-ssi fixups Mark Brown
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=1387545096-2334-1-git-send-email-mpa@pengutronix.de \
--to=mpa@pengutronix.de \
--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).