diff options
author | Danny Kukawka <danny.kukawka@bisect.de> | 2012-02-20 21:07:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-02-23 17:03:20 -0500 |
commit | 504f9b5a6bb5336ad434438d0cdd61a16db80129 (patch) | |
tree | 307356c1d47a8300fd4973159edaad130c5cda9f /drivers/net/ethernet/neterion | |
parent | 32d219ed61a97f6591a941e23673b2c1dd8628e7 (diff) |
ethernet: unify return value of .ndo_set_mac_address if address is invalid
Unify return value of .ndo_set_mac_address if the given address
isn't valid. Return -EADDRNOTAVAIL as eth_mac_addr() already does
if is_valid_ether_addr() fails.
Signed-off-by: Danny Kukawka <danny.kukawka@bisect.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/neterion')
-rw-r--r-- | drivers/net/ethernet/neterion/s2io.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/neterion/s2io.c b/drivers/net/ethernet/neterion/s2io.c index 44a6065794c3..22a8de00bf02 100644 --- a/drivers/net/ethernet/neterion/s2io.c +++ b/drivers/net/ethernet/neterion/s2io.c | |||
@@ -5248,7 +5248,7 @@ static int s2io_set_mac_addr(struct net_device *dev, void *p) | |||
5248 | struct sockaddr *addr = p; | 5248 | struct sockaddr *addr = p; |
5249 | 5249 | ||
5250 | if (!is_valid_ether_addr(addr->sa_data)) | 5250 | if (!is_valid_ether_addr(addr->sa_data)) |
5251 | return -EINVAL; | 5251 | return -EADDRNOTAVAIL; |
5252 | 5252 | ||
5253 | memcpy(dev->dev_addr, addr->sa_data, dev->addr_len); | 5253 | memcpy(dev->dev_addr, addr->sa_data, dev->addr_len); |
5254 | 5254 | ||