devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v4 0/2] ASoC: codecs: add support for TAS5720 digital amplifier
@ 2016-04-26 22:15 Andreas Dannenberg
  2016-04-26 22:15 ` [PATCH v4 1/2] ASoC: codecs: add TA5720 digital amplifier DT bindings Andreas Dannenberg
  2016-04-26 22:15 ` [PATCH v4 2/2] ASoC: codecs: add support for TAS5720 digital amplifier Andreas Dannenberg
  0 siblings, 2 replies; 4+ messages in thread
From: Andreas Dannenberg @ 2016-04-26 22:15 UTC (permalink / raw)
  To: alsa-devel, devicetree
  Cc: Andreas Dannenberg, Liam Girdwood, Mark Brown, Jaroslav Kysela,
	Takashi Iwai, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell,
	Kumar Gala, linux-kernel

Minor update to driver for TI's TAS5720L/M digital audio amplifiers
incorporating previous feedback (thanks Mark Brown and Andrew F Davis),
see below change log for details. Also re-tested different aspects of
the driver to make sure it still builds/works.

--

Andreas Dannenberg
Texas Instruments Inc


Changes since v3:
- Rework fault condition reporting to use dev_crit() to pay justice to
  the severity of the issues, also shorten the error string for more
  clarity/brevity
- Fix Kconfig help entry to no longer mention the interrupt signal that
  was used previously
- Simplify hw_params checks as the driver in its current form only
  supports left-justified audio formats and the codec is tolerant
  regarding the number of bits that are actually received.
- Clarify DT bindings to make the power supply properties required
- Remove .owner assignment from I2C driver struct as it is no longer
  needed (taken care off by the Kernel)

Changes since v2:
- Switched fault handling from using interrupts to polling mode
- Remove interrupt related description from DT bindings doc
- Remove unlikely() to simplify an expression
- Remove unnesseary tas5720_set_dai_sysclk() function stub

Changes since v1:
- Simplified DT interrupt documentation (Thanks Rob Herring)
- Fixed potential race condition during codec probe where deferred work
  that's used by the threaded IRQ handler was setup after the IRQ was
  initialized (would lead to an issue when the TAS5720 was already throwing
  interrupts at the time of probe)

Andreas Dannenberg (2):
  ASoC: codecs: add TA5720 digital amplifier DT bindings
  ASoC: codecs: add support for TAS5720 digital amplifier

 .../devicetree/bindings/sound/tas5720.txt          |  25 +
 sound/soc/codecs/Kconfig                           |   8 +
 sound/soc/codecs/Makefile                          |   2 +
 sound/soc/codecs/tas5720.c                         | 620 +++++++++++++++++++++
 sound/soc/codecs/tas5720.h                         |  90 +++
 5 files changed, 745 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/sound/tas5720.txt
 create mode 100644 sound/soc/codecs/tas5720.c
 create mode 100644 sound/soc/codecs/tas5720.h

-- 
2.6.4

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-04-27 16:34 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-26 22:15 [PATCH v4 0/2] ASoC: codecs: add support for TAS5720 digital amplifier Andreas Dannenberg
2016-04-26 22:15 ` [PATCH v4 1/2] ASoC: codecs: add TA5720 digital amplifier DT bindings Andreas Dannenberg
2016-04-27 16:34   ` Applied "ASoC: add TA5720 digital amplifier DT bindings" to the asoc tree Mark Brown
2016-04-26 22:15 ` [PATCH v4 2/2] ASoC: codecs: add support for TAS5720 digital amplifier Andreas Dannenberg

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).