From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,USER_AGENT_MUTT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E5F78C10F14 for ; Sun, 21 Apr 2019 08:21:23 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B537D2087F for ; Sun, 21 Apr 2019 08:21:23 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="QjB1w5ry"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Lb/UFZ0u" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B537D2087F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=d3ddPwVeRVUap0fo4r1Mfm/CURJtUqj+CmmTjCMpqVs=; b=QjB1w5rys/Y2ht INHZATVu8LF/XASOlJ2xDmgV2zhvQY6gsPCZyXBBw3TuasrJ9B6FCke4f4blRww3LSh5nw001FlHU ovmukIIiOn4W0MnWXLCuaarHawaY6T3m3JsgbiZVltNGG92KHTBdhBevvj3PIHgQImGpcquLvBjL3 ISa3fSTDQE+j/bgXmiif5mYaCshGYYOxalSqILRFC7QScwCGEYyEWOMMy5p8vO1uxbet3fBHe7j97 EWVlndk34Cacv76/66IdLDGeUWiMBbpEsYxfx8LIIv+yfmAh0kY1GddgZ1cWZYjWfjRGMgUhBw35v bus16aUVEoU3werg4zCg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hI7ix-0001He-S4; Sun, 21 Apr 2019 08:21:19 +0000 Received: from mail-pg1-x541.google.com ([2607:f8b0:4864:20::541]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hI7il-00010L-58 for linux-arm-kernel@lists.infradead.org; Sun, 21 Apr 2019 08:21:11 +0000 Received: by mail-pg1-x541.google.com with SMTP id j26so4486491pgl.5 for ; Sun, 21 Apr 2019 01:21:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=o6lc0ay5gHFR5swtwXOmViM7nIFzJo8K7yF0YlsWYOg=; b=Lb/UFZ0uxv3HebvlYtqdvSkTS29Xz8hxWdIwsbbMpu8C9N/iRhCNYm1K+TmfanEsMm cvArHVOtvPB2mp2UGDhFn0XVqB2+e3uzzdzCrndM7HCbkubjT5pBImuRm7BO29prjlrn mj2ykDW3QY8KXZZ2LfO3tiyyAt+NEOgHd/O6inrkJ0jBV0mN1alUE66BPIvj5Qdfak9a APHMyCZIDo9XWZKbSqQOIFOSf4V/rIhAeXy2TqqmSn4/ic4T75A9qJP3T0V+izWBHLh7 qE3Y3H+81XpBJf5U1HYbm/W0/xxcEAbnCMA3WXZFmayWksRoxFXViwUcSQRcXbYKEhZl XXHA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=o6lc0ay5gHFR5swtwXOmViM7nIFzJo8K7yF0YlsWYOg=; b=p2VUzfpgkjIGXjM/HSv1+NJyt4BhcXhc9snUbMH6ghg73MHZ0qW+CPLIDCUTx5CQJ6 rIArT8ywmXObAb1y+iO6OXFcZi3oyPlabh9WAestvlqUswQT8LFCBstlBtBqkjCtL+oj r1l5fYMlJuJjd8N01pABLq4Za7nE8vwRBnwHvXdT9ysC4WXoqrH4934bafBabiYbKFnE gLpfdy8G7Xaqv8EzkJlGOoujZsnztz/Yr33XIsIzEaXZokFBm7dhGEw5neqcdxf6PNJJ 7UQ7iaWw7+Ll3McmQwtsUt+bmDlpbvKSXMmP9Hf68nayQBq5tvMle9n7wSjgyK+2esxW 7Q6Q== X-Gm-Message-State: APjAAAUC6Ofh3H5aoQa6gN1YM6w6n6UwWLSjUVK/Vy1s3HoAOAOBxrNl 7/m/snNgtywif+8YEgYjBx4= X-Google-Smtp-Source: APXvYqwlm0+U1Ppd89QYq9KWm7eKJRFZ3kO4+wIUqMI3K2tvq7PCdrrvnWY2NN5zgRAX+2m/xkp14Q== X-Received: by 2002:a62:7648:: with SMTP id r69mr14095914pfc.114.1555834864277; Sun, 21 Apr 2019 01:21:04 -0700 (PDT) Received: from Asurada (c-98-248-47-108.hsd1.ca.comcast.net. [98.248.47.108]) by smtp.gmail.com with ESMTPSA id j20sm12366770pfn.84.2019.04.21.01.21.02 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 21 Apr 2019 01:21:03 -0700 (PDT) Date: Sun, 21 Apr 2019 01:20:57 -0700 From: Nicolin Chen To: Daniel Baluta Subject: Re: [PATCH] ARM: dts: imx: Add mclk0 clock for SAI Message-ID: <20190421082038.GA8304@Asurada> References: <20190420091239.3793-1-daniel.baluta@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20190420091239.3793-1-daniel.baluta@nxp.com> User-Agent: Mutt/1.5.22 (2013-10-16) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190421_012107_351124_84E09546 X-CRM114-Status: GOOD ( 19.64 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "mark.rutland@arm.com" , Aisheng Dong , Peng Fan , Anson Huang , "devicetree@vger.kernel.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" , dl-linux-imx , "kernel@pengutronix.de" , "festevam@gmail.com" , "S.j. Wang" , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sat, Apr 20, 2019 at 09:12:52AM +0000, Daniel Baluta wrote: > From: Shengjiu Wang > > SAI has 4 clock sources, which can be selected using MSEL > bit of SAI TCR2 register. I have a doubt at this statement. As far as I can understand, this MSEL is probably used by its internal clock MUX, so it's not really proving that SAI has 4 MCLK inputs. What I know is that SAI block itself only has 3 MCLK inputs as we defined in DT. It's just internally connects bus clock or MCLK1 to input0 of clock MUX's and connects MCLK[1-3] to input[1-3]. So adding an MCLK0 here doesn't sound a right way to me. Unless someone can justify for it, I think we should just fix it from driver side. Thanks Nicolin > > On imx6/7 mclk0 and mclk1 always point to the same clock > source. Anyhow, this is no longer true for imx8. > > For this reason, we need to add mclk0 and handle it > in a generic way in SAI driver. > > Signed-off-by: Shengjiu Wang > Signed-off-by: Daniel Baluta > --- > arch/arm/boot/dts/imx6sx.dtsi | 6 ++++-- > arch/arm/boot/dts/imx6ul.dtsi | 9 ++++++--- > arch/arm/boot/dts/imx7s.dtsi | 9 ++++++--- > 3 files changed, 16 insertions(+), 8 deletions(-) > > diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi > index b16a123990a2..682207b5d868 100644 > --- a/arch/arm/boot/dts/imx6sx.dtsi > +++ b/arch/arm/boot/dts/imx6sx.dtsi > @@ -1071,9 +1071,10 @@ > reg = <0x021d4000 0x4000>; > interrupts = ; > clocks = <&clks IMX6SX_CLK_SAI1_IPG>, > + <&clks IMX6SX_CLK_SAI1>, > <&clks IMX6SX_CLK_SAI1>, > <&clks 0>, <&clks 0>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dma-names = "rx", "tx"; > dmas = <&sdma 31 24 0>, <&sdma 32 24 0>; > status = "disabled"; > @@ -1090,9 +1091,10 @@ > reg = <0x021dc000 0x4000>; > interrupts = ; > clocks = <&clks IMX6SX_CLK_SAI2_IPG>, > + <&clks IMX6SX_CLK_SAI2>, > <&clks IMX6SX_CLK_SAI2>, > <&clks 0>, <&clks 0>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dma-names = "rx", "tx"; > dmas = <&sdma 33 24 0>, <&sdma 34 24 0>; > status = "disabled"; > diff --git a/arch/arm/boot/dts/imx6ul.dtsi b/arch/arm/boot/dts/imx6ul.dtsi > index bbf010c73336..e9691306f557 100644 > --- a/arch/arm/boot/dts/imx6ul.dtsi > +++ b/arch/arm/boot/dts/imx6ul.dtsi > @@ -304,9 +304,10 @@ > reg = <0x02028000 0x4000>; > interrupts = ; > clocks = <&clks IMX6UL_CLK_SAI1_IPG>, > + <&clks IMX6UL_CLK_SAI1>, > <&clks IMX6UL_CLK_SAI1>, > <&clks IMX6UL_CLK_DUMMY>, <&clks IMX6UL_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dmas = <&sdma 35 24 0>, > <&sdma 36 24 0>; > dma-names = "rx", "tx"; > @@ -319,9 +320,10 @@ > reg = <0x0202c000 0x4000>; > interrupts = ; > clocks = <&clks IMX6UL_CLK_SAI2_IPG>, > + <&clks IMX6UL_CLK_SAI2>, > <&clks IMX6UL_CLK_SAI2>, > <&clks IMX6UL_CLK_DUMMY>, <&clks IMX6UL_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dmas = <&sdma 37 24 0>, > <&sdma 38 24 0>; > dma-names = "rx", "tx"; > @@ -334,9 +336,10 @@ > reg = <0x02030000 0x4000>; > interrupts = ; > clocks = <&clks IMX6UL_CLK_SAI3_IPG>, > + <&clks IMX6UL_CLK_SAI3>, > <&clks IMX6UL_CLK_SAI3>, > <&clks IMX6UL_CLK_DUMMY>, <&clks IMX6UL_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dmas = <&sdma 39 24 0>, > <&sdma 40 24 0>; > dma-names = "rx", "tx"; > diff --git a/arch/arm/boot/dts/imx7s.dtsi b/arch/arm/boot/dts/imx7s.dtsi > index 0b01109ac0a9..693b88e37799 100644 > --- a/arch/arm/boot/dts/imx7s.dtsi > +++ b/arch/arm/boot/dts/imx7s.dtsi > @@ -884,10 +884,11 @@ > reg = <0x308a0000 0x10000>; > interrupts = ; > clocks = <&clks IMX7D_SAI1_IPG_CLK>, > + <&clks IMX7D_SAI1_ROOT_CLK>, > <&clks IMX7D_SAI1_ROOT_CLK>, > <&clks IMX7D_CLK_DUMMY>, > <&clks IMX7D_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dma-names = "rx", "tx"; > dmas = <&sdma 8 24 0>, <&sdma 9 24 0>; > status = "disabled"; > @@ -899,10 +900,11 @@ > reg = <0x308b0000 0x10000>; > interrupts = ; > clocks = <&clks IMX7D_SAI2_IPG_CLK>, > + <&clks IMX7D_SAI2_ROOT_CLK>, > <&clks IMX7D_SAI2_ROOT_CLK>, > <&clks IMX7D_CLK_DUMMY>, > <&clks IMX7D_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dma-names = "rx", "tx"; > dmas = <&sdma 10 24 0>, <&sdma 11 24 0>; > status = "disabled"; > @@ -914,10 +916,11 @@ > reg = <0x308c0000 0x10000>; > interrupts = ; > clocks = <&clks IMX7D_SAI3_IPG_CLK>, > + <&clks IMX7D_SAI3_ROOT_CLK>, > <&clks IMX7D_SAI3_ROOT_CLK>, > <&clks IMX7D_CLK_DUMMY>, > <&clks IMX7D_CLK_DUMMY>; > - clock-names = "bus", "mclk1", "mclk2", "mclk3"; > + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3"; > dma-names = "rx", "tx"; > dmas = <&sdma 12 24 0>, <&sdma 13 24 0>; > status = "disabled"; > -- > 2.17.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel