igb: change type for ring sizes to u16 in igb_set_ring_param
authorAlexander Duyck <alexander.h.duyck@intel.com>
Thu, 12 Nov 2009 18:36:41 +0000 (18:36 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 14 Nov 2009 04:46:48 +0000 (20:46 -0800)
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>
drivers/net/igb/igb_ethtool.c

index 90b89a81f6694c3cefef685b62578917160ccab0..f2b28254f2a05178690b8e4c367a6e684e4f07fb 100644 (file)
@@ -735,17 +735,17 @@ static int igb_set_ringparam(struct net_device *netdev,
        struct igb_adapter *adapter = netdev_priv(netdev);
        struct igb_ring *temp_ring;
        int i, err = 0;
-       u32 new_rx_count, new_tx_count;
+       u16 new_rx_count, new_tx_count;
 
        if ((ring->rx_mini_pending) || (ring->rx_jumbo_pending))
                return -EINVAL;
 
-       new_rx_count = min(ring->rx_pending, (u32)IGB_MAX_RXD);
-       new_rx_count = max(new_rx_count, (u32)IGB_MIN_RXD);
+       new_rx_count = min_t(u32, ring->rx_pending, IGB_MAX_RXD);
+       new_rx_count = max_t(u16, new_rx_count, IGB_MIN_RXD);
        new_rx_count = ALIGN(new_rx_count, REQ_RX_DESCRIPTOR_MULTIPLE);
 
-       new_tx_count = min(ring->tx_pending, (u32)IGB_MAX_TXD);
-       new_tx_count = max(new_tx_count, (u32)IGB_MIN_TXD);
+       new_tx_count = min_t(u32, ring->tx_pending, IGB_MAX_TXD);
+       new_tx_count = max_t(u16, new_tx_count, IGB_MIN_TXD);
        new_tx_count = ALIGN(new_tx_count, REQ_TX_DESCRIPTOR_MULTIPLE);
 
        if ((new_tx_count == adapter->tx_ring_count) &&