diff options
author | Alexander Duyck <alexander.h.duyck@intel.com> | 2009-11-12 13:36:41 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-11-13 23:46:48 -0500 |
commit | 0e15439ae5fefe438056a26a00aa3c6a9de454e9 (patch) | |
tree | e16c8a9fda2ffb48a9f6c49e9f1f36876617e385 /drivers/net/igb/igb_ethtool.c | |
parent | ddb1417529559810ec2024fd8ca21e4d497a3275 (diff) |
igb: change type for ring sizes to u16 in igb_set_ring_param
Change the type for the ring size values to u16 and use min/max_t instead of
min/max.
Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/igb/igb_ethtool.c')
-rw-r--r-- | drivers/net/igb/igb_ethtool.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/net/igb/igb_ethtool.c b/drivers/net/igb/igb_ethtool.c index 90b89a81f669..f2b28254f2a0 100644 --- a/drivers/net/igb/igb_ethtool.c +++ b/drivers/net/igb/igb_ethtool.c | |||
@@ -735,17 +735,17 @@ static int igb_set_ringparam(struct net_device *netdev, | |||
735 | struct igb_adapter *adapter = netdev_priv(netdev); | 735 | struct igb_adapter *adapter = netdev_priv(netdev); |
736 | struct igb_ring *temp_ring; | 736 | struct igb_ring *temp_ring; |
737 | int i, err = 0; | 737 | int i, err = 0; |
738 | u32 new_rx_count, new_tx_count; | 738 | u16 new_rx_count, new_tx_count; |
739 | 739 | ||
740 | if ((ring->rx_mini_pending) || (ring->rx_jumbo_pending)) | 740 | if ((ring->rx_mini_pending) || (ring->rx_jumbo_pending)) |
741 | return -EINVAL; | 741 | return -EINVAL; |
742 | 742 | ||
743 | new_rx_count = min(ring->rx_pending, (u32)IGB_MAX_RXD); | 743 | new_rx_count = min_t(u32, ring->rx_pending, IGB_MAX_RXD); |
744 | new_rx_count = max(new_rx_count, (u32)IGB_MIN_RXD); | 744 | new_rx_count = max_t(u16, new_rx_count, IGB_MIN_RXD); |
745 | new_rx_count = ALIGN(new_rx_count, REQ_RX_DESCRIPTOR_MULTIPLE); | 745 | new_rx_count = ALIGN(new_rx_count, REQ_RX_DESCRIPTOR_MULTIPLE); |
746 | 746 | ||
747 | new_tx_count = min(ring->tx_pending, (u32)IGB_MAX_TXD); | 747 | new_tx_count = min_t(u32, ring->tx_pending, IGB_MAX_TXD); |
748 | new_tx_count = max(new_tx_count, (u32)IGB_MIN_TXD); | 748 | new_tx_count = max_t(u16, new_tx_count, IGB_MIN_TXD); |
749 | new_tx_count = ALIGN(new_tx_count, REQ_TX_DESCRIPTOR_MULTIPLE); | 749 | new_tx_count = ALIGN(new_tx_count, REQ_TX_DESCRIPTOR_MULTIPLE); |
750 | 750 | ||
751 | if ((new_tx_count == adapter->tx_ring_count) && | 751 | if ((new_tx_count == adapter->tx_ring_count) && |