smsc911x: power-up phydev before doing a software reset.
authorEnric Balletbo i Serra <eballetbo@iseebcn.com>
Thu, 13 Nov 2014 08:14:34 +0000 (09:14 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 13 Nov 2014 20:09:28 +0000 (15:09 -0500)
commitccf899a27c08038db91765ff12bb0380dcd85887
tree83f4fbe419e32adce9844eade6babcda96aa8aa1
parent0c828f2f8395fb5e7faf0a116e476a3ce992a199
smsc911x: power-up phydev before doing a software reset.

With commit be9dad1f9f26604fb ("net: phy: suspend phydev when going
to HALTED"), the PHY device will be put in a low-power mode using
BMCR_PDOWN if the the interface is set down. The smsc911x driver does
a software_reset opening the device driver (ndo_open). In such case,
the PHY must be powered-up before access to any register and before
calling the software_reset function. Otherwise, as the PHY is powered
down the software reset fails and the interface can not be enabled
again.

This patch fixes this scenario that is easy to reproduce setting down
the network interface and setting up again.

    $ ifconfig eth0 down
    $ ifconfig eth0 up
    ifconfig: SIOCSIFFLAGS: Input/output error

Signed-off-by: Enric Balletbo i Serra <eballetbo@iseebcn.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/smsc/smsc911x.c