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

[Jason Liu]

Hi, Stefano,

2010/11/19 Jason Liu <liu.h.jason at gmail.com>:
> Hi, Stefano,
>
> 2010/11/18 Stefano Babic <sbabic at denx.de>:
>> On 11/18/2010 12:49 PM, Jason Liu wrote:
>>> The patch is to support getting FEC MAC address from fuse bank.
>>>
>>> Signed-off-by: Jason Liu <r64343 at freescale.com>
>>>
>>
>> Hi Jason,
>>
>> I applied your patch and I tested on a i.MX27 Lite board. The result
>> does not seem correct. According to the manual, the mac address is store
>> at address 0x10028814. Rather there is no stored MAC at this address on
>> my board, but your patch read the MAC from address 0x10028810 (I traced
>> the address of fuse->mac_addr).
>>
>> The reported address is what we can see at 0x10028810 (Word 4 of
>> fusebank 0):
>>
>> md 0x10028804
>> 10028804: 00000020 00000081 00000040 00000010     ....... at .......
>> 10028814: 00000000 00000000 00000000 00000000    ................
>>
>> And u-boot reports :
>> Address in SROM is         00:00:00:00:00:10
>> Address in environment is  00:aa:dd:cc:11:22
>>
>> It seems you read at a wrong address.
>
> I think you are right. I did not have the mx27 board to do test.
> Thanks for test.
>
> I only test it on mx51babbage board, the following is the output:
>
> Net:   got MAC address from fuse: 00:04:9f:00:f7:78
> which is the same as the MAC address programmed on the board.


Do you have any other comments about this patch?

BR,
Jason

>
>>
>> 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
>> =====================================================================
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>>
>