Commit 34ff0f95 authored by Jesper Juhl's avatar Jesper Juhl Committed by Mark Brown

ASoC: wm8994: Don't test for NULL before release_firmware()

release_firmware() does its own NULL ptr testing, it's redundant to
also test before calling it.
Signed-off-by: default avatarJesper Juhl <jj@chaosbits.net>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent bbdd3915
...@@ -3936,7 +3936,7 @@ static int wm8994_codec_probe(struct snd_soc_codec *codec) ...@@ -3936,7 +3936,7 @@ static int wm8994_codec_probe(struct snd_soc_codec *codec)
return ret; return ret;
} }
static int wm8994_codec_remove(struct snd_soc_codec *codec) static int wm8994_codec_remove(struct snd_soc_codec *codec)
{ {
struct wm8994_priv *wm8994 = snd_soc_codec_get_drvdata(codec); struct wm8994_priv *wm8994 = snd_soc_codec_get_drvdata(codec);
struct wm8994 *control = wm8994->wm8994; struct wm8994 *control = wm8994->wm8994;
...@@ -3977,14 +3977,10 @@ static int wm8994_codec_remove(struct snd_soc_codec *codec) ...@@ -3977,14 +3977,10 @@ static int wm8994_codec_remove(struct snd_soc_codec *codec)
free_irq(wm8994->micdet_irq, wm8994); free_irq(wm8994->micdet_irq, wm8994);
break; break;
} }
if (wm8994->mbc) release_firmware(wm8994->mbc);
release_firmware(wm8994->mbc); release_firmware(wm8994->mbc_vss);
if (wm8994->mbc_vss) release_firmware(wm8994->enh_eq);
release_firmware(wm8994->mbc_vss);
if (wm8994->enh_eq)
release_firmware(wm8994->enh_eq);
kfree(wm8994->retune_mobile_texts); kfree(wm8994->retune_mobile_texts);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment