The clk core now does something similar for us as part of of_clk_add_provider() and of_clk_add_hw_provider() and this i.MX specific call isn't necessary anymore. This should also plug a memory and clock reference leak due to multiple calls to imx_register_uart_clocks(), one for each clock unit. Signed-off-by: André Draszik <andre.draszik@xxxxxxxxxx> --- drivers/clk/imx/clk-imx7ulp.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/clk/imx/clk-imx7ulp.c b/drivers/clk/imx/clk-imx7ulp.c index f4a48a42637f..517fb388ce85 100644 --- a/drivers/clk/imx/clk-imx7ulp.c +++ b/drivers/clk/imx/clk-imx7ulp.c @@ -175,8 +175,6 @@ static void __init imx7ulp_clk_pcc2_init(struct device_node *np) imx_check_clk_hws(hws, clk_data->num); of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_data); - - imx_register_uart_clocks(); } CLK_OF_DECLARE(imx7ulp_clk_pcc2, "fsl,imx7ulp-pcc2", imx7ulp_clk_pcc2_init); @@ -222,8 +220,6 @@ static void __init imx7ulp_clk_pcc3_init(struct device_node *np) imx_check_clk_hws(hws, clk_data->num); of_clk_add_hw_provider(np, of_clk_hw_onecell_get, clk_data); - - imx_register_uart_clocks(); } CLK_OF_DECLARE(imx7ulp_clk_pcc3, "fsl,imx7ulp-pcc3", imx7ulp_clk_pcc3_init); -- 2.46.0.rc2.264.g509ed76dc8-goog