diff options
author | Guenter Roeck <linux@roeck-us.net> | 2015-03-02 20:03:27 (GMT) |
---|---|---|
committer | Honghua Yin <Hong-Hua.Yin@freescale.com> | 2015-03-17 06:50:24 (GMT) |
commit | 7a590b6c487881f36caf0369455c39b2b31ae5f6 (patch) | |
tree | 8a7a5919fdde75d803b78c55617fada80817fe5a /crypto/seed.c | |
parent | 753510e1db157ac23e45678522520ba637529bb4 (diff) | |
download | linux-fsl-qoriq-7a590b6c487881f36caf0369455c39b2b31ae5f6.tar.xz |
gianfar: Reduce logging noise seen due to phy polling if link is down
Commit 6ce29b0e2a04 ("gianfar: Avoid unnecessary reg accesses in adjust_link()")
eliminates unnecessary calls to adjust_link for phy devices which don't support
interrupts and need polling. As part of that work, the 'new_state' local flag,
which was used to reduce logging noise on the console, was eliminated.
Unfortunately, that means that a 'Link is Down' log message will now be
issued continuously if a link is configured as UP, the link state is down,
and the associated phy requires polling. This occurs because priv->oldduplex
is -1 in this case, which always differs from phydev->duplex. In addition,
phydev->speed may also differ from priv->oldspeed. gfar_update_link_state()
is therefore called each time a phy is polled, even if the link state did not
change.
Cc: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Claudiu Manoil <claudiu.manoil@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Change-Id: Iab3e64c4c5828133ff0e1441a087e55c5287408f
Reviewed-on: http://git.am.freescale.net:8181/32703
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Honghua Yin <Hong-Hua.Yin@freescale.com>
Diffstat (limited to 'crypto/seed.c')
0 files changed, 0 insertions, 0 deletions