diff options
author | Danny Kukawka <danny.kukawka@bisect.de> | 2012-02-15 01:45:40 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-02-15 15:34:17 -0500 |
commit | 7ce5d222190cb3ce3ae88bafde7c4fa52a5103e0 (patch) | |
tree | 2b284365b7a2e11d4ba286a589bd35bc39918805 /drivers/net/ethernet/micrel | |
parent | f2cedb63df14342ad40a8b5b324fc5d94a60b665 (diff) |
net: use eth_hw_addr_random() and reset addr_assign_type
Use eth_hw_addr_random() instead of calling random_ether_addr()
to set addr_assign_type correctly to NET_ADDR_RANDOM.
Reset the state to NET_ADDR_PERM as soon as the MAC get
changed via .ndo_set_mac_address.
v2: adapt to renamed eth_hw_addr_random()
Signed-off-by: Danny Kukawka <danny.kukawka@bisect.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/micrel')
-rw-r--r-- | drivers/net/ethernet/micrel/ks8842.c | 3 | ||||
-rw-r--r-- | drivers/net/ethernet/micrel/ks8851.c | 3 | ||||
-rw-r--r-- | drivers/net/ethernet/micrel/ks8851_mll.c | 1 |
3 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/ethernet/micrel/ks8842.c b/drivers/net/ethernet/micrel/ks8842.c index 0a85690a1321..0686b93f1857 100644 --- a/drivers/net/ethernet/micrel/ks8842.c +++ b/drivers/net/ethernet/micrel/ks8842.c | |||
@@ -1080,6 +1080,7 @@ static int ks8842_set_mac(struct net_device *netdev, void *p) | |||
1080 | if (!is_valid_ether_addr(addr->sa_data)) | 1080 | if (!is_valid_ether_addr(addr->sa_data)) |
1081 | return -EADDRNOTAVAIL; | 1081 | return -EADDRNOTAVAIL; |
1082 | 1082 | ||
1083 | netdev->addr_assign_type &= ~NET_ADDR_RANDOM; | ||
1083 | memcpy(netdev->dev_addr, mac, netdev->addr_len); | 1084 | memcpy(netdev->dev_addr, mac, netdev->addr_len); |
1084 | 1085 | ||
1085 | ks8842_write_mac_addr(adapter, mac); | 1086 | ks8842_write_mac_addr(adapter, mac); |
@@ -1211,7 +1212,7 @@ static int __devinit ks8842_probe(struct platform_device *pdev) | |||
1211 | ks8842_read_mac_addr(adapter, netdev->dev_addr); | 1212 | ks8842_read_mac_addr(adapter, netdev->dev_addr); |
1212 | 1213 | ||
1213 | if (!is_valid_ether_addr(netdev->dev_addr)) | 1214 | if (!is_valid_ether_addr(netdev->dev_addr)) |
1214 | random_ether_addr(netdev->dev_addr); | 1215 | eth_hw_addr_random(netdev); |
1215 | } | 1216 | } |
1216 | 1217 | ||
1217 | id = ks8842_read16(adapter, 32, REG_SW_ID_AND_ENABLE); | 1218 | id = ks8842_read16(adapter, 32, REG_SW_ID_AND_ENABLE); |
diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c index 0930b9c950d0..42a6a20efb15 100644 --- a/drivers/net/ethernet/micrel/ks8851.c +++ b/drivers/net/ethernet/micrel/ks8851.c | |||
@@ -439,7 +439,7 @@ static void ks8851_init_mac(struct ks8851_net *ks) | |||
439 | dev->dev_addr); | 439 | dev->dev_addr); |
440 | } | 440 | } |
441 | 441 | ||
442 | random_ether_addr(dev->dev_addr); | 442 | eth_hw_addr_random(dev); |
443 | ks8851_write_mac_addr(dev); | 443 | ks8851_write_mac_addr(dev); |
444 | } | 444 | } |
445 | 445 | ||
@@ -1050,6 +1050,7 @@ static int ks8851_set_mac_address(struct net_device *dev, void *addr) | |||
1050 | if (!is_valid_ether_addr(sa->sa_data)) | 1050 | if (!is_valid_ether_addr(sa->sa_data)) |
1051 | return -EADDRNOTAVAIL; | 1051 | return -EADDRNOTAVAIL; |
1052 | 1052 | ||
1053 | dev->addr_assign_type &= ~NET_ADDR_RANDOM; | ||
1053 | memcpy(dev->dev_addr, sa->sa_data, ETH_ALEN); | 1054 | memcpy(dev->dev_addr, sa->sa_data, ETH_ALEN); |
1054 | return ks8851_write_mac_addr(dev); | 1055 | return ks8851_write_mac_addr(dev); |
1055 | } | 1056 | } |
diff --git a/drivers/net/ethernet/micrel/ks8851_mll.c b/drivers/net/ethernet/micrel/ks8851_mll.c index 3d9a04e7b49d..180460f4e41f 100644 --- a/drivers/net/ethernet/micrel/ks8851_mll.c +++ b/drivers/net/ethernet/micrel/ks8851_mll.c | |||
@@ -1241,6 +1241,7 @@ static int ks_set_mac_address(struct net_device *netdev, void *paddr) | |||
1241 | struct sockaddr *addr = paddr; | 1241 | struct sockaddr *addr = paddr; |
1242 | u8 *da; | 1242 | u8 *da; |
1243 | 1243 | ||
1244 | netdev->addr_assign_type &= ~NET_ADDR_RANDOM; | ||
1244 | memcpy(netdev->dev_addr, addr->sa_data, netdev->addr_len); | 1245 | memcpy(netdev->dev_addr, addr->sa_data, netdev->addr_len); |
1245 | 1246 | ||
1246 | da = (u8 *)netdev->dev_addr; | 1247 | da = (u8 *)netdev->dev_addr; |