summaryrefslogtreecommitdiffstats
path: root/drivers/video/backlight/backlight.c
diff options
context:
space:
mode:
authorZhang Rui <rui.zhang@intel.com>2009-01-08 14:11:30 +0000
committerRichard Purdie <rpurdie@linux.intel.com>2009-01-08 14:11:30 +0000
commit9be1df98bca44dbe3769cd22f4ab8122b76c5313 (patch)
tree2bc256405bcdb4836446a2fd173c373f6d6d67e4 /drivers/video/backlight/backlight.c
parent0ec561f4b648260a46ace87acbc558241808455f (diff)
downloadlinux-9be1df98bca44dbe3769cd22f4ab8122b76c5313.tar.gz
linux-9be1df98bca44dbe3769cd22f4ab8122b76c5313.tar.bz2
linux-9be1df98bca44dbe3769cd22f4ab8122b76c5313.zip
bd->props.brightness doesn't reflect the actual backlight level.
Always invoke backlight_update_status when users want to change the backlight. For setups where brightness change is an expensive operation, this could be done in the driver rather than the core. http://bugzilla.kernel.org/show_bug.cgi?id=12249 Signed-off-by: Zhang Rui <rui.zhang@intel.com> Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
Diffstat (limited to 'drivers/video/backlight/backlight.c')
-rw-r--r--drivers/video/backlight/backlight.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/video/backlight/backlight.c b/drivers/video/backlight/backlight.c
index 9593fdc042f7..a9c013bb9f20 100644
--- a/drivers/video/backlight/backlight.c
+++ b/drivers/video/backlight/backlight.c
@@ -131,10 +131,8 @@ static ssize_t backlight_store_brightness(struct device *dev,
else {
pr_debug("backlight: set brightness to %lu\n",
brightness);
- if (bd->props.brightness != brightness) {
- bd->props.brightness = brightness;
- backlight_update_status(bd);
- }
+ bd->props.brightness = brightness;
+ backlight_update_status(bd);
rc = count;
}
}