<br><br><div class="gmail_quote">On Fri, Jan 18, 2013 at 6:14 AM, Mugunthan V N <span dir="ltr"><<a href="mailto:mugunthanvnm@ti.com" target="_blank">mugunthanvnm@ti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 1/18/2013 3:48 AM, Peter Korsgaard wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
When booting with CONFIG_ARM_APPENDED_DTB (either because of using an old<br>
U-Boot, not wanting the hassle of 2 files or when using Falcon fast boot<br>
mode in U-Boot), nothing updates the ethernet hwaddr specified for the<br>
CPSW slaves, causing the driver to use a random hwaddr, which is some times<br>
troublesome.<br>
<br>
The am33xx has unique ethernet hwaddrs programmed in the efuse, so it makes<br>
more sense to default to these rather than random ones. Add a fixup step<br>
which adds mac-address dt properties using the efuse addresses if the DTB<br>
didn't contain valid ones.<br>
<br>
Signed-off-by: Peter Korsgaard <<a href="mailto:jacmet@sunsite.dk" target="_blank">jacmet@sunsite.dk</a>><br>
<br>
</blockquote>
<br></div>
This implementation looks fine.<br>
Acked-by: Mugunthan V N <<a href="mailto:mugunthanvnm@ti.com" target="_blank">mugunthanvnm@ti.com</a>><br>
<br>
Regards<span class="HOEnZb"><font color="#888888"><br>
Mugunthan V N<br>
</font></span></blockquote></div><br>Thanks,<br>My approach for cpsw driver is posted here: <a href="https://patchwork.kernel.org/patch/1966481/">https://patchwork.kernel.org/patch/1966481/</a><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>