From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753564AbbAMTyg (ORCPT ); Tue, 13 Jan 2015 14:54:36 -0500 Received: from gw-1.arm.linux.org.uk ([78.32.30.217]:37762 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753450AbbAMTye (ORCPT ); Tue, 13 Jan 2015 14:54:34 -0500 Date: Tue, 13 Jan 2015 19:54:15 +0000 From: Russell King - ARM Linux To: Jyri Sarha Cc: Jean-Francois Moine , Philipp Zabel , Andrew Jackson , Mark Brown , Dave Airlie , "alsa-devel@alsa-project.org" , "devicetree@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio Message-ID: <20150113195415.GV12302@n2100.arm.linux.org.uk> References: <20150112140456.GI12302@n2100.arm.linux.org.uk> <20150112181341.4a53ea24@armhf> <20150112175705.GN12302@n2100.arm.linux.org.uk> <1421151718.4519.24.camel@pengutronix.de> <20150113122715.GP12302@n2100.arm.linux.org.uk> <20150113165411.5469930d@armhf> <20150113160313.GR12302@n2100.arm.linux.org.uk> <20150113200252.3dbfab14@armhf> <20150113192648.GU12302@n2100.arm.linux.org.uk> <54B574CD.9020905@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <54B574CD.9020905@ti.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 13, 2015 at 09:41:01PM +0200, Jyri Sarha wrote: > On 01/13/2015 09:26 PM, Russell King - ARM Linux wrote: > >SCLK: _~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_ > > WS: __~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~________________________________~ > >I2S1: llmm............................llmm............................llm > >I2S2: llmm............................llmm............................llm > >I2S3: llmm............................llmm............................llm > >I2S4: llmm............................llmm............................llm > > > >So, what I'm saying is that it is_impossible_ to drive the TDA998x using > >multiple I2S streams which are not produced by the same I2S block. > > This is besides the point, but it is possible that one of the multiple I2S > blocks is the bit-clock and frame-clock master to the i2s bus and the others > are slaves to it (banging their bits according to SCLK and WS of the I2S > master). However, in this situation there really is only one i2s bus with > multiple data pins. > > Just my 0.02€ to this discussion. Right, that's about the only way it could work. To represent that in DT, I would imagine we'd need something like this: #address-cells = <1>; #size-cells = <0>; ... port@1 { /* AP1,2 = I2S */ #address-cells = <1>; #size-cells = <0>; port-type = "i2s"; reg = <0x01>; /* WS */ tda998x_i2s1: endpoint@2 { reg = <0x02>; /* AP1 */ remote-endpoint = <&audio1_i2s>; }; tda998x_i2s2: endpoint@4 { reg = <0x04>; /* AP2 */ remote-endpoint = <&audio2_i2s>; }; }; where audio1_i2s is operating in master mode, and audio2_i2s is operating in slave mode for both WS and SCLK. If we can agree on that, then I'm happy with the proposed binding. (Remember that #address-cells and #size-cells are required in the parent where we have reg= in the child.) -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio Date: Tue, 13 Jan 2015 19:54:15 +0000 Message-ID: <20150113195415.GV12302@n2100.arm.linux.org.uk> References: <20150112140456.GI12302@n2100.arm.linux.org.uk> <20150112181341.4a53ea24@armhf> <20150112175705.GN12302@n2100.arm.linux.org.uk> <1421151718.4519.24.camel@pengutronix.de> <20150113122715.GP12302@n2100.arm.linux.org.uk> <20150113165411.5469930d@armhf> <20150113160313.GR12302@n2100.arm.linux.org.uk> <20150113200252.3dbfab14@armhf> <20150113192648.GU12302@n2100.arm.linux.org.uk> <54B574CD.9020905@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <54B574CD.9020905@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" , "devicetree@vger.kernel.org" , Andrew Jackson , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Mark Brown List-Id: devicetree@vger.kernel.org T24gVHVlLCBKYW4gMTMsIDIwMTUgYXQgMDk6NDE6MDFQTSArMDIwMCwgSnlyaSBTYXJoYSB3cm90 ZToKPiBPbiAwMS8xMy8yMDE1IDA5OjI2IFBNLCBSdXNzZWxsIEtpbmcgLSBBUk0gTGludXggd3Jv dGU6Cj4gPlNDTEs6IF9+X35ffl9+X35ffl9+X35ffl9+X35ffl9+X35ffl9+X35ffl9+X35ffl9+ X35ffl9+X35ffl9+X35ffl9+X35ffl8KPiA+ICAgV1M6IF9ffn5+fn5+fn5+fn5+fn5+fn5+fn5+ fn5+fn5+fn5+fn5fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX34KPiA+STJTMTogbGxt bS4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi5sbG1tLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4u Li4uLmxsbQo+ID5JMlMyOiBsbG1tLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLmxsbW0uLi4u Li4uLi4uLi4uLi4uLi4uLi4uLi4uLi4ubGxtCj4gPkkyUzM6IGxsbW0uLi4uLi4uLi4uLi4uLi4u Li4uLi4uLi4uLi4ubGxtbS4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi5sbG0KPiA+STJTNDog bGxtbS4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi4uLi5sbG1tLi4uLi4uLi4uLi4uLi4uLi4uLi4u Li4uLi4uLmxsbQo+ID4KPiA+U28sIHdoYXQgSSdtIHNheWluZyBpcyB0aGF0IGl0IGlzX2ltcG9z c2libGVfICB0byBkcml2ZSB0aGUgVERBOTk4eCB1c2luZwo+ID5tdWx0aXBsZSBJMlMgc3RyZWFt cyB3aGljaCBhcmUgbm90IHByb2R1Y2VkIGJ5IHRoZSBzYW1lIEkyUyBibG9jay4KPiAKPiBUaGlz IGlzIGJlc2lkZXMgdGhlIHBvaW50LCBidXQgaXQgaXMgcG9zc2libGUgdGhhdCBvbmUgb2YgdGhl IG11bHRpcGxlIEkyUwo+IGJsb2NrcyBpcyB0aGUgYml0LWNsb2NrIGFuZCBmcmFtZS1jbG9jayBt YXN0ZXIgdG8gdGhlIGkycyBidXMgYW5kIHRoZSBvdGhlcnMKPiBhcmUgc2xhdmVzIHRvIGl0IChi YW5naW5nIHRoZWlyIGJpdHMgYWNjb3JkaW5nIHRvIFNDTEsgYW5kIFdTIG9mIHRoZSBJMlMKPiBt YXN0ZXIpLiBIb3dldmVyLCBpbiB0aGlzIHNpdHVhdGlvbiB0aGVyZSByZWFsbHkgaXMgb25seSBv bmUgaTJzIGJ1cyB3aXRoCj4gbXVsdGlwbGUgZGF0YSBwaW5zLgo+IAo+IEp1c3QgbXkgMC4wMuKC rCB0byB0aGlzIGRpc2N1c3Npb24uCgpSaWdodCwgdGhhdCdzIGFib3V0IHRoZSBvbmx5IHdheSBp dCBjb3VsZCB3b3JrLgoKVG8gcmVwcmVzZW50IHRoYXQgaW4gRFQsIEkgd291bGQgaW1hZ2luZSB3 ZSdkIG5lZWQgc29tZXRoaW5nIGxpa2UgdGhpczoKCgkjYWRkcmVzcy1jZWxscyA9IDwxPjsKCSNz aXplLWNlbGxzID0gPDA+OwoJLi4uCiAgICAgICAgcG9ydEAxIHsgICAgICAgICAgICAgICAgICAg ICAgICAvKiBBUDEsMiA9IEkyUyAqLwoJCSNhZGRyZXNzLWNlbGxzID0gPDE+OwoJCSNzaXplLWNl bGxzID0gPDA+OwogICAgICAgICAgICAgICAgcG9ydC10eXBlID0gImkycyI7CiAgICAgICAgICAg ICAgICByZWcgPSA8MHgwMT47CQkvKiBXUyAqLwogICAgICAgICAgICAgICAgdGRhOTk4eF9pMnMx OiBlbmRwb2ludEAyIHsKCQkJcmVnID0gPDB4MDI+OwkvKiBBUDEgKi8KICAgICAgICAgICAgICAg ICAgICAgICAgcmVtb3RlLWVuZHBvaW50ID0gPCZhdWRpbzFfaTJzPjsKICAgICAgICAgICAgICAg IH07CiAgICAgICAgICAgICAgICB0ZGE5OTh4X2kyczI6IGVuZHBvaW50QDQgewoJCQlyZWcgPSA8 MHgwND47CS8qIEFQMiAqLwogICAgICAgICAgICAgICAgICAgICAgICByZW1vdGUtZW5kcG9pbnQg PSA8JmF1ZGlvMl9pMnM+OwogICAgICAgICAgICAgICAgfTsKICAgICAgICB9OwoKd2hlcmUgYXVk aW8xX2kycyBpcyBvcGVyYXRpbmcgaW4gbWFzdGVyIG1vZGUsIGFuZCBhdWRpbzJfaTJzIGlzCm9w ZXJhdGluZyBpbiBzbGF2ZSBtb2RlIGZvciBib3RoIFdTIGFuZCBTQ0xLLgoKSWYgd2UgY2FuIGFn cmVlIG9uIHRoYXQsIHRoZW4gSSdtIGhhcHB5IHdpdGggdGhlIHByb3Bvc2VkIGJpbmRpbmcuCihS ZW1lbWJlciB0aGF0ICNhZGRyZXNzLWNlbGxzIGFuZCAjc2l6ZS1jZWxscyBhcmUgcmVxdWlyZWQg aW4gdGhlCnBhcmVudCB3aGVyZSB3ZSBoYXZlIHJlZz0gaW4gdGhlIGNoaWxkLikKCi0tIApGVFRD IGJyb2FkYmFuZCBmb3IgMC44bWlsZSBsaW5lOiBjdXJyZW50bHkgYXQgMTAuNU1icHMgZG93biA0 MDBrYnBzIHVwCmFjY29yZGluZyB0byBzcGVlZHRlc3QubmV0LgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1k ZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo=