]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
net: netcp: fix cleanup interface list in netcp_remove()
authorKaricheri, Muralidharan <m-karicheri2@ti.com>
Tue, 28 Jul 2015 22:20:12 +0000 (18:20 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 30 Jul 2015 01:37:40 +0000 (18:37 -0700)
Currently if user do rmmod keystone_netcp.ko following warning is
seen :-

[   59.035891] ------------[ cut here ]------------
[   59.040535] WARNING: CPU: 2 PID: 1619 at drivers/net/ethernet/ti/
netcp_core.c:2127 netcp_remove)

This is because the interface list is not cleaned up in netcp_remove.
This patch fixes this. Also fix some checkpatch related warnings.

Signed-off-by: Murali Karicheri <m-karicheri2@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/ti/netcp_core.c

index ec8ed30196f3b8bd8e0ecf670d35a80c93ad4639..a1c6961323dc1ac1eb78b4c65afb1ba24340d411 100644 (file)
@@ -2112,6 +2112,7 @@ probe_quit:
 static int netcp_remove(struct platform_device *pdev)
 {
        struct netcp_device *netcp_device = platform_get_drvdata(pdev);
+       struct netcp_intf *netcp_intf, *netcp_tmp;
        struct netcp_inst_modpriv *inst_modpriv, *tmp;
        struct netcp_module *module;
 
@@ -2123,8 +2124,16 @@ static int netcp_remove(struct platform_device *pdev)
                list_del(&inst_modpriv->inst_list);
                kfree(inst_modpriv);
        }
-       WARN(!list_empty(&netcp_device->interface_head), "%s interface list not empty!\n",
-            pdev->name);
+
+       /* now that all modules are removed, clean up the interfaces */
+       list_for_each_entry_safe(netcp_intf, netcp_tmp,
+                                &netcp_device->interface_head,
+                                interface_list) {
+               netcp_delete_interface(netcp_device, netcp_intf->ndev);
+       }
+
+       WARN(!list_empty(&netcp_device->interface_head),
+            "%s interface list not empty!\n", pdev->name);
 
        devm_kfree(&pdev->dev, netcp_device);
        pm_runtime_put_sync(&pdev->dev);