]> asedeno.scripts.mit.edu Git - linux.git/commit
Revert "pwm: Let pwm_get_state() return the last implemented state"
authorThierry Reding <thierry.reding@gmail.com>
Mon, 21 Oct 2019 10:41:40 +0000 (12:41 +0200)
committerThierry Reding <thierry.reding@gmail.com>
Mon, 21 Oct 2019 14:48:52 +0000 (16:48 +0200)
commit40a6b9a00930fd6b59aa2eb6135abc2efe5440c3
treebaaaa4663cce1e5f33560d4d53e296f8e97dee95
parent54ecb8f7028c5eb3d740bb82b0f1d90f2df63c5c
Revert "pwm: Let pwm_get_state() return the last implemented state"

It turns out that commit 01ccf903edd6 ("pwm: Let pwm_get_state() return
the last implemented state") causes backlight failures on a number of
boards. The reason is that some of the drivers do not write the full
state through to the hardware registers, which means that ->get_state()
subsequently does not return the correct state. Consumers which rely on
pwm_get_state() returning the current state will therefore get confused
and subsequently try to program a bad state.

Before this change can be made, existing drivers need to be more
carefully audited and fixed to behave as the framework expects. Until
then, keep the original behaviour of returning the software state that
was applied rather than reading the state back from hardware.

Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tested-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Tested-by: Michal Vokáč <michal.vokac@ysoft.com>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
drivers/pwm/core.c