free_candev() already calls netif_napi_del(), so there is no need to call
it explicitly. It is harmless, but useless.
This makes the code mode consistent with the error handling path of
ctucan_probe_common().
While at it, remove a wrong comment about the return value of this
function.
Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
---
The comment went wrong after 45413bf75919 ("can: ctucanfd: Convert to platform remove callback returning void")
---
drivers/net/can/ctucanfd/ctucanfd_platform.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/can/ctucanfd/ctucanfd_platform.c b/drivers/net/can/ctucanfd/ctucanfd_platform.c
index 55bb10b157b4..8fe224b8dac0 100644
--- a/drivers/net/can/ctucanfd/ctucanfd_platform.c
+++ b/drivers/net/can/ctucanfd/ctucanfd_platform.c
@@ -84,7 +84,6 @@ static int ctucan_platform_probe(struct platform_device *pdev)
* @pdev: Handle to the platform device structure
*
* This function frees all the resources allocated to the device.
- * Return: 0 always
*/
static void ctucan_platform_remove(struct platform_device *pdev)
{
@@ -95,7 +94,6 @@ static void ctucan_platform_remove(struct platform_device *pdev)
unregister_candev(ndev);
pm_runtime_disable(&pdev->dev);
- netif_napi_del(&priv->napi);
free_candev(ndev);
}
--
2.34.1
