diff options
author | Arvind Yadav <arvind.yadav.cs@gmail.com> | 2017-07-25 06:15:37 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2017-07-26 08:01:42 -0400 |
commit | 260ea95cc027500ee5086e2ef568b8c915e66fb5 (patch) | |
tree | 0f0446e9ea3a0b4fa8414bf819b46dbb46491f83 /sound/atmel | |
parent | 5771a8c08880cdca3bfb4a3fc6d309d6bba20877 (diff) |
ASoC: atmel: ac97c: Handle return value of clk_prepare_enable.
clk_prepare_enable() can fail here and we must check its return value.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/atmel')
-rw-r--r-- | sound/atmel/ac97c.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/sound/atmel/ac97c.c b/sound/atmel/ac97c.c index 9d2c9d9af688..52b0522fda20 100644 --- a/sound/atmel/ac97c.c +++ b/sound/atmel/ac97c.c | |||
@@ -783,7 +783,9 @@ static int atmel_ac97c_probe(struct platform_device *pdev) | |||
783 | dev_dbg(&pdev->dev, "no peripheral clock\n"); | 783 | dev_dbg(&pdev->dev, "no peripheral clock\n"); |
784 | return PTR_ERR(pclk); | 784 | return PTR_ERR(pclk); |
785 | } | 785 | } |
786 | clk_prepare_enable(pclk); | 786 | retval = clk_prepare_enable(pclk); |
787 | if (retval) | ||
788 | return retval; | ||
787 | 789 | ||
788 | retval = snd_card_new(&pdev->dev, SNDRV_DEFAULT_IDX1, | 790 | retval = snd_card_new(&pdev->dev, SNDRV_DEFAULT_IDX1, |
789 | SNDRV_DEFAULT_STR1, THIS_MODULE, | 791 | SNDRV_DEFAULT_STR1, THIS_MODULE, |
@@ -897,9 +899,9 @@ static int atmel_ac97c_resume(struct device *pdev) | |||
897 | { | 899 | { |
898 | struct snd_card *card = dev_get_drvdata(pdev); | 900 | struct snd_card *card = dev_get_drvdata(pdev); |
899 | struct atmel_ac97c *chip = card->private_data; | 901 | struct atmel_ac97c *chip = card->private_data; |
902 | int ret = clk_prepare_enable(chip->pclk); | ||
900 | 903 | ||
901 | clk_prepare_enable(chip->pclk); | 904 | return ret; |
902 | return 0; | ||
903 | } | 905 | } |
904 | 906 | ||
905 | static SIMPLE_DEV_PM_OPS(atmel_ac97c_pm, atmel_ac97c_suspend, atmel_ac97c_resume); | 907 | static SIMPLE_DEV_PM_OPS(atmel_ac97c_pm, atmel_ac97c_suspend, atmel_ac97c_resume); |