Re: [1/1] ARC vmac ethernet driver.

[David Miller <davem@davemloft.net>]

This needs some changes.

First, you'll need to add some proper dependencies to the Kconfig for
this driver before we can apply it.

At a minimum you have to depend upon HAS_DMA otherwise the
dma_*() interface calls won't link on platforms such as s390.

Also:

+static char *mac_addr;
+module_param(mac_addr, charp, 0644);
+MODULE_PARM_DESC(mac_addr, "MAC address as colon separated hexadecimals");

Please remove this, we have interfaces for setting the ethernet
address.  If the device address can't be probed, use
a random ethernet address as a default (by using linux/etherdevice.h's
random_ether_addr())

If the user wants to override they can do so with the 'ifconfig'
command.

This is how we consistently handle this situation in every ethernet
driver, so please don't try to make arguments as to why your driver
should handle this differently.

Thanks.