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

[Hans de Goede]

Hi,

On 9/17/22 22:46, Sam Ravnborg wrote:
> 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>

Thank you.

I have just send out a v3 with your ack added and fixing the
2 unused variables when CONFIG_BACKLIGHT is not set spotted
by the lkp test robot.

I was sort of hoping that you could review / ack the entire
series ?

Regards,

Hans


>> ---
>> 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
>