aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSergey Ryazanov <ryazanov.s.a@gmail.com>2017-06-07 01:25:32 +0300
committerJohn Crispin <john@phrozen.org>2017-06-07 11:06:29 +0200
commit2cc61e6e8e951b1480a024fdb66bc539aa163bbc (patch)
treea439f2d5f7ffd9ef256bdb27e478c2d8371368d3
parent913b2290caad437bb523d6f1ee5a70707f7f7ef4 (diff)
downloadupstream-2cc61e6e8e951b1480a024fdb66bc539aa163bbc.tar.gz
upstream-2cc61e6e8e951b1480a024fdb66bc539aa163bbc.tar.bz2
upstream-2cc61e6e8e951b1480a024fdb66bc539aa163bbc.zip
ip17xx: correct aneg_done return value
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>
-rw-r--r--target/linux/generic/files/drivers/net/phy/ip17xx.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/target/linux/generic/files/drivers/net/phy/ip17xx.c b/target/linux/generic/files/drivers/net/phy/ip17xx.c
index 9eedfd4d28..85a9617a5d 100644
--- a/target/linux/generic/files/drivers/net/phy/ip17xx.c
+++ b/target/linux/generic/files/drivers/net/phy/ip17xx.c
@@ -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)