diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2021-05-04 15:25:37 +0200 |
|---|---|---|
| committer | Thierry Reding <thierry.reding@gmail.com> | 2021-06-30 19:12:18 +0200 |
| commit | 0512f0503b051a98608a9b0d33b75482b960a313 (patch) | |
| tree | 3ddcea33189dc2d043becf248f15bb61229affe8 | |
| parent | pwm: berlin: Implement .apply() callback (diff) | |
| download | linux-0512f0503b051a98608a9b0d33b75482b960a313.tar.xz linux-0512f0503b051a98608a9b0d33b75482b960a313.zip | |
pwm: berlin: Don't check the return code of pwmchip_remove()
pwmchip_remove() always returns 0. Don't use the value to make it
possible to eventually change the function to return void. This is a
good thing as pwmchip_remove() is usually called from a remove function
(mostly for platform devices) and their return value is ignored by the
device core anyhow.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
| -rw-r--r-- | drivers/pwm/pwm-berlin.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/pwm/pwm-berlin.c b/drivers/pwm/pwm-berlin.c index c17d7e8f0470..5537b5f6dd5d 100644 --- a/drivers/pwm/pwm-berlin.c +++ b/drivers/pwm/pwm-berlin.c @@ -256,12 +256,12 @@ static int berlin_pwm_probe(struct platform_device *pdev) static int berlin_pwm_remove(struct platform_device *pdev) { struct berlin_pwm_chip *bpc = platform_get_drvdata(pdev); - int ret; - ret = pwmchip_remove(&bpc->chip); + pwmchip_remove(&bpc->chip); + clk_disable_unprepare(bpc->clk); - return ret; + return 0; } #ifdef CONFIG_PM_SLEEP |