Hello Sascha,
On 10.06.2015 06:32, Sascha Hauer wrote:
Hi Wadim,
On Tue, Jun 09, 2015 at 09:04:25AM +0200, Wadim Egorov wrote:
Set the ethaddr for the current edev.
Signed-off-by: Wadim Egorov <w.egorov@xxxxxxxxx>
---
net/eth.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/net/eth.c b/net/eth.c
index 89bddba..03e0a2e 100644
--- a/net/eth.c
+++ b/net/eth.c
@@ -49,6 +49,7 @@ static void register_preset_mac_address(struct eth_device *edev, const char *eth
ethaddr_to_string(ethaddr, ethaddr_str);
if (is_valid_ether_addr(ethaddr)) {
+ memcpy(edev->ethaddr, ethaddr, 6);
dev_info(&edev->dev, "got preset MAC address: %s\n", ethaddr_str);
dev_set_param(&edev->dev, "ethaddr", ethaddr_str);
}
In which case is this necessary? Normally a dev_set_param on "ethaddr"
should already set edev->ethaddr, there should be no need to copy this
manually.
Sascha
when booting from ethernet on the AM335x, net_new() (called in net_udp_new)
will check if ethaddr is valid. This check fails, because ethaddr is not
set at this moment and a random MAC will be used.
Wadim
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
