diff options
| author | Johannes Stezenbach <js@sig21.net> | 2012-06-18 16:47:59 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2012-06-30 13:58:18 +0200 |
| commit | 69e07e2afb650702dad23bea882731fe02dbb964 (patch) | |
| tree | c9730ca246aa956990649606a4e30303c4b62423 /net | |
| parent | 8f2fcf65541d911c5de0bf9fcfbfda93a20bd723 (diff) | |
| download | barebox-69e07e2afb650702dad23bea882731fe02dbb964.tar.gz barebox-69e07e2afb650702dad23bea882731fe02dbb964.tar.xz | |
eth: eth MAC addresses are six bytes
Reduce confusion by making clear six bytes are passed,
not a string.
Signed-off-by: Johannes Stezenbach <js@sig21.net>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'net')
| -rw-r--r-- | net/eth.c | 2 | ||||
| -rw-r--r-- | net/net.c | 4 |
2 files changed, 3 insertions, 3 deletions
@@ -169,7 +169,7 @@ int eth_rx(void) static int eth_set_ethaddr(struct device_d *dev, struct param_d *param, const char *val) { struct eth_device *edev = dev_to_edev(dev); - char ethaddr[sizeof("xx:xx:xx:xx:xx:xx")]; + u8 ethaddr[6]; if (!val) return dev_param_set_generic(dev, param, NULL); @@ -159,7 +159,7 @@ void print_IPaddr (IPaddr_t x) puts(ip_to_string(x)); } -int string_to_ethaddr(const char *str, char *enetaddr) +int string_to_ethaddr(const char *str, u8 enetaddr[6]) { int reg; char *e; @@ -181,7 +181,7 @@ int string_to_ethaddr(const char *str, char *enetaddr) return 0; } -void ethaddr_to_string(const unsigned char *enetaddr, char *str) +void ethaddr_to_string(const u8 enetaddr[6], char *str) { sprintf(str, "%02X:%02X:%02X:%02X:%02X:%02X", enetaddr[0], enetaddr[1], enetaddr[2], enetaddr[3], |