From 91d10161226c1edfb1354441ad78b9ae679a6108 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Uwe=20Kleine-K=C3=B6nig?= Date: Wed, 30 Apr 2025 11:57:48 +0200 Subject: [PATCH] clk: pwm: Don't reconfigure running PWM at probe time MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit If the PWM is enabled already when .probe() is entered, period and duty_cycle are updated which essentially corresponds to a clock frequency change. This is unusual and surprising. So update the settings only when the clock gets prepared. Signed-off-by: Uwe Kleine-König Link: https://lore.kernel.org/r/a974f1147e1a080bf5ad7f1752c92b24516df284.1746006578.git.ukleinek@baylibre.com Signed-off-by: Stephen Boyd --- drivers/clk/clk-pwm.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/drivers/clk/clk-pwm.c b/drivers/clk/clk-pwm.c index f5e6fef3f4d5d..856828d5f58c4 100644 --- a/drivers/clk/clk-pwm.c +++ b/drivers/clk/clk-pwm.c @@ -109,12 +109,6 @@ static int clk_pwm_probe(struct platform_device *pdev) pwm_init_state(pwm, &clk_pwm->state); pwm_set_relative_duty_cycle(&clk_pwm->state, 1, 2); - - ret = pwm_apply_might_sleep(pwm, &clk_pwm->state); - if (ret < 0) - return ret; - - /* set enabled only now to not enable output above */ clk_pwm->state.enabled = true; clk_name = node->name; -- 2.47.2