On Mar 11, 2009, at 12:18 AM, Stephen Hemminger wrote:

quoted text
> On Wed, 11 Mar 2009 05:29:09 +0300
> Ilya Yanok <yanok@emcraft.com> wrote:
>
>> +static char ethaddr[18];
>> +
>> +module_param_string(ethaddr, ethaddr, sizeof(ethaddr), 0);
>> +
>> +static void get_mac_addr(struct net_device *nd, unsigned char *pmac)
>> +{
>> +	int i;
>> +	char *p = ethaddr;
>> +
>> +	ethaddr[17] = 0;
>> +	for (i = 0; i < ETH_ALEN; i++, p++) {
>> +		pmac[i] = simple_strtoul(p, &p, 16);
>> +		if (*p != ':')
>> +			break;
>> +	}
>> +	if (i != ETH_ALEN - 1) {
>> +		pr_err("Wrong MAC address format!\n");
>> +		memset(pmac, 0, ETH_ALEN);
>> +	}
>> +}
>> +
>
> Do you still need this? Only works with one board, configuration via
> module parameters is discouraged.  Same effect can be done by setting
> mac address with regular tools.

I'm confused by this. If I'm NFS-booting the board then how do I get  
to those userland tools if I don't have a valid MAC address in place  
to do the root mount?

-Mike

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html