diff options
author | Dominik Brodowski <linux@dominikbrodowski.net> | 2011-07-29 18:27:34 +0200 |
---|---|---|
committer | Dominik Brodowski <linux@dominikbrodowski.net> | 2011-07-29 18:27:34 +0200 |
commit | 74411c04540404a1ff29d254584ef393e150aeda (patch) | |
tree | ea8cfb194417e3dd1682b53201e12e38ffedc1c2 | |
parent | 49e58a932f6a35ad3d4888b5f66b27d4e20d990d (diff) | |
download | op-kernel-dev-74411c04540404a1ff29d254584ef393e150aeda.zip op-kernel-dev-74411c04540404a1ff29d254584ef393e150aeda.tar.gz |
smc91c92_cs.c: fix bogus compiler warning
On a recent compile, I got this warning:
CC [M] drivers/net/pcmcia/smc91c92_cs.o
drivers/net/pcmcia/smc91c92_cs.c: In function ‘smc91c92_probe’:
drivers/net/pcmcia/smc91c92_cs.c:812:12: warning: ‘j’ may be used uninitialized in this function
However, "j" is only used in a branch which has the same condition as
a previous branch, where j is set, e.g.
int j;
if (CONDITION)
j = VALUE
...
if (CONDITION)
printk(j)
Still, avoid this warning, as it is easy to circumvent.
Signed-off-by: Dominik Brodowski <linux@dominikbrodowski.net>
-rw-r--r-- | drivers/net/pcmcia/smc91c92_cs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/pcmcia/smc91c92_cs.c b/drivers/net/pcmcia/smc91c92_cs.c index 1cd9394..cffbc03 100644 --- a/drivers/net/pcmcia/smc91c92_cs.c +++ b/drivers/net/pcmcia/smc91c92_cs.c @@ -809,7 +809,7 @@ static int smc91c92_config(struct pcmcia_device *link) struct net_device *dev = link->priv; struct smc_private *smc = netdev_priv(dev); char *name; - int i, j, rev; + int i, rev, j = 0; unsigned int ioaddr; u_long mir; |