diff options
| author | Wang Chen <wangchen@cn.fujitsu.com> | 2008-09-26 10:20:32 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-10-09 01:02:40 +0200 |
| commit | e824b3eb4acad16c64c9ddf8e63b63e2469259b0 (patch) | |
| tree | e350547cda629039cdaa3d3995541985070eda64 | |
| parent | [10/21] driver/net/skge.c: restart the interface when it's options or (diff) | |
| download | linux-e824b3eb4acad16c64c9ddf8e63b63e2469259b0.tar.xz linux-e824b3eb4acad16c64c9ddf8e63b63e2469259b0.zip | |
skge: Fix skge_set_ring_param() losing error return
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>
| -rw-r--r-- | drivers/net/skge.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/skge.c b/drivers/net/skge.c index 3bca52c142fe..43f4c730be42 100644 --- a/drivers/net/skge.c +++ b/drivers/net/skge.c @@ -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) |