From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [PATCH] ASoC: tda998x: add a codec to the HDMI transmitter Date: Wed, 2 Jul 2014 19:51:54 +0200 Message-ID: <20140702195154.47d6f6b4@armhf> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20140702165628.GO20799@lunn.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Andrew Lunn Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, Russell King - ARM Linux , lgirdwood@gmail.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Mark Brown , linux-arm-kernel@lists.infradead.org List-Id: alsa-devel@alsa-project.org T24gV2VkLCAyIEp1bCAyMDE0IDE4OjU2OjI4ICswMjAwCkFuZHJldyBMdW5uIDxhbmRyZXdAbHVu bi5jaD4gd3JvdGU6Cgo+IEhvdyB3ZWxsIHdpbGwgdGhpcyB3b3JrIHdpdGggUnVzc2VsbCBjb25j ZXB0IG9mIGEgZnJvbnQgZW5kIGFuZCB0d28KPiBiYWNrZW5kcz8gQ2FuIHlvdSB1c2VzIHlvdXIg Q09ERUMgdHdpY2UsIG9uY2Ugd2l0aCB0aGUgSTJTIGJhY2tlbmQgYW5kCj4gYSBzZWNvbmQgdGlt ZSB3aXRoIHRoZSBTL1BESUYgYmFja2VuZD8KCkhpIEFuZHJldywKClRoZSBUREE5OTh4IENPREVD IGhhcyB0d28gZnVuY3Rpb25zOgotIGl0IHNldHMgdGhlIHBvc3NpYmxlIGZvcm1hdHMgYW5kIHJh dGVzIHdoZW4gdGhlIHNjcmVlbiBjb25uZWN0cyBhbmQKLSBpdCBzZXRzIHRoZSBhdWRpbyBpbnB1 dCBwb3J0IHdoZW4gYXVkaW8gc3RyZWFtaW5nIHN0YXJ0cy4KCkkgdGVzdGVkIHRoaXMgQ09ERUMg d2l0aCBib3RoIERBUE0gYW5kIERQQ00uIElmIHRoZSBhdWRpbyBzdWJzeXN0ZW0KYXNrcyBmb3Ig c3RyZWFtaW5nIG9uIGJvdGggSTJTIGFuZCBTL1BESUYsIG9ubHkgdGhlIGxhc3QgY2FsbCBpcyBz ZXJ2ZWQKKHRoaXMgZGVwZW5kcyBvbiB0aGUgb3JkZXIgb2YgdGhlIERBSSBsaW5rcyBpbiB0aGUg YXVkaW8gY2FyZCBjcmVhdGlvbgp0YWJsZSkuCgpBcyBJIHRvbGQgdG8gUnVzc2VsbCwgRFBDTSBq dXN0IGFza3MgZm9yIGEgJ3N5c3RlbScgREFJIChmcm9udC1lbmQpLApidXQgaXQgYWxzbyBhc2tz IGZvciBzb21lIGFkZGl0aW9uYWwgY29kZSBpbiB0aGUga2lya3dvb2QgRE1BIGRyaXZlcgpiZWNh dXNlIGFsbCBQQ01zIGFyZSBhY3RpdmF0ZWQgb24gc3RyZWFtaW5nIHN0YXJ0LgoKLS0gCktlbiBh ciBjJ2hlbnRhw7EJfAkgICAgICAqKiBCcmVpemggaGEgTGludXggYXRhdiEgKioKSmVmCQl8CQlo dHRwOi8vbW9pbmVqZi5mcmVlLmZyLwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVl ZGVza3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Wed, 2 Jul 2014 19:51:54 +0200 Subject: [PATCH] ASoC: tda998x: add a codec to the HDMI transmitter In-Reply-To: <20140702165628.GO20799@lunn.ch> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> Message-ID: <20140702195154.47d6f6b4@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 2 Jul 2014 18:56:28 +0200 Andrew Lunn wrote: > How well will this work with Russell concept of a front end and two > backends? Can you uses your CODEC twice, once with the I2S backend and > a second time with the S/PDIF backend? Hi Andrew, The TDA998x CODEC has two functions: - it sets the possible formats and rates when the screen connects and - it sets the audio input port when audio streaming starts. I tested this CODEC with both DAPM and DPCM. If the audio subsystem asks for streaming on both I2S and S/PDIF, only the last call is served (this depends on the order of the DAI links in the audio card creation table). As I told to Russell, DPCM just asks for a 'system' DAI (front-end), but it also asks for some additional code in the kirkwood DMA driver because all PCMs are activated on streaming start. -- Ken ar c'henta? | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932322AbaGBRuO (ORCPT ); Wed, 2 Jul 2014 13:50:14 -0400 Received: from smtp3-g21.free.fr ([212.27.42.3]:41846 "EHLO smtp3-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756711AbaGBRuJ convert rfc822-to-8bit (ORCPT ); Wed, 2 Jul 2014 13:50:09 -0400 Date: Wed, 2 Jul 2014 19:51:54 +0200 From: Jean-Francois Moine To: Andrew Lunn Cc: Mark Brown , Russell King - ARM Linux , devicetree@vger.kernel.org, alsa-devel@alsa-project.org, lgirdwood@gmail.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Rob Clark , Dave Airlie , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] ASoC: tda998x: add a codec to the HDMI transmitter Message-ID: <20140702195154.47d6f6b4@armhf> In-Reply-To: <20140702165628.GO20799@lunn.ch> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.24; arm-unknown-linux-gnueabihf) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2 Jul 2014 18:56:28 +0200 Andrew Lunn wrote: > How well will this work with Russell concept of a front end and two > backends? Can you uses your CODEC twice, once with the I2S backend and > a second time with the S/PDIF backend? Hi Andrew, The TDA998x CODEC has two functions: - it sets the possible formats and rates when the screen connects and - it sets the audio input port when audio streaming starts. I tested this CODEC with both DAPM and DPCM. If the audio subsystem asks for streaming on both I2S and S/PDIF, only the last call is served (this depends on the order of the DAI links in the audio card creation table). As I told to Russell, DPCM just asks for a 'system' DAI (front-end), but it also asks for some additional code in the kirkwood DMA driver because all PCMs are activated on streaming start. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/