Message ID | 20190806064614.20294-1-Anson.Huang@nxp.com (mailing list archive) |
---|---|
State | Mainlined |
Commit | ef7e6a1284374f22ca7016ae3f7c3dfccf7a3de2 |
Headers | show |
Series | [1/2] clk: imx8mm: Unregister clks when of_clk_add_provider failed | expand |
On Tue, 2019-08-06 at 14:46 +0800, Anson.Huang@nxp.com wrote: > From: Anson Huang <Anson.Huang@nxp.com> > > When of_clk_add_provider failed, all clks should be unregistered. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com> Thanks Anson for the patch!
On Tue, Aug 06, 2019 at 02:46:13PM +0800, Anson.Huang@nxp.com wrote: > From: Anson Huang <Anson.Huang@nxp.com> > > When of_clk_add_provider failed, all clks should be unregistered. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> Applied both, thanks.
diff --git a/drivers/clk/imx/clk-imx8mm.c b/drivers/clk/imx/clk-imx8mm.c index e494e99..49ad761 100644 --- a/drivers/clk/imx/clk-imx8mm.c +++ b/drivers/clk/imx/clk-imx8mm.c @@ -661,12 +661,17 @@ static int imx8mm_clocks_probe(struct platform_device *pdev) ret = of_clk_add_provider(np, of_clk_src_onecell_get, &clk_data); if (ret < 0) { pr_err("failed to register clks for i.MX8MM\n"); - return -EINVAL; + goto unregister_clks; } imx_register_uart_clocks(uart_clks); return 0; + +unregister_clks: + imx_unregister_clocks(clks, ARRAY_SIZE(clks)); + + return ret; } static const struct of_device_id imx8mm_clk_of_match[] = {