Linaro Android

Bug #817315
Comment #3

Comment 3 for bug 817315

Revision history for this message

Eric Miao (eric.y.miao) wrote on 2011-08-17:

0001-net-eth.c-fix-eth_write_hwaddr-to-use-dev-enetaddr-a.patch Edit (1.1 KiB, text/plain)

The bug is caused by behavior changed in commit:

commit 48506a2cde2458fa1f8c5993afc98e5a4617e1d3
Author: Simon Glass <email address hidden>
Date: Thu May 5 09:52:07 2011 -0700

Add Ethernet hardware MAC address framework to usbnet

Built-in Ethernet adapters support setting the mac address by means of a
ethaddr environment variable for each interface (ethaddr, eth1addr, eth2addr).

    This adds similar support to the USB network side, using the names
    usbethaddr, usbeth1addr, etc. They are kept separate since we don't want
    a USB device taking the MAC address of a built-in device or vice versa.

TEST=build, test on harmony, with setenv usbethaddr c0:c1:c0:13:0b:b8, bootp,
tftp ...

Signed-off-by: Simon Glass <email address hidden>

And can be fixed simply by ignoring the return value of eth_getenv_enetaddr_by_index() which should be the right fix, as it allows use of dev->enetaddr as fall-back, fec_mxc.c will fill this field up with the correct MAC address obtained from fuse. The attached patch fixed this issue and has been submitted to upstream.