Re: [PATCH v2] backlight: lm3630a: turn off both led strings when display is blank

From: Lee Jones
Date: Thu May 18 2023 - 06:16:43 EST


On Wed, 10 May 2023, Maximilian Weigand wrote:

> From: Maximilian Weigand <mweigand@xxxxxxxxxxxx>
>
> Use backlight_is_blank() to determine if the led strings should be turned
> off in the update_status() functions of both strings.
>
> Reviewed-by: Daniel Thompson <daniel.thompson@xxxxxxxxxx>

I moved this tag to the correct place and ...

> Signed-off-by: Maximilian Weigand <mweigand@xxxxxxxxxxxx>
> ---
> Changes in v2:
> - fix description, replace display_is_black() with backlight_is_black()
> - use backlight_get_brightness() instead of accessing the property
> directly
>
> drivers/video/backlight/lm3630a_bl.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)

Applied, thanks

--
Lee Jones [李琼斯]