]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
atlx: fix warning in drivers/net/atlx/atl2.c
authorIngo Molnar <mingo@elte.hu>
Wed, 26 Nov 2008 01:00:05 +0000 (17:00 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 26 Nov 2008 01:00:05 +0000 (17:00 -0800)
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>
drivers/net/atlx/atl2.c

index 0326a84503e3eb9d9aa5b7ac73c2df2c8b643af2..bc394491b63bb45397bc15df3d058987c0bec0d6 100644 (file)
@@ -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);