On 18/01/13 05:14, Mugunthan V N wrote: > On 1/18/2013 3:48 AM, Peter Korsgaard wrote: >> When booting with CONFIG_ARM_APPENDED_DTB (either because of using an old >> U-Boot, not wanting the hassle of 2 files or when using Falcon fast boot >> mode in U-Boot), nothing updates the ethernet hwaddr specified for the >> CPSW slaves, causing the driver to use a random hwaddr, which is some times >> troublesome. >> >> The am33xx has unique ethernet hwaddrs programmed in the efuse, so it makes >> more sense to default to these rather than random ones. Add a fixup step >> which adds mac-address dt properties using the efuse addresses if the DTB >> didn't contain valid ones. >> >> Signed-off-by: Peter Korsgaard <jacmet@xxxxxxxxxx> >> > > This implementation looks fine. > Acked-by: Mugunthan V N <mugunthanvnm@xxxxxx> Tested-by: Mark Jackson <mpfj-list@xxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html