diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-06-16 09:19:27 -0400 |
---|---|---|
committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-06-17 15:48:44 -0400 |
commit | 9f0ed7a7c547efbce2c15b5017744809e9bba23a (patch) | |
tree | 51b4e3209c347e072af70ba41c056fb3a6164e82 /sound/soc/codecs/ab8500-codec.c | |
parent | e0690385a86cac5403a62d91dc146f2508416ded (diff) |
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.c | 3 |
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, | |||
1235 | req != ANC_APPLY_IIR) { | 1235 | req != ANC_APPLY_IIR) { |
1236 | dev_err(dev, "%s: ERROR: Unsupported status to set '%s'!\n", | 1236 | dev_err(dev, "%s: ERROR: Unsupported status to set '%s'!\n", |
1237 | __func__, enum_anc_state[req]); | 1237 | __func__, enum_anc_state[req]); |
1238 | return -EINVAL; | 1238 | status = -EINVAL; |
1239 | goto cleanup; | ||
1239 | } | 1240 | } |
1240 | apply_fir = req == ANC_APPLY_FIR || req == ANC_APPLY_FIR_IIR; | 1241 | apply_fir = req == ANC_APPLY_FIR || req == ANC_APPLY_FIR_IIR; |
1241 | apply_iir = req == ANC_APPLY_IIR || req == ANC_APPLY_FIR_IIR; | 1242 | apply_iir = req == ANC_APPLY_IIR || req == ANC_APPLY_FIR_IIR; |