mvswitch: fix autonegotiation issue
authorSergey Ryazanov <ryazanov.s.a@gmail.com>
Tue, 6 Jun 2017 22:25:31 +0000 (01:25 +0300)
committerJo-Philipp Wich <jo@mein.io>
Wed, 13 Dec 2017 14:52:49 +0000 (15:52 +0100)
The Marvel 88E6060 switch has an MDIO interface, but does not emulate
regular PHY behavior for the host. The network core can not detect using
the generic code, whether the connection via the attached PHY can be
used or not. The PHY's state machine is stuck in a state of
auto-negotiation and does not go any further so the Ethernet interface
of the router stay forever in the not-runing state.

Fix this issue by implementing the aneg_done callback to be able to
inform the network core that the Ethernet interface link to which the
switch is connected can be marked as RUNNING.

Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
(cherry picked from commit 913b2290caad437bb523d6f1ee5a70707f7f7ef4)

target/linux/generic/files/drivers/net/phy/mvswitch.c

index af31d0b..b2dfe0b 100644 (file)
@@ -341,6 +341,12 @@ mvswitch_read_status(struct phy_device *pdev)
        return 0;
 }
 
+static int
+mvswitch_aneg_done(struct phy_device *phydev)
+{
+       return 1;       /* Return any positive value */
+}
+
 static int
 mvswitch_config_aneg(struct phy_device *phydev)
 {
@@ -412,6 +418,7 @@ static struct phy_driver mvswitch_driver = {
        .detach         = &mvswitch_detach,
        .config_init    = &mvswitch_config_init,
        .config_aneg    = &mvswitch_config_aneg,
+       .aneg_done      = &mvswitch_aneg_done,
        .read_status    = &mvswitch_read_status,
        .driver         = { .owner = THIS_MODULE,},
 };