RE: [RFC PATCH 08/17] drivers/net/bfin_mac: Don't unnecessarilyreset the PHY

From: Zhang, Sonic
Date: Tue Oct 25 2011 - 03:26:46 EST


Acked-by: Sonic Zhang <sonic.zhang@xxxxxxxxxx>


Sonic

>-----Original Message-----
>From: Kyle Moffett [mailto:Kyle.D.Moffett@xxxxxxxxxx]
>Sent: Friday, October 21, 2011 5:00 AM
>To: linux-kernel@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx
>Cc: Kyle Moffett; David S. Miller; Mike Frysinger; Zhang,
>Sonic; Tobias Klauser; uclinux-dist-devel@xxxxxxxxxxxxxxxxxxxx
>Subject: [RFC PATCH 08/17] drivers/net/bfin_mac: Don't
>unnecessarily reset the PHY
>
>The PHY is already reset during driver probing, and this
>manual reset afterwards will wipe out board-specific PHY
>fixups and driver-specific
>phy->drv->config_init() register tweaks.
>
>Signed-off-by: Kyle Moffett <Kyle.D.Moffett@xxxxxxxxxx>
>---
> drivers/net/bfin_mac.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
>diff --git a/drivers/net/bfin_mac.c b/drivers/net/bfin_mac.c
>index 6c019e1..1427ec3 100644
>--- a/drivers/net/bfin_mac.c
>+++ b/drivers/net/bfin_mac.c
>@@ -1402,7 +1402,6 @@ static int bfin_mac_open(struct net_device *dev)
> return ret;
>
> phy_start(lp->phydev);
>- phy_write(lp->phydev, MII_BMCR, BMCR_RESET);
> setup_system_regs(dev);
> setup_mac_addr(dev->dev_addr);
>
>--
>1.7.2.5
>
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/