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 | } |
