diff options
author | Jose Alonso <joalonsof@gmail.com> | 2011-07-10 14:46:51 -0400 |
---|---|---|
committer | Matthew Garrett <mjg@redhat.com> | 2011-07-11 09:52:31 -0400 |
commit | b486742a12a474a01d1acb1a5924af11d9b32b68 (patch) | |
tree | e2ea6e330e16219b60a349e7c5cbbf5ef829729d /drivers/platform/x86 | |
parent | be65dde82a4a402e9607c2f306f343bf0912623c (diff) |
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>
Diffstat (limited to 'drivers/platform/x86')
-rw-r--r-- | drivers/platform/x86/dell-laptop.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c index 04c34f1a265c..e39ab1d3ed87 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) | |||
540 | else | 540 | else |
541 | dell_send_request(buffer, 0, 1); | 541 | dell_send_request(buffer, 0, 1); |
542 | 542 | ||
543 | ret = buffer->output[1]; | ||
544 | |||
543 | out: | 545 | out: |
544 | release_buffer(); | 546 | release_buffer(); |
545 | if (ret) | 547 | return ret; |
546 | return ret; | ||
547 | return buffer->output[1]; | ||
548 | } | 548 | } |
549 | 549 | ||
550 | static const struct backlight_ops dell_ops = { | 550 | static const struct backlight_ops dell_ops = { |