Re: [PATCH v2] net: Add eth_platform_get_mac_address() helper.

Joe Perches [off-list ref] writes:

On Wed, 2016-01-06 at 16:33 -0500, David Miller wrote:

quoted

A repeating pattern in drivers has become to use OF node information
and, if not found, platform specific host information to extract the
ethernet address for a given device.

[]

quoted

diff --git a/include/linux/etherdevice.h b/include/linux/etherdevice.h

[]

quoted

@@ -485,3 +487,32 @@ static int __init eth_offload_init(void)

 }
 
 fs_initcall(eth_offload_init);
+
+unsigned char * __weak arch_get_platform_mac_address(void)
+{
+       return NULL;

WARN_ON_ONCE ?

That would prevent a driver from using this with additional fallback
methods.  For what reason?

I don't have a specific usecase, but I can imagine drivers falling back
to e.g a random address without wanting to be noisy about it.


Bjørn