Commit 2a9df742 authored by Florian Fainelli's avatar Florian Fainelli Committed by David S. Miller

net: phy: bcm7xxx: drop A0 revision workaround and fix B0 workaround

bcm7445_config_init() was working around non-production version of the
PHY HW block, so just remove it entirely.

bcm7xxx_28nm_afe_config_init() was running for all PHY revisions greater
than B0, but this workaround sequence is really specific to the B0 PHY
revision, so rename the function accordingly and update the GPHY macro
to use the generic config_init callback.
Signed-off-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6ec259c1
...@@ -45,39 +45,6 @@ ...@@ -45,39 +45,6 @@
#define CORE_EXPB0 0xb0 #define CORE_EXPB0 0xb0
static int bcm7445_config_init(struct phy_device *phydev)
{
int ret;
const struct bcm7445_regs {
int reg;
u16 value;
} bcm7445_regs_cfg[] = {
/* increases ADC latency by 24ns */
{ MII_BCM54XX_EXP_SEL, 0x0038 },
{ MII_BCM54XX_EXP_DATA, 0xAB95 },
/* increases internal 1V LDO voltage by 5% */
{ MII_BCM54XX_EXP_SEL, 0x2038 },
{ MII_BCM54XX_EXP_DATA, 0xBB22 },
/* reduce RX low pass filter corner frequency */
{ MII_BCM54XX_EXP_SEL, 0x6038 },
{ MII_BCM54XX_EXP_DATA, 0xFFC5 },
/* reduce RX high pass filter corner frequency */
{ MII_BCM54XX_EXP_SEL, 0x003a },
{ MII_BCM54XX_EXP_DATA, 0x2002 },
};
unsigned int i;
for (i = 0; i < ARRAY_SIZE(bcm7445_regs_cfg); i++) {
ret = phy_write(phydev,
bcm7445_regs_cfg[i].reg,
bcm7445_regs_cfg[i].value);
if (ret)
return ret;
}
return 0;
}
static void phy_write_exp(struct phy_device *phydev, static void phy_write_exp(struct phy_device *phydev,
u16 reg, u16 value) u16 reg, u16 value)
{ {
...@@ -102,7 +69,7 @@ static void phy_write_misc(struct phy_device *phydev, ...@@ -102,7 +69,7 @@ static void phy_write_misc(struct phy_device *phydev,
phy_write(phydev, MII_BCM54XX_EXP_DATA, value); phy_write(phydev, MII_BCM54XX_EXP_DATA, value);
} }
static int bcm7xxx_28nm_afe_config_init(struct phy_device *phydev) static int bcm7xxx_28nm_b0_afe_config_init(struct phy_device *phydev)
{ {
/* Increase VCO range to prevent unlocking problem of PLL at low /* Increase VCO range to prevent unlocking problem of PLL at low
* temp * temp
...@@ -204,12 +171,10 @@ static int bcm7xxx_28nm_config_init(struct phy_device *phydev) ...@@ -204,12 +171,10 @@ static int bcm7xxx_28nm_config_init(struct phy_device *phydev)
dev_name(&phydev->dev), phydev->drv->name, rev, patch); dev_name(&phydev->dev), phydev->drv->name, rev, patch);
switch (rev) { switch (rev) {
case 0xa0:
case 0xb0: case 0xb0:
ret = bcm7445_config_init(phydev); ret = bcm7xxx_28nm_b0_afe_config_init(phydev);
break; break;
default: default:
ret = bcm7xxx_28nm_afe_config_init(phydev);
break; break;
} }
...@@ -337,7 +302,7 @@ static int bcm7xxx_dummy_config_init(struct phy_device *phydev) ...@@ -337,7 +302,7 @@ static int bcm7xxx_dummy_config_init(struct phy_device *phydev)
.features = PHY_GBIT_FEATURES | \ .features = PHY_GBIT_FEATURES | \
SUPPORTED_Pause | SUPPORTED_Asym_Pause, \ SUPPORTED_Pause | SUPPORTED_Asym_Pause, \
.flags = PHY_IS_INTERNAL, \ .flags = PHY_IS_INTERNAL, \
.config_init = bcm7xxx_28nm_afe_config_init, \ .config_init = bcm7xxx_28nm_config_init, \
.config_aneg = genphy_config_aneg, \ .config_aneg = genphy_config_aneg, \
.read_status = genphy_read_status, \ .read_status = genphy_read_status, \
.resume = bcm7xxx_28nm_resume, \ .resume = bcm7xxx_28nm_resume, \
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment