From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?R2HDq3RhbiBDYXJsaWVy?= Subject: Re: mx27_3ds : Sound doesn't work anymore on linux-next Date: Tue, 21 Aug 2012 17:14:47 +0200 Message-ID: <5033A5E7.2030701@gmail.com> References: <502E6033.4010805@gmail.com> <502E6329.60308@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-ey0-f179.google.com (mail-ey0-f179.google.com [209.85.215.179]) by alsa0.perex.cz (Postfix) with ESMTP id 1B12E266003 for ; Tue, 21 Aug 2012 17:15:40 +0200 (CEST) Received: by eaa13 with SMTP id 13so1886261eaa.38 for ; Tue, 21 Aug 2012 08:15:41 -0700 (PDT) In-Reply-To: <502E6329.60308@gmail.com> 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: Fabio Estevam Cc: alsa-devel@alsa-project.org, "linux-arm-kernel@lists.infradead.org" List-Id: alsa-devel@alsa-project.org SGksCk9uIDA4LzE3LzIwMTIgMDU6MjggUE0sIEdhw6t0YW4gQ2FybGllciB3cm90ZToKPiBKdXN0 IHNvbWUgbW9yZSBpbmZvcm1hdGlvbnMsIHByb2Nlc3MgZW50ZXIgaW4gRCsgc3RhdGUgYW5kIHRo aXMgaXMgc28KPiBpbXBvc3NpYmxlIHRvIGtpbGwgdGhlbS4KPiBPbiAwOC8xNy8yMDEyIDA1OjE2 IFBNLCBHYcOrdGFuIENhcmxpZXIgd3JvdGU6Cj4+IEhpLAo+PiBJdCBpcyBzZXZlcmFsIGRheXMg dGhhdCBJIHRyaWVkIHRvIG1ha2Ugc291bmQgd29yayBvbiBteDI3XzNkcyBwbGF0Zm9ybQo+PiBm cm9tIGxpbnV4LW5leHQgKGRhaWx5IGNsb25lZC9wdWxsZWQpIGJ1dCBlYWNoIHRpbWUgSSB0cnkg c3RhcnQgc3RhcnQKPj4gYXBsYXkgb3IgYWxzYW1peGVyLCB0aGUgcHJvY2VzcyBmcmVlemUgYW5k IENUUkwrQyBoYXMgbm90IGVmZmVjdC4gSSBjYW4KPj4gb3BlbiBhbm90aGVyIFNTSCBzZXNzaW9u IGFuZCBleGVjdXRlIG90aGVyIGNvbW1hbmRzIGJ1dCBhbHNhIGNvbW1hbmRzCj4+IGFsd2F5cyBm cmVlemUuCj4+IFRoZSBzb3VuZCBjYXJkIGlzIGNvcnJlY3RseSBpbnN0YWxsZXIgYXQgYm9vdCA6 Cj4+Cj4+IGlteF9tYzEzNzgzIGlteF9tYzEzNzgzLjA6ICBtYzEzNzgzLWhpZmkgPC0+IGlteC1z c2kuMCBtYXBwaW5nIG9rCj4+IEFMU0EgZGV2aWNlIGxpc3Q6Cj4+ICAgICMwOiBpbXhfbWMxMzc4 Mwo+Pgo+PiBJIG9ubHkgYXBwbHkgdGhlIHBhdGNoICJbUEFUQ0ggMS8yXSBBUk06IG14MjdwZGs6 IEFkZCBhdWRpbyBzdXBwb3J0Ii4gSQo+PiBhdHRhY2ggbXkgY29uZmlnIGZpbGUuCj4+Cj4+IEkg dHJ5IHRvIGFwcGx5IHBhdGNoIG9uIGEgcHJldmlvdXMgY29tbWl0IHdoZW4gSSB0b2xkIHRoYXQg dGhpcyBwYXRjaAo+PiB3YXMgd29ya2luZyBidXQgaXQgZGlkbid0IHN1Y2NlZWQgdGhlIGlzIHN0 aWxsIGZyZWV6aW5nLgo+PiBJIHRyeSB0byBtYWtlIGl0IHdvcmtpbmcgb24gYW5vdGhlciBteDI3 LWJhc2VkIHBsYXRmb3JtIGJ1dCBzYW1lCj4+IHByb2JsZW0uIEkgdHJ5IHRvIHVwZGF0ZSBBbHNh IGZyb20gdmVyc2lvbiAxLjAuMjMgdG8gMS4wLjI1IGJ1dCBzYW1lCj4+IGJlaGF2aW91ci4KPj4g V2hhdCBpcyB3cm9uZyA/Cj4+Cj4+IFJlZ2FyZHMsCj4+IEdhw6t0YW4gQ2FybGllci4KPgo+IFJl Z2FyZHMsCj4gR2HDq3Rhbi4KPgpJIGhhdmUgbW9yZSBpbmZvcm1hdGlvbnMgYWJvdXQgdGhlIHBy b2JsZW0gOiBmcmVlemUgb25seSBvY2N1cnMgd2hlbiBJIApkaXNhYmxlIE1NQyBkZXZpY2Ugc3Vw cG9ydCAoQ09ORklHX01NQ19NWEM9bikuCkkgZG9uJ3Qga25vdyB3aHkgdGhlc2UgdHdvIGRyaXZl cnMgYWZmZWN0IGVhY2ggb3RoZXIgKGV4Y2VwdCB0aGF0IENTUEkgCmZvciBNQzEzNzgzIGFuZCBT REhDIHVzZSB0aGUgc2FtZSBQRVJESVYxIGNsb2NrKS4KU28gSSBlbmFibGUgaXQgYmFjayBhbmQg dHJ5IHB1dCBhICJyZXR1cm4gMCIgaW4gbXhjbW1jLmMgKG9ubHkgZm9yIGRlYnVnIApwdXJwb3Nl KSBhcyBmb2xsb3dpbmcgOgoKIyMjIFRoaXMgbWFrZXMgYWxzYW1peGVyIHdvcmtpbmcKQEAgLTkx MywxMCArOTEzLDExIEBAIHN0YXRpYyBpbnQgbXhjbWNpX3Byb2JlKHN0cnVjdCBwbGF0Zm9ybV9k ZXZpY2UgKnBkZXYpCiAgCQlnb3RvIG91dF9pb3VubWFwOwogIAl9CgogIAljbGtfcHJlcGFyZV9l bmFibGUoaG9zdC0+Y2xrX3Blcik7CiAgCWNsa19wcmVwYXJlX2VuYWJsZShob3N0LT5jbGtfaXBn KTsKK3JldHVybiAwOwoKICAJbXhjbWNpX3NvZnRyZXNldChob3N0KTsKCiAgCWhvc3QtPnJldl9u byA9IHJlYWR3KGhvc3QtPmJhc2UgKyBNTUNfUkVHX1JFVl9OTyk7CiAgCWlmIChob3N0LT5yZXZf bm8gIT0gMHg0MDApIHsKCiMjIyBUaGlzIG1ha2VzIGFsc2FtaXhlciBmcmVlemluZwpAQCAtODk4 LDEwICs4OTgsMTEgQEAgc3RhdGljIGludCBteGNtY2lfcHJvYmUoc3RydWN0IHBsYXRmb3JtX2Rl dmljZSAqcGRldikKICAJZWxzZQogIAkJaG9zdC0+ZGVmYXVsdF9pcnFfbWFzayA9IDA7CgogIAlo b3N0LT5yZXMgPSByOwogIAlob3N0LT5pcnEgPSBpcnE7CityZXR1cm4gMDsKCiAgCWhvc3QtPmNs a19pcGcgPSBkZXZtX2Nsa19nZXQoJnBkZXYtPmRldiwgImlwZyIpOwogIAlpZiAoSVNfRVJSKGhv c3QtPmNsa19pcGcpKSB7CiAgCQlyZXQgPSBQVFJfRVJSKGhvc3QtPmNsa19pcGcpOwogIAkJZ290 byBvdXRfaW91bm1hcDsKCgpTbyBJIHN1cHBvc2VkIHRoYXQgdGhlcmUgaXMgYSBwcm9ibGVtIGlu IGNsb2NrIGRlZmluaXRpb24gYnV0IEkgZG9uJ3QgCm5vdyBob3cgdG8gc29sdmUgaXQuIEkgdHJ5 IHRvIHJlcGxhY2UgaW4gY2xrLWlteDI3LmMKY2xrX3JlZ2lzdGVyX2Nsa2RldihjbGtbc3NpMV9p cGdfZ2F0ZV0sIE5VTEwsICJpbXgtc3NpLjAiKTsKYnkKY2xrX3JlZ2lzdGVyX2Nsa2RldihjbGtb c3NpMV9pcGdfZ2F0ZV0sICJpcGciLCAiaW14LXNzaS4wIik7CgphbmQgaW4gaW14LXNzaS5jIGlu IGlteF9zc2lfcHJvYmUoKSwgcmVwbGFjZWQKc3NpLT5jbGsgPSBkZXZtX2Nsa19nZXQoJnBkZXYt PmRldiwgImlwZyIpOwpieQpzc2ktPmNsayA9IGRldm1fY2xrX2dldCgmcGRldi0+ZGV2LCAiaXBn Iik7CgpUaGlzIGhhcyBubyBlZmZlY3QuIEkgZG9uJ3QgZnVsbHkga25vdyBob3cgY2xvY2sgbWVj aGFuaXNtIGZvciBpbXgyNyAKd29ya3Mgc28gSSBhbSBub3QgYWJsZSB0byBzb2x2ZSB0aGlzIHBy b2JsZW0gYW5kIHNlbmQgYSBwYXRjaC4KUmVnYXJkcywKR2HDq3RhbiBDYXJsaWVyLgoKX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KQWxzYS1kZXZlbCBtYWls aW5nIGxpc3QKQWxzYS1kZXZlbEBhbHNhLXByb2plY3Qub3JnCmh0dHA6Ly9tYWlsbWFuLmFsc2Et cHJvamVjdC5vcmcvbWFpbG1hbi9saXN0aW5mby9hbHNhLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: gcembed@gmail.com (=?UTF-8?B?R2HDq3RhbiBDYXJsaWVy?=) Date: Tue, 21 Aug 2012 17:14:47 +0200 Subject: mx27_3ds : Sound doesn't work anymore on linux-next In-Reply-To: <502E6329.60308@gmail.com> References: <502E6033.4010805@gmail.com> <502E6329.60308@gmail.com> Message-ID: <5033A5E7.2030701@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 08/17/2012 05:28 PM, Ga?tan Carlier wrote: > Just some more informations, process enter in D+ state and this is so > impossible to kill them. > On 08/17/2012 05:16 PM, Ga?tan Carlier wrote: >> Hi, >> It is several days that I tried to make sound work on mx27_3ds platform >> from linux-next (daily cloned/pulled) but each time I try start start >> aplay or alsamixer, the process freeze and CTRL+C has not effect. I can >> open another SSH session and execute other commands but alsa commands >> always freeze. >> The sound card is correctly installer at boot : >> >> imx_mc13783 imx_mc13783.0: mc13783-hifi <-> imx-ssi.0 mapping ok >> ALSA device list: >> #0: imx_mc13783 >> >> I only apply the patch "[PATCH 1/2] ARM: mx27pdk: Add audio support". I >> attach my config file. >> >> I try to apply patch on a previous commit when I told that this patch >> was working but it didn't succeed the is still freezing. >> I try to make it working on another mx27-based platform but same >> problem. I try to update Alsa from version 1.0.23 to 1.0.25 but same >> behaviour. >> What is wrong ? >> >> Regards, >> Ga?tan Carlier. > > Regards, > Ga?tan. > I have more informations about the problem : freeze only occurs when I disable MMC device support (CONFIG_MMC_MXC=n). I don't know why these two drivers affect each other (except that CSPI for MC13783 and SDHC use the same PERDIV1 clock). So I enable it back and try put a "return 0" in mxcmmc.c (only for debug purpose) as following : ### This makes alsamixer working @@ -913,10 +913,11 @@ static int mxcmci_probe(struct platform_device *pdev) goto out_iounmap; } clk_prepare_enable(host->clk_per); clk_prepare_enable(host->clk_ipg); +return 0; mxcmci_softreset(host); host->rev_no = readw(host->base + MMC_REG_REV_NO); if (host->rev_no != 0x400) { ### This makes alsamixer freezing @@ -898,10 +898,11 @@ static int mxcmci_probe(struct platform_device *pdev) else host->default_irq_mask = 0; host->res = r; host->irq = irq; +return 0; host->clk_ipg = devm_clk_get(&pdev->dev, "ipg"); if (IS_ERR(host->clk_ipg)) { ret = PTR_ERR(host->clk_ipg); goto out_iounmap; So I supposed that there is a problem in clock definition but I don't now how to solve it. I try to replace in clk-imx27.c clk_register_clkdev(clk[ssi1_ipg_gate], NULL, "imx-ssi.0"); by clk_register_clkdev(clk[ssi1_ipg_gate], "ipg", "imx-ssi.0"); and in imx-ssi.c in imx_ssi_probe(), replaced ssi->clk = devm_clk_get(&pdev->dev, "ipg"); by ssi->clk = devm_clk_get(&pdev->dev, "ipg"); This has no effect. I don't fully know how clock mechanism for imx27 works so I am not able to solve this problem and send a patch. Regards, Ga?tan Carlier.