linux-stable.git/drivers/net/phy/phy_device.c, branch v3.14

net: phy: fix uninitalized ethtool_wolinfo in phy_suspend

2014-03-15T02:38:54+00:00

Callers of phy_ethtool_get_wol are supposed to provide a properly
cleared struct ethtool_wolinfo. Therefore, fix phy_suspend to clear
it before passing it to phy_ethtool_get_wol.

Signed-off-by: Sebastian Hesselbarth 
Reviewed-by: Ben Hutchings 
Signed-off-by: David S. Miller

phy: unmask link partner capabilities

2014-02-25T23:39:09+00:00

Masking the link partner's capabilities with local capabilities can be
misleading in autonegotiation scenarios such as PAUSE frame
autonegotiation.
This patch calculates the join between the local capabilities and the
link parner capabilities, when it determines the speed and duplex
settings, but does not mask any of the link partner capabilities when
it calculates PAUSE frame settings.

Signed-off-by: Cristian Bercaru 
Signed-off-by: David S. Miller

net: phy: ensure Gigabit features are masked off if requested

2014-02-05T04:15:51+00:00

When a Gigabit PHY device is connected to a 10/100Mbits capable Ethernet
MAC, the driver will restrict the phydev->supported modes to mask off
Gigabit. If the Gigabit PHY comes out of reset with the Gigabit features
set by default in MII_CTRL1000, it will keep advertising these feature,
so by the time we call genphy_config_advert(), the condition on
phydev->supported having the Gigabit features on is false, and we do not
update MII_CTRL1000 with updated values, and we keep advertising Gigabit
features, eventually configuring the PHY for Gigabit whilst the Ethernet
MAC does not support that.

This patches fixes the problem by ensuring that the Gigabit feature bits
are always cleared in MII_CTRL1000, if the PHY happens to be a Gigabit
PHY, and then, if Gigabit features are supported, setting those and
updating MII_CTRL1000 accordingly.

Reported-by: Max Filippov 
Signed-off-by: Florian Fainelli 
Tested-by: Max Filippov 
Signed-off-by: David S. Miller

phy: cleanup 10g code

2014-01-20T03:55:50+00:00

Code should avoid needless exports, don't export something unless it used.
Make local functions static and remove unused stubs.

Signed-off-by: Stephen Hemminger 
Signed-off-by: David S. Miller

phylib: Support attaching to generic 10g driver

2014-01-13T22:29:49+00:00

phy_attach_direct() may now attach to a generic 10G driver. It can
also be used exactly as phy_connect_direct(), which will be useful
when using of_mdio, as phy_connect (and therefore of_phy_connect)
start the PHY state machine, which is currently irrelevant for 10G
PHYs.

Signed-off-by: Andy Fleming 
Signed-off-by: Shaohui Xie 
Acked-by: Florian Fainelli 
Signed-off-by: David S. Miller

phylib: Add generic 10G driver

2014-01-13T22:29:49+00:00

Very incomplete, but will allow for binding an ethernet controller
to it.

Signed-off-by: Andy Fleming 
Signed-off-by: Shaohui Xie 
Signed-off-by: David S. Miller

phylib: turn genphy_driver to an array

2014-01-13T22:29:49+00:00

Then other generic phy driver such as generic 10g phy driver can join it.

Signed-off-by: Shaohui Xie 
Signed-off-by: David S. Miller

phylib: make phy_scan_fixups() static

2014-01-05T00:27:58+00:00

phy_scan_fixups()  isn't and shouldn't be called by the drivers directly, so
unexport it. And since Florian Fainelli's recent patches, the function is only
called locally, so we can make it static as well.

Signed-off-by: Sergei Shtylyov 
Signed-off-by: David S. Miller

phylib: remove unused adjust_state() callback

2014-01-05T00:27:58+00:00

Remove adjust_state() callback from 'struct phy_device' since it seems to have
never been really used from the inception: phy_start_machine() has been always
called with 2nd argument equal to NULL.

Signed-off-by: Sergei Shtylyov 
Signed-off-by: David S. Miller

phy: kill excess empty lines

2014-01-05T00:27:58+00:00

Remove excess empty lines such as those between a function call and its result
check and just duplicate ones between functions.

Signed-off-by: Sergei Shtylyov 
Signed-off-by: David S. Miller