Thanks Tom,

On 08/02/2013 07:16 AM, Tom Rini wrote:
On Fri, Aug 02, 2013 at 03:08:36PM +0200, Stefano Babic wrote:
Hi Eric,

On 02/08/2013 01:28, Eric Nelson wrote:
Hi all,

While testing Troy's usbnet patches, I was reminded of
this mysterious warning issued when the environment has
no mac address stored, but the device provides one:

     printf("\nWarning: %s using MAC address from net device\n",
             dev->name);

Why is this a warning? It's the out-of-the box default
for new boards who haven't saved a mac address to the
persistent environment.

Well, I have found it very useful in case I set explicitely the ethaddr
variable, so I know where the MAC address comes from. Do you find it
disturbing ? Maybe we could replace "warning" with "info".

On am335x the message we use is:
<ethaddr> not set. Validating first E-fuse MAC
And then validate it, and setenv.

I think this points out the issue. The fec_mxc driver isn't
setting a default ethaddr environment variable.

Stefano, we've had lots of questions from customers because
ethaddr is empty in the normal case (no environment saved).

I'll figure out where this goes and submit a patch.

Regards,


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

Reply via email to