[U-Boot] Mac address warning

[Eric Nelson]

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