> -----Original Message----- > From: Sebastian Hesselbarth [mailto:sebastian.hesselba...@gmail.com] > Sent: 17 January 2013 00:55 > To: Sebastian Hesselbarth > Cc: u-boot@lists.denx.de; Rabeeh Khoury; Albert Aribaud; Prafulla > Wadaskar; Andy Fleming; Joe Hershberger; Daniel Stodden; Luka Perkov > Subject: [PATCH v3 07/10] NET: mvgbe: add phylib support > > This add phylib support to the Marvell GBE driver. > > Signed-off-by: Sebastian Hesselbarth <sebastian.hesselba...@gmail.com> > --- > Cc: u-boot@lists.denx.de > Cc: Sebastian Hesselbarth <sebastian.hesselba...@gmail.com> > Cc: Rabeeh Khoury <rab...@solid-run.com> > Cc: Albert Aribaud <albert.u.b...@aribaud.net> > Cc: Prafulla Wadaskar <prafu...@marvell.com> > Cc: Andy Fleming <aflem...@gmail.com> > Cc: Joe Hershberger <joe.hershber...@gmail.com> > Cc: Daniel Stodden <daniel.stod...@gmail.com> > Cc: Luka Perkov <l...@openwrt.org> > --- > drivers/net/mvgbe.c | 68 > ++++++++++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 64 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/mvgbe.c b/drivers/net/mvgbe.c > index 47bf27c..192c989 100644 > --- a/drivers/net/mvgbe.c > +++ b/drivers/net/mvgbe.c > @@ -52,7 +52,7 @@ DECLARE_GLOBAL_DATA_PTR; > #define MV_PHY_ADR_REQUEST 0xee > #define MVGBE_SMI_REG (((struct mvgbe_registers *)MVGBE0_BASE)->smi) > > -#if defined(CONFIG_MII) || defined(CONFIG_CMD_MII) > +#if defined(CONFIG_PHYLIB) || defined(CONFIG_MII) || > defined(CONFIG_CMD_MII) > /* > * smi_reg_read - miiphy_read callback function. > * > @@ -184,6 +184,24 @@ static int smi_reg_write(const char *devname, u8 > phy_adr, u8 reg_ofs, u16 data) > } > #endif > > +#if defined(CONFIG_PHYLIB) > +int mvgbe_phy_read(struct mii_dev *bus, int phyAddr, int devAddr, int > regAddr) > +{ > + u16 data; > + int ret; > + ret = smi_reg_read(bus->name, phyAddr, regAddr, &data); > + if (ret) > + return ret; > + return data; > +} > + > +int mvgbe_phy_write(struct mii_dev *bus, int phyAddr, int devAddr, > int regAddr, > + u16 data) > +{ > + return smi_reg_write(bus->name, phyAddr, regAddr, data); > +} > +#endif > + > /* Stop and checks all queues */ > static void stop_queue(u32 * qreg) > { > @@ -467,8 +485,9 @@ static int mvgbe_init(struct eth_device *dev) > /* Enable port Rx. */ > MVGBE_REG_WR(regs->rqc, (1 << RXUQ)); > > -#if (defined (CONFIG_MII) || defined (CONFIG_CMD_MII)) \ > - && defined (CONFIG_SYS_FAULT_ECHO_LINK_DOWN) > +#if (defined(CONFIG_MII) || defined(CONFIG_CMD_MII)) \ > + && !defined(CONFIG_PHYLIB) \ > + && defined(CONFIG_SYS_FAULT_ECHO_LINK_DOWN) > /* Wait up to 5s for the link status */ > for (i = 0; i < 5; i++) { > u16 phyadr; > @@ -647,6 +666,45 @@ static int mvgbe_recv(struct eth_device *dev) > return 0; > } > > +#if defined(CONFIG_PHYLIB) > +int mvgbe_phylib_init(struct eth_device *dev, int phyid) > +{ > + struct mii_dev *bus; > + struct phy_device *phydev; > + int ret; > + > + bus = mdio_alloc(); > + if (!bus) { > + printf("mdio_alloc failed\n"); > + return -ENOMEM; > + } > + bus->read = mvgbe_phy_read; > + bus->write = mvgbe_phy_write; > + sprintf(bus->name, dev->name); > + > + ret = mdio_register(bus); > + if (ret) { > + printf("mdio_register failed\n"); > + free(bus); > + return -ENOMEM; > + } > + > + /* Set phy address of the port */ > + mvgbe_phy_write(bus, MV_PHY_ADR_REQUEST, 0, MV_PHY_ADR_REQUEST, > phyid); > + > + phydev = phy_connect(bus, phyid, dev, PHY_INTERFACE_MODE_RGMII); > + if (!phydev) { > + printf("phy_connect failed\n"); > + return -ENODEV; > + } > + > + phy_config(phydev); > + phy_startup(phydev); > + > + return 0; > +} > +#endif > + > int mvgbe_initialize(bd_t *bis) > { > struct mvgbe_device *dmvgbe; > @@ -729,7 +787,9 @@ error1: > > eth_register(dev); > > -#if defined(CONFIG_MII) || defined(CONFIG_CMD_MII) > +#if defined(CONFIG_PHYLIB) > + mvgbe_phylib_init(dev, PHY_BASE_ADR + devnum); > +#elif defined(CONFIG_MII) || defined(CONFIG_CMD_MII) > miiphy_register(dev->name, smi_reg_read, smi_reg_write); > /* Set phy address of the port */ > miiphy_write(dev->name, MV_PHY_ADR_REQUEST, > --
Acked-By: Prafulla Wadaskar <prafu...@marvell.com> This patch do not have any dependency. I suggest to detach this patch and send it as standalone patch so that Joe can pull it earliest. Regards... Prafulla . . . _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot