-----Original Message-----
From: Stephen Boyd [mailto:sboyd@kernel.org]
Sent: Saturday, April 17, 2021 7:57 AM
To: Jacky Bai <ping.bai@nxp.com>; Abel Vesa <redacted>;
robh+dt@kernel.org; s.hauer@pengutronix.de; shawnguo@kernel.org
Cc: festevam@gmail.com; devicetree@vger.kernel.org;
linux-clk@vger.kernel.org; dl-linux-imx [off-list ref]
Subject: Re: [PATCH 2/2] clk: imx: Remove the audio ipg clock from imx8mp

Quoting Jacky Bai (2021-04-16 00:37:03)

quoted

diff --git a/include/dt-bindings/clock/imx8mp-clock.h

b/include/dt-bindings/clock/imx8mp-clock.h
index 43927a1b9e94..235c7a00d379 100644

--- a/include/dt-bindings/clock/imx8mp-clock.h
+++ b/include/dt-bindings/clock/imx8mp-clock.h

@@ -117,7 +117,6 @@
 #define IMX8MP_CLK_AUDIO_AHB                   108
 #define IMX8MP_CLK_MIPI_DSI_ESC_RX             109
 #define IMX8MP_CLK_IPG_ROOT                    110
-#define IMX8MP_CLK_IPG_AUDIO_ROOT              111

Maybe just add a comment saying it isn't there? Or remove it in three months
time once DTS no longer references it?

Ok, will resolve this comments in V2. I can add some comments and keep it here for a while.

Removing this here means that it has to merge whenever the DTS doesn't
reference it anymore which causes a cross tree dependency.

Right, this delete will lead to the dependency with the dts tree for now.

BR
Jacky Bai

quoted

 #define IMX8MP_CLK_DRAM_ALT                    112
 #define IMX8MP_CLK_DRAM_APB                    113
 #define IMX8MP_CLK_VPU_G1                      114