[U-Boot] [PATCH 1/2] imx: Get fec mac address from fuse

[Stefano Babic <sbabic@denx.de>]

On 10/26/2010 12:35 PM, Jason Liu wrote:
>>>  static int fec_set_hwaddr(struct eth_device *dev)
>>> @@ -414,9 +410,6 @@ static int fec_init(struct eth_device *dev, bd_t* bd)
>>>       uint32_t base;
>>>       struct fec_priv *fec = (struct fec_priv *)dev->priv;
>>>
>>> -     /* Initialize MAC address */
>>> -     fec_set_hwaddr(dev);
>>> -
>>
>> Not sure. Why is this call removed ? We have changed the storage for the
>> MAC address, but we need always to set the FEC controller. What is the
>> reason to drop it ?
> 
> It's because, it will print some floating  value  of MAC address when
> bootup and in fact, we don't need re-set it again here..
> The net framework will do that. This is why I remove. it.

The call was added one month ago. It sets the MAC address any time the
network is used, to be sure that the correct MAC is taken even if the
ethaddr variable is changed. You can take a look at the patch:

http://www.mail-archive.com/u-boot at lists.denx.de/msg37629.html

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================