From: Daniel Hellstrom Date: Fri, 14 Jan 2011 03:02:41 +0000 (+0000) Subject: GRETH: avoid writing bad speed/duplex when setting transfer mode X-Git-Tag: v3.0~1794^2~168 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2436af8ca7a6c4679cf7da7e3867f1d5cd8528b7;p=platform%2Fkernel%2Flinux-amlogic.git GRETH: avoid writing bad speed/duplex when setting transfer mode Signed-off-by: Daniel Hellstrom Signed-off-by: David S. Miller --- diff --git a/drivers/net/greth.c b/drivers/net/greth.c index 9386bce..e48d182 100644 --- a/drivers/net/greth.c +++ b/drivers/net/greth.c @@ -1232,29 +1232,26 @@ static void greth_link_change(struct net_device *dev) struct greth_private *greth = netdev_priv(dev); struct phy_device *phydev = greth->phy; unsigned long flags; - int status_change = 0; + u32 ctrl; spin_lock_irqsave(&greth->devlock, flags); if (phydev->link) { if ((greth->speed != phydev->speed) || (greth->duplex != phydev->duplex)) { - - GRETH_REGANDIN(greth->regs->control, - ~(GRETH_CTRL_FD | GRETH_CTRL_SP | GRETH_CTRL_GB)); + ctrl = GRETH_REGLOAD(greth->regs->control) & + ~(GRETH_CTRL_FD | GRETH_CTRL_SP | GRETH_CTRL_GB); if (phydev->duplex) - GRETH_REGORIN(greth->regs->control, GRETH_CTRL_FD); - - if (phydev->speed == SPEED_100) { - - GRETH_REGORIN(greth->regs->control, GRETH_CTRL_SP); - } + ctrl |= GRETH_CTRL_FD; + if (phydev->speed == SPEED_100) + ctrl |= GRETH_CTRL_SP; else if (phydev->speed == SPEED_1000) - GRETH_REGORIN(greth->regs->control, GRETH_CTRL_GB); + ctrl |= GRETH_CTRL_GB; + GRETH_REGSAVE(greth->regs->control, ctrl); greth->speed = phydev->speed; greth->duplex = phydev->duplex; status_change = 1;