aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJose Alonso <joalonsof@gmail.com>2011-07-10 15:46:51 -0300
committerMatthew Garrett <mjg@redhat.com>2011-07-11 09:52:31 -0400
commitb486742a12a474a01d1acb1a5924af11d9b32b68 (patch)
treee2ea6e330e16219b60a349e7c5cbbf5ef829729d
parentbe65dde82a4a402e9607c2f306f343bf0912623c (diff)
downloadlinux-linaro-b486742a12a474a01d1acb1a5924af11d9b32b68.tar.gz
dell-laptop - using buffer without mutex_lock
Using buffer->output[1] without mutex_lock() Signed-off-by: Jose Alonso <joalonsof@gmail.com> Signed-off-by: Matthew Garrett <mjg@redhat.com>
-rw-r--r--drivers/platform/x86/dell-laptop.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c
index 04c34f1a265..e39ab1d3ed8 100644
--- a/drivers/platform/x86/dell-laptop.c
+++ b/drivers/platform/x86/dell-laptop.c
@@ -540,11 +540,11 @@ static int dell_get_intensity(struct backlight_device *bd)
else
dell_send_request(buffer, 0, 1);
+ ret = buffer->output[1];
+
out:
release_buffer();
- if (ret)
- return ret;
- return buffer->output[1];
+ return ret;
}
static const struct backlight_ops dell_ops = {