Hi, Stefano, 2010/11/19 Jason Liu <liu.h.ja...@gmail.com>: > Hi, Stefano, > > 2010/11/18 Stefano Babic <sba...@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 <r64...@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 .......@....... >> 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: off...@denx.de >> ===================================================================== >> _______________________________________________ >> U-Boot mailing list >> U-Boot@lists.denx.de >> http://lists.denx.de/mailman/listinfo/u-boot >> > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot