With the following commit I now see:

commit 7616e7850804c7c69e0a22c179dfcba9e8f3f587
Author: Simon Glass <s...@chromium.org>
Date:   Mon Jun 13 16:13:10 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.

   Signed-off-by: Simon Glass <s...@chromium.org>
   Tested-by: Eric Bénard <e...@eukrea.com>

messages like:

Warning: failed to set MAC address

However, I don't see how this ever works properly as we do:

                       if (eth_write_hwaddr(dev, NULL, eth_number))

this ends up calling:

eth_getenv_enetaddr_by_index(base_name, eth_number, env_enetaddr)

but base_name was passed in as NULL.

So we lookup env strings like:

<NULL>1addr
<NULL>2addr
<NULL>3addr

etc., seems wrong.

- k

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to