diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-01-03 13:25:55 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2015-01-05 14:03:00 -0500 |
commit | 180cf794d4f5e0e82a4df30b8bacd9b472f80f44 (patch) | |
tree | 2df63fe09f1c4fbe60e7a049f9d3c56597eebe58 | |
parent | 97bf6af1f928216fd6c5a66e8a57bfa95a659672 (diff) |
ASoC: fsi: Deletion of unnecessary checks before the function call "clk_enable"
The clk_enable() 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 8869971d7884..d49f25f9efd3 100644 --- a/sound/soc/sh/fsi.c +++ b/sound/soc/sh/fsi.c | |||
@@ -820,12 +820,9 @@ static int fsi_clk_enable(struct device *dev, | |||
820 | return ret; | 820 | return ret; |
821 | } | 821 | } |
822 | 822 | ||
823 | if (clock->xck) | 823 | clk_enable(clock->xck); |
824 | clk_enable(clock->xck); | 824 | clk_enable(clock->ick); |
825 | if (clock->ick) | 825 | clk_enable(clock->div); |
826 | clk_enable(clock->ick); | ||
827 | if (clock->div) | ||
828 | clk_enable(clock->div); | ||
829 | 826 | ||
830 | clock->count++; | 827 | clock->count++; |
831 | } | 828 | } |