Commit 868f13bd authored by Uwe Kleine-König's avatar Uwe Kleine-König Committed by Thierry Reding

pwm: vt8500: Only unprepare the clock after the pwmchip was removed

Until pwmchip_remove() returns the PWM is supposed to work, so
pwmchip_remove() must be called before the clock is stopped.

The return value of pwmchip_remove doesn't need to be checked because
it returns zero anyhow and I plan to make it return void soon.
Signed-off-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarThierry Reding <thierry.reding@gmail.com>
parent fb2cb3bf
...@@ -240,10 +240,11 @@ static int vt8500_pwm_remove(struct platform_device *pdev) ...@@ -240,10 +240,11 @@ static int vt8500_pwm_remove(struct platform_device *pdev)
{ {
struct vt8500_chip *chip = platform_get_drvdata(pdev); struct vt8500_chip *chip = platform_get_drvdata(pdev);
pwmchip_remove(&chip->chip);
clk_unprepare(chip->clk); clk_unprepare(chip->clk);
return pwmchip_remove(&chip->chip); return 0;
} }
static struct platform_driver vt8500_pwm_driver = { static struct platform_driver vt8500_pwm_driver = {
......
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