Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932513AbbD0Ld7 (ORCPT ); Mon, 27 Apr 2015 07:33:59 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:41716 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752920AbbD0Ld4 (ORCPT ); Mon, 27 Apr 2015 07:33:56 -0400 Message-ID: <553E1E99.9020101@ti.com> Date: Mon, 27 Apr 2015 14:33:45 +0300 From: Jyri Sarha User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Jean-Francois Moine CC: , , Subject: Re: [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter References: In-Reply-To: Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4253 Lines: 96 Hi, Have you done anything about the tda998x audio support lately? I was thinking of taking a shot at this now that I finally seem to have some time for it. However, if you are just about to send another series I'll wait for that first and see what makes the most sense after that. My plan is to do something really simple first. No graph bindings or anything fancy like that. Just a minimal dt-binding to for audio pin configuration and use simple-card for the rest. I still try to make it possible to support spdif and i2s at the same time, but I can not test it as I do not have such HW. I also try to make the ASoC part as generic as possible, so that it could be reused by other HDMI encoders with spdif or i2s interface. Best regards, Jyri On 01/20/15 21:35, Jean-Francois Moine wrote: > Based on broonie/ASoC > > v10: > - add the generic dt-card > - define the audio ports from a DT graph of ports (Russell King) > - reuse HDMI constants (Andrew Jackson - Jyri Sarha) > - alloc rate_constraints in codec (Jyri Sarha) > - fix bad number of channels (Jyri Sarha) > - correct codec generation from config (Russell King - Jyri Sarha) > - no module init/exit (Russell King) > v9: > - back to a TDA998x specific CODEC > - more comments > - change magic values to constants > v8: > - change some comments about the patches > v7: > - remove the change of the K predivider (Jyri Sarha) > - add S24_3LE and S32_LE as possible audio formats (Jyri Sarha) > - don't move the struct priv2 definition and use the > slave encoder private data as the device private data > (Russell King) > - remove the useless request_module (Russell King/Mark Brown) > - don't lock the HDMI module (Russell King) > - use platform_device_unregister to remove the codec > (Russell King) > v6: > - extend the HDMI CODEC instead of using a specific CODEC > v5: > - use the TDA998x private data instead of a specific area > for the CODEC interface > - the CODEC is TDA998x specific (Mark Brown) > v4: > - remove all the TDA998x specific stuff from the CODEC > - move the EDID scan from the CODEC to the TDA998x > - move the CODEC to sound/soc (Mark Brown) > - update the audio_sample_rate from the EDID (Andrew Jackson) > v3: fix bad rate (Andrew Jackson) > v2: check double stream start (Mark Brown) > > Jean-Francois Moine (9): > ASoC: kirkwood: dynamically build the DAI array > ASoC: kirkwood: check the DAI type from the DAI name > ASoC: kirkwood: accept the DAI definitions from a graph of ports > drm/i2c: tda998x: Add support of a DT graph of ports > drm/i2c: tda998x: Change drvdata for audio extension > ASoC: tda998x: add a codec to the HDMI transmitter > drm/i2c: tda998x: set cts_n according to the sample width > ASoC: core: export snd_soc_get_dai_name > ASoC: add generic dt-card support > > .../devicetree/bindings/drm/i2c/tda998x.txt | 51 ++++ > .../devicetree/bindings/sound/mvebu-audio.txt | 30 +++ > drivers/gpu/drm/i2c/tda998x_drv.c | 237 +++++++++++++++++- > include/sound/soc.h | 2 + > include/sound/tda998x.h | 32 +++ > sound/soc/codecs/Kconfig | 5 + > sound/soc/codecs/Makefile | 2 + > sound/soc/codecs/tda998x.c | 175 +++++++++++++ > sound/soc/generic/Kconfig | 2 + > sound/soc/generic/Makefile | 2 + > sound/soc/generic/dt-card.c | 275 +++++++++++++++++++++ > sound/soc/kirkwood/kirkwood-i2s.c | 141 ++++++----- > sound/soc/kirkwood/kirkwood.h | 4 +- > sound/soc/soc-core.c | 5 +- > 14 files changed, 877 insertions(+), 86 deletions(-) > create mode 100644 include/sound/tda998x.h > create mode 100644 sound/soc/codecs/tda998x.c > create mode 100644 sound/soc/generic/dt-card.c > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/