wl12xx: wrong values are returned in gpio_power_write()

Return values were assigned to incorrect var / weren't assigned.
fix it, and defer mutex_lock after the sanity checks.

Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>

1 files changed, 4 insertions, 6 deletions

diff --git a/drivers/net/wireless/wl12xx/debugfs.c b/drivers/net/wireless/wl12xx/debugfs.c
index dc3ca0031817..bebfa28a171a 100644
--- a/drivers/net/wireless/wl12xx/debugfs.c
+++ b/drivers/net/wireless/wl12xx/debugfs.c
@@ -261,27 +261,25 @@ static ssize_t gpio_power_write(struct file *file,
         unsigned long value;
         int ret;
 
-        mutex_lock(&wl->mutex);
-
         len = min(count, sizeof(buf) - 1);
         if (copy_from_user(buf, user_buf, len)) {
-                ret = -EFAULT;
-                goto out;
+                return -EFAULT;
         }
         buf[len] = '\0';
 
         ret = strict_strtoul(buf, 0, &value);
         if (ret < 0) {
                 wl1271_warning("illegal value in gpio_power");
-                goto out;
+                return -EINVAL;
         }
 
+        mutex_lock(&wl->mutex);
+
         if (value)
                 wl1271_power_on(wl);
         else
                 wl1271_power_off(wl);
 
-out:
         mutex_unlock(&wl->mutex);
         return count;
 }