Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Abel Vesa <abel.vesa@nxp.com>
To:     Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Mike Turquette <mturquette@baylibre.com>
Cc:     Pengutronix Kernel Team <kernel@pengutronix.de>,
        Peng Fan <peng.fan@nxp.com>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Frank Li <frank.li@nxp.com>, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-clk@vger.kernel.org
Subject: [PATCH 1/2] arm: dts: imx7: Use audio_mclk_post_div instead audio_mclk_root_clk
Date:   Thu, 27 Jan 2022 16:10:51 +0200
Message-Id: <20220127141052.1900174-1-abel.vesa@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?zxUNcF3zaNScPAgzBr2sBVyY45/DXRuGt/ZCUq2bF9ZzTV1P3TuqipUp+8yC?=
 =?us-ascii?Q?w7G9gQgiECVICX7ZGtl4NAM/kljYsDAm/cAjJaeSAieUTxqQsBaNCCsD76rk?=
 =?us-ascii?Q?ymyNnkAh3dBEU42Nd7oFmpL+/MG4Ze0j5GoD8ki/FWg35fSFEXlskSmRieE0?=
 =?us-ascii?Q?pfR8y8v5a7sYJahqt/Bso4ZXzEbHPjHkrUAl9DlJ5gZ1Icg9J3bKgF6Wp2gi?=
 =?us-ascii?Q?G2O9lCxUca0/UVLJyR63CpNKx60y4mfaUilJRe82tuvPD6906mxnZkAGmF9k?=
 =?us-ascii?Q?/ywwNM3EE8Uql+1l8lun9YOqna1OSgMfkLM83fFVcip8ALcQ7aOfMKRQ4Ana?=
 =?us-ascii?Q?UpKx6fFwXU3ql2aY+AvrXdEkYwnwgxQ6+Jt9MtusCDGKmMCSZ53rE2GNe8DK?=
 =?us-ascii?Q?Fi2MxMv7Mo7utLYM8GLaa07bh/XotkGj3qoCoCkYjNaGn3rH2XndFRMgh/ig?=
 =?us-ascii?Q?6FZIjE3ukzw6IvIoqpasWegO9G6zre9hgFwoYnKvyNfLeDhRao3GzGEFPZPy?=
 =?us-ascii?Q?gQX/xOm02zHmXmK4Lv7xwKbQcItjx1CnsNJB1nWqbsAmJlhEC2Bh5eU1+zYm?=
 =?us-ascii?Q?COcJUaljVq7uVC5sY6waKKuIUckKmw8okp5UQAiibX89/yiMkm+cNSfsNqss?=
 =?us-ascii?Q?aHUSeIAbpg94YzYagjK3z08MjBliUryYDzkMUOH5yicMmJlJ/5bMYKEllnwa?=
 =?us-ascii?Q?kGb11DXcaMRq8Xz0l6jniw6rJPfpt4BTUMPTHRZRmbcQM1i/ResyHGMXJocR?=
 =?us-ascii?Q?aIb9O7dCGU28x2ByDVC3z+aFOdPGGxCzs49qttb2SgD9LRS0vlne2EmQ5a/N?=
 =?us-ascii?Q?kW0Z1VWF554aEBKZZJvKm8td0zmIrMZRdZeO2TSY0PN3FvT7NwAx/ELyv0wz?=
 =?us-ascii?Q?sK8yMXw4xhZ+pElEd5/WvZ1L+V6q5OwhF0mowBXrFpC8kh42+PSFjQxKdgbo?=
 =?us-ascii?Q?uFAut994ODsmX2ZoagNFriUxqacIKsjPWp/M4aG8Iy5XybaxmhDfhnV61iLn?=
 =?us-ascii?Q?X0oKu9NNqJsOuLP5WWrmnWcz2S5kYPCZ0js51pM/gom72RgahVW/UcO0Wgd2?=
 =?us-ascii?Q?oMkzkohnUet7bpsFS62JgKo0Tb6kh8GWyKq4o4pN9rMU7doKEQVwXl9tenNO?=
 =?us-ascii?Q?ueP1trvr1dqCLuCycGPl4M+E3FAK7bcyEYlFU0Ua0nIqYcn8XtH1aDmlbhQ3?=
 =?us-ascii?Q?MPesaIA/iu3RnLES3Ate7K7RHWlkLM+UeMsIBPKoBn9dMCpTYzzVHXepZDPg?=
 =?us-ascii?Q?8qxS/fWbwhU0Rs9NGBjMLHMlKQceHqTlxzou2xC/q3C3kM8Jg/F5hNpl+a14?=
 =?us-ascii?Q?kDWZuFsx/5VXRdxVjZAieJnLVad6o6TvnI8zDAYxQOv4xnFSJmTPEDrkTWcw?=
 =?us-ascii?Q?OdrOb6BWC3NY+JEvoewtxMmMJYMnVLmKgTU+GnJOyTmMzv7XQHE9NUmzwlUe?=
 =?us-ascii?Q?mhz3fDu4cvHJ1zTyh+r86FNAbQJC9sJp1tD8C6RGwtRV6fqzw7JwnVO2dBVw?=
 =?us-ascii?Q?WSDN2wxy08tGlZuYTqzjQCFaZZvLRneq+io/18E3nS1jWB1x1kBNZBz1q1Bc?=
 =?us-ascii?Q?WbDHo8TnB8V9vKvqoPkD5hHhRgZTWy8K4KaCw++Ej/a12FrznYbGF//4i5Fw?=
 =?us-ascii?Q?ZzxVozQYYsUcV4gjUCJ9F7w=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 56c2441e-6d44-4a1c-4f37-08d9e19ef36f
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB4688.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jan 2022 14:11:45.8626
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: c5PNfeNte9mo6q9bPmiSBCpzdY5/9KTO+yOVzp358V2ZfcBLOhgYGR3Oa0XWxrUUQFBuOlLaNO7uF7w33xy/3g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB9208
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The audio_mclk_root_clk was added as a gate with the CCGR121 (0x4790),
but according to the reference manual, there is no such gate. Moreover,
the consumer driver of the mentioned clock might gate it and leave
the ECSPI2 (the true owner of that gate) hanging. So lets use the
audio_mclk_post_div, which is the parent.

Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
---
 arch/arm/boot/dts/imx7-colibri.dtsi     | 4 ++--
 arch/arm/boot/dts/imx7-mba7.dtsi        | 2 +-
 arch/arm/boot/dts/imx7d-nitrogen7.dts   | 2 +-
 arch/arm/boot/dts/imx7d-pico-hobbit.dts | 4 ++--
 arch/arm/boot/dts/imx7d-pico-pi.dts     | 4 ++--
 arch/arm/boot/dts/imx7d-sdb.dts         | 4 ++--
 arch/arm/boot/dts/imx7s-warp.dts        | 4 ++--
 7 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/arch/arm/boot/dts/imx7-colibri.dtsi b/arch/arm/boot/dts/imx7-colibri.dtsi
index 62b771c1d5a9..f1c60b0cb143 100644
--- a/arch/arm/boot/dts/imx7-colibri.dtsi
+++ b/arch/arm/boot/dts/imx7-colibri.dtsi
@@ -40,7 +40,7 @@ simple-audio-card,cpu {
 
 		dailink_master: simple-audio-card,codec {
 			sound-dai = <&codec>;
-			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		};
 	};
 };
@@ -293,7 +293,7 @@ codec: sgtl5000@a {
 		compatible = "fsl,sgtl5000";
 		#sound-dai-cells = <0>;
 		reg = <0x0a>;
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_sai1_mclk>;
 		VDDA-supply = <&reg_module_3v3_avdd>;
diff --git a/arch/arm/boot/dts/imx7-mba7.dtsi b/arch/arm/boot/dts/imx7-mba7.dtsi
index 49086c6b6a0a..3df6dff7734a 100644
--- a/arch/arm/boot/dts/imx7-mba7.dtsi
+++ b/arch/arm/boot/dts/imx7-mba7.dtsi
@@ -302,7 +302,7 @@ &i2c2 {
 	tlv320aic32x4: audio-codec@18 {
 		compatible = "ti,tlv320aic32x4";
 		reg = <0x18>;
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		clock-names = "mclk";
 		ldoin-supply = <&reg_audio_3v3>;
 		iov-supply = <&reg_audio_3v3>;
diff --git a/arch/arm/boot/dts/imx7d-nitrogen7.dts b/arch/arm/boot/dts/imx7d-nitrogen7.dts
index e0751e6ba3c0..a31de900139d 100644
--- a/arch/arm/boot/dts/imx7d-nitrogen7.dts
+++ b/arch/arm/boot/dts/imx7d-nitrogen7.dts
@@ -288,7 +288,7 @@ &i2c4 {
 	codec: wm8960@1a {
 		compatible = "wlf,wm8960";
 		reg = <0x1a>;
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		clock-names = "mclk";
 		wlf,shared-lrclk;
 	};
diff --git a/arch/arm/boot/dts/imx7d-pico-hobbit.dts b/arch/arm/boot/dts/imx7d-pico-hobbit.dts
index 7b2198a9372c..d917dc4f2f22 100644
--- a/arch/arm/boot/dts/imx7d-pico-hobbit.dts
+++ b/arch/arm/boot/dts/imx7d-pico-hobbit.dts
@@ -31,7 +31,7 @@ simple-audio-card,cpu {
 
 		dailink_master: simple-audio-card,codec {
 			sound-dai = <&sgtl5000>;
-			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		};
 	};
 };
@@ -41,7 +41,7 @@ sgtl5000: codec@a {
 		#sound-dai-cells = <0>;
 		reg = <0x0a>;
 		compatible = "fsl,sgtl5000";
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		VDDA-supply = <&reg_2p5v>;
 		VDDIO-supply = <&reg_vref_1v8>;
 	};
diff --git a/arch/arm/boot/dts/imx7d-pico-pi.dts b/arch/arm/boot/dts/imx7d-pico-pi.dts
index 70bea95c06d8..f263e391e24c 100644
--- a/arch/arm/boot/dts/imx7d-pico-pi.dts
+++ b/arch/arm/boot/dts/imx7d-pico-pi.dts
@@ -31,7 +31,7 @@ simple-audio-card,cpu {
 
 		dailink_master: simple-audio-card,codec {
 			sound-dai = <&sgtl5000>;
-			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		};
 	};
 };
@@ -41,7 +41,7 @@ sgtl5000: codec@a {
 		#sound-dai-cells = <0>;
 		reg = <0x0a>;
 		compatible = "fsl,sgtl5000";
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		VDDA-supply = <&reg_2p5v>;
 		VDDIO-supply = <&reg_vref_1v8>;
 	};
diff --git a/arch/arm/boot/dts/imx7d-sdb.dts b/arch/arm/boot/dts/imx7d-sdb.dts
index 7813ef960f6e..f053f5122741 100644
--- a/arch/arm/boot/dts/imx7d-sdb.dts
+++ b/arch/arm/boot/dts/imx7d-sdb.dts
@@ -385,14 +385,14 @@ &i2c4 {
 	codec: wm8960@1a {
 		compatible = "wlf,wm8960";
 		reg = <0x1a>;
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		clock-names = "mclk";
 		wlf,shared-lrclk;
 		wlf,hp-cfg = <2 2 3>;
 		wlf,gpio-cfg = <1 3>;
 		assigned-clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_SRC>,
 				  <&clks IMX7D_PLL_AUDIO_POST_DIV>,
-				  <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+				  <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		assigned-clock-parents = <&clks IMX7D_PLL_AUDIO_POST_DIV>;
 		assigned-clock-rates = <0>, <884736000>, <12288000>;
 	};
diff --git a/arch/arm/boot/dts/imx7s-warp.dts b/arch/arm/boot/dts/imx7s-warp.dts
index 4f1edef06c92..e8734d218b9d 100644
--- a/arch/arm/boot/dts/imx7s-warp.dts
+++ b/arch/arm/boot/dts/imx7s-warp.dts
@@ -75,7 +75,7 @@ simple-audio-card,cpu {
 
 		dailink_master: simple-audio-card,codec {
 			sound-dai = <&codec>;
-			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+			clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		};
 	};
 };
@@ -232,7 +232,7 @@ codec: sgtl5000@a {
 		#sound-dai-cells = <0>;
 		reg = <0x0a>;
 		compatible = "fsl,sgtl5000";
-		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_CLK>;
+		clocks = <&clks IMX7D_AUDIO_MCLK_ROOT_DIV>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_sai1_mclk>;
 		VDDA-supply = <&vgen4_reg>;
-- 
2.31.1