diff options
| author | Zhang Rui <rui.zhang@intel.com> | 2009-01-08 17:11:30 +0300 |
|---|---|---|
| committer | Richard Purdie <rpurdie@linux.intel.com> | 2009-01-08 17:11:30 +0300 |
| commit | 9be1df98bca44dbe3769cd22f4ab8122b76c5313 (patch) | |
| tree | 2bc256405bcdb4836446a2fd173c373f6d6d67e4 /drivers/video | |
| parent | 0ec561f4b648260a46ace87acbc558241808455f (diff) | |
| download | linux-9be1df98bca44dbe3769cd22f4ab8122b76c5313.tar.xz | |
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')
| -rw-r--r-- | drivers/video/backlight/backlight.c | 6 |
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; } } |