net: phy: broadcom: Allow enabling or disabling of EEE

In preparation for adding support for Wirespeed/downshift, we need to
change bcm_phy_eee_enable() to allow enabling or disabling EEE, so make
the function take an extra enable/disable boolean parameter and rename
it to illustrate it sets EEE, not necessarily just enables it.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Florian Fainelli 2016-11-22 11:40:56 -08:00 committed by David S. Miller
parent d06f78c423
commit 99cec8a4dd
4 changed files with 13 additions and 7 deletions

View File

@ -104,7 +104,7 @@ static int bcm_cygnus_config_init(struct phy_device *phydev)
return rc; return rc;
/* Advertise EEE */ /* Advertise EEE */
rc = bcm_phy_enable_eee(phydev); rc = bcm_phy_set_eee(phydev, true);
if (rc) if (rc)
return rc; return rc;

View File

@ -195,7 +195,7 @@ int bcm_phy_enable_apd(struct phy_device *phydev, bool dll_pwr_down)
} }
EXPORT_SYMBOL_GPL(bcm_phy_enable_apd); EXPORT_SYMBOL_GPL(bcm_phy_enable_apd);
int bcm_phy_enable_eee(struct phy_device *phydev) int bcm_phy_set_eee(struct phy_device *phydev, bool enable)
{ {
int val; int val;
@ -205,7 +205,10 @@ int bcm_phy_enable_eee(struct phy_device *phydev)
if (val < 0) if (val < 0)
return val; return val;
val |= LPI_FEATURE_EN | LPI_FEATURE_EN_DIG1000X; if (enable)
val |= LPI_FEATURE_EN | LPI_FEATURE_EN_DIG1000X;
else
val &= ~(LPI_FEATURE_EN | LPI_FEATURE_EN_DIG1000X);
phy_write_mmd_indirect(phydev, BRCM_CL45VEN_EEE_CONTROL, phy_write_mmd_indirect(phydev, BRCM_CL45VEN_EEE_CONTROL,
MDIO_MMD_AN, (u32)val); MDIO_MMD_AN, (u32)val);
@ -216,14 +219,17 @@ int bcm_phy_enable_eee(struct phy_device *phydev)
if (val < 0) if (val < 0)
return val; return val;
val |= (MDIO_AN_EEE_ADV_100TX | MDIO_AN_EEE_ADV_1000T); if (enable)
val |= (MDIO_AN_EEE_ADV_100TX | MDIO_AN_EEE_ADV_1000T);
else
val &= ~(MDIO_AN_EEE_ADV_100TX | MDIO_AN_EEE_ADV_1000T);
phy_write_mmd_indirect(phydev, BCM_CL45VEN_EEE_ADV, phy_write_mmd_indirect(phydev, BCM_CL45VEN_EEE_ADV,
MDIO_MMD_AN, (u32)val); MDIO_MMD_AN, (u32)val);
return 0; return 0;
} }
EXPORT_SYMBOL_GPL(bcm_phy_enable_eee); EXPORT_SYMBOL_GPL(bcm_phy_set_eee);
int bcm_phy_downshift_get(struct phy_device *phydev, u8 *count) int bcm_phy_downshift_get(struct phy_device *phydev, u8 *count)
{ {

View File

@ -36,7 +36,7 @@ int bcm_phy_config_intr(struct phy_device *phydev);
int bcm_phy_enable_apd(struct phy_device *phydev, bool dll_pwr_down); int bcm_phy_enable_apd(struct phy_device *phydev, bool dll_pwr_down);
int bcm_phy_enable_eee(struct phy_device *phydev); int bcm_phy_set_eee(struct phy_device *phydev, bool enable);
int bcm_phy_downshift_get(struct phy_device *phydev, u8 *count); int bcm_phy_downshift_get(struct phy_device *phydev, u8 *count);

View File

@ -199,7 +199,7 @@ static int bcm7xxx_28nm_config_init(struct phy_device *phydev)
if (ret) if (ret)
return ret; return ret;
ret = bcm_phy_enable_eee(phydev); ret = bcm_phy_set_eee(phydev, true);
if (ret) if (ret)
return ret; return ret;