Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964937AbcDYUSU (ORCPT ); Mon, 25 Apr 2016 16:18:20 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:42601 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933472AbcDYUSQ (ORCPT ); Mon, 25 Apr 2016 16:18:16 -0400 From: Andreas Dannenberg To: , CC: Andreas Dannenberg , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Subject: [PATCH v3 0/2] ASoC: codecs: add support for TAS5720 digital amplifier Date: Mon, 25 Apr 2016 15:17:34 -0500 Message-ID: <1461615456-19510-1-git-send-email-dannenberg@ti.com> X-Mailer: git-send-email 2.7.2 MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1963 Lines: 45 Updated driver for TI's TAS5720L/M digital audio amplifiers incorporating previous feedback (thanks Mark Brown). Biggest change is that the driver is no longer using interrupts for codec fault reporting, but instead uses a polling-based approach to check for (and clear) codec fault conditions every 200ms. This is much more graceful and actually lower overhead than trying to keep the actual TAS5720 fault interrupts (that come in every 300us!) under control which is what v2 of the driver was doing. Because of the removal of interrupts the DT doc had to be updated as well, so I did not carry forward Rob Herring's earlier Acked-by because of said change. -- Andreas Dannenberg Texas Instruments Inc 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 | 26 + sound/soc/codecs/Kconfig | 10 + sound/soc/codecs/Makefile | 2 + sound/soc/codecs/tas5720.c | 638 +++++++++++++++++++++ sound/soc/codecs/tas5720.h | 90 +++ 5 files changed, 766 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