aboutsummaryrefslogtreecommitdiffstats
path: root/target/linux/layerscape/patches-5.4/701-net-0297-staging-fsl_ppfe-eth-unregister-netdev-after-pfe_phy.patch
blob: 07313293845c31bc1b90d21b8430e821e9f6b1de (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
From 4ebb4e490dd924d1a8ca7fe693a0dd1cfa150687 Mon Sep 17 00:00:00 2001
From: Calvin Johnson <calvin.johnson@nxp.com>
Date: Mon, 30 Apr 2018 11:40:01 +0530
Subject: [PATCH] staging: fsl_ppfe/eth: unregister netdev after pfe_phy_exit

rmmod pfe.ko throws below warning:

kernfs: can not remove 'phydev', no directory
------------[ cut here ]------------
WARNING: CPU: 0 PID: 2230 at fs/kernfs/dir.c:1481
kernfs_remove_by_name_ns+0x90/0xa0

This is caused when the unregistered netdev structure is accessed to
disconnect phy.

Resolve the issue by unregistering netdev after disconnecting phy.

Signed-off-by: Calvin Johnson <calvin.johnson@nxp.com>
---
 drivers/staging/fsl_ppfe/pfe_eth.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

--- a/drivers/staging/fsl_ppfe/pfe_eth.c
+++ b/drivers/staging/fsl_ppfe/pfe_eth.c
@@ -2464,15 +2464,15 @@ static void pfe_eth_exit_one(struct pfe_
 {
 	netif_info(priv, probe, priv->ndev, "%s\n", __func__);
 
-	if (!us) {
+	if (!us)
 		pfe_eth_sysfs_exit(priv->ndev);
 
-		unregister_netdev(priv->ndev);
-	}
-
 	if (!(priv->einfo->phy_flags & GEMAC_NO_PHY))
 		pfe_phy_exit(priv->ndev);
 
+	if (!us)
+		unregister_netdev(priv->ndev);
+
 	if (priv->mii_bus)
 		pfe_eth_mdio_exit(priv->mii_bus);