ravb: Fixed to be able to unload modules
authorYuusuke Ashizuka <ashiduka@fujitsu.com>
Thu, 20 Aug 2020 09:43:07 +0000 (18:43 +0900)
committerDavid S. Miller <davem@davemloft.net>
Mon, 24 Aug 2020 22:38:02 +0000 (15:38 -0700)
commit1838d6c62f57836639bd3d83e7855e0ee4f6defc
tree2998c83924b7b4e963187a8fb2ba1510f6b5a8d6
parent3622adb02623cde94ea85c68e9a968d0059de545
ravb: Fixed to be able to unload modules

When this driver is built as a module, I cannot rmmod it after insmoding
it.
This is because that this driver calls ravb_mdio_init() at the time of
probe, and module->refcnt is incremented by alloc_mdio_bitbang() called
after that.
Therefore, even if ifup is not performed, the driver is in use and rmmod
cannot be performed.

$ lsmod
Module                  Size  Used by
ravb                   40960  1
$ rmmod ravb
rmmod: ERROR: Module ravb is in use

Call ravb_mdio_init() at open and free_mdio_bitbang() at close, thereby
rmmod is possible in the ifdown state.

Fixes: c156633f1353 ("Renesas Ethernet AVB driver proper")
Signed-off-by: Yuusuke Ashizuka <ashiduka@fujitsu.com>
Reviewed-by: Sergei Shtylyov <sergei.shtylyov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/renesas/ravb_main.c