[PATCH v2 3/5] drm/gma500: Use backlight_get_brightness() to get the brightness

Sam Ravnborg sam at ravnborg.org
Sat Sep 17 20:46:18 UTC 2022


On Sat, Sep 17, 2022 at 04:03:50PM +0200, Hans de Goede wrote:
> Use backlight_get_brightness() instead of directly referencing
> bd->props.brightness. This will take backlight_is_blank() into account,
> properly setting brightness to 0 when screen-blanking has been requested
> through the backlight sysfs interface.
> 
> Suggested-by: Sam Ravnborg <sam at ravnborg.org>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
Acked-by: Sam Ravnborg <sam at ravnborg.org>
> ---
> Changes in v2 of the patch-set:
> - New patch in v2 of the patch-set
> ---
>  drivers/gpu/drm/gma500/backlight.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/gma500/backlight.c b/drivers/gpu/drm/gma500/backlight.c
> index d113c5810ca5..9be28dc0bdd1 100644
> --- a/drivers/gpu/drm/gma500/backlight.c
> +++ b/drivers/gpu/drm/gma500/backlight.c
> @@ -52,7 +52,7 @@ static int gma_backlight_get_brightness(struct backlight_device *bd)
>  static int gma_backlight_update_status(struct backlight_device *bd)
>  {
>  	struct drm_device *dev = bl_get_data(bd);
> -	int level = bd->props.brightness;
> +	int level = backlight_get_brightness(bd);
>  
>  	/* Percentage 1-100% being valid */
>  	if (level < 1)
> -- 
> 2.37.3


More information about the dri-devel mailing list