auxdisplay: ht16k33: Remove struct backlight_ops.check_fb
authorThomas Zimmermann <tzimmermann@suse.de>
Tue, 5 Mar 2024 16:22:35 +0000 (17:22 +0100)
committerLee Jones <lee@kernel.org>
Thu, 28 Mar 2024 10:08:54 +0000 (10:08 +0000)
The driver sets struct fb_info.bl_dev to the correct backlight
device. Thus rely on the backlight core code to match backlight
and framebuffer devices, and remove the extra check_fb() function
from struct backlight_ops.

v3:
* use 'check_fb()' in commit message (Andy)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Robin van der Gracht <robin@protonic.nl>
Acked-by: Robin van der Gracht <robin@protonic.nl>
Acked-by: Andy Shevchenko <andy@kernel.org>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://lore.kernel.org/r/20240305162425.23845-3-tzimmermann@suse.de
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/auxdisplay/ht16k33.c

index 96acfb2b58cd9244fda53113fa54cfaa11e2ccfd..82e55d61893bb5682e90a43720ff8cc4c8ed62bc 100644 (file)
@@ -295,16 +295,8 @@ static int ht16k33_bl_update_status(struct backlight_device *bl)
        return ht16k33_brightness_set(priv, brightness);
 }
 
-static int ht16k33_bl_check_fb(struct backlight_device *bl, struct fb_info *fi)
-{
-       struct ht16k33_priv *priv = bl_get_data(bl);
-
-       return (fi == NULL) || (fi->par == priv);
-}
-
 static const struct backlight_ops ht16k33_bl_ops = {
        .update_status  = ht16k33_bl_update_status,
-       .check_fb       = ht16k33_bl_check_fb,
 };
 
 /*