From: Michał Kępień Date: Wed, 5 Apr 2017 06:49:09 +0000 (+0200) Subject: platform/x86: fujitsu-laptop: account for backlight power when determining brightness X-Git-Tag: v4.12-rc1~101^2~39 X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=5959ddd02be4e435da866ee065187f5c8bccea28;p=karo-tx-linux.git platform/x86: fujitsu-laptop: account for backlight power when determining brightness The comment for the get_brightness backlight device callback in include/linux/backlight.h states that it should "return the current backlight brightness (accounting for power, fb_blank etc.)". fujitsu-laptop violates that premise by simply returning the value to which ACPI function GBLL evaluates to. Make sure 0 is returned when backlight power is turned off. Signed-off-by: Michał Kępień Reviewed-by: Jonathan Woithe Signed-off-by: Darren Hart (VMware) --- diff --git a/drivers/platform/x86/fujitsu-laptop.c b/drivers/platform/x86/fujitsu-laptop.c index af36b15515ed..b88172134063 100644 --- a/drivers/platform/x86/fujitsu-laptop.c +++ b/drivers/platform/x86/fujitsu-laptop.c @@ -423,7 +423,7 @@ static int get_max_brightness(void) static int bl_get_brightness(struct backlight_device *b) { - return get_lcd_level(); + return b->props.power == FB_BLANK_POWERDOWN ? 0 : get_lcd_level(); } static int bl_update_status(struct backlight_device *b)