From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964972AbbD0SXj (ORCPT ); Mon, 27 Apr 2015 14:23:39 -0400 Received: from smtp5-g21.free.fr ([212.27.42.5]:51384 "EHLO smtp5-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932941AbbD0SXg convert rfc822-to-8bit (ORCPT ); Mon, 27 Apr 2015 14:23:36 -0400 X-Greylist: delayed 7200 seconds by postgrey-1.27 at vger.kernel.org; Mon, 27 Apr 2015 14:23:36 EDT Date: Mon, 27 Apr 2015 20:25:02 +0200 From: Jean-Francois Moine To: Jyri Sarha Cc: , , Subject: Re: [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter Message-ID: <20150427202502.2573af29@armhf> In-Reply-To: <553E1E99.9020101@ti.com> References: <553E1E99.9020101@ti.com> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.25; 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 Mon, 27 Apr 2015 14:33:45 +0300 Jyri Sarha wrote: > 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. Hi Jyri, I was busy on an other work, so, I did not advance on the tda998x. If you look at the patch series, it contains 3 independant parts: - dynamic building of the DAIs in the kirkwood audio controller, - audio CODEC of the tda998x - DT card based on audio graph It seems that only the last part raised a problem. Otherwise, the simple card works fine with the 2 first parts. I am merging my patches in 4.1-rc1, and, as soon as I get an image on my screen :), I will resubmit a patch series but about the tda998x codec only. Using i2s and s/pdif at the same time with the simple card asks for a patch as the one I submitted in february 2014 (ASoC: simple-card: DT fix and multi DAI links extension). As I am using my DT card, I will not resubmit it. Eventually, the kirkwood patches are not critical. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter Date: Mon, 27 Apr 2015 20:25:02 +0200 Message-ID: <20150427202502.2573af29@armhf> References: <553E1E99.9020101@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <553E1E99.9020101@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Jyri Sarha Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: alsa-devel@alsa-project.org T24gTW9uLCAyNyBBcHIgMjAxNSAxNDozMzo0NSArMDMwMApKeXJpIFNhcmhhIDxqc2FyaGFAdGku Y29tPiB3cm90ZToKCj4gSGF2ZSB5b3UgZG9uZSBhbnl0aGluZyBhYm91dCB0aGUgdGRhOTk4eCBh dWRpbyBzdXBwb3J0IGxhdGVseT8KPiAKPiBJIHdhcyB0aGlua2luZyBvZiB0YWtpbmcgYSBzaG90 IGF0IHRoaXMgbm93IHRoYXQgSSBmaW5hbGx5IHNlZW0gdG8gaGF2ZSAKPiBzb21lIHRpbWUgZm9y IGl0LiBIb3dldmVyLCBpZiB5b3UgYXJlIGp1c3QgYWJvdXQgdG8gc2VuZCBhbm90aGVyIHNlcmll cyAKPiBJJ2xsIHdhaXQgZm9yIHRoYXQgZmlyc3QgYW5kIHNlZSB3aGF0IG1ha2VzIHRoZSBtb3N0 IHNlbnNlIGFmdGVyIHRoYXQuCj4gCj4gTXkgcGxhbiBpcyB0byBkbyBzb21ldGhpbmcgcmVhbGx5 IHNpbXBsZSBmaXJzdC4gTm8gZ3JhcGggYmluZGluZ3Mgb3IgCj4gYW55dGhpbmcgZmFuY3kgbGlr ZSB0aGF0LiBKdXN0IGEgbWluaW1hbCBkdC1iaW5kaW5nIHRvIGZvciBhdWRpbyBwaW4gCj4gY29u ZmlndXJhdGlvbiBhbmQgdXNlIHNpbXBsZS1jYXJkIGZvciB0aGUgcmVzdC4gSSBzdGlsbCB0cnkg dG8gbWFrZSBpdCAKPiBwb3NzaWJsZSB0byBzdXBwb3J0IHNwZGlmIGFuZCBpMnMgYXQgdGhlIHNh bWUgdGltZSwgYnV0IEkgY2FuIG5vdCB0ZXN0IAo+IGl0IGFzIEkgZG8gbm90IGhhdmUgc3VjaCBI Vy4KPiAKPiBJIGFsc28gdHJ5IHRvIG1ha2UgdGhlIEFTb0MgcGFydCBhcyBnZW5lcmljIGFzIHBv c3NpYmxlLCBzbyB0aGF0IGl0IAo+IGNvdWxkIGJlIHJldXNlZCBieSBvdGhlciBIRE1JIGVuY29k ZXJzIHdpdGggc3BkaWYgb3IgaTJzIGludGVyZmFjZS4KCkhpIEp5cmksCgpJIHdhcyBidXN5IG9u IGFuIG90aGVyIHdvcmssIHNvLCBJIGRpZCBub3QgYWR2YW5jZSBvbiB0aGUgdGRhOTk4eC4KCklm IHlvdSBsb29rIGF0IHRoZSBwYXRjaCBzZXJpZXMsIGl0IGNvbnRhaW5zIDMgaW5kZXBlbmRhbnQg cGFydHM6Ci0gZHluYW1pYyBidWlsZGluZyBvZiB0aGUgREFJcyBpbiB0aGUga2lya3dvb2QgYXVk aW8gY29udHJvbGxlciwKLSBhdWRpbyBDT0RFQyBvZiB0aGUgdGRhOTk4eAotIERUIGNhcmQgYmFz ZWQgb24gYXVkaW8gZ3JhcGgKCkl0IHNlZW1zIHRoYXQgb25seSB0aGUgbGFzdCBwYXJ0IHJhaXNl ZCBhIHByb2JsZW0uCk90aGVyd2lzZSwgdGhlIHNpbXBsZSBjYXJkIHdvcmtzIGZpbmUgd2l0aCB0 aGUgMiBmaXJzdCBwYXJ0cy4KCkkgYW0gbWVyZ2luZyBteSBwYXRjaGVzIGluIDQuMS1yYzEsIGFu ZCwgYXMgc29vbiBhcyBJIGdldCBhbiBpbWFnZSBvbgpteSBzY3JlZW4gOiksIEkgd2lsbCByZXN1 Ym1pdCBhIHBhdGNoIHNlcmllcyBidXQgYWJvdXQgdGhlIHRkYTk5OHgKY29kZWMgb25seS4KClVz aW5nIGkycyBhbmQgcy9wZGlmIGF0IHRoZSBzYW1lIHRpbWUgd2l0aCB0aGUgc2ltcGxlIGNhcmQg YXNrcyBmb3IgYQpwYXRjaCBhcyB0aGUgb25lIEkgc3VibWl0dGVkIGluIGZlYnJ1YXJ5IDIwMTQg KEFTb0M6IHNpbXBsZS1jYXJkOiBEVApmaXggYW5kIG11bHRpIERBSSBsaW5rcyBleHRlbnNpb24p LiBBcyBJIGFtIHVzaW5nIG15IERUIGNhcmQsIEkgd2lsbApub3QgcmVzdWJtaXQgaXQuIEV2ZW50 dWFsbHksIHRoZSBraXJrd29vZCBwYXRjaGVzIGFyZSBub3QgY3JpdGljYWwuCgotLSAKS2VuIGFy IGMnaGVudGHDsQl8CSAgICAgICoqIEJyZWl6aCBoYSBMaW51eCBhdGF2ISAqKgpKZWYJCXwJCWh0 dHA6Ly9tb2luZWpmLmZyZWUuZnIvCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVk ZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8v ZHJpLWRldmVsCg==