From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [PATCH 2/2] ARM: Dove: Add the audio device to the Cubox DT Date: Sun, 1 Sep 2013 09:04:20 +0200 Message-ID: <20130901090420.526e3bc3@armhf> References: <20130828113521.35c53365@armhf> <521DCD80.9010105@gmail.com> <20130831125128.0e3a23c7@armhf> <20130831112430.GD6617@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from smtp1-g21.free.fr (smtp1-g21.free.fr [212.27.42.1]) by alsa0.perex.cz (Postfix) with ESMTP id D5DF22607B6 for ; Sun, 1 Sep 2013 09:03:13 +0200 (CEST) In-Reply-To: <20130831112430.GD6617@n2100.arm.linux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Russell King - ARM Linux Cc: Mark Rutland , alsa-devel@alsa-project.org, Jason Cooper , Pawel Moll , Stephen Warren , Rob Herring , Ian Campbell , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: alsa-devel@alsa-project.org T24gU2F0LCAzMSBBdWcgMjAxMyAxMjoyNDozMSArMDEwMApSdXNzZWxsIEtpbmcgLSBBUk0gTGlu dXggPGxpbnV4QGFybS5saW51eC5vcmcudWs+IHdyb3RlOgoKPiA+IEFjY29yZGluZyB0byB0aGUg RG92ZSBzcGVjaWZpY2F0aW9uLCB0aGUgYXVkaW8gY29udHJvbGxlciB3b3JrcyB3aXRoCj4gPiB0 aGUgc2FtcGxlcyByYXRlcyA0NC4xLCA0OCBhbmQgOTYga0h6LCBzbywgSSBkb24ndCBzZWUgdGhl IHVzYWdlIG9mIHRoZQo+ID4gZXh0ZXJuYWwgY2xvY2ssIGV4Y2VwdCB3aGVuIHVzaW5nIHRoZSB0 d28gYXVkaW8gY29udHJvbGxlcnMgd2l0aAo+ID4gZGlmZmVyZW50IHNhbXBsZSByYXRlcy4gIAo+ IAo+IEkgZG9uJ3Qgc2VlIHdoYXQgdGhlIERvdmUgc3BlY2lmaWNhdGlvbiBoYXMgdG8gZG8gd2l0 aCB0aGF0IHN0YXRlbWVudDoKPiB3aGF0IHRoZSBEb3ZlIHNwZWMgc2F5cyBpcyB0aGF0IGlmIHlv dSB1c2UganVzdCB0aGUgaW50ZXJuYWwgRENPLCB0aGVuCj4gb25seSA0NC4xa0h6LCA0OGtIeiBh bmQgOTZrSHogYXJlIHN1cHBvcnRlZCAod2l0aCBzb21lIHRyaW1taW5nIG9mIHRoYXQuKQo+IEhv d2V2ZXIsIHRoZSB1c2Ugb2YgYW4gZXh0ZXJuYWwgY2xvY2sgYWxsb3dzIGZ1cnRoZXIgcmF0ZXMg dG8gYmUgc3VwcG9ydGVkLgo+IElmIHlvdSBoYXZlIGFuIGV4dGVybmFsIGNsb2NrLCB0aGVyZSBp cyBubyByZXF1aXJlbWVudCB0byB1c2UgdGhlIERDTyBmb3IKPiB0aG9zZSBzYW1wbGUgcmF0ZXMg LSB5b3UgY2FuIGlmIHlvdSB3aXNoLCBvciB5b3UgY2FuIHVzZSB0aGUgZXh0ZXJuYWwgY2xvY2su Cj4gCj4gVGhlIG1haW5saW5lIGRyaXZlciBpbXBsZW1lbnRzIHRoZSB1c2Ugb2YgdGhlIERDTyBm b3IgdGhlIHN0YW5kYXJkIDQ0LjEsCj4gNDggYW5kIDk2a0h6IHJhdGVzLCBvdGhlcndpc2UgaXQg dXNlcyB0aGUgZXh0ZXJuYWwgY2xvY2sgaWYgcHJlc2VudC4gIFRoaXMKPiBpcyBlbnRpcmVseSBj b25mb3JtYW50IHdpdGggdGhlIERvdmUgc3BlYy4KClJ1c3NlbGwsCgpJIGxvb2tlZCBhZ2FpbiBh dCB0aGUgQXJtYWRhLTUxMC1GdW5jdGlvbmFsLVNwZWMsIGFuZCBJIGZvdW5kOgoKLSBzZWN0aW9u IDEyLjEKCS4uLgoJSTJTIFN1cHBvcnRlZCBGZWF0dXJlcwoJCUFuIGF1ZGlvIHNhbXBsZSByYXRl IChGcykgb2YgNDQuMS80OC85NiBrSHouCgkJLi4uCglTL1BESUYgU3VwcG9ydGVkIEZlYXR1cmVz CgkJVGhlIElFQzYwOTU4LTEsIElFQzYwOTU4LTMsIGFuZCBJRUM2MTkzNyBzcGVjaWZpY2F0aW9u cy4KCQlBbiBhdWRpbyBzYW1wbGUgcmF0ZSAoRnMpIG9mIDQ0LjEvNDgvOTYga0h6LgoJCS4uLgoK LSBUYWJsZSA0MzY6IENsb2NrcyBDb250cm9sIFJlZ2lzdGVyCgkgICAgIE9mZnNldDogVW5pdDA6 IDB4MDAwQjEyMzAsIFVuaXQxOiAweDAwMEI1MjMwCgkuLi4KCTE6MCBNQ0xLIHNvdXJjZSBSVyAg TUNMSyA9IDI1NiBGcywgd2hlcmUgRnMgPSA0NC4xIGtIeiwgNDgga0h6LCA5NiBrSHouCgkJCTB4 MCBQbGF5YmFjayBzaG91bGQgbm90IGJlIGFjdGl2ZSB3aGVuIHNlbGVjdGluZyBhbiBleHRlcm5h bCBNQ0xLLCBvciBtb3ZpbmcKCQkJICAgIGZyb20gb25lIGV4dGVybmFsIE1DTEsgdG8gYW5vdGhl ciBNQ0xLIHNvdXJjZS4KCQkJICAgIDAgPSBEQ086IE1DTEsgaXMgZnJvbSBEQ08uCgkJCSAgICAx ID0gUmVzZXJ2ZWQKCQkJICAgIDIgPSBSZXNlcnZlZAoJCQkgICAgMyA9IEV4dGVybmFsOiBNQ0xL IGlzIGZyb20gYW4gZXh0ZXJuYWwgc291cmNlLgoKQXMgdGhleSBkb24ndCB3b3JrIHdpdGggbXkg VFYgc2V0LCBtYXkgeW91IGNvbmZpcm0gdGhhdCB0aGVzZSB3ZWJyYWRpb3MKCglodHRwOi8va2Vy bmUub25saW5lLnN0YWxpZy5uZXQvbGl2ZS1vcmkub2dnCglodHRwOi8vd3d3LnR2LXJhZGlvLmNv bS9zdGF0aW9uL2ZyYW5jZV9ibGV1X2JyZWl6X2l6ZWxfbXAzL2ZyYW5jZV9ibGV1X2JyZWl6X2l6 ZWxfbXAzLTMyay5tM3UKCWh0dHA6Ly9wbHVtZm0ub25saW5lLnN0YWxpZy5uZXQvbGl2ZS1vcmku b2dnCgp3b3JrIGZvciB5b3Ugd2l0aCB0aGUgZXh0ZXJuYWwgY2xvY2s/IChzYW1wbGUgcmF0ZXMg cmVzcC4gMzMuMDc1LCAyNAphbmQgMjIuMDUga0h6KQoKLS0gCktlbiBhciBjJ2hlbnRhw7EJfAkg ICAgICAqKiBCcmVpemggaGEgTGludXggYXRhdiEgKioKSmVmCQl8CQlodHRwOi8vbW9pbmVqZi5m cmVlLmZyLwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpB bHNhLWRldmVsIG1haWxpbmcgbGlzdApBbHNhLWRldmVsQGFsc2EtcHJvamVjdC5vcmcKaHR0cDov L21haWxtYW4uYWxzYS1wcm9qZWN0Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Fsc2EtZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Sun, 1 Sep 2013 09:04:20 +0200 Subject: [PATCH 2/2] ARM: Dove: Add the audio device to the Cubox DT In-Reply-To: <20130831112430.GD6617@n2100.arm.linux.org.uk> References: <20130828113521.35c53365@armhf> <521DCD80.9010105@gmail.com> <20130831125128.0e3a23c7@armhf> <20130831112430.GD6617@n2100.arm.linux.org.uk> Message-ID: <20130901090420.526e3bc3@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, 31 Aug 2013 12:24:31 +0100 Russell King - ARM Linux wrote: > > According to the Dove specification, the audio controller works with > > the samples rates 44.1, 48 and 96 kHz, so, I don't see the usage of the > > external clock, except when using the two audio controllers with > > different sample rates. > > I don't see what the Dove specification has to do with that statement: > what the Dove spec says is that if you use just the internal DCO, then > only 44.1kHz, 48kHz and 96kHz are supported (with some trimming of that.) > However, the use of an external clock allows further rates to be supported. > If you have an external clock, there is no requirement to use the DCO for > those sample rates - you can if you wish, or you can use the external clock. > > The mainline driver implements the use of the DCO for the standard 44.1, > 48 and 96kHz rates, otherwise it uses the external clock if present. This > is entirely conformant with the Dove spec. Russell, I looked again at the Armada-510-Functional-Spec, and I found: - section 12.1 ... I2S Supported Features An audio sample rate (Fs) of 44.1/48/96 kHz. ... S/PDIF Supported Features The IEC60958-1, IEC60958-3, and IEC61937 specifications. An audio sample rate (Fs) of 44.1/48/96 kHz. ... - Table 436: Clocks Control Register Offset: Unit0: 0x000B1230, Unit1: 0x000B5230 ... 1:0 MCLK source RW MCLK = 256 Fs, where Fs = 44.1 kHz, 48 kHz, 96 kHz. 0x0 Playback should not be active when selecting an external MCLK, or moving from one external MCLK to another MCLK source. 0 = DCO: MCLK is from DCO. 1 = Reserved 2 = Reserved 3 = External: MCLK is from an external source. As they don't work with my TV set, may you confirm that these webradios http://kerne.online.stalig.net/live-ori.ogg http://www.tv-radio.com/station/france_bleu_breiz_izel_mp3/france_bleu_breiz_izel_mp3-32k.m3u http://plumfm.online.stalig.net/live-ori.ogg work for you with the external clock? (sample rates resp. 33.075, 24 and 22.05 kHz) -- Ken ar c'henta? | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/