backlight: pwm_bl: Improve bootloader/kernel device handover
authorDaniel Thompson <daniel.thompson@linaro.org>
Thu, 22 Jul 2021 14:46:23 +0000 (15:46 +0100)
committerLee Jones <lee.jones@linaro.org>
Thu, 19 Aug 2021 09:59:03 +0000 (10:59 +0100)
commit79fad92f2e596f5a8dd085788a24f540263ef887
treed44fa52b10483f936cd3e787e316fa9396590f25
parentdaa37361518bf2d1f591bbdaa7c68b2a43d7af48
backlight: pwm_bl: Improve bootloader/kernel device handover

Currently there are (at least) two problems in the way pwm_bl starts
managing the enable_gpio pin. Both occur when the backlight is initially
off and the driver finds the pin not already in output mode and, as a
result, unconditionally switches it to output-mode and asserts the signal.

Problem 1: This could cause the backlight to flicker since, at this stage
in driver initialisation, we have no idea what the PWM and regulator are
doing (an unconfigured PWM could easily "rest" at 100% duty cycle).

Problem 2: This will cause us not to correctly honour the
post_pwm_on_delay (which also risks flickers).

Fix this by moving the code to configure the GPIO output mode until after
we have examines the handover state. That allows us to initialize
enable_gpio to off if the backlight is currently off and on if the
backlight is on.

Cc: stable@vger.kernel.org
Reported-by: Marek Vasut <marex@denx.de>
Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
Acked-by: Marek Vasut <marex@denx.de>
Tested-by: Marek Vasut <marex@denx.de>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
drivers/video/backlight/pwm_bl.c