pwm: imx27: Fix clock handling in pwm_imx27_apply()
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Sun, 9 Feb 2020 21:31:06 +0000 (22:31 +0100)
committerThierry Reding <thierry.reding@gmail.com>
Mon, 30 Mar 2020 14:55:25 +0000 (16:55 +0200)
commit15d4dbd601591858611184f9ddeb5bf21569159c
treef90ec7570c283eb4f19aa5e94541c237876dd38a
parent2c25b07e5ec119cab609e41407a1fb3fa61442f5
pwm: imx27: Fix clock handling in pwm_imx27_apply()

pwm_imx27_apply() enables the clocks if the previous PWM state was
disabled. Given that the clocks are supposed to be left on iff the PWM
is running, the decision to disable the clocks at the end of the
function must not depend on the previous state.

Without this fix the enable count of the two affected clocks increases
by one whenever ->apply() changes from one disabled state to another.

Fixes: bd88d319abe9 ("pwm: imx27: Unconditionally write state to hardware")
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
drivers/pwm/pwm-imx27.c