Commit 5988c269 authored by Xu Wang's avatar Xu Wang Committed by Thomas Zimmermann

video: fbdev: sh_mobile_lcdcfb: Remove redundant null check before...

video: fbdev: sh_mobile_lcdcfb: Remove redundant null check before clk_prepare_enable/clk_disable_unprepare

Because clk_prepare_enable() and clk_disable_unprepare() already checked
NULL clock parameter, so the additional checks are unnecessary, just
remove them.
Signed-off-by: default avatarXu Wang <vulab@iscas.ac.cn>
Signed-off-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20201014085722.26069-1-vulab@iscas.ac.cn
parent f5707418
...@@ -341,7 +341,6 @@ static void lcdc_wait_bit(struct sh_mobile_lcdc_priv *priv, ...@@ -341,7 +341,6 @@ static void lcdc_wait_bit(struct sh_mobile_lcdc_priv *priv,
static void sh_mobile_lcdc_clk_on(struct sh_mobile_lcdc_priv *priv) static void sh_mobile_lcdc_clk_on(struct sh_mobile_lcdc_priv *priv)
{ {
if (atomic_inc_and_test(&priv->hw_usecnt)) { if (atomic_inc_and_test(&priv->hw_usecnt)) {
if (priv->dot_clk)
clk_prepare_enable(priv->dot_clk); clk_prepare_enable(priv->dot_clk);
pm_runtime_get_sync(priv->dev); pm_runtime_get_sync(priv->dev);
} }
...@@ -351,7 +350,6 @@ static void sh_mobile_lcdc_clk_off(struct sh_mobile_lcdc_priv *priv) ...@@ -351,7 +350,6 @@ static void sh_mobile_lcdc_clk_off(struct sh_mobile_lcdc_priv *priv)
{ {
if (atomic_sub_return(1, &priv->hw_usecnt) == -1) { if (atomic_sub_return(1, &priv->hw_usecnt) == -1) {
pm_runtime_put(priv->dev); pm_runtime_put(priv->dev);
if (priv->dot_clk)
clk_disable_unprepare(priv->dot_clk); clk_disable_unprepare(priv->dot_clk);
} }
} }
......
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