On Thu, 2016-01-07 at 00:26 +0100, Bjørn Mork wrote: > Joe Perches <j...@perches.com> writes: > > On Wed, 2016-01-06 at 16:33 -0500, David Miller wrote: > > > 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. > > [] > > > diff --git a/include/linux/etherdevice.h b/include/linux/etherdevice.h > > [] > > > @@ -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?
It's declared __weak and should be overridden by an arch specific function. A NULL address would cause a fault when using a function like copy_ether_addr. > 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. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html