drivers: net: tulip: Fix set but not used with W=1
authorAndrew Lunn <andrew@lunn.ch>
Sat, 31 Oct 2020 00:54:45 +0000 (01:54 +0100)
committerJakub Kicinski <kuba@kernel.org>
Mon, 2 Nov 2020 23:50:04 +0000 (15:50 -0800)
When compiled for platforms other than __i386__ or __x86_64__:

drivers/net/ethernet/dec/tulip/tulip_core.c: In function ‘tulip_init_one’:
drivers/net/ethernet/dec/tulip/tulip_core.c:1296:13: warning: variable ‘last_irq’ set but not used [-Wunused-but-set-variable]
 1296 |  static int last_irq;

Add more #if defined() to totally remove the code when not needed.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/20201031005445.1060112-1-andrew@lunn.ch
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/dec/tulip/tulip_core.c

index e7b0d7d..c1dcd6c 100644 (file)
@@ -1293,7 +1293,9 @@ static int tulip_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
        static unsigned char last_phys_addr[ETH_ALEN] = {
                0x00, 'L', 'i', 'n', 'u', 'x'
        };
+#if defined(__i386__) || defined(__x86_64__)   /* Patch up x86 BIOS bug. */
        static int last_irq;
+#endif
        int i, irq;
        unsigned short sum;
        unsigned char *ee_data;
@@ -1617,7 +1619,9 @@ static int tulip_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 
        for (i = 0; i < 6; i++)
                last_phys_addr[i] = dev->dev_addr[i];
+#if defined(__i386__) || defined(__x86_64__)   /* Patch up x86 BIOS bug. */
        last_irq = irq;
+#endif
 
        /* The lower four bits are the media type. */
        if (board_idx >= 0  &&  board_idx < MAX_UNITS) {