diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2021-11-12 15:53:52 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2021-11-15 13:15:07 +0000 |
| commit | e99fa4230fa89f9e76073dd6af3dd85c33ba70b8 (patch) | |
| tree | 812e08a150d7783c9ee48bcf12ced4f481e7c08a | |
| parent | ac746c8520d9d056b6963ecca8ff1da9929d02f1 (diff) | |
| download | linux-e99fa4230fa89f9e76073dd6af3dd85c33ba70b8.tar.bz2 | |
net: dsa: vsc73xxx: Make vsc73xx_remove() return void
vsc73xx_remove() returns zero unconditionally and no caller checks the
returned value. So convert the function to return no value.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Vladimir Oltean <olteanv@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/dsa/vitesse-vsc73xx-core.c | 4 | ||||
| -rw-r--r-- | drivers/net/dsa/vitesse-vsc73xx.h | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/dsa/vitesse-vsc73xx-core.c b/drivers/net/dsa/vitesse-vsc73xx-core.c index a4b1447ff055..4c18f619ec02 100644 --- a/drivers/net/dsa/vitesse-vsc73xx-core.c +++ b/drivers/net/dsa/vitesse-vsc73xx-core.c @@ -1216,12 +1216,10 @@ int vsc73xx_probe(struct vsc73xx *vsc) } EXPORT_SYMBOL(vsc73xx_probe); -int vsc73xx_remove(struct vsc73xx *vsc) +void vsc73xx_remove(struct vsc73xx *vsc) { dsa_unregister_switch(vsc->ds); gpiod_set_value(vsc->reset, 1); - - return 0; } EXPORT_SYMBOL(vsc73xx_remove); diff --git a/drivers/net/dsa/vitesse-vsc73xx.h b/drivers/net/dsa/vitesse-vsc73xx.h index 30b951504e65..30b1f0a36566 100644 --- a/drivers/net/dsa/vitesse-vsc73xx.h +++ b/drivers/net/dsa/vitesse-vsc73xx.h @@ -26,5 +26,5 @@ struct vsc73xx_ops { int vsc73xx_is_addr_valid(u8 block, u8 subblock); int vsc73xx_probe(struct vsc73xx *vsc); -int vsc73xx_remove(struct vsc73xx *vsc); +void vsc73xx_remove(struct vsc73xx *vsc); void vsc73xx_shutdown(struct vsc73xx *vsc); |