diff options
| author | Ingo Molnar | 2008-11-26 02:00:05 +0100 |
|---|---|---|
| committer | David S. Miller | 2008-11-26 02:00:05 +0100 |
| commit | a098397d562e2ce5aca7b9b007a4954d88ef8f5e (patch) | |
| tree | 3aa91ccc9bd3019cf5bfd22fad4bb4a819e9863c /drivers/net/atlx | |
| parent | bluetooth: fix warning in net/bluetooth/rfcomm/sock.c (diff) | |
| download | kernel-qcow2-linux-a098397d562e2ce5aca7b9b007a4954d88ef8f5e.tar.gz kernel-qcow2-linux-a098397d562e2ce5aca7b9b007a4954d88ef8f5e.tar.xz kernel-qcow2-linux-a098397d562e2ce5aca7b9b007a4954d88ef8f5e.zip | |
atlx: fix warning in drivers/net/atlx/atl2.c
fix this warning:
drivers/net/atlx/atl2.c: In function ‘atl2_request_irq’:
drivers/net/atlx/atl2.c:644: warning: unused variable ‘err’
'err' is unused in the !CONFIG_PCI_MSI case.
Instead of further increasing the #ifdeffery in this function,
restructure the code a bit and get rid of the #ifdef. This
relies on the fact that pci_enable_msi() will always fail in
the !CONFIG_PCI_MSI case.
There should be no change in driver behavior.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/atlx')
| -rw-r--r-- | drivers/net/atlx/atl2.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/atlx/atl2.c b/drivers/net/atlx/atl2.c index 0326a84503e3..bc394491b63b 100644 --- a/drivers/net/atlx/atl2.c +++ b/drivers/net/atlx/atl2.c @@ -644,7 +644,6 @@ static int atl2_request_irq(struct atl2_adapter *adapter) int flags, err = 0; flags = IRQF_SHARED; -#ifdef CONFIG_PCI_MSI adapter->have_msi = true; err = pci_enable_msi(adapter->pdev); if (err) @@ -652,7 +651,6 @@ static int atl2_request_irq(struct atl2_adapter *adapter) if (adapter->have_msi) flags &= ~IRQF_SHARED; -#endif return request_irq(adapter->pdev->irq, &atl2_intr, flags, netdev->name, netdev); |