Commit e13647c1 authored by Richard Cochran's avatar Richard Cochran Committed by David S. Miller

phylib: Add support for the LXT973 phy.

This patch implements a work around for Erratum 5, "3.3 V Fiber Speed
Selection." If the hardware wiring does not respect this erratum, then
fiber optic mode will not work properly.
Signed-off-by: default avatarRichard Cochran <richard.cochran@omicron.at>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 327723ed
...@@ -53,6 +53,9 @@ ...@@ -53,6 +53,9 @@
#define MII_LXT971_ISR 19 /* Interrupt Status Register */ #define MII_LXT971_ISR 19 /* Interrupt Status Register */
/* register definitions for the 973 */
#define MII_LXT973_PCR 16 /* Port Configuration Register */
#define PCR_FIBER_SELECT 1
MODULE_DESCRIPTION("Intel LXT PHY driver"); MODULE_DESCRIPTION("Intel LXT PHY driver");
MODULE_AUTHOR("Andy Fleming"); MODULE_AUTHOR("Andy Fleming");
...@@ -119,6 +122,33 @@ static int lxt971_config_intr(struct phy_device *phydev) ...@@ -119,6 +122,33 @@ static int lxt971_config_intr(struct phy_device *phydev)
return err; return err;
} }
static int lxt973_probe(struct phy_device *phydev)
{
int val = phy_read(phydev, MII_LXT973_PCR);
if (val & PCR_FIBER_SELECT) {
/*
* If fiber is selected, then the only correct setting
* is 100Mbps, full duplex, and auto negotiation off.
*/
val = phy_read(phydev, MII_BMCR);
val |= (BMCR_SPEED100 | BMCR_FULLDPLX);
val &= ~BMCR_ANENABLE;
phy_write(phydev, MII_BMCR, val);
/* Remember that the port is in fiber mode. */
phydev->priv = lxt973_probe;
} else {
phydev->priv = NULL;
}
return 0;
}
static int lxt973_config_aneg(struct phy_device *phydev)
{
/* Do nothing if port is in fiber mode. */
return phydev->priv ? 0 : genphy_config_aneg(phydev);
}
static struct phy_driver lxt970_driver = { static struct phy_driver lxt970_driver = {
.phy_id = 0x78100000, .phy_id = 0x78100000,
.name = "LXT970", .name = "LXT970",
...@@ -146,6 +176,18 @@ static struct phy_driver lxt971_driver = { ...@@ -146,6 +176,18 @@ static struct phy_driver lxt971_driver = {
.driver = { .owner = THIS_MODULE,}, .driver = { .owner = THIS_MODULE,},
}; };
static struct phy_driver lxt973_driver = {
.phy_id = 0x00137a10,
.name = "LXT973",
.phy_id_mask = 0xfffffff0,
.features = PHY_BASIC_FEATURES,
.flags = 0,
.probe = lxt973_probe,
.config_aneg = lxt973_config_aneg,
.read_status = genphy_read_status,
.driver = { .owner = THIS_MODULE,},
};
static int __init lxt_init(void) static int __init lxt_init(void)
{ {
int ret; int ret;
...@@ -157,9 +199,15 @@ static int __init lxt_init(void) ...@@ -157,9 +199,15 @@ static int __init lxt_init(void)
ret = phy_driver_register(&lxt971_driver); ret = phy_driver_register(&lxt971_driver);
if (ret) if (ret)
goto err2; goto err2;
ret = phy_driver_register(&lxt973_driver);
if (ret)
goto err3;
return 0; return 0;
err2: err3:
phy_driver_unregister(&lxt971_driver);
err2:
phy_driver_unregister(&lxt970_driver); phy_driver_unregister(&lxt970_driver);
err1: err1:
return ret; return ret;
...@@ -169,6 +217,7 @@ static void __exit lxt_exit(void) ...@@ -169,6 +217,7 @@ static void __exit lxt_exit(void)
{ {
phy_driver_unregister(&lxt970_driver); phy_driver_unregister(&lxt970_driver);
phy_driver_unregister(&lxt971_driver); phy_driver_unregister(&lxt971_driver);
phy_driver_unregister(&lxt973_driver);
} }
module_init(lxt_init); module_init(lxt_init);
......
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