From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-bk0-x229.google.com (mail-bk0-x229.google.com [IPv6:2a00:1450:4008:c01::229]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority" (not verified)) by ozlabs.org (Postfix) with ESMTPS id 8253B2C024A for ; Sat, 17 Aug 2013 22:28:13 +1000 (EST) Received: by mail-bk0-f41.google.com with SMTP id na10so941878bkb.14 for ; Sat, 17 Aug 2013 05:28:09 -0700 (PDT) From: Tomasz Figa To: Sascha Hauer Subject: Re: [PATCH v5 1/2] ASoC: fsl: Add S/PDIF CPU DAI driver Date: Sat, 17 Aug 2013 14:28:04 +0200 Message-ID: <2805432.jMVhHxhr1m@flatron> In-Reply-To: <20130816101151.GQ26614@pengutronix.de> References: <20130816095357.GA4694@MrMyself> <20130816101151.GQ26614@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, alsa-devel@alsa-project.org, lars@metafoo.de, ian.campbell@citrix.com, pawel.moll@arm.com, swarren@wwwdotorg.org, festevam@gmail.com, Nicolin Chen , timur@tabi.org, rob.herring@calxeda.com, broonie@kernel.org, p.zabel@pengutronix.de, galak@codeaurora.org, shawn.guo@linaro.org, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Friday 16 of August 2013 12:11:51 Sascha Hauer wrote: > On Fri, Aug 16, 2013 at 05:53:58PM +0800, Nicolin Chen wrote: > > On Fri, Aug 16, 2013 at 10:56:32AM +0200, Sascha Hauer wrote: > > > > "tx<0-8>" Optional Tx clock source for spdif playback. > > > > > > > > If absent, will use core clock. > > > > The index from 0 to 8 is identical > > > > to the clock source list described > > > > in TxClk_Source bit of register STC. > > > > Multiple clock source are allowed > > > > for this tx clock source. The driver > > > > will select one source from them for > > > > each supported sample rate according > > > > to the clock rates of these provided > > > > clock sources. > > > > > > You mean tx<0-7> > > > > Yes. Thank you. > > > > > Also I would make this option required. Use a dummy clock for mux > > > inputs that are grounded for a specific SoC. > > > > Some clocks are not from CCM and we haven't defined in imx6q-clk.txt, > > so in most cases we can't provide a phandle for them, eg: spdif_ext. > > I think it's a bit hard to force it to be 'required'. An 'optional' > > looks more flexible to me and a default one is ensured even if it's > > missing. > > <&clks 0> is the dummy clock. This can be used for all input clocks not > defined by the SoC. Where does this assumption come from? Is it documented anywhere? What about cases when you have full description of clocks in device tree, with one node per clock and #clock-cells = <0>? Best regards, Tomasz