diff options
| author |   Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2017-03-01 17:24:47 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-03-02 14:33:49 -0800 |
| commit | be12502e2e64854dbe0a2ddff6d26ec1143d6890 (patch) | |
| tree | 147e0110de4d5228f09def2d8c112f69fb62b87c /drivers/net/ethernet/broadcom/sb1250-mac.c | |
| parent | tcp: fix potential double free issue for fastopen_req (diff) | |
| download | linux-dev-be12502e2e64854dbe0a2ddff6d26ec1143d6890.tar.xz linux-dev-be12502e2e64854dbe0a2ddff6d26ec1143d6890.zip | |
drivers: net: ethernet: remove incorrect __exit markups
Even if bus is not hot-pluggable, devices can be unbound from the
driver via sysfs, so we should not be using __exit annotations on
remove() methods. The only exception is drivers registered with
platform_driver_probe() which specifically disables sysfs bind/unbind
attributes.
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/broadcom/sb1250-mac.c')
| -rw-r--r-- | drivers/net/ethernet/broadcom/sb1250-mac.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/broadcom/sb1250-mac.c b/drivers/net/ethernet/broadcom/sb1250-mac.c index 89d4feba1a9a..55c8e25b43d9 100644 --- a/drivers/net/ethernet/broadcom/sb1250-mac.c +++ b/drivers/net/ethernet/broadcom/sb1250-mac.c @@ -2617,7 +2617,7 @@ out_out: return err; } -static int __exit sbmac_remove(struct platform_device *pldev) +static int sbmac_remove(struct platform_device *pldev) { struct net_device *dev = platform_get_drvdata(pldev); struct sbmac_softc *sc = netdev_priv(dev); @@ -2634,7 +2634,7 @@ static int __exit sbmac_remove(struct platform_device *pldev) static struct platform_driver sbmac_driver = { .probe = sbmac_probe, - .remove = __exit_p(sbmac_remove), + .remove = sbmac_remove, .driver = { .name = sbmac_string, }, |