diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-04-28 15:22:10 +0200 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2017-04-30 14:30:52 +0200 |
| commit | 65ed0a8d1f24abd79be149253025de8949321900 (patch) | |
| tree | 182b3eca2d541ed5ad57de3b799ea24298bf6cc5 /sound | |
| parent | ASoC: STI: Fix null ptr deference in IRQ handler (diff) | |
| download | linux-65ed0a8d1f24abd79be149253025de8949321900.tar.xz linux-65ed0a8d1f24abd79be149253025de8949321900.zip | |
ASoC: sti: Fix error handling if of_clk_get() fails
We intended to return here. The current code has a static checker
warning because we set "ret" but don't use it.
Fixes: 76c2145ded6b ("ASoC: sti: Add CPU DAI driver for playback")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Arnaud POULIQUEN <arnaud.pouliquen@st.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
| -rw-r--r-- | sound/soc/sti/uniperif_player.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/sti/uniperif_player.c b/sound/soc/sti/uniperif_player.c index d7e8dd46d2cc..d8b6936e544e 100644 --- a/sound/soc/sti/uniperif_player.c +++ b/sound/soc/sti/uniperif_player.c @@ -1074,7 +1074,7 @@ int uni_player_init(struct platform_device *pdev, player->clk = of_clk_get(pdev->dev.of_node, 0); if (IS_ERR(player->clk)) { dev_err(player->dev, "Failed to get clock\n"); - ret = PTR_ERR(player->clk); + return PTR_ERR(player->clk); } /* Select the frequency synthesizer clock */ |