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: Thu, 3 Jul 2014 07:49:59 +0200 Message-ID: <20140703074959.7c489912@armhf> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> <20140702195154.47d6f6b4@armhf> <20140702194252.GN410@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20140702194252.GN410@sirena.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Mark Brown Cc: Andrew Lunn , alsa-devel@alsa-project.org, Russell King - ARM Linux , devicetree@vger.kernel.org, lgirdwood@gmail.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: alsa-devel@alsa-project.org T24gV2VkLCAyIEp1bCAyMDE0IDIwOjQyOjUyICswMTAwCk1hcmsgQnJvd24gPGJyb29uaWVAa2Vy bmVsLm9yZz4gd3JvdGU6Cgo+ID4gSSB0ZXN0ZWQgdGhpcyBDT0RFQyB3aXRoIGJvdGggREFQTSBh bmQgRFBDTS4gSWYgdGhlIGF1ZGlvIHN1YnN5c3RlbQo+ID4gYXNrcyBmb3Igc3RyZWFtaW5nIG9u IGJvdGggSTJTIGFuZCBTL1BESUYsIG9ubHkgdGhlIGxhc3QgY2FsbCBpcyBzZXJ2ZWQKPiA+ICh0 aGlzIGRlcGVuZHMgb24gdGhlIG9yZGVyIG9mIHRoZSBEQUkgbGlua3MgaW4gdGhlIGF1ZGlvIGNh cmQgY3JlYXRpb24KPiA+IHRhYmxlKS4gIAo+IAo+IEknZCBleHBlY3QgdGhpcyB0byByZXR1cm4g YW4gZXJyb3IgZm9yIHRoZSBidXN5IERBSSByYXRoZXIgdGhhbiBqdXN0Cj4gc2lsZW50bHkgaWdu b3JlIGl0IGZhaWxpbmcgdG8gc3RhcnQgb3IgKGJldHRlcikgaW1wbGVtZW50IHNvbWUgY29udHJv bAo+IHRvIGxldCB0aGUgdXNlciBzZWxlY3Qgd2hpY2ggb2YgdGhlIERBSXMgaXMgYWN0aXZlLgoK TWFyaywKClRoaXMgaXMgbm90IGFuIGVycm9yLiBJZiB0aGUgYXVkaW8gc3Vic3lzdGVtIChEUENN LCBub3QgdGhlIHVzZXIpCmNob29zZXMgdG8gYWN0aXZhdGUgYm90aCBJMlMgYW5kIFMvUERJRiwg dGhpcyBtZWFucyB0aGUgSERNSSBhdWRpbyBtYXkKYmUgdGFrZW4gZWl0aGVyIGZyb20gSTJTIG9y IGZyb20gUy9QRElGOiBib3RoIGlucHV0cyBoYXZlIHRoZSByaWdodApmb3JtYXQgYW5kIHJhdGUu CgotLSAKS2VuIGFyIGMnaGVudGHDsQl8CSAgICAgICoqIEJyZWl6aCBoYSBMaW51eCBhdGF2ISAq KgpKZWYJCXwJCWh0dHA6Ly9tb2luZWpmLmZyZWUuZnIvCl9fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVs QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Thu, 3 Jul 2014 07:49:59 +0200 Subject: [PATCH] ASoC: tda998x: add a codec to the HDMI transmitter In-Reply-To: <20140702194252.GN410@sirena.org.uk> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> <20140702195154.47d6f6b4@armhf> <20140702194252.GN410@sirena.org.uk> Message-ID: <20140703074959.7c489912@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, 2 Jul 2014 20:42:52 +0100 Mark Brown wrote: > > 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). > > I'd expect this to return an error for the busy DAI rather than just > silently ignore it failing to start or (better) implement some control > to let the user select which of the DAIs is active. Mark, This is not an error. If the audio subsystem (DPCM, not the user) chooses to activate both I2S and S/PDIF, this means the HDMI audio may be taken either from I2S or from S/PDIF: both inputs have the right format and rate. -- 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 S1755399AbaGCFsN (ORCPT ); Thu, 3 Jul 2014 01:48:13 -0400 Received: from smtp2-g21.free.fr ([212.27.42.2]:7840 "EHLO smtp2-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751311AbaGCFsL convert rfc822-to-8bit (ORCPT ); Thu, 3 Jul 2014 01:48:11 -0400 Date: Thu, 3 Jul 2014 07:49:59 +0200 From: Jean-Francois Moine To: Mark Brown Cc: Andrew Lunn , 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: <20140703074959.7c489912@armhf> In-Reply-To: <20140702194252.GN410@sirena.org.uk> References: <20140702183841.7c964832@armhf> <20140702165628.GO20799@lunn.ch> <20140702195154.47d6f6b4@armhf> <20140702194252.GN410@sirena.org.uk> 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 20:42:52 +0100 Mark Brown wrote: > > 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). > > I'd expect this to return an error for the busy DAI rather than just > silently ignore it failing to start or (better) implement some control > to let the user select which of the DAIs is active. Mark, This is not an error. If the audio subsystem (DPCM, not the user) chooses to activate both I2S and S/PDIF, this means the HDMI audio may be taken either from I2S or from S/PDIF: both inputs have the right format and rate. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/