summaryrefslogtreecommitdiffstats
path: root/sound/soc/codecs/ab8500-codec.c
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-06-16 15:19:27 +0200
committerMark Brown <broonie@opensource.wolfsonmicro.com>2012-06-17 21:48:44 +0200
commit9f0ed7a7c547efbce2c15b5017744809e9bba23a (patch)
tree51b4e3209c347e072af70ba41c056fb3a6164e82 /sound/soc/codecs/ab8500-codec.c
parentASoC: Ux500: Add machine-driver (diff)
downloadlinux-9f0ed7a7c547efbce2c15b5017744809e9bba23a.tar.xz
linux-9f0ed7a7c547efbce2c15b5017744809e9bba23a.zip
ASoC: Ux500: unlock on an error path
There is a missing mutex_unlock() here. The cleanup path also has more debug output. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/codecs/ab8500-codec.c')
-rw-r--r--sound/soc/codecs/ab8500-codec.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/soc/codecs/ab8500-codec.c b/sound/soc/codecs/ab8500-codec.c
index 389dd660b511..3c795921c5f6 100644
--- a/sound/soc/codecs/ab8500-codec.c
+++ b/sound/soc/codecs/ab8500-codec.c
@@ -1235,7 +1235,8 @@ static int anc_status_control_put(struct snd_kcontrol *kcontrol,
req != ANC_APPLY_IIR) {
dev_err(dev, "%s: ERROR: Unsupported status to set '%s'!\n",
__func__, enum_anc_state[req]);
- return -EINVAL;
+ status = -EINVAL;
+ goto cleanup;
}
apply_fir = req == ANC_APPLY_FIR || req == ANC_APPLY_FIR_IIR;
apply_iir = req == ANC_APPLY_IIR || req == ANC_APPLY_FIR_IIR;