[PATCH] arm64: dts: imx8mn-beacon: Remove unnecessary clock configuration

Adam Ford posted 1 patch 1 year, 12 months ago
[PATCH] arm64: dts: imx8mn-beacon: Remove unnecessary clock configuration
Posted by Adam Ford 1 year, 12 months ago
Since commit 8208181fe536 ("clk: imx: composite-8m:
Add imx8m_divider_determine_rate") the lcdif controller has
had the ability to set the disp_pixel_clk rate which propagates
up the tree and sets the video_pll rate automatically.

As such, there is no need to define it in the board file.

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
index 35b8d2060cd9..bbd80896db96 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mn-beacon-kit.dts
@@ -99,8 +99,6 @@ adv7535_out: endpoint {
 };
 
 &lcdif {
-	assigned-clocks = <&clk IMX8MN_VIDEO_PLL1>;
-	assigned-clock-rates = <594000000>;
 	status = "okay";
 };
 
-- 
2.43.0
Re: [PATCH] arm64: dts: imx8mn-beacon: Remove unnecessary clock configuration
Posted by Shawn Guo 1 year, 11 months ago
On Sun, Feb 11, 2024 at 05:18:02PM -0600, Adam Ford wrote:
> Since commit 8208181fe536 ("clk: imx: composite-8m:
> Add imx8m_divider_determine_rate") the lcdif controller has
> had the ability to set the disp_pixel_clk rate which propagates
> up the tree and sets the video_pll rate automatically.
> 
> As such, there is no need to define it in the board file.
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>

Applied, thanks!