macb: Keep driver's speed/duplex in sync with actual NCFGR
authorVitalii Demianets <vitas@nppfactor.kiev.ua>
Fri, 2 Nov 2012 07:09:24 +0000 (07:09 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 3 Nov 2012 19:37:35 +0000 (15:37 -0400)
When underlying phy driver restores its state very fast after being brought
down and up so that macb driver function macb_handle_link_change() was never
called with link state "down", driver's internal representation of phy speed
and duplex (bp->speed and bp->duplex) didn't change. So, macb driver sees no
reason to perform actual write to the NCFGR register, although the speed and
duplex settings in that register were reset when interface was brought down
and up. In that case actual phy speed and duplex differ from NCFGR settings.
The patch fixes that by keeping internal driver representation of speed and
duplex in sync with actual content of NCFGR.

Signed-off-by: Vitalii Demianets <vitas@nppfactor.kiev.ua>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cadence/macb.c

index c3748753512578577d155dd866ca2f6cc08e822f..13c3c332f58d1e10425ffc849e8b4efebf04e2ed 100644 (file)
@@ -1031,6 +1031,8 @@ static void macb_init_hw(struct macb *bp)
                config |= MACB_BIT(NBC);        /* No BroadCast */
        config |= macb_dbw(bp);
        macb_writel(bp, NCFGR, config);
+       bp->speed = SPEED_10;
+       bp->duplex = DUPLEX_HALF;
 
        macb_configure_dma(bp);