ip17xx: correct aneg_done return value
authorSergey Ryazanov <ryazanov.s.a@gmail.com>
Tue, 6 Jun 2017 22:25:32 +0000 (01:25 +0300)
committerJo-Philipp Wich <jo@mein.io>
Wed, 13 Dec 2017 14:53:16 +0000 (15:53 +0100)
PHY core treats any positive return value as the auto-negotiation done
indication. Since we do not actually check any device register in this
callback then update it to return positive value with a neutral meaning
instead of the register flag to avoid  confusing for future readers.

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

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

index a4fe53dc5bdd1177121a2980a6245ea9ae080174..dbb867e30ea85c3de19f2b5bf9f85d80c20b8dda 100644 (file)
@@ -1334,7 +1334,7 @@ static int ip17xx_config_aneg(struct phy_device *pdev)
 
 static int ip17xx_aneg_done(struct phy_device *pdev)
 {
-       return BMSR_ANEGCOMPLETE;
+       return 1;       /* Return any positive value */
 }
 
 static int ip17xx_update_link(struct phy_device *pdev)