diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2013-01-16 03:46:49 -0500 |
---|---|---|
committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2013-02-01 01:28:39 -0500 |
commit | 8bb628697fb05172f10a0960385b8369be15cb6c (patch) | |
tree | 641f3b8c29c7d6b283c831071a571dc5b1da5a37 /drivers/net/ethernet/intel/e1000e/netdev.c | |
parent | 9e019901816ca4bad83bf4922b791610e6dd529e (diff) |
e1000e: resolve -Wunused-parameter compile warnings
Remove the unused parameter when possible, otherwise use __always_unused
attribute.
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/e1000e/netdev.c')
-rw-r--r-- | drivers/net/ethernet/intel/e1000e/netdev.c | 33 |
1 files changed, 14 insertions, 19 deletions
diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ethernet/intel/e1000e/netdev.c index 3bb7b014dd0e..eb01a55f60a6 100644 --- a/drivers/net/ethernet/intel/e1000e/netdev.c +++ b/drivers/net/ethernet/intel/e1000e/netdev.c | |||
@@ -1747,7 +1747,7 @@ static void e1000e_downshift_workaround(struct work_struct *work) | |||
1747 | * @irq: interrupt number | 1747 | * @irq: interrupt number |
1748 | * @data: pointer to a network interface device structure | 1748 | * @data: pointer to a network interface device structure |
1749 | **/ | 1749 | **/ |
1750 | static irqreturn_t e1000_intr_msi(int irq, void *data) | 1750 | static irqreturn_t e1000_intr_msi(int __always_unused irq, void *data) |
1751 | { | 1751 | { |
1752 | struct net_device *netdev = data; | 1752 | struct net_device *netdev = data; |
1753 | struct e1000_adapter *adapter = netdev_priv(netdev); | 1753 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -1813,7 +1813,7 @@ static irqreturn_t e1000_intr_msi(int irq, void *data) | |||
1813 | * @irq: interrupt number | 1813 | * @irq: interrupt number |
1814 | * @data: pointer to a network interface device structure | 1814 | * @data: pointer to a network interface device structure |
1815 | **/ | 1815 | **/ |
1816 | static irqreturn_t e1000_intr(int irq, void *data) | 1816 | static irqreturn_t e1000_intr(int __always_unused irq, void *data) |
1817 | { | 1817 | { |
1818 | struct net_device *netdev = data; | 1818 | struct net_device *netdev = data; |
1819 | struct e1000_adapter *adapter = netdev_priv(netdev); | 1819 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -1888,7 +1888,7 @@ static irqreturn_t e1000_intr(int irq, void *data) | |||
1888 | return IRQ_HANDLED; | 1888 | return IRQ_HANDLED; |
1889 | } | 1889 | } |
1890 | 1890 | ||
1891 | static irqreturn_t e1000_msix_other(int irq, void *data) | 1891 | static irqreturn_t e1000_msix_other(int __always_unused irq, void *data) |
1892 | { | 1892 | { |
1893 | struct net_device *netdev = data; | 1893 | struct net_device *netdev = data; |
1894 | struct e1000_adapter *adapter = netdev_priv(netdev); | 1894 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -1920,8 +1920,7 @@ no_link_interrupt: | |||
1920 | return IRQ_HANDLED; | 1920 | return IRQ_HANDLED; |
1921 | } | 1921 | } |
1922 | 1922 | ||
1923 | 1923 | static irqreturn_t e1000_intr_msix_tx(int __always_unused irq, void *data) | |
1924 | static irqreturn_t e1000_intr_msix_tx(int irq, void *data) | ||
1925 | { | 1924 | { |
1926 | struct net_device *netdev = data; | 1925 | struct net_device *netdev = data; |
1927 | struct e1000_adapter *adapter = netdev_priv(netdev); | 1926 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -1939,7 +1938,7 @@ static irqreturn_t e1000_intr_msix_tx(int irq, void *data) | |||
1939 | return IRQ_HANDLED; | 1938 | return IRQ_HANDLED; |
1940 | } | 1939 | } |
1941 | 1940 | ||
1942 | static irqreturn_t e1000_intr_msix_rx(int irq, void *data) | 1941 | static irqreturn_t e1000_intr_msix_rx(int __always_unused irq, void *data) |
1943 | { | 1942 | { |
1944 | struct net_device *netdev = data; | 1943 | struct net_device *netdev = data; |
1945 | struct e1000_adapter *adapter = netdev_priv(netdev); | 1944 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -2496,9 +2495,7 @@ void e1000e_free_rx_resources(struct e1000_ring *rx_ring) | |||
2496 | * while increasing bulk throughput. This functionality is controlled | 2495 | * while increasing bulk throughput. This functionality is controlled |
2497 | * by the InterruptThrottleRate module parameter. | 2496 | * by the InterruptThrottleRate module parameter. |
2498 | **/ | 2497 | **/ |
2499 | static unsigned int e1000_update_itr(struct e1000_adapter *adapter, | 2498 | static unsigned int e1000_update_itr(u16 itr_setting, int packets, int bytes) |
2500 | u16 itr_setting, int packets, | ||
2501 | int bytes) | ||
2502 | { | 2499 | { |
2503 | unsigned int retval = itr_setting; | 2500 | unsigned int retval = itr_setting; |
2504 | 2501 | ||
@@ -2558,18 +2555,16 @@ static void e1000_set_itr(struct e1000_adapter *adapter) | |||
2558 | goto set_itr_now; | 2555 | goto set_itr_now; |
2559 | } | 2556 | } |
2560 | 2557 | ||
2561 | adapter->tx_itr = e1000_update_itr(adapter, | 2558 | adapter->tx_itr = e1000_update_itr(adapter->tx_itr, |
2562 | adapter->tx_itr, | 2559 | adapter->total_tx_packets, |
2563 | adapter->total_tx_packets, | 2560 | adapter->total_tx_bytes); |
2564 | adapter->total_tx_bytes); | ||
2565 | /* conservative mode (itr 3) eliminates the lowest_latency setting */ | 2561 | /* conservative mode (itr 3) eliminates the lowest_latency setting */ |
2566 | if (adapter->itr_setting == 3 && adapter->tx_itr == lowest_latency) | 2562 | if (adapter->itr_setting == 3 && adapter->tx_itr == lowest_latency) |
2567 | adapter->tx_itr = low_latency; | 2563 | adapter->tx_itr = low_latency; |
2568 | 2564 | ||
2569 | adapter->rx_itr = e1000_update_itr(adapter, | 2565 | adapter->rx_itr = e1000_update_itr(adapter->rx_itr, |
2570 | adapter->rx_itr, | 2566 | adapter->total_rx_packets, |
2571 | adapter->total_rx_packets, | 2567 | adapter->total_rx_bytes); |
2572 | adapter->total_rx_bytes); | ||
2573 | /* conservative mode (itr 3) eliminates the lowest_latency setting */ | 2568 | /* conservative mode (itr 3) eliminates the lowest_latency setting */ |
2574 | if (adapter->itr_setting == 3 && adapter->rx_itr == lowest_latency) | 2569 | if (adapter->itr_setting == 3 && adapter->rx_itr == lowest_latency) |
2575 | adapter->rx_itr = low_latency; | 2570 | adapter->rx_itr = low_latency; |
@@ -4115,7 +4110,7 @@ static int e1000_sw_init(struct e1000_adapter *adapter) | |||
4115 | * @irq: interrupt number | 4110 | * @irq: interrupt number |
4116 | * @data: pointer to a network interface device structure | 4111 | * @data: pointer to a network interface device structure |
4117 | **/ | 4112 | **/ |
4118 | static irqreturn_t e1000_intr_msi_test(int irq, void *data) | 4113 | static irqreturn_t e1000_intr_msi_test(int __always_unused irq, void *data) |
4119 | { | 4114 | { |
4120 | struct net_device *netdev = data; | 4115 | struct net_device *netdev = data; |
4121 | struct e1000_adapter *adapter = netdev_priv(netdev); | 4116 | struct e1000_adapter *adapter = netdev_priv(netdev); |
@@ -6249,7 +6244,7 @@ static void e1000_shutdown(struct pci_dev *pdev) | |||
6249 | 6244 | ||
6250 | #ifdef CONFIG_NET_POLL_CONTROLLER | 6245 | #ifdef CONFIG_NET_POLL_CONTROLLER |
6251 | 6246 | ||
6252 | static irqreturn_t e1000_intr_msix(int irq, void *data) | 6247 | static irqreturn_t e1000_intr_msix(int __always_unused irq, void *data) |
6253 | { | 6248 | { |
6254 | struct net_device *netdev = data; | 6249 | struct net_device *netdev = data; |
6255 | struct e1000_adapter *adapter = netdev_priv(netdev); | 6250 | struct e1000_adapter *adapter = netdev_priv(netdev); |