Commit 3b8b571f authored by Uwe Kleine-König's avatar Uwe Kleine-König Committed by Thierry Reding

pwm: tegra: Don't needlessly enable and disable the clock in .remove()

There is no reason to enable the PWM clock just to assert the reset
control. (If the reset control depends on the clock this is a bug and
probably it doesn't because in .probe() the reset is deasserted without
the clock being enabled.)
Signed-off-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: default avatarThierry Reding <thierry.reding@gmail.com>
parent 86f7fa71
...@@ -300,14 +300,8 @@ static int tegra_pwm_probe(struct platform_device *pdev) ...@@ -300,14 +300,8 @@ static int tegra_pwm_probe(struct platform_device *pdev)
static int tegra_pwm_remove(struct platform_device *pdev) static int tegra_pwm_remove(struct platform_device *pdev)
{ {
struct tegra_pwm_chip *pc = platform_get_drvdata(pdev); struct tegra_pwm_chip *pc = platform_get_drvdata(pdev);
int err;
err = clk_prepare_enable(pc->clk);
if (err < 0)
return err;
reset_control_assert(pc->rst); reset_control_assert(pc->rst);
clk_disable_unprepare(pc->clk);
return pwmchip_remove(&pc->chip); return pwmchip_remove(&pc->chip);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment