From: Dan Carpenter <dan.carpenter@oracle.com>
Date: Sat, 16 Jun 2012 13:19:27 +0000 (+0300)
Subject: ASoC: Ux500: unlock on an error path
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=9f0ed7a7c547efbce2c15b5017744809e9bba23a;p=~shefty%2Frdma-dev.git

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>
---

diff --git a/sound/soc/codecs/ab8500-codec.c b/sound/soc/codecs/ab8500-codec.c
index 389dd660b51..3c795921c5f 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;