From patchwork Sun Jun 15 23:39:45 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Anssi Hannula X-Patchwork-Id: 4355561 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id A4D349F1C4 for ; Sun, 15 Jun 2014 23:42:55 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id C735520221 for ; Sun, 15 Jun 2014 23:42:54 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id EE2FB2020E for ; Sun, 15 Jun 2014 23:42:53 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WwK2S-0001cR-QF; Sun, 15 Jun 2014 23:40:40 +0000 Received: from mail3.ippnet.fi ([62.197.168.3]) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WwK2P-0001En-8B for linux-arm-kernel@lists.infradead.org; Sun, 15 Jun 2014 23:40:38 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by mail3.ippnet.fi (Postfix) with ESMTP id 34E355D808; Mon, 16 Jun 2014 02:41:46 +0300 (EEST) X-Virus-Scanned: amavisd-new at kemia.fi Received: from mail3.ippnet.fi ([127.0.0.1]) by localhost (kone16815.ippnet.fi [127.0.0.1]) (amavisd-new, port 10030) with ESMTP id LAo6ywv2URyZ; Mon, 16 Jun 2014 02:41:42 +0300 (EEST) Received: from webmail.tpnet.fi (webmail.tpnet.fi [62.106.63.33]) by mail3.ippnet.fi (Postfix) with ESMTP id C86E35D790; Mon, 16 Jun 2014 02:41:39 +0300 (EEST) Received: from mail.onse.fi (host-109-204-178-176.tp-fne.tampereenpuhelin.net [109.204.178.176]) by webmail.tpnet.fi (Postfix) with ESMTP id D6BBB2073E; Mon, 16 Jun 2014 02:39:58 +0300 (EEST) Received: by mail.onse.fi (Postfix, from userid 501) id 95C3F40219; Mon, 16 Jun 2014 02:39:58 +0300 (EEST) From: Anssi Hannula To: Shawn Guo Subject: [PATCH 2/2] ARM: dts: imx6: remove non-working spdif rxtx4 and rxtx6 clocks Date: Mon, 16 Jun 2014 02:39:45 +0300 Message-Id: <1402875585-29025-2-git-send-email-anssi.hannula@iki.fi> X-Mailer: git-send-email 1.8.4.5 In-Reply-To: <1402875585-29025-1-git-send-email-anssi.hannula@iki.fi> References: <1402875585-29025-1-git-send-email-anssi.hannula@iki.fi> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20140615_164037_661669_4CF3F699 X-CRM114-Status: GOOD ( 12.28 ) X-Spam-Score: -0.7 (/) Cc: alsa-devel@alsa-project.org, Mark Brown , linux-arm-kernel@lists.infradead.org, Nicolin Chen X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-2.5 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP The S/PDIF rxtx4 and rxtx6 clock inputs are "ESAI_HCKT" and "MLB clock", respectively, according to the SoC documentation, and they are currently mapped to clocks "esai" and "mlb". However, they do not seem to actually work correctly. Testing on a Cubox-i system with fsl_spdif driver forced to select one of those as input will result in I/O errors on audio playback, which I believe means missing clock signal. Possibly the "ESAI_HCKT" and "MLB clock" refer to some other clocks related to ESAI and MLB, or we are missing something else. Since audio playback will not work if fsl_spdif selects these clocks (which happens rarely), set the inputs do dummy clocks, at least for now. Signed-off-by: Anssi Hannula Cc: Nicolin Chen Cc: Mark Brown --- Does anyone know what is wrong here? i.e. are the clocks actually wrong or are we maybe doing something wrong with their setup? Also, is this the correct way to "fix" the issue, or should we rather modify the driver to not select rxtx4/rxtx6 for now (assuming no one knows how to fix this properly, of course)? (I haven't been involved with device trees before) arch/arm/boot/dts/imx6qdl.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index 3aef9a2f5fda..f0d1c2334952 100644 --- a/arch/arm/boot/dts/imx6qdl.dtsi +++ b/arch/arm/boot/dts/imx6qdl.dtsi @@ -182,8 +182,8 @@ dma-names = "rx", "tx"; clocks = <&clks 197>, <&clks 3>, <&clks 197>, <&clks 0>, - <&clks 0>, <&clks 118>, - <&clks 0>, <&clks 139>, + <&clks 0>, <&clks 0>, + <&clks 0>, <&clks 0>, <&clks 0>; clock-names = "core", "rxtx0", "rxtx1", "rxtx2",