diff options
| author | Hannes Eder <hannes@hanneseder.net> | 2009-02-14 12:47:30 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-02-18 04:34:46 +0100 |
| commit | de2f19daac017301b0bfd73bdecff89e7ea30eb7 (patch) | |
| tree | 7a52754f9976b26ea826a0231802474946ff2c18 /drivers/net/tulip/de4x5.c | |
| parent | drivers/net/tulip: fix warning: format not a string literal and no ... (diff) | |
| download | linux-de2f19daac017301b0bfd73bdecff89e7ea30eb7.tar.xz linux-de2f19daac017301b0bfd73bdecff89e7ea30eb7.zip | |
drivers/net/tulip: fix sparse warning: symbol shadows an earlier one
Impact: Rename a local variable resp. remove a redundant variable and
while being at it use more unform loop constructs.
Fix this sparse warning:
drivers/net/tulip/de4x5.c:3944:13: warning: symbol 'i' shadows an earlier one
drivers/net/tulip/de4x5.c:3938:9: originally declared here
drivers/net/tulip/media.c:72:21: warning: symbol 'i' shadows an earlier one
drivers/net/tulip/media.c:54:13: originally declared here
drivers/net/tulip/media.c:134:21: warning: symbol 'i' shadows an earlier one
drivers/net/tulip/media.c:117:13: originally declared here
Signed-off-by: Hannes Eder <hannes@hanneseder.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tulip/de4x5.c')
| -rw-r--r-- | drivers/net/tulip/de4x5.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/tulip/de4x5.c b/drivers/net/tulip/de4x5.c index 98e86826b2c7..6430a2ec6db1 100644 --- a/drivers/net/tulip/de4x5.c +++ b/drivers/net/tulip/de4x5.c @@ -3942,8 +3942,8 @@ PCI_signature(char *name, struct de4x5_private *lp) strcpy(name, "DE434/5"); return status; } else { /* Search for a DEC name in the SROM */ - int i = *((char *)&lp->srom + 19) * 3; - strncpy(name, (char *)&lp->srom + 26 + i, 8); + int tmp = *((char *)&lp->srom + 19) * 3; + strncpy(name, (char *)&lp->srom + 26 + tmp, 8); } name[8] = '\0'; for (i=0; i<siglen; i++) { |