Commit cb1dab0e authored by Chen-Yu Tsai's avatar Chen-Yu Tsai Committed by Maxime Ripard

drm/sun4i: hdmi: Disable clks in bind function error path and unbind function

The HDMI driver enables the bus and mod clocks in the bind function, but
does not disable them if it then bails our due to any errors. Neither
does it disable the clocks in the unbind function.

Fix this by adding a proper error path to the bind function, and
clk_disable_unprepare calls to the unbind function.

Also rename the err_cleanup_connector label to err_cleanup_encoder,
since it is the encoder that gets cleaned up.

Fixes: 9c568101 ("drm/sun4i: Add HDMI support")
Signed-off-by: default avatarChen-Yu Tsai <wens@csie.org>
Acked-by: default avatarMaxime Ripard <maxime.ripard@free-electrons.com>
Signed-off-by: default avatarMaxime Ripard <maxime.ripard@free-electrons.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170929082306.16193-6-wens@csie.org
parent 9e66317d
...@@ -302,26 +302,29 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master, ...@@ -302,26 +302,29 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master,
hdmi->mod_clk = devm_clk_get(dev, "mod"); hdmi->mod_clk = devm_clk_get(dev, "mod");
if (IS_ERR(hdmi->mod_clk)) { if (IS_ERR(hdmi->mod_clk)) {
dev_err(dev, "Couldn't get the HDMI mod clock\n"); dev_err(dev, "Couldn't get the HDMI mod clock\n");
return PTR_ERR(hdmi->mod_clk); ret = PTR_ERR(hdmi->mod_clk);
goto err_disable_bus_clk;
} }
clk_prepare_enable(hdmi->mod_clk); clk_prepare_enable(hdmi->mod_clk);
hdmi->pll0_clk = devm_clk_get(dev, "pll-0"); hdmi->pll0_clk = devm_clk_get(dev, "pll-0");
if (IS_ERR(hdmi->pll0_clk)) { if (IS_ERR(hdmi->pll0_clk)) {
dev_err(dev, "Couldn't get the HDMI PLL 0 clock\n"); dev_err(dev, "Couldn't get the HDMI PLL 0 clock\n");
return PTR_ERR(hdmi->pll0_clk); ret = PTR_ERR(hdmi->pll0_clk);
goto err_disable_mod_clk;
} }
hdmi->pll1_clk = devm_clk_get(dev, "pll-1"); hdmi->pll1_clk = devm_clk_get(dev, "pll-1");
if (IS_ERR(hdmi->pll1_clk)) { if (IS_ERR(hdmi->pll1_clk)) {
dev_err(dev, "Couldn't get the HDMI PLL 1 clock\n"); dev_err(dev, "Couldn't get the HDMI PLL 1 clock\n");
return PTR_ERR(hdmi->pll1_clk); ret = PTR_ERR(hdmi->pll1_clk);
goto err_disable_mod_clk;
} }
ret = sun4i_tmds_create(hdmi); ret = sun4i_tmds_create(hdmi);
if (ret) { if (ret) {
dev_err(dev, "Couldn't create the TMDS clock\n"); dev_err(dev, "Couldn't create the TMDS clock\n");
return ret; goto err_disable_mod_clk;
} }
writel(SUN4I_HDMI_CTRL_ENABLE, hdmi->base + SUN4I_HDMI_CTRL_REG); writel(SUN4I_HDMI_CTRL_ENABLE, hdmi->base + SUN4I_HDMI_CTRL_REG);
...@@ -362,7 +365,7 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master, ...@@ -362,7 +365,7 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master,
ret = sun4i_hdmi_i2c_create(dev, hdmi); ret = sun4i_hdmi_i2c_create(dev, hdmi);
if (ret) { if (ret) {
dev_err(dev, "Couldn't create the HDMI I2C adapter\n"); dev_err(dev, "Couldn't create the HDMI I2C adapter\n");
return ret; goto err_disable_mod_clk;
} }
drm_encoder_helper_add(&hdmi->encoder, drm_encoder_helper_add(&hdmi->encoder,
...@@ -422,6 +425,10 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master, ...@@ -422,6 +425,10 @@ static int sun4i_hdmi_bind(struct device *dev, struct device *master,
drm_encoder_cleanup(&hdmi->encoder); drm_encoder_cleanup(&hdmi->encoder);
err_del_i2c_adapter: err_del_i2c_adapter:
i2c_del_adapter(hdmi->i2c); i2c_del_adapter(hdmi->i2c);
err_disable_mod_clk:
clk_disable_unprepare(hdmi->mod_clk);
err_disable_bus_clk:
clk_disable_unprepare(hdmi->bus_clk);
return ret; return ret;
} }
...@@ -434,6 +441,8 @@ static void sun4i_hdmi_unbind(struct device *dev, struct device *master, ...@@ -434,6 +441,8 @@ static void sun4i_hdmi_unbind(struct device *dev, struct device *master,
drm_connector_cleanup(&hdmi->connector); drm_connector_cleanup(&hdmi->connector);
drm_encoder_cleanup(&hdmi->encoder); drm_encoder_cleanup(&hdmi->encoder);
i2c_del_adapter(hdmi->i2c); i2c_del_adapter(hdmi->i2c);
clk_disable_unprepare(hdmi->mod_clk);
clk_disable_unprepare(hdmi->bus_clk);
} }
static const struct component_ops sun4i_hdmi_ops = { static const struct component_ops sun4i_hdmi_ops = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment