pwm: sifive: Don't check the return code of pwmchip_remove()
[ Upstream commit ceb2c2842f3664dcc4e6d8cb317e1e83bb81b1e5 ] pwmchip_remove() returns always 0. Don't use the value to make it possible to eventually change the function to return void. Also the driver core ignores the return value of pwm_sifive_remove(). Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> Signed-off-by: Thierry Reding <thierry.reding@gmail.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:

committed by
Greg Kroah-Hartman

parent
b7e2d64d67
commit
6d7f7ffbcd
@@ -296,7 +296,7 @@ static int pwm_sifive_remove(struct platform_device *dev)
|
|||||||
struct pwm_sifive_ddata *ddata = platform_get_drvdata(dev);
|
struct pwm_sifive_ddata *ddata = platform_get_drvdata(dev);
|
||||||
bool is_enabled = false;
|
bool is_enabled = false;
|
||||||
struct pwm_device *pwm;
|
struct pwm_device *pwm;
|
||||||
int ret, ch;
|
int ch;
|
||||||
|
|
||||||
for (ch = 0; ch < ddata->chip.npwm; ch++) {
|
for (ch = 0; ch < ddata->chip.npwm; ch++) {
|
||||||
pwm = &ddata->chip.pwms[ch];
|
pwm = &ddata->chip.pwms[ch];
|
||||||
@@ -309,10 +309,10 @@ static int pwm_sifive_remove(struct platform_device *dev)
|
|||||||
clk_disable(ddata->clk);
|
clk_disable(ddata->clk);
|
||||||
|
|
||||||
clk_disable_unprepare(ddata->clk);
|
clk_disable_unprepare(ddata->clk);
|
||||||
ret = pwmchip_remove(&ddata->chip);
|
pwmchip_remove(&ddata->chip);
|
||||||
clk_notifier_unregister(ddata->clk, &ddata->notifier);
|
clk_notifier_unregister(ddata->clk, &ddata->notifier);
|
||||||
|
|
||||||
return ret;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static const struct of_device_id pwm_sifive_of_match[] = {
|
static const struct of_device_id pwm_sifive_of_match[] = {
|
||||||
|
Reference in New Issue
Block a user