<br><br><div class="gmail_quote">On Thu, Jan 17, 2013 at 11:19 PM, Peter Korsgaard <span dir="ltr"><<a href="mailto:jacmet@sunsite.dk" target="_blank">jacmet@sunsite.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>>>>> "Tony" == Tony Lindgren <<a href="mailto:tony@atomide.com">tony@atomide.com</a>> writes:<br>
<br>
Hi,<br>
<div class="im"><br>
 >> The way I've handled it is similar to how it is done on atleast one<br>
 >> other arm subarchicture, see<br>
 >> arch/arm/mach-mxs/mach-mxs.c::update_fec_mac_prop()<br>
<br>
</div> Tony> Cool this seems like the cleanest way to deal with it so far. Can<br>
 Tony> you please resend with also linux-net and DT list Cc:d too?<br>
<br>
 Tony> Also the arch_initcall should be omap_arch_initcall in linux next<br>
 Tony> for consistency if we ever have SoC specific sections for those.<br>
<br>
Sure, resent with the initcall change and netdev/devicetree-discuss added.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Bye, Peter Korsgaard<br>
</font></span></blockquote></div><br>Thanks,<br>I added DT option mac-address-source because there is still possiblility to <br>have mac address set by the u-boot in the future. Beaglebone have special <br>eeprom on board, were you can have stored user MAC and u-boot can read eeprom <br>
data. Maybe some users will want this feature.  Also in cpsw dt binding <br>documentation, there is mention  /* Filled in by U-Boot */.  For that purpose, <br>code I proposed can easily deal with any driver exteds. <br><br>
Reading CPU mac address registers with (my approach) or without (your <br>approach) DT should be more discussed. Also your way seems to be fine. <br><br>Regards<br>Michal<br>