skge: Fix skge_set_ring_param() losing error return
authorWang Chen <wangchen@cn.fujitsu.com>
Fri, 26 Sep 2008 08:20:32 +0000 (16:20 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 8 Oct 2008 23:02:40 +0000 (16:02 -0700)
The error return is useful to caller, driver shouldn't miss it.

Signed-off-by: Wang Chen <wangchen@cn.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/skge.c

index 3bca52c..43f4c73 100644 (file)
@@ -501,7 +501,7 @@ static int skge_set_ring_param(struct net_device *dev,
                               struct ethtool_ringparam *p)
 {
        struct skge_port *skge = netdev_priv(dev);
-       int err;
+       int err = 0;
 
        if (p->rx_pending == 0 || p->rx_pending > MAX_RX_RING_SIZE ||
            p->tx_pending < TX_LOW_WATER || p->tx_pending > MAX_TX_RING_SIZE)
@@ -517,7 +517,7 @@ static int skge_set_ring_param(struct net_device *dev,
                        dev_close(dev);
        }
 
-       return 0;
+       return err;
 }
 
 static u32 skge_get_msglevel(struct net_device *netdev)