diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-12-02 16:34:30 +0300 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2014-12-03 21:29:04 +0300 |
commit | e98c89e05e8b33ad40efff2012c1404e39d12ad8 (patch) | |
tree | cb3f9f547ff8947649f3b0638538ab37f075c29f | |
parent | f114040e3ea6e07372334ade75d1ee0775c355e1 (diff) | |
download | linux-e98c89e05e8b33ad40efff2012c1404e39d12ad8.tar.xz |
ASoC: fsi: Deletion of unnecessary checks before the function call "clk_disable"
The clk_disable() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | sound/soc/sh/fsi.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/sound/soc/sh/fsi.c b/sound/soc/sh/fsi.c index 66fddec9543d..eefb16100cd1 100644 --- a/sound/soc/sh/fsi.c +++ b/sound/soc/sh/fsi.c @@ -842,12 +842,9 @@ static int fsi_clk_disable(struct device *dev, return -EINVAL; if (1 == clock->count--) { - if (clock->xck) - clk_disable(clock->xck); - if (clock->ick) - clk_disable(clock->ick); - if (clock->div) - clk_disable(clock->div); + clk_disable(clock->xck); + clk_disable(clock->ick); + clk_disable(clock->div); } return 0; |