From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Ujfalusi Subject: Re: [PATCH 5/7] ARM: dts: Add nodes for missing omap4 interconnect target modules Date: Thu, 12 Oct 2017 12:10:01 +0300 Message-ID: <78eff2f9-1fe5-66f8-e754-344cb1ea0d97@ti.com> References: <20170929223411.9691-1-tony@atomide.com> <20170929223411.9691-6-tony@atomide.com> <20171011165850.jnaievrgcszcawh3@earth> <20171012084030.lshobr3ewu3t5goc@squirrel.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20171012084030.lshobr3ewu3t5goc@squirrel.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sebastian Reichel , Tony Lindgren , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, =?UTF-8?Q?Beno=c3=aet_Cousson?= , devicetree@vger.kernel.org, Laurent Pinchart , Liam Girdwood , Mark Brown , Mark Rutland , Mauro Carvalho Chehab , Nishanth Menon , Paul Walmsley , Rob Herring , Sakari Ailus , Tero Kristo , Tomi Valkeinen List-Id: devicetree@vger.kernel.org 77u/CgoKVGV4YXMgSW5zdHJ1bWVudHMgRmlubGFuZCBPeSwgUG9ya2thbGFua2F0dSAyMiwgMDAx ODAgSGVsc2lua2kuIFktdHVubnVzL0J1c2luZXNzIElEOiAwNjE1NTIxLTQuIEtvdGlwYWlra2Ev RG9taWNpbGU6IEhlbHNpbmtpCgpPbiAyMDE3LTEwLTEyIDExOjQwLCBNYXR0aGlqcyB2YW4gRHVp biB3cm90ZToKPiBPbiBUaHUsIE9jdCAxMiwgMjAxNyBhdCAwOTowNzo1NUFNICswMzAwLCBQZXRl ciBVamZhbHVzaSB3cm90ZToKPj4gSSBkb24ndCBzZWUgbXVjaCBiZW5lZml0IG9mIHVzaW5nIE1j QVNQIG9uIE9NQVA0LCBvbmx5IGRyYXdiYWNrcy4uLiBJdAo+PiBsYWNrcyBBRklGTyBzbyBpdCBp cyBwcm9uZSB0byB1bmRlcnJ1biBhbmQgaWYgaXQgdW5kZXJydW5zIHlvdSBuZWVkIHRvCj4+IHJl c2V0IE1jQVNQLiBCZWVuIHRoZXJlIHdpdGggdGhlIFNhbXN1bmcgR2FsYXh5IE5leHVzLgo+IAo+ IEFFU1MvQVRDIGlzbid0IGFibGUgdG8gY29uc2lzdGVudGx5IHJlc3BvbmQgdG8gYSBETUEgcmVx dWVzdCB3aXRoaW4gMjAKPiB1cyA/ICBTdHJhbmdlLi4gSSB3b3VsZCBoYXZlIGV4cGVjdGVkIGl0 IHRvIGhhdmUgbm8gdHJvdWJsZSB3aXRoIHRoYXQuCj4gKEFuZCBwcmVzdW1hYmx5IHNvIGRpZCBU SSwgb3IgdGhleSB3b3VsZCBoYXZlIGluY2x1ZGVkIEFGSUZPPykKPiAKPj4gSSB3b25kZXIgaG93 IE1vdG9yb2xhIGlzIHVzaW5nIHRoZSBNY0FTUCBhcyB0aGUgT01BUDQgdmVyc2lvbgo+PiAob2Zm aWNpYWxseSkgZG9lcyBub3Qgc3VwcG9ydCBJMlMuCj4gCj4gSG1tPyBJIGNoZWNrZWQgdGhlIG9t YXA0NDMwIFRSTSBhbmQgSSBkb24ndCBzZWUgdGhhdCBtZW50aW9uZWQuIEl0IHNlZW1zCj4gdG8g aGF2ZSBwaW5zIGZvciBpMnMgdHJhbnNtaXQ6Cj4gCj4gCWFiZV9tY2FzcF9heHIgICAgICAgICAg ICAgIE8gICAgIEF1ZGlvIHRyYW5zbWl0IGRhdGEKPiAJYWJlX21jYXNwX2FjbGt4ICAgICAgICAg ICAgTyAgICAgVHJhbnNtaXQgYml0IGNsb2NrCj4gCWFiZV9tY2FzcF9haGNsa3ggICAgICAgICAg IE8gICAgIFRyYW5zbWl0IGhpZ2gtZnJlcXVlbmN5IG1hc3RlciBjbG9jawo+IAlhYmVfbWNhc3Bf YWZzeCAgICAgICAgICAgICBPICAgICBUcmFuc21pdCBmcmFtZSBzeW5jaHJvbml6YXRpb24KPiAJ YWJlX21jYXNwX2FtdXRlICAgICAgICAgICAgTyAgICAgTXV0ZSBvdXQgdG8gZXh0ZXJuYWwKPiAJ YWJlX21jYXNwX2FtdXRlaW4gICAgICAgICAgSSAgICAgTXV0ZSBpbiBmcm9tIGV4dGVybmFsCj4g Cj4gSWYgaXQgc3VwcG9ydGVkIHMvcGRpZiB0eCBvbmx5LCB0aGVuIGFsbCBvZiB0aGVzZSBleGNl cHQgYXhyIHdvdWxkIGJlCj4gdXNlbGVzcy4KClRoZSBNY0FTUCB0cmFuc21pdCBzZWN0aW9uIHVz ZXMgdGhlIFMvUERJRiBpbmR1c3RyeSBmb3JtYXQsIHdoaWNoIGlzCnN1cHBvcnRlZCBieSB0aGUg RElUIHRyYW5zZmVyIG1vZGUgb2YgdGhlIG1vZHVsZS4KCkFuZCBsb29rIGZvciBUcmFuc2ZlciBN b2RlcyBjaGFwdGVyIG9mIE1jU1NQIGluIHRoZSBPTUFQNCBUUk0uCgpJdCBpcyBESVQgb25seS4K Cj4gCj4gTWF0dGhpanMKPiAKCi0gUMOpdGVyCgoKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgt YXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3Jn L21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: peter.ujfalusi@ti.com (Peter Ujfalusi) Date: Thu, 12 Oct 2017 12:10:01 +0300 Subject: [PATCH 5/7] ARM: dts: Add nodes for missing omap4 interconnect target modules In-Reply-To: <20171012084030.lshobr3ewu3t5goc@squirrel.local> References: <20170929223411.9691-1-tony@atomide.com> <20170929223411.9691-6-tony@atomide.com> <20171011165850.jnaievrgcszcawh3@earth> <20171012084030.lshobr3ewu3t5goc@squirrel.local> Message-ID: <78eff2f9-1fe5-66f8-e754-344cb1ea0d97@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org ? Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki On 2017-10-12 11:40, Matthijs van Duin wrote: > On Thu, Oct 12, 2017 at 09:07:55AM +0300, Peter Ujfalusi wrote: >> I don't see much benefit of using McASP on OMAP4, only drawbacks... It >> lacks AFIFO so it is prone to underrun and if it underruns you need to >> reset McASP. Been there with the Samsung Galaxy Nexus. > > AESS/ATC isn't able to consistently respond to a DMA request within 20 > us ? Strange.. I would have expected it to have no trouble with that. > (And presumably so did TI, or they would have included AFIFO?) > >> I wonder how Motorola is using the McASP as the OMAP4 version >> (officially) does not support I2S. > > Hmm? I checked the omap4430 TRM and I don't see that mentioned. It seems > to have pins for i2s transmit: > > abe_mcasp_axr O Audio transmit data > abe_mcasp_aclkx O Transmit bit clock > abe_mcasp_ahclkx O Transmit high-frequency master clock > abe_mcasp_afsx O Transmit frame synchronization > abe_mcasp_amute O Mute out to external > abe_mcasp_amutein I Mute in from external > > If it supported s/pdif tx only, then all of these except axr would be > useless. The McASP transmit section uses the S/PDIF industry format, which is supported by the DIT transfer mode of the module. And look for Transfer Modes chapter of McSSP in the OMAP4 TRM. It is DIT only. > > Matthijs > - P?ter