]> git.baikalelectronics.ru Git - kernel.git/commit
net: stmmac: Move debugfs init/exit to ->probe()/->remove()
authorThierry Reding <treding@nvidia.com>
Tue, 27 Nov 2018 13:21:43 +0000 (14:21 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 30 Nov 2018 21:16:19 +0000 (13:16 -0800)
commitda1b1c28537ce7973cee3f20d42fe85e537efa21
tree3c152f2f43c58818525c36139fdfc2681b520f64
parentbd12291fd970d661d7c8a65463a3c850f13c64b3
net: stmmac: Move debugfs init/exit to ->probe()/->remove()

Setting up and tearing down debugfs is current unbalanced, as seen by
this error during resume from suspend:

    [  752.134067] dwc-eth-dwmac 2490000.ethernet eth0: ERROR failed to create debugfs directory
    [  752.134347] dwc-eth-dwmac 2490000.ethernet eth0: stmmac_hw_setup: failed debugFS registration

The imbalance happens because the driver creates the debugfs hierarchy
when the device is opened and tears it down when the device is closed.
There's little gain in that, and it could be argued that it is even
surprising because it's not usually done for other devices. Fix the
imbalance by moving the debugfs creation and teardown to the driver's
->probe() and ->remove() implementations instead.

Note that the ring descriptors cannot be read while the interface is
down, so make sure to return an empty file when the descriptors_status
debugfs file is read.

Signed-off-by: Thierry Reding <treding@nvidia.com>
Acked-by: Jose Abreu <joabreu@synopsys.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c