[PATCH] phylib: m88e1111: enable SGMII mode
From: Kim Phillips <hidden>
Date: 2007-05-03 20:13:07
Subsystem:
ethernet phy library, networking drivers, the rest · Maintainers:
Andrew Lunn, Heiner Kallweit, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds
From: Kapil Juneja <redacted> If connected via SGMII, initialize with SGMII mode configured. Signed-off-by: Kapil Juneja <redacted> Signed-off-by: Andy Fleming <redacted> Signed-off-by: Kim Phillips <redacted> --- please consider for 2.6.22 This patch depends on: [PATCH v2] phylib: enable RGMII-ID on the Marvell m88e1111 PHY - http://marc.info/?l=linux-netdev&m=117648831716153&w=2 - submitted 13 apr 2007 I am not sure if initializing for SGMII without clock should be done unconditionally..the spec doesn't mention under what conditions the clock data are able to be recovered from the data stream. I'm assuming always. drivers/net/phy/marvell.c | 16 ++++++++++++++++ 1 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index b87f8d2..d4cc952 100644
--- a/drivers/net/phy/marvell.c
+++ b/drivers/net/phy/marvell.c@@ -60,6 +60,7 @@ #define MII_M1111_PHY_EXT_SR 0x1b #define MII_M1111_HWCFG_MODE_MASK 0xf #define MII_M1111_HWCFG_MODE_RGMII 0xb +#define MII_M1111_HWCFG_MODE_SGMII_NO_CLK 0x4 MODULE_DESCRIPTION("Marvell PHY driver"); MODULE_AUTHOR("Andy Fleming");
@@ -169,6 +170,21 @@ static int m88e1111_config_init(struct phy_device *phydev) return err; } + if (phydev->interface == PHY_INTERFACE_MODE_SGMII) + int temp; + + temp = phy_read(phydev, MII_M1111_PHY_EXT_SR); + if (temp < 0) + return temp; + + temp &= ~(MII_M1111_HWCFG_MODE_MASK); + temp |= MII_M1111_HWCFG_MODE_SGMII_NO_CLK; + + err = phy_write(phydev, MII_M1111_PHY_EXT_SR, temp); + if (err < 0) + return err; + } + err = phy_write(phydev, MII_BMCR, BMCR_RESET); if (err < 0) return err;
--
1.5.0.3