Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: broonie@kernel.org, lgirdwood@gmail.com, dmurphy@ti.com
Cc: alsa-devel@alsa-project.org
Subject: [PATCH v3 0/9] ASoC: tas2552: Fixes, cleanups and improvements
Date: Mon, 8 Jun 2015 15:19:47 +0300	[thread overview]
Message-ID: <1433765996-28930-1-git-send-email-peter.ujfalusi@ti.com> (raw)

Hi,

Changes since v02:
- Dropped already applied patches from v02
- Fix bit shift in the PLL configuration patch (1/9) and add macro for it for
  readability
- New patches since v02:
 - fix module removing and loading issue by calling pm_runtime_disable when the
   module is removed
 - Remove extra semicolon
 - Consitent naming for 'struct tas2552_data': tas_data -> tas2552
 - Binding document update regarding to the added header file for clock
   selection

Changes since v01:
- missing patch found. It was merged with the previous patch while fixing
  checkpatch errors (patch 19 got squashed to patch 18)
- Sorry for this.

I have used the TAS2552-EVM to verify that my setup is in working condition for
a work on other audio product.
During the bringup of the EVM I have faced some issues and limitation with the
driver. The result is this series against the tas2552 driver.

Notable changes:
- Fixes for kernel crashes
- The enable-gpio was not optional as it was documented in the DTS
- Register bit definitions were not correct in several places
- Adding recommended startup/stop sequence
- Handling of sample formats the driver claims to support
- PLL configuration fixes and reference clock selection
- TDM support
- Gain control fix and control to select DIN source

Regards,
Peter
---
Peter Ujfalusi (9):
  ASoC: tas2552: Correct the PLL configuration
  ASoC: tas2552: Add control for selecting DIN source
  ASoC: tas2552: Correct Output Data register usage
  ASoC: tas2552: Correct Boost Auto-Pass Through Control register usage
  ASoC: tas2552: Code, define alignment changes for uniformity
  ASoC: tas2552: Call pm_runtime_disable when the module is removed
  ASoC: tas2552: Remove unneeded semicolon
  ASoC: tas2552: Use consistent name for 'struct tas2552_data'
  ASoC: tas2552: Update DT binding document regarding clkock
    configuration

 .../devicetree/bindings/sound/tas2552.txt          |   6 +
 sound/soc/codecs/tas2552.c                         | 173 ++++++++++++++-------
 sound/soc/codecs/tas2552.h                         |  61 ++++----
 3 files changed, 155 insertions(+), 85 deletions(-)

-- 
2.4.2

             reply	other threads:[~2015-06-08 12:20 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-08 12:19 Peter Ujfalusi [this message]
2015-06-08 12:19 ` [PATCH v3 1/9] ASoC: tas2552: Correct the PLL configuration Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 2/9] ASoC: tas2552: Add control for selecting DIN source Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 3/9] ASoC: tas2552: Correct Output Data register usage Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 4/9] ASoC: tas2552: Correct Boost Auto-Pass Through Control " Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 5/9] ASoC: tas2552: Code, define alignment changes for uniformity Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 6/9] ASoC: tas2552: Call pm_runtime_disable when the module is removed Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 7/9] ASoC: tas2552: Remove unneeded semicolon Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 8/9] ASoC: tas2552: Use consistent name for 'struct tas2552_data' Peter Ujfalusi
2015-06-08 12:19 ` [PATCH v3 9/9] ASoC: tas2552: Update DT binding document regarding clkock configuration Peter Ujfalusi
2015-06-08 17:53 ` [PATCH v3 0/9] ASoC: tas2552: Fixes, cleanups and improvements 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=1433765996-28930-1-git-send-email-peter.ujfalusi@ti.com \
    --to=peter.ujfalusi@ti.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=dmurphy@ti.com \
    --cc=lgirdwood@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox