On Wed, 2016-02-03 at 16:23 +0100, Arnd Bergmann wrote: > On Wednesday 03 February 2016 16:02:38 Lubomir Rintel wrote: > > From: Arnd Bergmann <a...@arndb.de> > > > > This takes the MAC address for smsc75xx/smsc95xx USB network > > devices > > from a the device tree. This is required to get a usable persistent > > address on the popular beagleboard, whose hardware designers > > accidentally forgot that an ethernet device really requires an a > > MAC address to be functional. > > > > The smsc75xx and smsc95xx drivers are just two copies of the > > same code, so better fix both. > > > > Tested-by: Lubomir Rintel <lkund...@v3.sk> > > Signed-off-by: Arnd Bergmann <a...@arndb.de> > > > > I have no memory of writing this patch, where did you find it?
2011's discussion: https://lkml.org/lkml/2011/3/17/416 (Link also in the cover letter). > The changelog sounds like I wrote it, so I assume it was me after > all. > > > + address = of_get_property(dev->udev->dev.of_node, > > + "local-mac-address", NULL); > > + if (address) { > > + memcpy(dev->net->dev_addr, address, ETH_ALEN); > > + return; > > + } > > This should use of_get_mac_address(), not an open-coded property > lookup. The function was probably added after I wrote the > the original patch. Okay. Will fix that up once I get feedback for the devicetree part. > Arnd Thanks, Lubo