Commit 36c71a73 authored by Rasmus Villemoes's avatar Rasmus Villemoes Committed by David S. Miller

net: tulip: Remove private "strncmp"

The comment says that the built-in strncmp didn't work. That is not
surprising, as apparently "str" semantics are not really what is
wanted (hint: de4x5_strncmp only stops when two different bytes are
encountered or the end is reached; not if either byte happens to be
0). de4x5_strncmp is actually a memcmp (except for the signature and
that bytes are not necessarily treated as unsigned char); since only
the boolean value of the result is used we can just replace
de4x5_strncmp with memcmp.
Signed-off-by: default avatarRasmus Villemoes <linux@rasmusvillemoes.dk>
Acked-by: default avatarGrant Grundler <grundler@parisc-linux.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c8a73a35
...@@ -995,7 +995,6 @@ static void de4x5_dbg_mii(struct net_device *dev, int k); ...@@ -995,7 +995,6 @@ static void de4x5_dbg_mii(struct net_device *dev, int k);
static void de4x5_dbg_media(struct net_device *dev); static void de4x5_dbg_media(struct net_device *dev);
static void de4x5_dbg_srom(struct de4x5_srom *p); static void de4x5_dbg_srom(struct de4x5_srom *p);
static void de4x5_dbg_rx(struct sk_buff *skb, int len); static void de4x5_dbg_rx(struct sk_buff *skb, int len);
static int de4x5_strncmp(char *a, char *b, int n);
static int dc21041_infoleaf(struct net_device *dev); static int dc21041_infoleaf(struct net_device *dev);
static int dc21140_infoleaf(struct net_device *dev); static int dc21140_infoleaf(struct net_device *dev);
static int dc21142_infoleaf(struct net_device *dev); static int dc21142_infoleaf(struct net_device *dev);
...@@ -4102,8 +4101,7 @@ get_hw_addr(struct net_device *dev) ...@@ -4102,8 +4101,7 @@ get_hw_addr(struct net_device *dev)
} }
/* /*
** Test for enet addresses in the first 32 bytes. The built-in strncmp ** Test for enet addresses in the first 32 bytes.
** didn't seem to work here...?
*/ */
static int static int
de4x5_bad_srom(struct de4x5_private *lp) de4x5_bad_srom(struct de4x5_private *lp)
...@@ -4111,8 +4109,8 @@ de4x5_bad_srom(struct de4x5_private *lp) ...@@ -4111,8 +4109,8 @@ de4x5_bad_srom(struct de4x5_private *lp)
int i, status = 0; int i, status = 0;
for (i = 0; i < ARRAY_SIZE(enet_det); i++) { for (i = 0; i < ARRAY_SIZE(enet_det); i++) {
if (!de4x5_strncmp((char *)&lp->srom, (char *)&enet_det[i], 3) && if (!memcmp(&lp->srom, &enet_det[i], 3) &&
!de4x5_strncmp((char *)&lp->srom+0x10, (char *)&enet_det[i], 3)) { !memcmp((char *)&lp->srom+0x10, &enet_det[i], 3)) {
if (i == 0) { if (i == 0) {
status = SMC; status = SMC;
} else if (i == 1) { } else if (i == 1) {
...@@ -4125,18 +4123,6 @@ de4x5_bad_srom(struct de4x5_private *lp) ...@@ -4125,18 +4123,6 @@ de4x5_bad_srom(struct de4x5_private *lp)
return status; return status;
} }
static int
de4x5_strncmp(char *a, char *b, int n)
{
int ret=0;
for (;n && !ret; n--) {
ret = *a++ - *b++;
}
return ret;
}
static void static void
srom_repair(struct net_device *dev, int card) srom_repair(struct net_device *dev, int card)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment