[PATCH] driver/net/skge.c: Restart the interface when it's options or pauseparam is set

[Xiaoming.Zhang]

Hi,

We have an issue of the skge driver: The card won't work when it's options are
changed. That's the hardware info:
# lspci -v
05:04.0 Ethernet controller: Marvell Technology Group Ltd. 88E8001 Gigabit Ethernet Controller (rev 13)
        Subsystem: Marvell Technology Group Ltd. Marvell RDK-8001
        Flags: bus master, 66MHz, medium devsel, latency 32, IRQ 16
        Memory at d042c000 (32-bit, non-prefetchable) [size=16K]
        I/O ports at d000 [size=256]
        [virtual] Expansion ROM at 20400000 [disabled] [size=128K]
        Capabilities: [48] Power Management version 2
        Capabilities: [50] Vital Product Data

The happens in both Linux-2.6.26(skge version 1.23) and RHEL5.2(skge version 1.6).


For example, at first it is set to "speed 1000 duplex full auto-neg on" and
it works, then run 
       ethtool -s <ethx> autoneg off
or     ethtool -s <ethx> speed 100 duplex full autoneg off

Then it will stop working. After that if we restart the interface:
	ifconifg <ethx> down
	ifconfig <ethx> up
It will work again. And `ethtool -A' has the same issue.

So we think after setting the options, the interface should be restarted.
That's the patch, thank you.


skge: Restart the interface when it's options or pauseparam is set.

Signed-off-by: Zhang Xiaoming <xiaoming.zhang@xxxxxxxxxxxxxx>
---
diff -uprN linux-2.6.26/drivers/net/skge.c.orig linux-2.6.26/drivers/net/skge.c
--- linux-2.6.26/drivers/net/skge.c.orig        2008-08-07 01:26:34.000000000 +0800
+++ linux-2.6.26/drivers/net/skge.c     2008-08-07 01:29:31.000000000 +0800
@@ -367,8 +367,10 @@ static int skge_set_settings(struct net_
        skge->autoneg = ecmd->autoneg;
        skge->advertising = ecmd->advertising;

-       if (netif_running(dev))
-               skge_phy_reset(skge);
+       if (netif_running(dev)) {
+               skge_down(dev);
+               skge_up(dev);
+       }

        return (0);
 }
@@ -609,8 +611,10 @@ static int skge_set_pauseparam(struct ne
                        skge->flow_control = FLOW_MODE_NONE;
        }

-       if (netif_running(dev))
-               skge_phy_reset(skge);
+       if (netif_running(dev)) {
+               skge_down(dev);
+               skge_up(dev);
+       }

        return 0;
 }



-- 
Have fun,
Xiaoming (Mark) Zhang

--
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/