alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Markus Pargmann <mpa@pengutronix.de>
To: Mark Brown <broonie@kernel.org>,
	Liam Girdwood <lgirdwood@gmail.com>, Timur Tabi <timur@tabi.org>
Cc: alsa-devel@alsa-project.org, kernel@pengutronix.de,
	Markus Pargmann <mpa@pengutronix.de>,
	Shawn Guo <shawn.guo@linaro.org>,
	Fabio Estevam <festevam@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/9] ASoC: fsl-ssi: offline/online configuration and cleanups
Date: Mon, 25 Nov 2013 12:13:36 +0100	[thread overview]
Message-ID: <1385378025-1262-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 imx50 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 imx50 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.

I tested this series on mx53.

The series is based on Mark Brown's topic/fsl branch.

Regards,

Markus

Changes in v2:
 - Enable fsl-ssi interrupts independent of CONFIG_DEBUG_FS.
 - Move irq mapping fix into seperate patch (was in "Move sysfs stats to debugfs")
 - Rebased on v3.13-rc1 topic/fsl branch.

Changes in v1:
 - Not removing imx21-ssi from the compatible array in the DTS patch.
 - Dropped "ASoC: fsl-ssi: Drop AC97 debug register usage" for the moment as I
   don't have time to setup AC97 hardware for some tests

Markus Pargmann (9):
  ASoC: fsl-ssi: Move sysfs stats to debugfs
  ASoC: fsl-ssi: Fix interrupt mapping and release
  ASoC: fsl-ssi: Add imx50-ssi and of_device_id matching
  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 imx50-ssi
  ASoC: fsl-ssi: Drop AC97 debug register usage

 arch/arm/boot/dts/imx51.dtsi   |  10 +-
 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        | 521 +++++++++++++++++++++++++++--------------
 5 files changed, 371 insertions(+), 194 deletions(-)

-- 
1.8.4.2

             reply	other threads:[~2013-11-25 11:14 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-25 11:13 Markus Pargmann [this message]
2013-11-25 11:13 ` [PATCH v2 1/9] ASoC: fsl-ssi: Move sysfs stats to debugfs Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 2/9] ASoC: fsl-ssi: Fix interrupt mapping and release Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 3/9] ASoC: fsl-ssi: Add imx50-ssi and of_device_id matching Markus Pargmann
2013-11-26  4:33   ` Shawn Guo
2013-11-26  9:35     ` Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 4/9] ASoC: fsl-ssi: Add offline_config flag Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 5/9] ASoC: fsl-ssi: Add configuration helper functions Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 6/9] ASoC: fsl-ssi: Move RX/TX configuration to seperate functions Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 7/9] ASoC: fsl-ssi: Drop ac97 specific trigger function Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 8/9] ARM: DTS: imx5* imx6*, use imx50-ssi Markus Pargmann
2013-11-25 11:13 ` [PATCH v2 9/9] ASoC: fsl-ssi: Drop AC97 debug register usage 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=1385378025-1262-1-git-send-email-mpa@pengutronix.de \
    --to=mpa@pengutronix.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=festevam@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --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 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).