Re: [PATCH] of_net: Implement of_get_nvmem_mac_address helper

[Mike Looijmans]

ïOn 25-03-18 10:17, Mike Looijmans wrote:
> On 24-03-18 19:53, Andrew Lunn wrote:
> > > A quick survey for the of_get_mac_address users learns that most of them do
> > > a memcpy (or similar) right after it, so for these drivers the
> > > "of_get_nvmem_mac_address" style signature that performs the memcpy (or
> > > better, ether_addr_copy) is a better fit, e.g.:
> > >
> > > int of_get_mac_address(struct device_node *np, void *addr)
> >
> > Hi Mike
> >
> > This is a nicer solution, but it is quite a lot of work, there are a
> > lot of users. Maybe Coccinelle can help?
>
> About 58 of them, yeah. And this looked like such a simple thing when I 
> started it...
> https://elixir.bootlin.com/linux/v4.16-rc6/ident/of_get_mac_address
> I have no experience with Coccinelle though.

I'll at least post a v2 patch with what we've discussed so far.

Another thing that just occurred to me is that the nvmem interface would also 
allow a MAC address to be written to NV storage. Haven't looked at that path, 
but it would be nice to be able to permanently program the MAC address using 
standard interfaces.


Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijmans@xxxxxxxxxxxxxxxxx
Website: www.topicproducts.com

Please consider the environment before printing this e-mail