net: bcmgenet: Delay PHY initialization to bcmgenet_open()
authorFlorian Fainelli <f.fainelli@gmail.com>
Thu, 16 Jul 2015 22:51:18 +0000 (15:51 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 21 Jul 2015 03:48:14 +0000 (20:48 -0700)
commit6cc8e6d4dcb3651eea9b01db3e195fffb19fb24f
treec620a5621936dff721865cb0d721203fcdca69a5
parentc624f89121020882b3db0a33cac8daf151d2930f
net: bcmgenet: Delay PHY initialization to bcmgenet_open()

We are currently doing a full PHY initialization and even starting the
pHY state machine during bcmgenet_mii_init() which is executed in the
driver's probe function. This is convenient to determine whether we can
attach to a proper PHY device but comes at the expense of spending up to
10ms per MDIO transactions (to reach the waitqueue timeout), which slows
things down.

This also creates a sitaution where we end-up attaching twice to the
PHY, which is not quite correct either.

Fix this by moving bcmgenet_mii_probe() into bcmgenet_open() and update
its error path accordingly.

Avoid printing the message "attached PHY at address 1 [...]" every time
we bring up/down the interface and remove this print since it duplicates
what the PHY driver already does for us.

Fixes: 1c1008c793fa4 ("net: bcmgenet: add main driver file")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/genet/bcmgenet.c
drivers/net/ethernet/broadcom/genet/bcmgenet.h
drivers/net/ethernet/broadcom/genet/bcmmii.c