platform/x86: compal-laptop: Use backlight helper
authorStephen Kitt <steve@sk2.org>
Tue, 7 Jun 2022 18:46:34 +0000 (20:46 +0200)
committerHans de Goede <hdegoede@redhat.com>
Wed, 22 Jun 2022 09:53:56 +0000 (11:53 +0200)
Instead of manually checking the power state in struct
backlight_properties, use backlight_is_blank().

Signed-off-by: Stephen Kitt <steve@sk2.org>
Cc: Cezary Jackiewicz <cezary.jackiewicz@gmail.com>
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: Mark Gross <markgross@kernel.org>
Cc: platform-driver-x86@vger.kernel.org
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Link: https://lore.kernel.org/r/20220607184635.1127913-4-steve@sk2.org
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/compal-laptop.c

index ab61037..0942f50 100644 (file)
@@ -324,9 +324,7 @@ static int bl_update_status(struct backlight_device *b)
        if (ret)
                return ret;
 
-       set_backlight_state((b->props.power == FB_BLANK_UNBLANK)
-               &&    !(b->props.state & BL_CORE_SUSPENDED)
-               &&    !(b->props.state & BL_CORE_FBBLANK));
+       set_backlight_state(!backlight_is_blank(b));
        return 0;
 }