From: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Date: Thu, 17 Jun 2021 09:51:42 +0000 (+0200)
Subject: pwm: tegra: Don't needlessly enable and disable the clock in .remove()
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=3b8b571fcc49e794703eb01a97e9ed8b85e88304;p=linux.git

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: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---

diff --git a/drivers/pwm/pwm-tegra.c b/drivers/pwm/pwm-tegra.c
index a051cf43e7d71..e865743e59890 100644
--- a/drivers/pwm/pwm-tegra.c
+++ b/drivers/pwm/pwm-tegra.c
@@ -300,14 +300,8 @@ static int tegra_pwm_probe(struct platform_device *pdev)
 static int tegra_pwm_remove(struct platform_device *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);
-	clk_disable_unprepare(pc->clk);
 
 	return pwmchip_remove(&pc->chip);
 }