kernel: backport a few PHY layer fixes from upstream
authorFelix Fietkau <nbd@openwrt.org>
Fri, 26 Dec 2014 12:36:30 +0000 (12:36 +0000)
committerFelix Fietkau <nbd@openwrt.org>
Fri, 26 Dec 2014 12:36:30 +0000 (12:36 +0000)
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
SVN-Revision: 43776

target/linux/generic/patches-3.14/072-net-phy-Check-for-aneg-completion-before-setting-sta.patch [new file with mode: 0644]
target/linux/generic/patches-3.14/073-net-phy-resume-phydev-when-going-to-RESUMING.patch [new file with mode: 0644]

diff --git a/target/linux/generic/patches-3.14/072-net-phy-Check-for-aneg-completion-before-setting-sta.patch b/target/linux/generic/patches-3.14/072-net-phy-Check-for-aneg-completion-before-setting-sta.patch
new file mode 100644 (file)
index 0000000..8e25f8c
--- /dev/null
@@ -0,0 +1,35 @@
+From: Balakumaran Kannan <kumaran.4353@gmail.com>
+Date: Thu, 24 Apr 2014 08:22:47 +0530
+Subject: [PATCH] net phy: Check for aneg completion before setting state to
+ PHY_RUNNING
+
+phy_state_machine should check whether auto-negotiatin is completed
+before changing phydev->state from PHY_NOLINK to PHY_RUNNING. If
+auto-negotiation is not completed phydev->state should be set to
+PHY_AN.
+
+Signed-off-by: Balakumaran Kannan <kumaran.4353@gmail.com>
+Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
+Signed-off-by: David S. Miller <davem@davemloft.net>
+---
+
+--- a/drivers/net/phy/phy.c
++++ b/drivers/net/phy/phy.c
+@@ -744,6 +744,17 @@ void phy_state_machine(struct work_struc
+                       break;
+               if (phydev->link) {
++                      if (AUTONEG_ENABLE == phydev->autoneg) {
++                              err = phy_aneg_done(phydev);
++                              if (err < 0)
++                                      break;
++
++                              if (!err) {
++                                      phydev->state = PHY_AN;
++                                      phydev->link_timeout = PHY_AN_TIMEOUT;
++                                      break;
++                              }
++                      }
+                       phydev->state = PHY_RUNNING;
+                       netif_carrier_on(phydev->attached_dev);
+                       phydev->adjust_link(phydev->attached_dev);
diff --git a/target/linux/generic/patches-3.14/073-net-phy-resume-phydev-when-going-to-RESUMING.patch b/target/linux/generic/patches-3.14/073-net-phy-resume-phydev-when-going-to-RESUMING.patch
new file mode 100644 (file)
index 0000000..bbe09f2
--- /dev/null
@@ -0,0 +1,68 @@
+--- a/drivers/net/phy/phy.c
++++ b/drivers/net/phy/phy.c
+@@ -690,7 +690,7 @@ void phy_state_machine(struct work_struc
+       struct delayed_work *dwork = to_delayed_work(work);
+       struct phy_device *phydev =
+                       container_of(dwork, struct phy_device, state_queue);
+-      int needs_aneg = 0, do_suspend = 0;
++      bool needs_aneg = false, do_suspend = false, do_resume = false;
+       int err = 0;
+       mutex_lock(&phydev->lock);
+@@ -702,7 +702,7 @@ void phy_state_machine(struct work_struc
+       case PHY_PENDING:
+               break;
+       case PHY_UP:
+-              needs_aneg = 1;
++              needs_aneg = true;
+               phydev->link_timeout = PHY_AN_TIMEOUT;
+@@ -732,7 +732,7 @@ void phy_state_machine(struct work_struc
+                       phydev->adjust_link(phydev->attached_dev);
+               } else if (0 == phydev->link_timeout--) {
+-                      needs_aneg = 1;
++                      needs_aneg = true;
+                       /* If we have the magic_aneg bit, we try again */
+                       if (phydev->drv->flags & PHY_HAS_MAGICANEG)
+                               break;
+@@ -770,7 +770,7 @@ void phy_state_machine(struct work_struc
+                       netif_carrier_on(phydev->attached_dev);
+               } else {
+                       if (0 == phydev->link_timeout--)
+-                              needs_aneg = 1;
++                              needs_aneg = true;
+               }
+               phydev->adjust_link(phydev->attached_dev);
+@@ -806,7 +806,7 @@ void phy_state_machine(struct work_struc
+                       phydev->link = 0;
+                       netif_carrier_off(phydev->attached_dev);
+                       phydev->adjust_link(phydev->attached_dev);
+-                      do_suspend = 1;
++                      do_suspend = true;
+               }
+               break;
+       case PHY_RESUMING:
+@@ -855,6 +855,7 @@ void phy_state_machine(struct work_struc
+                       }
+                       phydev->adjust_link(phydev->attached_dev);
+               }
++              do_resume = true;
+               break;
+       }
+@@ -862,9 +863,10 @@ void phy_state_machine(struct work_struc
+       if (needs_aneg)
+               err = phy_start_aneg(phydev);
+-
+-      if (do_suspend)
++      else if (do_suspend)
+               phy_suspend(phydev);
++      else if (do_resume)
++              phy_resume(phydev);
+       if (err < 0)
+               phy_error(phydev);